From: khilman@ti•com (Kevin Hilman)
To: linux-arm-kernel@lists•infradead.org
Subject: ARM: OMAP2xxx: PM: remove obsolete timer disable code in the suspend path
Date: Fri, 10 Feb 2012 11:26:53 -0800 [thread overview]
Message-ID: <87y5saxzeq.fsf@ti.com> (raw)
In-Reply-To: <alpine.DEB.2.00.1202100140430.2553@utopia.booyaka.com> (Paul Walmsley's message of "Fri, 10 Feb 2012 01:42:09 -0700 (MST)")
Paul Walmsley <paul@pwsan•com> writes:
> On Thu, 9 Feb 2012, Paul Walmsley wrote:
>
>>
>> Remove omap_{read,write}l() from the 24xx PM code. The clocksource
>> code should now handle what this was supposed to do.
>>
>> Tested on N800 -- but it's hard to say whether this fixes anything.
>> OMAP24xx static suspend path is currently broken, and this patch
>> doesn't change that.
>>
>> Signed-off-by: Paul Walmsley <paul@pwsan•com>
>> Cc: Kevin Hilman <khilman@ti•com>
>> Cc: Rob Herring <robherring2@gmail•com>
>> Cc: Tony Lindgren <tony@atomide•com>
>
> Just realized that a stray change made it into the patch that was posted.
> Below is a version with the stray change removed. It applies on
> Tony's v3.3-rc3 testing branch, plus the OMAP2420 serial fix patch that
> was posted earlier.
>
>
> - Paul
>
> From: Paul Walmsley <paul@pwsan•com>
> Date: Fri, 10 Feb 2012 01:40:04 -0700
> Subject: [PATCH v2] ARM: OMAP2xxx: PM: remove obsolete timer disable code
> in the suspend path
>
> Remove omap_{read,write}l() from the 24xx PM code. The clocksource
> code should now handle what this was supposed to do.
>
> Tested on N800 -- but it's hard to say whether this fixes anything.
> OMAP24xx static suspend path is currently broken, and this patch
> doesn't change that.
>
> Signed-off-by: Paul Walmsley <paul@pwsan•com>
> Cc: Kevin Hilman <khilman@ti•com>
> Cc: Rob Herring <robherring2@gmail•com>
> Cc: Tony Lindgren <tony@atomide•com>
Acked-by: Kevin Hilman <khilman@ti•com>
> ---
> arch/arm/mach-omap2/pm24xx.c | 22 +---------------------
> 1 files changed, 1 insertions(+), 21 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/pm24xx.c b/arch/arm/mach-omap2/pm24xx.c
> index 23de98d..18eda53 100644
> --- a/arch/arm/mach-omap2/pm24xx.c
> +++ b/arch/arm/mach-omap2/pm24xx.c
> @@ -254,26 +254,6 @@ static int omap2_pm_begin(suspend_state_t state)
> return 0;
> }
>
> -static int omap2_pm_suspend(void)
> -{
> - u32 wken_wkup, mir1;
> -
> - wken_wkup = omap2_prm_read_mod_reg(WKUP_MOD, PM_WKEN);
> - wken_wkup &= ~OMAP24XX_EN_GPT1_MASK;
> - omap2_prm_write_mod_reg(wken_wkup, WKUP_MOD, PM_WKEN);
> -
> - /* Mask GPT1 */
> - mir1 = omap_readl(0x480fe0a4);
> - omap_writel(1 << 5, 0x480fe0ac);
> -
> - omap2_enter_full_retention();
> -
> - omap_writel(mir1, 0x480fe0a4);
> - omap2_prm_write_mod_reg(wken_wkup, WKUP_MOD, PM_WKEN);
> -
> - return 0;
> -}
> -
> static int omap2_pm_enter(suspend_state_t state)
> {
> int ret = 0;
> @@ -281,7 +261,7 @@ static int omap2_pm_enter(suspend_state_t state)
> switch (state) {
> case PM_SUSPEND_STANDBY:
> case PM_SUSPEND_MEM:
> - ret = omap2_pm_suspend();
> + omap2_enter_full_retention();
> break;
> default:
> ret = -EINVAL;
next prev parent reply other threads:[~2012-02-10 19:26 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-02-10 3:08 ARM: OMAP2xxx: PM: remove obsolete timer disable code in the suspend path Paul Walmsley
2012-02-10 8:42 ` Paul Walmsley
2012-02-10 19:26 ` Kevin Hilman [this message]
2012-02-20 22:08 ` Tony Lindgren
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=87y5saxzeq.fsf@ti.com \
--to=khilman@ti$(echo .)com \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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