> -----Original Message----- > From: Russell King - ARM Linux [mailto:li...@arm.linux.org.uk] > Sent: Tuesday, January 25, 2011 9:43 PM > To: Santosh Shilimkar > Cc: Thomas Gleixner; Colin Cross; 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 > > 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?
Sure. Tested-by: Santosh Shilimkar <santosh.shilim...@ti.com> -- 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