Hi Luciano, On Mon, 29 Aug 2011 21:41:47 +0300, Luciano Coelho wrote: > From: Randy Dunlap <rdunlap@xxxxxxxxxxxx> > > Combine MFD_SUPPORT (which only enabled the remainder of the MFD > menu) and MFD_CORE. This allows other drivers to select MFD_CORE > without needing to also select MFD_SUPPORT, which fixes some > kconfig unmet dependency warnings. Modeled after I2C kconfig. > > [Forward-ported to 3.1-rc4. This fixes a warning when some drivers, > such as RADIO_WL1273, are selected, but MFD_SUPPORT is not. -- Luca] I like the idea in general, this makes things much simpler. There is at least one issue with your current implementation though. "make oldconfig" has this to complain about: drivers/mfd/Kconfig:5:error: recursive dependency detected! drivers/mfd/Kconfig:5: symbol MFD_CORE is selected by OLPC_XO1_PM arch/x86/Kconfig:2028: symbol OLPC_XO1_PM depends on MFD_CS5535 drivers/mfd/Kconfig:613: symbol MFD_CS5535 depends on MFD_CORE Not sure if it is really caused by your patch or only revealed by it, but it should be fixed anyway. The following should fix it, please consider folding in your patch: --- linux-3.1-rc4.orig/arch/x86/Kconfig 2011-08-16 11:49:42.000000000 +0200 +++ linux-3.1-rc4/arch/x86/Kconfig 2011-08-31 16:54:09.000000000 +0200 @@ -2028,7 +2028,6 @@ config OLPC config OLPC_XO1_PM bool "OLPC XO-1 Power Management" depends on OLPC && MFD_CS5535 && PM_SLEEP - select MFD_CORE ---help--- Add support for poweroff and suspend of the OLPC XO-1 laptop. @@ -2044,7 +2043,6 @@ config OLPC_XO1_SCI depends on OLPC && OLPC_XO1_PM select POWER_SUPPLY select GPIO_CS5535 - select MFD_CORE ---help--- Add support for SCI-based features of the OLPC XO-1 laptop: - EC-driven system wakeups > > Signed-off-by: Randy Dunlap <rdunlap@xxxxxxxxxxxx> > Reported-by: Johannes Berg <johannes@xxxxxxxxxxxxxxxx> > Cc: Jean Delvare <khali@xxxxxxxxxxxx> > Cc: Tony Lindgren <tony@xxxxxxxxxxx> > Cc: Grant Likely <grant.likely@xxxxxxxxxxxx> > Signed-off-by: Luciano Coelho <coelho@xxxxxx> > --- > > I guess this should fix the problem. I've simple forward-ported > Randy's patch to the latest mainline kernel. I don't know via which > tree this should go in, though. Samuel Ortiz is the maintainer of the mfd subsystem, so his tree would be an obvious choice. > NOTE: I have *not* tested this very thoroughly. But at least > omap2plus stuff seems to work okay with this change. MFD_SUPPORT is > also selected by a couple of "tile" platforms defconfigs, but I guess > the Kconfig system should take care of it. I can't test it either, but it looks sane to me. If you merge the proposed changes above, you can add: Acked-by: Jean Delvare <khali@xxxxxxxxxxxx> -- Jean Delvare -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
