michaelh/zephyr
1
0
Fork 0
Code Releases Activity

kconfig: Replace defconfig singe-symbol 'if's with 'depends on'

Browse source 
Same deal as in commit eddd98f ("kconfig: Replace some single-symbol
'if's with 'depends on'"), for all symbols defined within defconfig
files. See that commit for an explanation.

Maybe 'if's were used originally to mirror the 'if's in the main Kconfig
files, and then it got copied around by people assuming 'if' must work
differently from 'depends on'. It doesn't match in every spot at least.
Better to keep it simple and just consistently use 'depends on' when
it's a single symbol/choice I think. Helps reinforce that 'if' isn't
magic too.

Verified by printing all Kconfig menu nodes (symbols, choices, menus,
etc.) before and after the change and diffing (should show no
difference).

Signed-off-by: Ulf Magnusson <Ulf.Magnusson@nordicsemi.no>
This commit is contained in:
Ulf Magnusson 2020-02-10 04:18:50 +01:00 • committed by Kumar Gala
commit a42a42cd5a
272 changed files with 752 additions and 2949 deletions
Download patch file Download diff file Expand all files Collapse all files

5
soc/xtensa/intel_apl_adsp/Kconfig.defconfig
View file

@ -32,13 +32,10 @@ config DYNAMIC_INTERRUPTS
config LOG
default y
if TEST
# To prevent test uses TEST_LOGGING_MINIMAL
config TEST_LOGGING_DEFAULTS
default n
endif # TEST
depends on TEST
if LOG