Hi Jean On Wed, 21 Sep 2011, jean.pihet@xxxxxxxxxxxxxx wrote: > Figures are added to the power domains structs for RET and OFF modes. > > Note: the latency figures for MPU, PER, CORE, NEON have been obtained > from actual measurements. > The latency figures for the other power domains are preliminary and > shall be added. > > Cf. http://www.omappedia.org/wiki/Power_Management_Device_Latencies_Measurement > for a detailed explanation on where are the numbers magically coming from. > > Tested on OMAP3 Beagleboard in RET/OFF using wake-up latency constraints > on MPU, CORE and PER. A few more question about these numbers. - Are these numbers the ones right off the hardware, or rounded, or worst-case plus a certain percentage? - What frequency/voltage were these measured at? - Also, you mention OMAP3 Beagleboard. Is that 34xx or 36xx Beagleboard? > Signed-off-by: Jean Pihet <j-pihet@xxxxxx> > --- > arch/arm/mach-omap2/powerdomains3xxx_data.c | 78 +++++++++++++++++++++++++++ > 1 files changed, 78 insertions(+), 0 deletions(-) > > diff --git a/arch/arm/mach-omap2/powerdomains3xxx_data.c b/arch/arm/mach-omap2/powerdomains3xxx_data.c > index 469a920..32922bb 100644 > --- a/arch/arm/mach-omap2/powerdomains3xxx_data.c > +++ b/arch/arm/mach-omap2/powerdomains3xxx_data.c > @@ -31,6 +31,14 @@ > > /* > * Powerdomains > + * > + * The wakeup_lat values are derived from measurements on > + * the actual target. > + * > + * Note: the latency figures for MPU, PER, CORE, NEON have been obtained > + * from actual measurements. > + * The latency figures for the other power domains are preliminary and > + * shall be added. > */ > > static struct powerdomain iva2_pwrdm = { > @@ -52,6 +60,13 @@ static struct powerdomain iva2_pwrdm = { > [2] = PWRSTS_OFF_ON, > [3] = PWRSTS_ON, > }, > + .wakeup_lat = { > + [PWRDM_FUNC_PWRST_OFF] = 1100, > + [PWRDM_FUNC_PWRST_OSWR] = UNSUP_STATE, > + [PWRDM_FUNC_PWRST_CSWR] = 350, > + [PWRDM_FUNC_PWRST_INACTIVE] = UNSUP_STATE, > + [PWRDM_FUNC_PWRST_ON] = 0, > + }, > }; > > static struct powerdomain mpu_3xxx_pwrdm = { > @@ -68,6 +83,13 @@ static struct powerdomain mpu_3xxx_pwrdm = { > .pwrsts_mem_on = { > [0] = PWRSTS_OFF_ON, > }, > + .wakeup_lat = { > + [PWRDM_FUNC_PWRST_OFF] = 1830, > + [PWRDM_FUNC_PWRST_OSWR] = UNSUP_STATE, > + [PWRDM_FUNC_PWRST_CSWR] = 121, > + [PWRDM_FUNC_PWRST_INACTIVE] = UNSUP_STATE, > + [PWRDM_FUNC_PWRST_ON] = 0, > + }, > }; > > /* > @@ -98,6 +120,13 @@ static struct powerdomain core_3xxx_pre_es3_1_pwrdm = { > [0] = PWRSTS_OFF_RET_ON, /* MEM1ONSTATE */ > [1] = PWRSTS_OFF_RET_ON, /* MEM2ONSTATE */ > }, > + .wakeup_lat = { > + [PWRDM_FUNC_PWRST_OFF] = 3082, > + [PWRDM_FUNC_PWRST_OSWR] = UNSUP_STATE, > + [PWRDM_FUNC_PWRST_CSWR] = 153, > + [PWRDM_FUNC_PWRST_INACTIVE] = UNSUP_STATE, > + [PWRDM_FUNC_PWRST_ON] = 0, > + }, > }; > > static struct powerdomain core_3xxx_es3_1_pwrdm = { > @@ -121,6 +150,13 @@ static struct powerdomain core_3xxx_es3_1_pwrdm = { > [0] = PWRSTS_OFF_RET_ON, /* MEM1ONSTATE */ > [1] = PWRSTS_OFF_RET_ON, /* MEM2ONSTATE */ > }, > + .wakeup_lat = { > + [PWRDM_FUNC_PWRST_OFF] = 3082, > + [PWRDM_FUNC_PWRST_OSWR] = UNSUP_STATE, > + [PWRDM_FUNC_PWRST_CSWR] = 153, > + [PWRDM_FUNC_PWRST_INACTIVE] = UNSUP_STATE, > + [PWRDM_FUNC_PWRST_ON] = 0, > + }, > }; > > static struct powerdomain dss_pwrdm = { > @@ -136,6 +172,13 @@ static struct powerdomain dss_pwrdm = { > .pwrsts_mem_on = { > [0] = PWRSTS_ON, /* MEMONSTATE */ > }, > + .wakeup_lat = { > + [PWRDM_FUNC_PWRST_OFF] = 70, > + [PWRDM_FUNC_PWRST_OSWR] = UNSUP_STATE, > + [PWRDM_FUNC_PWRST_CSWR] = 20, > + [PWRDM_FUNC_PWRST_INACTIVE] = UNSUP_STATE, > + [PWRDM_FUNC_PWRST_ON] = 0, > + }, > }; > > /* > @@ -157,6 +200,13 @@ static struct powerdomain sgx_pwrdm = { > .pwrsts_mem_on = { > [0] = PWRSTS_ON, /* MEMONSTATE */ > }, > + .wakeup_lat = { > + [PWRDM_FUNC_PWRST_OFF] = 1000, > + [PWRDM_FUNC_PWRST_OSWR] = UNSUP_STATE, > + [PWRDM_FUNC_PWRST_CSWR] = UNSUP_STATE, > + [PWRDM_FUNC_PWRST_INACTIVE] = UNSUP_STATE, > + [PWRDM_FUNC_PWRST_ON] = 0, > + }, > }; > > static struct powerdomain cam_pwrdm = { > @@ -172,6 +222,13 @@ static struct powerdomain cam_pwrdm = { > .pwrsts_mem_on = { > [0] = PWRSTS_ON, /* MEMONSTATE */ > }, > + .wakeup_lat = { > + [PWRDM_FUNC_PWRST_OFF] = 850, > + [PWRDM_FUNC_PWRST_OSWR] = UNSUP_STATE, > + [PWRDM_FUNC_PWRST_CSWR] = 35, > + [PWRDM_FUNC_PWRST_INACTIVE] = UNSUP_STATE, > + [PWRDM_FUNC_PWRST_ON] = 0, > + }, > }; > > static struct powerdomain per_pwrdm = { > @@ -187,6 +244,13 @@ static struct powerdomain per_pwrdm = { > .pwrsts_mem_on = { > [0] = PWRSTS_ON, /* MEMONSTATE */ > }, > + .wakeup_lat = { > + [PWRDM_FUNC_PWRST_OFF] = 671, > + [PWRDM_FUNC_PWRST_OSWR] = UNSUP_STATE, > + [PWRDM_FUNC_PWRST_CSWR] = 31, > + [PWRDM_FUNC_PWRST_INACTIVE] = UNSUP_STATE, > + [PWRDM_FUNC_PWRST_ON] = 0, > + }, > }; > > static struct powerdomain emu_pwrdm = { > @@ -201,6 +265,13 @@ static struct powerdomain neon_pwrdm = { > .omap_chip = OMAP_CHIP_INIT(CHIP_IS_OMAP3430), > .pwrsts = PWRSTS_OFF_RET_ON, > .pwrsts_logic_ret = PWRSTS_RET, > + .wakeup_lat = { > + [PWRDM_FUNC_PWRST_OFF] = 0, > + [PWRDM_FUNC_PWRST_OSWR] = UNSUP_STATE, > + [PWRDM_FUNC_PWRST_CSWR] = 0, > + [PWRDM_FUNC_PWRST_INACTIVE] = UNSUP_STATE, > + [PWRDM_FUNC_PWRST_ON] = 0, > + }, > }; > > static struct powerdomain usbhost_pwrdm = { > @@ -223,6 +294,13 @@ static struct powerdomain usbhost_pwrdm = { > .pwrsts_mem_on = { > [0] = PWRSTS_ON, /* MEMONSTATE */ > }, > + .wakeup_lat = { > + [PWRDM_FUNC_PWRST_OFF] = 800, > + [PWRDM_FUNC_PWRST_OSWR] = UNSUP_STATE, > + [PWRDM_FUNC_PWRST_CSWR] = 150, > + [PWRDM_FUNC_PWRST_INACTIVE] = UNSUP_STATE, > + [PWRDM_FUNC_PWRST_ON] = 0, > + }, > }; > > static struct powerdomain dpll1_pwrdm = { > -- > 1.7.4.1 > > -- > 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 > - 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