Skip to content

Commit

Permalink
Default enable CFG_NOTIF_TEST_WD only when dependencies are enabled
Browse files Browse the repository at this point in the history
Set CFG_NOTIF_TEST_WD ?= y only when the features it needs are enabled.
Fixes the following warning on platforms that enable
CFG_ENABLE_EMBEDDED_TESTS but not CFG_CORE_ASYNC_NOTIF or CFG_CALLOUT:

 mk/config.mk:1039: Warning: Disabling CFG_NOTIF_TEST_WD [requires CFG_CALLOUT CFG_CORE_ASYNC_NOTIF]

Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
  • Loading branch information
jforissier committed Oct 3, 2024
1 parent 718cc2b commit 02725f0
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion mk/config.mk
Original file line number Diff line number Diff line change
Expand Up @@ -1035,7 +1035,9 @@ endif
CFG_CALLOUT ?= $(CFG_CORE_ASYNC_NOTIF)

# Enable notification based test watchdog
CFG_NOTIF_TEST_WD ?= $(CFG_ENABLE_EMBEDDED_TESTS)
ifeq (yy,$(CFG_ENABLE_EMBEDDED_TESTS)$(CFG_CALLOUT)$(CFG_CORE_ASYNC_NOTIF))
CFG_NOTIF_TEST_WD ?= y
endif
$(eval $(call cfg-depends-all,CFG_NOTIF_TEST_WD,CFG_CALLOUT \
CFG_CORE_ASYNC_NOTIF))

Expand Down

0 comments on commit 02725f0

Please sign in to comment.