michaelh/zephyr
1
0
Fork 0
Code Releases Activity

kconfig: Remove some redundant single-item menus and ifs

Browse source 
A single menu within an if like

    if FOO

    menu "blah"

    ...

    endmenu

    endif

can be replaced with

    menu "blah"
            depends on FOO

    ...

    endmenu

Fix up all existing instances.

Also remove redundant extra menus underneath 'menuconfig' symbols.
'menuconfig' already creates a menu.

Also remove the menu in arch/arm/core/aarch32/Kconfig around the
"Floating point ABI" choice. The choice depends on FLOAT, which depends
on CPU_HAS_CPU, so remove the 'depends on CPU_HAS_FPU' too.

Piggyback removing a redundant 'default n' for BME280.

Signed-off-by: Ulf Magnusson <Ulf.Magnusson@nordicsemi.no>
This commit is contained in:
Ulf Magnusson 2019-12-21 07:00:23 +01:00 • committed by Anas Nashif
commit 6ef29c0250
8 changed files with 3 additions and 34 deletions
Download patch file Download diff file Expand all files Collapse all files

5
drivers/sensor/bme280/Kconfig
View file

@ -7,14 +7,11 @@
menuconfig BME280
bool "BME280/BMP280 sensor"
depends on I2C || SPI
default n
help
Enable driver for BME280 I2C-based or SPI-based temperature and pressure sensor.
if BME280
menu "Attributes"
choice
prompt "BME280 temperature oversampling"
default BME280_TEMP_OVER_2X
@ -110,6 +107,4 @@ config BME280_FILTER_16
bool "16"
endchoice
endmenu
endif # BME280