On 06/03/2013 07:15 AM, Heiko Stübner wrote: > dw_apb_timer_init used to search the devicetree for matching timer > devices, making calls to it from board files necessary. > > Change the dw_apb_timer_init to work with CLOCKSOURCE_OF_DECLARE. > With this change the function gets called once for each timer node > and tracks these number of calls to attach clockevent and clocksource > devices to the nodes. > > Also convert all previous users of dw_apb_timer_init to use > clocksource_of_init. > > Tested on the upcoming rk3066 code. > > Signed-off-by: Heiko Stuebner <heiko@xxxxxxxxx> Acked-by: Rob Herring <rob.herring@xxxxxxxxxxx> But 1 comment: > @@ -88,7 +88,7 @@ DT_MACHINE_START(PICOXCELL, "Picochip picoXcell") > .map_io = picoxcell_map_io, > .nr_irqs = NR_IRQS_LEGACY, > .init_irq = irqchip_init, > - .init_time = dw_apb_timer_init, > + .init_time = clocksource_of_init, clocksource_of_init is the default now, so you can just remove this and the one in socfpga. Same for irqchip_init BTW, but that's another patch. Rob -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html