Hi Jean

On Wed, 21 Sep 2011, jean.pi...@newoldbits.com 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-pi...@ti.com>
> ---
>  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 majord...@vger.kernel.org
> 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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to