On Mon, 17 Jan 2011, Tony Lindgren wrote: > * Paul Walmsley <[email protected]> [110115 20:31]: > > --- a/arch/arm/mach-omap1/time.c > > +++ b/arch/arm/mach-omap1/time.c > > @@ -244,6 +244,13 @@ static void __init omap_timer_init(void) > > > > omap_init_mpu_timer(rate); > > omap_init_clocksource(rate); > > + /* > > + * XXX Since this file seems to deal mostly with the MPU timer, > > + * this doesn't seem like the correct place for the sync timer > > + * clocksource init. > > + */ > > + if (!cpu_is_omap7xx() && !cpu_is_omap15xx()) > > + omap_init_clocksource_32k(); > > } > > > > struct sys_timer omap_timer = { > > To me it looks like the omap_init_clocksource_32k() call should be > in arch/arm/mach-omap1/timer32k.c instead.
Just FYI for the lists; Tony and I talked about this off-list; he'll deal with this in a subsequent patch to avoid changing the OMAP1 clocksource behavior during this patch. - Paul -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
