On Tue, Jan 25, 2011 at 09:34:15PM +0530, Santosh Shilimkar wrote: > Russell, Thomas > > -----Original Message----- > > From: Russell King - ARM Linux [mailto:li...@arm.linux.org.uk] > > Sent: Tuesday, January 25, 2011 7:08 PM > > To: Thomas Gleixner > > Cc: Colin Cross; Santosh Shilimkar; catalin.mari...@arm.com; > > linus.ml.wall...@gmail.com; linux-omap@vger.kernel.org; linux-arm- > > ker...@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? -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html