On Mon, 17 Jan 2011, Tony Lindgren wrote:
* Paul Walmsley [off-list ref] [110115 20:31]:
quoted
--- 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