| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Mon, 4 Oct 2010 18:03:46 +0200 From: Jean Delvare <khali@...ux-fr.org> To: Michal Marek <mmarek@...e.cz> Cc: linux-i2c@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH] i2c: Fix Kconfig dependencies Hi Michal, On Mon, 4 Oct 2010 15:31:33 +0200, Michal Marek wrote: > drivers/i2c/algos/Kconfig makes all the algorithms dependent on > !I2C_HELPER_AUTO, which triggers a Kconfig warning about broken > dependencies when some driver selects one of the algorithms. Make only > the prompts dependent on !I2C_HELPER_AUTO, not the complete symbols. > This moves the entries out of the "I2C Algorithms" submenu, but most > users do not unset I2C_HELPER_AUTO, so they will not see it anyway. You have bad luck, I'm afraid. Sure, most users do not unset I2C_HELPER_AUTO, so they will not see the extra entries in the main I2C menu, however _I_ unset it and guess what, _I_ am the one deciding whether your patch can be applied or not ;) I don't want the algo drivers listed in the main I2C menu, sorry. It is much clearer the way it is now. > > Signed-off-by: Michal Marek <mmarek@...e.cz> > --- > > Please consider for 2.6.36, as the warning is caused by a kconfig fix > that I'm about to send to Linus for 2.6.36. And it is triggered by a > simple 'make defconfig' in i386/x84_64: You should reconsider then. Kernel 2.6.36 will be released in a few days, is your fix so important that it has to go in, even when you have evidence that it breaks other parts of the configuration, which in turn will have to be fixed? It would help if you could explain in detail what your fix is doing and why it is urgently needed (if it really is.) I borrowed the idea of I2C_HELPER_AUTO from CONFIG_VIDEO_HELPER_CHIPS_AUTO, so that part will break too, and these are a lot more configuration options and drivers. Also look at I2C_SMBUS, I presume it needs a fix too if selecting undeclared symbols is no longer allowed. > $ make defconfig > *** Default configuration is based on 'x86_64_defconfig' > warning: (IEEE1394_PCILYNX && (PCI || BROKEN) && PCI && IEEE1394 && I2C || SFC && NETDEVICES && NETDEV_10000 && PCI && INET || I2C_VIA && I2C && PCI && EXPERIMENTAL || I2C_HYDRA && I2C && PCI && PPC_CHRP && EXPERIMENTAL || I2C_GPIO && I2C && GENERIC_GPIO || I2C_IXP2000 && I2C && ARCH_IXP2000 || I2C_SIMTEC && I2C || I2C_VERSATILE && I2C && (ARCH_VERSATILE || ARCH_REALVIEW || ARCH_VEXPRESS) || I2C_PARPORT && I2C && PARPORT || I2C_PARPORT_LIGHT && I2C || I2C_ACORN && I2C && ARCH_ACORN || SCx200_I2C && I2C && SCx200_GPIO || VIDEO_BT848 && MEDIA_SUPPORT && VIDEO_CAPTURE_DRIVERS && VIDEO_DEV && PCI && I2C && VIDEO_V4L2 && INPUT && VIDEO_IR || VIDEO_CX88 && MEDIA_SUPPORT && VIDEO_CAPTURE_DRIVERS && VIDEO_V4L2 && VIDEO_DEV && PCI && I2C && INPUT && VIDEO_IR || VIDEO_CX23885 && MEDIA_SUPPORT && VIDEO_CAPTURE_DRIVERS && VIDEO_V4L2 && DVB_CORE && VIDEO_DEV && PCI && I2C && INPUT && IR_CORE || VIDEO_AU0828 && MEDIA_SUPPORT && VIDEO_CAPTURE_DRIVERS && I2C && INPUT && DVB_CORE && USB && VI > DEO_V4L2 || VIDEO_IVTV && MEDIA_SUPPORT && VIDEO_CAPTURE_DRIVERS && VIDEO_V4L2 && PCI && I2C && INPUT && VIDEO_IR || VIDEO_CX18 && MEDIA_SUPPORT && VIDEO_CAPTURE_DRIVERS && VIDEO_V4L2 && DVB_CORE && PCI && I2C && EXPERIMENTAL && INPUT && VIDEO_IR || VIDEO_SAA7164 && MEDIA_SUPPORT && VIDEO_CAPTURE_DRIVERS && VIDEO_V4L2 && DVB_CORE && PCI && I2C || DVB_PLUTO2 && MEDIA_SUPPORT && DVB_CAPTURE_DRIVERS && DVB_CORE && PCI && I2C || DRM && HAS_IOMEM && (AGP || AGP=n) && !EMULATED_CMPXCHG && MMU || FB_DDC && HAS_IOMEM && FB || FB_VIA && HAS_IOMEM && FB && PCI && X86 || VIDEO_CX25821 && STAGING && !STAGING_EXCLUDE_BUILD && DVB_CORE && VIDEO_DEV && PCI && I2C && INPUT) selects I2C_ALGOBIT which has unmet direct dependencies (I2C && !I2C_HELPER_AUTO) > [...] This message is obscure. The actual problem is that I2C_ALGOBIT is selected but doesn't exist, isn't it?. > > --- > drivers/i2c/algos/Kconfig | 13 ++++++------- > 1 files changed, 6 insertions(+), 7 deletions(-) > > diff --git a/drivers/i2c/algos/Kconfig b/drivers/i2c/algos/Kconfig > index 7b2ce4a..9ced1a0 100644 > --- a/drivers/i2c/algos/Kconfig > +++ b/drivers/i2c/algos/Kconfig > @@ -2,16 +2,15 @@ > # I2C algorithm drivers configuration > # > > -menu "I2C Algorithms" > - depends on !I2C_HELPER_AUTO > - > config I2C_ALGOBIT > - tristate "I2C bit-banging interfaces" > + tristate > + prompt "I2C bit-banging interfaces" if !I2C_HELPER_AUTO > > config I2C_ALGOPCF > - tristate "I2C PCF 8584 interfaces" > + tristate > + prompt "I2C PCF 8584 interfaces" if !I2C_HELPER_AUTO > > config I2C_ALGOPCA > - tristate "I2C PCA 9564 interfaces" > + tristate > + prompt "I2C PCA 9564 interfaces" if !I2C_HELPER_AUTO > > -endmenu I would prefer the fix below, which seems to work for me (but I am no Kconfig expert) and preserves the submenu for algo drivers. Only the indentation is missing, would be nice to solve. --- drivers/i2c/Kconfig | 3 ++- drivers/i2c/algos/Kconfig | 15 ++++++++++++--- 2 files changed, 14 insertions(+), 4 deletions(-) --- linux-2.6.36-rc6.orig/drivers/i2c/algos/Kconfig 2010-08-02 00:11:14.000000000 +0200 +++ linux-2.6.36-rc6/drivers/i2c/algos/Kconfig 2010-10-04 17:33:10.000000000 +0200 @@ -2,16 +2,25 @@ # I2C algorithm drivers configuration # +config I2C_ALGOBIT + tristate + +config I2C_ALGOPCF + tristate + +config I2C_ALGOPCA + tristate + menu "I2C Algorithms" depends on !I2C_HELPER_AUTO config I2C_ALGOBIT - tristate "I2C bit-banging interfaces" + prompt "I2C bit-banging interfaces" config I2C_ALGOPCF - tristate "I2C PCF 8584 interfaces" + prompt "I2C PCF 8584 interfaces" config I2C_ALGOPCA - tristate "I2C PCA 9564 interfaces" + prompt "I2C PCA 9564 interfaces" endmenu --- linux-2.6.36-rc6.orig/drivers/i2c/Kconfig 2010-09-21 11:07:14.000000000 +0200 +++ linux-2.6.36-rc6/drivers/i2c/Kconfig 2010-10-04 17:54:29.000000000 +0200 @@ -75,7 +75,8 @@ config I2C_HELPER_AUTO In doubt, say Y. config I2C_SMBUS - tristate "SMBus-specific protocols" if !I2C_HELPER_AUTO + tristate + prompt "SMBus-specific protocols" if !I2C_HELPER_AUTO help Say Y here if you want support for SMBus extensions to the I2C specification. At the moment, the only supported extension is -- Jean Delvare -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists