michaelh/zephyr
1
0
Fork 0
Code Releases Activity

kconfig: drivers: pwm: Remove redundant deps.

Browse source 
drivers/pwm/Kconfig.esp32 is 'source'd within an 'if PWM' in
drivers/pwm/Kconfig.

'if FOO' is just shorthand for adding 'depends on FOO' to each item
within the 'if'. Dependencies on menus work similarly. There are no
"conditional includes" in Kconfig, so 'if FOO' has no special meaning
around a source. Conditional includes wouldn't be possible, because an
if condition could include (directly or indirectly) forward references
to symbols not defined yet.

Tip: When adding a symbol, check its dependencies in the menuconfig
('ninja menuconfig', then / to jump to the symbol). The menuconfig also
shows how the file with the symbol got included, so if you see
duplicated dependencies, it's easy to hunt down where they come from.

Signed-off-by: Ulf Magnusson <Ulf.Magnusson@nordicsemi.no>
This commit is contained in:
Ulf Magnusson 2019-03-07 06:44:34 +01:00 • committed by Anas Nashif
commit 46c19ff18f
2 changed files with 0 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

1
drivers/pwm/Kconfig.esp32
View file

@ -16,7 +16,6 @@ if PWM_LED_ESP32
config PWM_LED_ESP32_DEV_NAME_0
string "ESP32 PWM LED Name"
depends on PWM_LED_ESP32
default "PWM_LED"
help
Specify the device name for the PWM driver.

2
drivers/pwm/Kconfig.pca9685
View file

@ -20,14 +20,12 @@ if PWM_PCA9685
config PWM_PCA9685_INIT_PRIORITY
int "Init priority"
depends on PWM_PCA9685
default 70
help
Device driver initialization priority.
config PWM_PCA9685_0
bool "PCA9685 PWM chip #0"
depends on PWM_PCA9685
help
Enable config options for the PCA9685 I2C-based PWM chip #0.