* [PATCH] ARM: imx6: enable WAIT mode hardware workaround for imx6sx
@ 2016-08-22 17:03 Anson Huang
2016-08-25 1:26 ` Peter Chen
2016-08-29 5:44 ` Shawn Guo
0 siblings, 2 replies; 4+ messages in thread
From: Anson Huang @ 2016-08-22 17:03 UTC (permalink / raw)
To: linux-arm-kernel
Need to enable INT_MEM_CLK_LPM bit in CCM_CGPR for WAIT mode,
without this bit set, if there is pending interrupt during
ARM platform entering WAIT mode without power gating, cache
data will be corrupted, this is a hardware workaround for WAIT
mode and must be enabled.
Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
---
arch/arm/mach-imx/cpuidle-imx6sx.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/mach-imx/cpuidle-imx6sx.c b/arch/arm/mach-imx/cpuidle-imx6sx.c
index 3c6672b..41cdce6 100644
--- a/arch/arm/mach-imx/cpuidle-imx6sx.c
+++ b/arch/arm/mach-imx/cpuidle-imx6sx.c
@@ -90,6 +90,7 @@ static struct cpuidle_driver imx6sx_cpuidle_driver = {
int __init imx6sx_cpuidle_init(void)
{
+ imx6q_set_int_mem_clk_lpm(true);
imx6_enable_rbc(false);
/*
* set ARM power up/down timing to the fastest,
--
1.9.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH] ARM: imx6: enable WAIT mode hardware workaround for imx6sx
2016-08-22 17:03 [PATCH] ARM: imx6: enable WAIT mode hardware workaround for imx6sx Anson Huang
@ 2016-08-25 1:26 ` Peter Chen
2016-08-29 5:44 ` Shawn Guo
1 sibling, 0 replies; 4+ messages in thread
From: Peter Chen @ 2016-08-25 1:26 UTC (permalink / raw)
To: linux-arm-kernel
>Need to enable INT_MEM_CLK_LPM bit in CCM_CGPR for WAIT mode, without
>this bit set, if there is pending interrupt during ARM platform entering WAIT mode
>without power gating, cache data will be corrupted, this is a hardware workaround for
>WAIT mode and must be enabled.
>
>Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
>---
> arch/arm/mach-imx/cpuidle-imx6sx.c | 1 +
> 1 file changed, 1 insertion(+)
>
>diff --git a/arch/arm/mach-imx/cpuidle-imx6sx.c b/arch/arm/mach-imx/cpuidle-
>imx6sx.c
>index 3c6672b..41cdce6 100644
>--- a/arch/arm/mach-imx/cpuidle-imx6sx.c
>+++ b/arch/arm/mach-imx/cpuidle-imx6sx.c
>@@ -90,6 +90,7 @@ static struct cpuidle_driver imx6sx_cpuidle_driver = {
>
> int __init imx6sx_cpuidle_init(void)
> {
>+ imx6q_set_int_mem_clk_lpm(true);
> imx6_enable_rbc(false);
> /*
> * set ARM power up/down timing to the fastest,
>--
>1.9.1
It fixed the system unstable problem at my imx6sx-sdb board. So:
Tested-by: Peter Chen <peter.chen@nxp.com>
Peter
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] ARM: imx6: enable WAIT mode hardware workaround for imx6sx
2016-08-22 17:03 [PATCH] ARM: imx6: enable WAIT mode hardware workaround for imx6sx Anson Huang
2016-08-25 1:26 ` Peter Chen
@ 2016-08-29 5:44 ` Shawn Guo
2016-08-29 6:02 ` Yongcai Huang
1 sibling, 1 reply; 4+ messages in thread
From: Shawn Guo @ 2016-08-29 5:44 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Aug 23, 2016 at 01:03:20AM +0800, Anson Huang wrote:
> Need to enable INT_MEM_CLK_LPM bit in CCM_CGPR for WAIT mode,
> without this bit set, if there is pending interrupt during
> ARM platform entering WAIT mode without power gating, cache
> data will be corrupted, this is a hardware workaround for WAIT
> mode and must be enabled.
>
> Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
> ---
> arch/arm/mach-imx/cpuidle-imx6sx.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm/mach-imx/cpuidle-imx6sx.c b/arch/arm/mach-imx/cpuidle-imx6sx.c
> index 3c6672b..41cdce6 100644
> --- a/arch/arm/mach-imx/cpuidle-imx6sx.c
> +++ b/arch/arm/mach-imx/cpuidle-imx6sx.c
> @@ -90,6 +90,7 @@ static struct cpuidle_driver imx6sx_cpuidle_driver = {
>
> int __init imx6sx_cpuidle_init(void)
> {
> + imx6q_set_int_mem_clk_lpm(true);
It might be the time to rename the function to imx6_set_int_mem_clk_lpm()
with a beforehand patch.
Shawn
> imx6_enable_rbc(false);
> /*
> * set ARM power up/down timing to the fastest,
> --
> 1.9.1
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] ARM: imx6: enable WAIT mode hardware workaround for imx6sx
2016-08-29 5:44 ` Shawn Guo
@ 2016-08-29 6:02 ` Yongcai Huang
0 siblings, 0 replies; 4+ messages in thread
From: Yongcai Huang @ 2016-08-29 6:02 UTC (permalink / raw)
To: linux-arm-kernel
Best Regards!
Anson Huang
> -----Original Message-----
> From: Shawn Guo [mailto:shawnguo at kernel.org]
> Sent: 2016-08-29 1:44 PM
> To: Yongcai Huang <anson.huang@nxp.com>
> Cc: linux-arm-kernel at lists.infradead.org; Fabio Estevam
> <fabio.estevam@nxp.com>; Peter Chen <peter.chen@nxp.com>;
> kernel at pengutronix.de
> Subject: Re: [PATCH] ARM: imx6: enable WAIT mode hardware workaround for
> imx6sx
>
> On Tue, Aug 23, 2016 at 01:03:20AM +0800, Anson Huang wrote:
> > Need to enable INT_MEM_CLK_LPM bit in CCM_CGPR for WAIT mode,
> without
> > this bit set, if there is pending interrupt during ARM platform
> > entering WAIT mode without power gating, cache data will be corrupted,
> > this is a hardware workaround for WAIT mode and must be enabled.
> >
> > Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
> > ---
> > arch/arm/mach-imx/cpuidle-imx6sx.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/arch/arm/mach-imx/cpuidle-imx6sx.c
> > b/arch/arm/mach-imx/cpuidle-imx6sx.c
> > index 3c6672b..41cdce6 100644
> > --- a/arch/arm/mach-imx/cpuidle-imx6sx.c
> > +++ b/arch/arm/mach-imx/cpuidle-imx6sx.c
> > @@ -90,6 +90,7 @@ static struct cpuidle_driver imx6sx_cpuidle_driver =
> > {
> >
> > int __init imx6sx_cpuidle_init(void)
> > {
> > + imx6q_set_int_mem_clk_lpm(true);
>
> It might be the time to rename the function to imx6_set_int_mem_clk_lpm()
> with a beforehand patch.
>
> Shawn
Agree, already send out a V2 patch set to rename the function firstly, thanks.
Anson.
>
> > imx6_enable_rbc(false);
> > /*
> > * set ARM power up/down timing to the fastest,
> > --
> > 1.9.1
> >
> >
> > _______________________________________________
> > linux-arm-kernel mailing list
> > linux-arm-kernel at lists.infradead.org
> > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2016-08-29 6:02 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-22 17:03 [PATCH] ARM: imx6: enable WAIT mode hardware workaround for imx6sx Anson Huang
2016-08-25 1:26 ` Peter Chen
2016-08-29 5:44 ` Shawn Guo
2016-08-29 6:02 ` Yongcai Huang
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.