[PATCH 3/5] ARM: twd: Add context save restore support
Russell King - ARM Linux
linux at arm.linux.org.uk
Tue Jan 25 11:13:16 EST 2011
On Tue, Jan 25, 2011 at 09:34:15PM +0530, Santosh Shilimkar wrote:
> Russell, Thomas
> > -----Original Message-----
> > From: Russell King - ARM Linux [mailto:linux at arm.linux.org.uk]
> > Sent: Tuesday, January 25, 2011 7:08 PM
> > To: Thomas Gleixner
> > Cc: Colin Cross; Santosh Shilimkar; catalin.marinas at arm.com;
> > linus.ml.walleij at gmail.com; linux-omap at vger.kernel.org; linux-arm-
> > kernel at lists.infradead.org
> > Subject: Re: [PATCH 3/5] ARM: twd: Add context save restore support
> > I can't say because these patches only add the hooks, there's no
> > implementation yet which uses the hooks.
> > Given the description about _why_ those hooks are necessary, it
> > seems
> > that something is required. Either we start adding custom hacks to
> > each clockevent driver as is done with this patch, or we get some
> > generic help in place.
> > I'm not thrilled by the custom hack approach - and I thought the
> > clockevent stuff was created to stop this kind of thing happening.
> > I suggest we defer this until there's a visible use case available.
> Got some time to debug this issue. OMAP idle code already has the
> broad-cast notifiers working for sometime. So I removed the existing
> save restore twd code and just used the below patch which Russell
> posted on this thread.
> And things just work and I guess we are done with this issue !!
> Timer framework is doing all right things with notifiers.
> Am going to use this patch for my further work and drop the
> save/restore patch. Will update you if there is any other
> issue on this. I should have validated this patch on my own
> earlier :(
> Thanks for the discussion.
Thanks - I guess that's a tested-by then?
More information about the linux-arm-kernel