Re: [PATCH 1/5] ARM: OMAP2/3: Remove OMAP2_32KSYNCT_BASE

[Tony Lindgren <tony@atomide.com>]

* Russell King - ARM Linux <linux@arm.linux.org.uk> [090528 08:25]:
> On Thu, May 14, 2009 at 04:23:06PM -0700, Tony Lindgren wrote:
> > Use processor specific defines instead.
> > 
> > As an extra bonus, this patch fixes the problem of CONFIG_DEBUG_SPINLOCK
> > calling sched_clock before we have things initialized:
> 
> Hmm.  I don't think sched_clock peculiarities should be handed by the
> clock source implementation.
> 
> This is really a generic kernel bug: lots of ARM architecture sched_clock
> implementations can not be called before things like IO memory has been
> setup - and that's something that mainline people have been made aware
> of repeatedly, and they've fixed the problems when they've come up.

Yeah agreed.
 
> Moreover, the issue of using clocksources for sched_clock is being
> discussed on the main ARM lists, with the possibility of the generic
> kernel-wide sched_clock implementation being based upon clock sources.
> 
> Whether that changes anything with this patch, I'm not entirely sure.

The clocksource and sched_clock thing is a separate issue, this patch
is still needed to remove the conflicting defines.

Tony