* Kevin Hilman <khil...@deeprootsystems.com> [091014 15:15]:
> From: Rajendra Nayak <rna...@ti.com>

Might be worth checking the descriptions in the whole series :)
 
> Signed-off-by: Rajendra Nayak <rna...@ti.com>
> Signed-off-by: Kevin Hilman <khil...@deeprootsystems.com>
> ---
>  arch/arm/mach-omap2/pm34xx.c |   64 +++++++++++++++++++++++++++++++++++------
>  1 files changed, 54 insertions(+), 10 deletions(-)
> 
> diff --git a/arch/arm/mach-omap2/pm34xx.c b/arch/arm/mach-omap2/pm34xx.c
> index 9dbb5a0..7eab539 100644
> --- a/arch/arm/mach-omap2/pm34xx.c
> +++ b/arch/arm/mach-omap2/pm34xx.c
> @@ -49,7 +49,10 @@ static LIST_HEAD(pwrst_list);
>  
>  static void (*_omap_sram_idle)(u32 *addr, int save_state);
>  
> -static struct powerdomain *mpu_pwrdm;
> +static struct powerdomain *mpu_pwrdm, *neon_pwrdm;
> +static struct powerdomain *core_pwrdm, *per_pwrdm;
> +
> +static int set_pwrdm_state(struct powerdomain *pwrdm, u32 state);
>  
>  /*
>   * PRCM Interrupt Handler Helper Function
> @@ -169,13 +172,22 @@ static void omap_sram_idle(void)
>       /* save_state = 1 => Only L1 and logic lost */
>       /* save_state = 2 => Only L2 lost */
>       /* save_state = 3 => L1, L2 and logic lost */
> -     int save_state = 0, mpu_next_state;
> +     int save_state = 0;
> +     int mpu_next_state = PWRDM_POWER_ON;
> +     int per_next_state = PWRDM_POWER_ON;
> +     int core_next_state = PWRDM_POWER_ON;
>  
>       if (!_omap_sram_idle)
>               return;
>  
> +     pwrdm_clear_all_prev_pwrst(mpu_pwrdm);
> +     pwrdm_clear_all_prev_pwrst(neon_pwrdm);
> +     pwrdm_clear_all_prev_pwrst(core_pwrdm);
> +     pwrdm_clear_all_prev_pwrst(per_pwrdm);
> +
>       mpu_next_state = pwrdm_read_next_pwrst(mpu_pwrdm);
>       switch (mpu_next_state) {
> +     case PWRDM_POWER_ON:
>       case PWRDM_POWER_RET:
>               /* No need to save context */
>               save_state = 0;
> @@ -187,18 +199,37 @@ static void omap_sram_idle(void)
>       }
>       pwrdm_pre_transition();
>  
> -     omap2_gpio_prepare_for_retention();
> -     omap_uart_prepare_idle(0);
> -     omap_uart_prepare_idle(1);
> -     omap_uart_prepare_idle(2);
> +     /* NEON control */
> +     if (pwrdm_read_pwrst(neon_pwrdm) == PWRDM_POWER_ON)
> +             set_pwrdm_state(neon_pwrdm, mpu_next_state);
> +
> +     /* CORE & PER */
> +     core_next_state = pwrdm_read_next_pwrst(core_pwrdm);
> +     if (core_next_state < PWRDM_POWER_ON) {
> +             omap2_gpio_prepare_for_retention();
> +             omap_uart_prepare_idle(0);
> +             omap_uart_prepare_idle(1);
> +             /* PER changes only with core */
> +             per_next_state = pwrdm_read_next_pwrst(per_pwrdm);
> +             if (per_next_state < PWRDM_POWER_ON)
> +                     omap_uart_prepare_idle(2);
> +             /* Enable IO-PAD wakeup */
> +             prm_set_mod_reg_bits(OMAP3430_EN_IO, WKUP_MOD, PM_WKEN);
> +     }
>  
>       _omap_sram_idle(NULL, save_state);
>       cpu_init();
>  
> -     omap_uart_resume_idle(2);
> -     omap_uart_resume_idle(1);
> -     omap_uart_resume_idle(0);
> -     omap2_gpio_resume_after_retention();
> +     if (core_next_state < PWRDM_POWER_ON) {
> +             if (per_next_state < PWRDM_POWER_ON)
> +                     omap_uart_resume_idle(2);
> +             omap_uart_resume_idle(1);
> +             omap_uart_resume_idle(0);
> +
> +             /* Disable IO-PAD wakeup */
> +             prm_clear_mod_reg_bits(OMAP3430_EN_IO, WKUP_MOD, PM_WKEN);
> +             omap2_gpio_resume_after_retention();
> +     }
>  
>       pwrdm_post_transition();
>  
> @@ -791,6 +822,10 @@ static int __init omap3_pm_init(void)
>               goto err2;
>       }
>  
> +     neon_pwrdm = pwrdm_lookup("neon_pwrdm");
> +     per_pwrdm = pwrdm_lookup("per_pwrdm");
> +     core_pwrdm = pwrdm_lookup("core_pwrdm");
> +
>       omap_push_sram_idle();
>  #ifdef CONFIG_SUSPEND
>       suspend_set_ops(&omap_pm_ops);
> @@ -798,6 +833,15 @@ static int __init omap3_pm_init(void)
>  
>       pm_idle = omap3_pm_idle;
>  
> +     pwrdm_add_wkdep(neon_pwrdm, mpu_pwrdm);
> +     /*
> +      * REVISIT: This wkdep is only necessary when GPIO2-6 are enabled for
> +      * IO-pad wakeup.  Otherwise it will unnecessarily waste power
> +      * waking up PER with every CORE wakeup - see
> +      * http://marc.info/?l=linux-omap&m=121852150710062&w=2
> +     */
> +     pwrdm_add_wkdep(per_pwrdm, core_pwrdm);
> +
>  err1:
>       return ret;
>  err2:
> -- 
> 1.6.4.3
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-ker...@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
--
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