From: Kevin Hilman <khilman@ti.com>
To: Paul Walmsley <paul@pwsan.com>
Cc: linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH] OMAP2/3: PM: remove unnecessary wakeup/sleep dependency clear
Date: Thu, 27 Jan 2011 10:25:13 -0800 [thread overview]
Message-ID: <871v3yp5km.fsf@ti.com> (raw)
In-Reply-To: <alpine.DEB.2.00.1101270252010.16671@utopia.booyaka.com> (Paul Walmsley's message of "Thu, 27 Jan 2011 02:52:55 -0700 (MST)")
Paul Walmsley <paul@pwsan.com> writes:
> The OMAP2 and OMAP3 PM code clears clockdomain wakeup and sleep
> dependencies. This is unnecessary after commit
> 6f7f63cc9adf3192e6fcac4e8bed5cc10fd924aa ("OMAP clockdomain:
> initialize clockdomain registers when the clockdomain layer starts")
> which clears these dependencies during clockdomain init.
>
> Signed-off-by: Paul Walmsley <paul@pwsan.com>
> Cc: Kevin Hilman <khilman@deeprootsystems.com>
> ---
> Full-chip retention idle entry tested on OMAP35xx Beagleboard.
Also tested on 3630/Zoom3.
Queueing for 2.6.39. Thanks,
Kevin
> arch/arm/mach-omap2/pm24xx.c | 8 +-------
> arch/arm/mach-omap2/pm34xx.c | 15 ---------------
> 2 files changed, 1 insertions(+), 22 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/pm24xx.c b/arch/arm/mach-omap2/pm24xx.c
> index 97feb3a..10f8747 100644
> --- a/arch/arm/mach-omap2/pm24xx.c
> +++ b/arch/arm/mach-omap2/pm24xx.c
> @@ -363,9 +363,6 @@ static const struct platform_suspend_ops __initdata omap_pm_ops;
> /* XXX This function should be shareable between OMAP2xxx and OMAP3 */
> static int __init clkdms_setup(struct clockdomain *clkdm, void *unused)
> {
> - clkdm_clear_all_wkdeps(clkdm);
> - clkdm_clear_all_sleepdeps(clkdm);
> -
> if (clkdm->flags & CLKDM_CAN_ENABLE_AUTO)
> omap2_clkdm_allow_idle(clkdm);
> else if (clkdm->flags & CLKDM_CAN_FORCE_SLEEP &&
> @@ -411,10 +408,7 @@ static void __init prcm_setup_regs(void)
> pwrdm_set_next_pwrst(pwrdm, PWRDM_POWER_OFF);
> omap2_clkdm_sleep(gfx_clkdm);
>
> - /*
> - * Clear clockdomain wakeup dependencies and enable
> - * hardware-supervised idle for all clkdms
> - */
> + /* Enable hardware-supervised idle for all clkdms */
> clkdm_for_each(clkdms_setup, NULL);
> clkdm_add_wkdep(mpu_clkdm, wkup_clkdm);
>
> diff --git a/arch/arm/mach-omap2/pm34xx.c b/arch/arm/mach-omap2/pm34xx.c
> index a4aa192..0eb9738 100644
> --- a/arch/arm/mach-omap2/pm34xx.c
> +++ b/arch/arm/mach-omap2/pm34xx.c
> @@ -694,21 +694,6 @@ static void __init prcm_setup_regs(void)
> u32 omap3630_grpsel_uart4_mask = cpu_is_omap3630() ?
> OMAP3630_GRPSEL_UART4_MASK : 0;
>
> -
> - /* XXX Reset all wkdeps. This should be done when initializing
> - * powerdomains */
> - omap2_prm_write_mod_reg(0, OMAP3430_IVA2_MOD, PM_WKDEP);
> - omap2_prm_write_mod_reg(0, MPU_MOD, PM_WKDEP);
> - omap2_prm_write_mod_reg(0, OMAP3430_DSS_MOD, PM_WKDEP);
> - omap2_prm_write_mod_reg(0, OMAP3430_NEON_MOD, PM_WKDEP);
> - omap2_prm_write_mod_reg(0, OMAP3430_CAM_MOD, PM_WKDEP);
> - omap2_prm_write_mod_reg(0, OMAP3430_PER_MOD, PM_WKDEP);
> - if (omap_rev() > OMAP3430_REV_ES1_0) {
> - omap2_prm_write_mod_reg(0, OMAP3430ES2_SGX_MOD, PM_WKDEP);
> - omap2_prm_write_mod_reg(0, OMAP3430ES2_USBHOST_MOD, PM_WKDEP);
> - } else
> - omap2_prm_write_mod_reg(0, GFX_MOD, PM_WKDEP);
> -
> /*
> * Enable interface clock autoidle for all modules.
> * Note that in the long run this should be done by clockfw
next prev parent reply other threads:[~2011-01-27 18:25 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-01-27 9:52 [PATCH] OMAP2/3: PM: remove unnecessary wakeup/sleep dependency clear Paul Walmsley
2011-01-27 18:25 ` Kevin Hilman [this message]
2011-02-10 23:11 ` Kevin Hilman
2011-02-10 23:17 ` Paul Walmsley
2011-02-11 1:39 ` Paul Walmsley
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=871v3yp5km.fsf@ti.com \
--to=khilman@ti.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-omap@vger.kernel.org \
--cc=paul@pwsan.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox