Skip to content

Commit

Permalink
mk: config: default enable CFG_NOTIF_TEST_WD only when dependencies a…
Browse files Browse the repository at this point in the history
…re enabled

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 4, 2024
1 parent 718cc2b commit acf64d2
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion mk/config.mk
Original file line number Diff line number Diff line change
Expand Up @@ -1035,7 +1035,8 @@ endif
CFG_CALLOUT ?= $(CFG_CORE_ASYNC_NOTIF)

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

Expand Down

0 comments on commit acf64d2

Please sign in to comment.