On Mon, 17 Jan 2011, Tony Lindgren wrote:
> * Paul Walmsley <paul@xxxxxxxxx> [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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
