[PATCH v2 6/9] omap: dpll: Add allow_idle/deny_idle support for all DPLL's

[Paul Walmsley]

On Tue, 8 Feb 2011, Rajendra Nayak wrote:

> > -----Original Message-----
> > From: Paul Walmsley [mailto:paul at pwsan.com]
> > 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