On Wed, Sep 5, 2012 at 8:41 AM, Lee Jones <lee.jones@xxxxxxxxxx> wrote: > On Tue, Sep 04, 2012 at 07:27:10PM +0200, Linus Walleij wrote: >> On Tue, Sep 4, 2012 at 4:28 PM, Arnd Bergmann <arnd@xxxxxxxx> wrote: >> >> > In this particular case, we don't have a single board file providing a >> > struct nmk_i2c_controller definition for platform data, so the best way >> > to handle this IMHO is to remove the header file with the platform >> > data definition, and just encode the defaults in the driver. >> >> Alessandro Rubini is actively working on bridging this (and >> other amba_device primecells) to PCI, that is the reason why it >> was recently converted to an amba_device. How is he then supposed to >> get the proper parameters into the driver? Note that the PCI ID >> is no help at all since the parameters depend on what is connected >> to the I2C bus, not on what it itself is connected to. Isn't platform data >> used in such cases? > > So why can't Alessandro continue to use Platform Data in the normal way? He probably can, this is not an argument about that, what I am worried about is Arnd's suggestion to delete the platform data header if there are potential users of it. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html