[PATCH 09/12] ARM: OMAP2+: powerdomain: skip register reads for powerdomains known to be on

Bedia, Vaibhav vaibhav.bedia at ti.com
Wed Dec 26 01:31:14 EST 2012


On Wed, Dec 26, 2012 at 11:51:46, Bedia, Vaibhav wrote:
> Hi Paul,
> 
> On Mon, Dec 10, 2012 at 01:33:28, Paul Walmsley wrote:
> > There's no need to determine the current power state for powerdomains
> > that must be on while the kernel is running.  We mark these
> > powerdomains with a new flag, PWRDM_ACTIVE_WITH_KERNEL.  Any
> > powerdomain marked with that flag is reported as being in the ON power
> > state while the kernel is running.
> [...]
> > diff --git a/arch/arm/mach-omap2/powerdomains33xx_data.c b/arch/arm/mach-omap2/powerdomains33xx_data.c
> > index 869adb8..acb148a 100644
> > --- a/arch/arm/mach-omap2/powerdomains33xx_data.c
> > +++ b/arch/arm/mach-omap2/powerdomains33xx_data.c
> > @@ -123,7 +123,8 @@ static struct powerdomain mpu_33xx_pwrdm = {
> >  	.pwrstst_offs		= AM33XX_PM_MPU_PWRSTST_OFFSET,
> >  	.pwrsts			= PWRSTS_OFF_RET_ON,
> >  	.pwrsts_logic_ret	= PWRSTS_OFF_RET,
> > -	.flags			= PWRDM_HAS_LOWPOWERSTATECHANGE,
> > +	.flags			= (PWRDM_HAS_LOWPOWERSTATECHANGE |
> > +				   PWRDM_ACTIVE_WITH_KERNEL),
> 
> In case of AM33xx, this flag should be set for PER and WKUP pwrdms also.
> EMIF, L3, L4 etc come under PER so this domain can't transition on an active
> system. PRCM and Control module come under WKUP, so

... the kernel should not attempt to change the WKUP power domain state.

Regards,
Vaibhav 




More information about the linux-arm-kernel mailing list