Skip to content

Commit

Permalink
Fix Kconfig dependency problems wrt boolean menuconfigs
Browse files Browse the repository at this point in the history
If one has a dependency chain (tristate)FOO depends on (bool)BAR depends on
(tristate)BAZ, build problems will result.  If BAZ=m, then BAR can be set
y, which allows FOO=y.  It's possible to have FOO=y && BAZ=m, which
wouldn't be allowed if FOO depended directly on BAZ.  In effect, the bool
promotes the tristate from m to y.

This ends up causing a problem with several menuconfigs that look like:

menuconfig BAR
	bool
	depends on BAZ [tristate]
if BAR
config FOO
	tristate
endif

The solution used here is to add the dependencies of BAR to the if
statement, so that items in the if block will gain a direct
non-bool-promoted dependency on BAZ.  This is how it would work if a menu
was used instead of an if block.

Signed-off-by: Trent Piepho <xyzzy@speakeasy.org>
Acked-by: Mauro Carvalho Chehab <mchehab@infradead.org>
Cc: "David S. Miller" <davem@davemloft.net>
Acked-by: Jeff Garzik <jeff@garzik.org>
Cc: Dominik Brodowski <linux@dominikbrodowski.net>
Cc: Chas Williams <chas@cmf.nrl.navy.mil>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
  • Loading branch information
Trent Piepho authored and Linus Torvalds committed Jun 28, 2007
1 parent 30acbab commit 59faba1
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion drivers/atm/Kconfig
Expand Up @@ -7,7 +7,7 @@ menuconfig ATM_DRIVERS
depends on NETDEVICES && ATM
default y

if ATM_DRIVERS
if ATM_DRIVERS && NETDEVICES && ATM

config ATM_DUMMY
tristate "Dummy ATM driver"
Expand Down
2 changes: 1 addition & 1 deletion drivers/media/dvb/Kconfig
Expand Up @@ -11,7 +11,7 @@ menuconfig DVB_CAPTURE_DRIVERS
---help---
Say Y to select Digital TV adapters

if DVB_CAPTURE_DRIVERS
if DVB_CAPTURE_DRIVERS && DVB_CORE

comment "Supported SAA7146 based PCI Adapters"
depends on DVB_CORE && PCI && I2C
Expand Down
2 changes: 1 addition & 1 deletion drivers/media/radio/Kconfig
Expand Up @@ -9,7 +9,7 @@ menuconfig RADIO_ADAPTERS
---help---
Say Y here to enable selecting AM/FM radio adapters.

if RADIO_ADAPTERS
if RADIO_ADAPTERS && VIDEO_DEV

config RADIO_CADET
tristate "ADS Cadet AM/FM Tuner"
Expand Down
6 changes: 3 additions & 3 deletions drivers/media/video/Kconfig
Expand Up @@ -11,7 +11,7 @@ menuconfig VIDEO_CAPTURE_DRIVERS
webcams, analog TV, and hybrid analog/digital TV.
Some of those devices also supports FM radio.

if VIDEO_CAPTURE_DRIVERS
if VIDEO_CAPTURE_DRIVERS && VIDEO_DEV

config VIDEO_ADV_DEBUG
bool "Enable advanced debug functionality"
Expand Down Expand Up @@ -347,7 +347,7 @@ endmenu # encoder / decoder chips

config VIDEO_VIVI
tristate "Virtual Video Driver"
depends on VIDEO_V4L2 && !SPARC32 && !SPARC64 && PCI && VIDEO_DEV
depends on VIDEO_V4L2 && !SPARC32 && !SPARC64 && PCI
select VIDEO_BUF
default n
---help---
Expand Down Expand Up @@ -691,7 +691,7 @@ menuconfig V4L_USB_DRIVERS
depends on USB
default y

if V4L_USB_DRIVERS
if V4L_USB_DRIVERS && USB

source "drivers/media/video/pvrusb2/Kconfig"

Expand Down
2 changes: 1 addition & 1 deletion drivers/net/pcmcia/Kconfig
Expand Up @@ -19,7 +19,7 @@ menuconfig NET_PCMCIA

If unsure, say N.

if NET_PCMCIA
if NET_PCMCIA && PCMCIA

config PCMCIA_3C589
tristate "3Com 3c589 PCMCIA support"
Expand Down

0 comments on commit 59faba1

Please sign in to comment.