On Mon, Jul 11, 2011 at 09:41:24AM -0700, Dmitry Torokhov wrote: > On Mon, Jul 11, 2011 at 01:20:04PM +0100, Mark Brown wrote: > > This isn't a good fix, the driver should depend on the MFD core not > > select it. > The problem with such dependencies is that user has to first enable MFD > core and then hunt through all other menus in search of the function > drivers for that chip... > Depending on what other things TPS6507X depends on it might be good idea > to go ahead and select it. If this is an issue it's going to be much more useful to have the boards select their MFD cores than have random function drivers do that, in most situations it's not actually going to be useful to select the core driver as the core driver itself needs something to select either the I2C or SPI bus interface for the device in order to actually communicate with it. Remember that this is for embedded systems where the user will normally have a pretty good idea what's on their system, someone will have had to explicitly plumb the device into the board anyway. -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html