On 11/21/2013 09:51 PM, Felipe Balbi wrote: > Hi, > > On Thu, Nov 21, 2013 at 07:56:51PM -0600, Joel Fernandes wrote: > > [...] > >> New functions for OF-only boot are introduced, and we can soon delete the old >> versions once we migrate all platforms. Currently only AM335x is migrated and > > actually, you don't need to initialize .init_timer at all in DT boot. Actually we still do, because the plan is to keep the hwmod stuff that's required in timer.c in a custom .init_time, and then of_clocksource_init maybe called to do what you're suggesting but (not yet) more on that below. > Just use CLKSOURCE_OF_DECLARE() and pass your omap_generic_timer_init() > as argument (although, I'd call it omap_of_timer_init()). > That will put of_device_id structure on a special section > (__clksource_of_table) and pass your function as a data argument. That > function will be called automatically during init. > I thought of doing that, but currently the timer selection for clocksource is not a simple matching of compatible string, rather it is selecting the timer based on properties such as ti,timer-alwon and such. In omap3 for example, there are needs for specific timers and such have been provided with the macros passing in timer id etc. Right now, this can be forced through DT with the "ti,timer-clocksource" property I introduced. All this selection algorithm is too complex to be handle directly by the CLOCKSOURCE_OF_DECLARE / clocksource_of_init matching mechanism. thanks, -Joel -- 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
