On Tue, 8 Feb 2011, Rajendra Nayak wrote: > > -----Original Message----- > > From: Paul Walmsley [mailto:paul@xxxxxxxxx] > > Sent: Tuesday, February 08, 2011 8:27 AM > > > > > arch/arm/mach-omap2/clock.c | 8 +++++++- > > > arch/arm/mach-omap2/clock.h | 1 + > > > arch/arm/mach-omap2/clock3xxx_data.c | 2 +- > > > arch/arm/mach-omap2/clock44xx_data.c | 2 +- > > > arch/arm/mach-omap2/pm44xx.c | 4 ++++ > > > > Looks like you add the allow_idle/deny_idle functions for the OMAP3 CORE > > DPLL, but you don't change mach-omap2/pm34xx.c. Any reason for this? > > Nothing specifically. My initial patchset was enabling these only for > OMAP4, So I thought will keep it that way. That's fine with me, but you might want to drop the OMAP3 part of the changes, or put them into a separate patch. Also, when you resend the autoidle patches, could you put them into a separate series? There's an autoidle series that I've been working on here, and I will add those into that series, and keep the other patch series separate. thanks, - Paul -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html