> -----Original Message-----
> From: Gulati, Shweta
> Sent: Wednesday, March 24, 2010 7:17 PM
> To: linux-omap@vger.kernel.org
> Cc: Gulati, Shweta; Sripathy, Vishwanath
> Subject: [PATCH L23.I3 PM 1/2 ]: To put all the powerdomains in OFF mode 
> instead
> of Ret
Subject should not have TI internal release names. 
Also Cover letter for the patch series is missing
> 
> From: Shweta Gulati <shweta.gul...@ti.com>
> 
> According to Defect OMAPS00210499 CAM, DSS and USBHOST should stay
> in OFF state instead of Retention in OS Idle Path.
> This Patch puts all the powerdomain in OFF state by default
> in pm init code and ensures that Mpu next state is OFF only
> before it goes to Suspend/Idle.
> 
> Signed-off-by: Vishwanath BS <vishwanath...@ti.com>
> Signed-off-by: Shweta Gulati <shweta.gul...@ti.com>
> ---
> 
> Index: kernel-omap3-dev/arch/arm/mach-omap2/cpuidle34xx.c
> =====================================================
> ==============
> --- kernel-omap3-dev.orig/arch/arm/mach-omap2/cpuidle34xx.c
> +++ kernel-omap3-dev/arch/arm/mach-omap2/cpuidle34xx.c
> @@ -144,11 +144,14 @@ static int omap3_enter_idle(struct cpuid
>       local_irq_disable();
>       local_fiq_disable();
> 
> -     if (!enable_off_mode) {
> +     if (disable_off_mode) {
>               if (mpu_state < PWRDM_POWER_RET)
>                       mpu_state = PWRDM_POWER_RET;
>               if (core_state < PWRDM_POWER_RET)
>                       core_state = PWRDM_POWER_RET;
> +     } else {
> +             mpu_state = PWRDM_POWER_OFF;
> +             core_state = PWRDM_POWER_OFF;
>       }
> 
>       /* For any state above inactive set the logic and memory retention
> Index: kernel-omap3-dev/arch/arm/mach-omap2/pm-debug.c
> =====================================================
> ==============
> --- kernel-omap3-dev.orig/arch/arm/mach-omap2/pm-debug.c
> +++ kernel-omap3-dev/arch/arm/mach-omap2/pm-debug.c
> @@ -548,7 +548,7 @@ static int option_set(void *data, u64 va
> 
>       *option = val;
> 
> -     if (option == &enable_off_mode)
> +     if (option == &disable_off_mode)
>               omap3_pm_off_mode_enable(val);
>       return 0;
>  }
> @@ -597,8 +597,8 @@ static int __init pm_dbg_init(void)
> 
>               }
> 
> -     (void) debugfs_create_file("enable_off_mode", S_IRUGO | S_IWUGO, d,
> -                                &enable_off_mode, &pm_dbg_option_fops);
> +     (void) debugfs_create_file("disable_off_mode", S_IRUGO | S_IWUGO, d,
> +                                &disable_off_mode, &pm_dbg_option_fops);
>       (void) debugfs_create_file("sleep_while_idle", S_IRUGO | S_IWUGO, d,
>                                  &sleep_while_idle, &pm_dbg_option_fops);
>       (void) debugfs_create_file("enable_oswr", S_IRUGO | S_IWUGO, d,
> Index: kernel-omap3-dev/arch/arm/mach-omap2/pm.h
> =====================================================
> ==============
> --- kernel-omap3-dev.orig/arch/arm/mach-omap2/pm.h
> +++ kernel-omap3-dev/arch/arm/mach-omap2/pm.h
> @@ -13,7 +13,7 @@
> 
>  #include <plat/powerdomain.h>
> 
> -extern u32 enable_off_mode;
> +extern u32 disable_off_mode;
>  extern u32 sleep_while_idle;
>  extern u32 enable_oswr;
>  extern u32 voltage_off_while_idle;
> Index: kernel-omap3-dev/arch/arm/mach-omap2/pm34xx.c
> =====================================================
> ==============
> --- kernel-omap3-dev.orig/arch/arm/mach-omap2/pm34xx.c
> +++ kernel-omap3-dev/arch/arm/mach-omap2/pm34xx.c
> @@ -75,7 +75,7 @@ static inline bool is_suspending(void)
>  #define OMAP3630_EFUSE_CNTRL         0x48002A8C
>  #define ABO_LDO_TRANXDONE_TIMEOUT    100
> 
> -u32 enable_off_mode;
> +u32 disable_off_mode;
>  u32 sleep_while_idle;
>  u32 enable_oswr;
>  u32 wakeup_timer_seconds;
> @@ -1197,15 +1197,15 @@ static void __init prcm_setup_regs(void)
>       omap3_d2d_idle();
>  }
> 
> -void omap3_pm_off_mode_enable(int enable)
> +void omap3_pm_off_mode_enable(int disable)
>  {
>       struct power_state *pwrst;
>       u32 state;
> 
> -     if (enable)
> -             state = PWRDM_POWER_OFF;
> -     else
> +     if (disable)
>               state = PWRDM_POWER_RET;
> +     else
> +             state = PWRDM_POWER_OFF;
> 
>  #ifdef CONFIG_OMAP_PM_SRF
>       resource_lock_opp(VDD1_OPP);
> @@ -1303,7 +1303,8 @@ static int __init pwrdms_setup(struct po
>       if (!pwrst)
>               return -ENOMEM;
>       pwrst->pwrdm = pwrdm;
> -     if (strcmp("iva2_pwrdm", pwrdm->name))
> +     if (!strcmp("mpu_pwrdm", pwrdm->name) ||
> +                      !strcmp("core_pwrdm", pwrdm->name))
>               pwrst->next_state = PWRDM_POWER_RET;
>       else
>                pwrst->next_state = PWRDM_POWER_OFF;
> Index: kernel-omap3-dev/arch/arm/mach-omap2/resource34xx.c
> =====================================================
> ==============
> --- kernel-omap3-dev.orig/arch/arm/mach-omap2/resource34xx.c
> +++ kernel-omap3-dev/arch/arm/mach-omap2/resource34xx.c
> @@ -106,7 +106,7 @@ void init_pd_latency(struct shared_resou
>       struct pd_latency_db *pd_lat_db;
> 
>       resp->no_of_users = 0;
> -     if (enable_off_mode)
> +     if (!disable_off_mode)
>               resp->curr_level = PD_LATENCY_OFF;
>       else
>               resp->curr_level = PD_LATENCY_RET;
> @@ -144,7 +144,7 @@ int set_pd_latency(struct shared_resourc
>               }
>       }
> 
> -     if (!enable_off_mode && pd_lat_level == PD_LATENCY_OFF)
> +     if (disable_off_mode && pd_lat_level == PD_LATENCY_OFF)
>               pd_lat_level = PD_LATENCY_RET;
> 
>       resp->curr_level = pd_lat_level;
--
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