Hi Tony, On Fri, Jun 15, 2012 at 18:15:20, Tony Lindgren wrote: > * Mohammed, Afzal <afzal@xxxxxx> [120615 03:26]: > > Here clock is required even before driver is probed, i.e for platform to > > calculate timings, that has to be passed through platform data. > > Eventually we should be able to move the gpmc registration to the driver > probe, especially with device tree. There's no need to set up gpmc > before the driver probe runs for the device using gpmc. Just how the > gpmc init gets called from the driver probe is still a bit open though.. Sorry, I did not get you, can you please clarify By gpmc registration, if you meant registering platform device for gpmc peripherals, for a board that uses the new gpmc driver interface*, it will be done in probe only. All the responsibilities of old gpmc init is now taken care by probe; probe in addition does setting up gpmc for each peripheral, registering platform device (if the board makes use of new driver interface). If a board uses new gpmc driver interface, gpmc for that device is not setup before gpmc probe. > It may require some bus level hooks, or wrapper drivers for the generic > device drivers like smsc911x. This too, not sure whether I follow you > > > I understand the necessity for clk rate information in driver, but seems > > unless we have a generic way to scale timings for all the kinds of > > peripheral, having it may not be of much help. > > We should not need to pass clock handles around. It's better to > export some helper functions in the gpmc code for the calculation. Currently we have helper function in gpmc.c for the same, were you referring those ? Regards Afzal *: [1] converts omap3evm & beagle board to use new interface, in addition to [1], similar to it, one more series would be posted to convert remaining boards also to use the new gpmc driver interface. As these cannot be tested by me and as it depends on final shape of this basic driver conversion series (or the present series), they have not yet been converted [1] http://www.mail-archive.com/linux-omap@xxxxxxxxxxxxxxx/msg69917.html -- 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
