Hi Paul, On Monday 30 September 2013 03:57 PM, Paul Walmsley wrote: > On Thu, 26 Sep 2013, Afzal Mohammed wrote: >> From: Ambresh K <ambresh@xxxxxx> >> >> Add the data file to describe all power domains in AM43x SoC. >> OMAP4 powerdomain operations is being reused here. >> >> Signed-off-by: Ambresh K <ambresh@xxxxxx> >> Signed-off-by: Afzal Mohammed <afzal@xxxxxx> > > ... > >> --- /dev/null >> +++ b/arch/arm/mach-omap2/powerdomains43xx_data.c >> @@ -0,0 +1,142 @@ > > ... > >> +static struct powerdomain per_43xx_pwrdm = { >> + .name = "per_pwrdm", >> + .voltdm = { .name = "core" }, >> + .prcm_offs = AM43XX_PRM_PER_INST, >> + .prcm_partition = AM43XX_PRM_PARTITION, >> + .pwrsts = PWRSTS_OFF_RET_ON, >> + .pwrsts_logic_ret = PWRSTS_OFF_RET, >> + .banks = 4, >> + .pwrsts_mem_ret = { >> + [0] = PWRSTS_OFF_RET, /* icss_mem */ >> + [1] = PWRSTS_OFF_RET, /* per_mem */ >> + [2] = PWRSTS_OFF_RET, /* ram1_mem */ >> + [3] = PWRSTS_OFF_RET, /* ram2_mem */ >> + }, >> + .pwrsts_mem_on = { >> + [0] = PWRSTS_OFF_RET, /* icss_mem */ >> + [1] = PWRSTS_ON, /* per_mem */ >> + [2] = PWRSTS_OFF_RET, /* ram1_mem */ >> + [3] = PWRSTS_OFF_RET, /* ram2_mem */ >> + }, >> + .flags = PWRDM_HAS_LOWPOWERSTATECHANGE, >> +}; > > The pwrsts_mem_on flags don't make any sense here for banks 0, 2, and 3. > They claim that those banks can never be turned on. Could you please > doublecheck these? Checked it, they should be PWRSTS_ON, I will fix it. Regards Afzal -- 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