> -----Original Message----- > From: Nishanth Menon [mailto:menon.nishanth@xxxxxxxxx] > Sent: Tuesday, February 17, 2009 9:33 AM > To: Curran, Dominic > Cc: linux-omap; Aguirre Rodriguez, Sergio Alberto > Subject: Re: [OMAPZOOM][PATCH] Testing for TWL4030 in board file change. > > Curran, Dominic said the following on 02/17/2009 04:27 PM: > >> -----Original Message----- > >> From: Nishanth Menon [mailto:menon.nishanth@xxxxxxxxx] > >> Sent: Tuesday, February 17, 2009 2:12 AM > >> To: Curran, Dominic > >> Cc: linux-omap; Aguirre Rodriguez, Sergio Alberto > >> Subject: Re: [OMAPZOOM][PATCH] Testing for TWL4030 in board file change. > >> > >> Would a solution as follows be appropriate? > >> in arch/arm/mach-omap2/Kconfig: > >> config MACH_OMAP_ZOOM2 > >> bool "OMAP3 ZOOM2 board" > >> depends on ARCH_OMAP3 && ARCH_OMAP34XX > >> + select TWL4030_CORE if VIDEO_OMAP3 > >> > >> A similar strategy has been implemented for N800, albeit for other > >> peripherals. > >> > > > > If 'reverse dependencies' set the symbol and its dependencies I would > say this would be a good solution, but unfortunately it does not. > > The documentation for reverse dependencies warns: > > > > 107 select should be used with care. select will force > > 108 a symbol to a value without visiting the dependencies. > > 109 By abusing select you are able to select a symbol FOO even > > 110 if FOO depends on BAR that is not set. > > 111 In general use select only for non-visible symbols > > > > Since TWL4030 has dependencies then I don't think this is appropriate > Mach-omap && video_omap3 depends on twl4030 and twl4030 is not reverse > dependent on mach-omap. twl4030 is dependent on i2c - I believe based on > drivers/mfd/Kconfig how do we handle this? Maybe a select of i2c is > appropriate there? or how about: > > + select TWL4030_CORE if VIDEO_OMAP3 > + select I2C if TWL4030_CORE > > > You may want to see arch/arm/mach-omap2/Kconfig -> MACH_NOKIA_N800. I > think we have a similar condition there with CBUS and VIDEO_TCM825X I > think.. Honestly, I'm a bit lost about the goal of this. I don't see how this is solving our problem. - 3430SDP and Zoom1/2 uses TWL4030 for power handling the camera sensors - Current Omap3 camera driver can be used in other platforms than the above that uses OMAP3x30 chip, so we cannot add dependencies on camera driver to platform specific powering methods. - On above platforms, there exists the need for compiling without T2 support sometimes and you don't want a broken build because of this How is above helping for solving this out? > > my 2cents: i like select better than #ifdef ;).. but then, that is just > me :).. > > Regards, > Nishanth Menon -- 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