Re: State of LDP3430 platform

Paul Walmsley
2011-01-18T19:24:09Z

On Mon, 17 Jan 2011, Tony Lindgren wrote:

> * Paul Walmsley <paul@pwsan.com> [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