michaelh/zephyr
1
0
Fork 0
Code Releases Activity

kconfig: Turn pointless/confusing 'menuconfig's into 'config's

Browse source 
Defining a symbol with 'menuconfig' just tells the menuconfig to display
any dependent symbols that immediately follow it in a separate menu.
'menuconfig' has no effect on symbol values.

Making a symbol that doesn't have any dependent symbols after it a
'menuconfig' should be avoided, because then you end up with an empty
menu, which is shown as e.g.

    [*] Enable foo ---

This is how it would be shown if there were children but they all
happened to be invisible as well.

With a regular 'config', it turns into

    [*] Enable foo

Change all pointless 'menuconfig's to 'config's.

See the section on 'menuconfig' on the Kconfig - Tips and Best Practices
page as well.

Signed-off-by: Ulf Magnusson <Ulf.Magnusson@nordicsemi.no>
This commit is contained in:
Ulf Magnusson 2019-10-22 01:04:04 +02:00 • committed by Maureen Helm
commit 677f1e6db9
71 changed files with 90 additions and 97 deletions
Download patch file Download diff file Expand all files Collapse all files

2
drivers/sensor/opt3001/Kconfig
View file

@ -5,7 +5,7 @@
# SPDX-License-Identifier: Apache-2.0
#
menuconfig OPT3001
config OPT3001
bool "OPT3001 Light Sensor"
depends on (I2C && HAS_DTS_I2C)
help