| Message ID | 20190513031531.7879-1-festevam@gmail.com |
|---|---|
| State | New |
| Headers | show |
| Series | [v2] ARM: imx: cpuidle-imx6sx: Restrict the SW2ISO increase toi.MX6SX | expand |
On Mon, May 13, 2019 at 12:15:31AM -0300, Fabio Estevam wrote: > Since commit 1e434b703248 ("ARM: imx: update the cpu power up timing > setting on i.mx6sx") some characters loss is noticed on i.MX6ULL UART > as reported by Christoph Niedermaier. > > The intention of such commit was to increase the SW2ISO field for i.MX6SX > only, but since cpuidle-imx6sx is also used on i.MX6UL/i.MX6ULL this caused > unintended side effects on other SoCs. > > Fix this problem by keeping the original SW2ISO value for i.MX6UL/i.MX6ULL > and only increase SW2ISO in the i.MX6SX case. > > Cc: stable@vger.kernel.org > Fixes: 1e434b703248 ("ARM: imx: update the cpu power up timing setting on i.mx6sx") > Reported-by: Christoph Niedermaier <cniedermaier@dh-electronics.com> > Signed-off-by: Fabio Estevam <festevam@gmail.com> > Tested-by: Sébastien Szymanski <sebastien.szymanski@armadeus.com> > Tested-by: Christoph Niedermaier <cniedermaier@dh-electronics.com> Applied, thanks.
diff --git a/arch/arm/mach-imx/cpuidle-imx6sx.c b/arch/arm/mach-imx/cpuidle-imx6sx.c index fd0053e47a15..4016b717da1b 100644 --- a/arch/arm/mach-imx/cpuidle-imx6sx.c +++ b/arch/arm/mach-imx/cpuidle-imx6sx.c @@ -15,6 +15,7 @@ #include "common.h" #include "cpuidle.h" +#include "hardware.h" static int imx6sx_idle_finish(unsigned long val) { @@ -110,7 +111,7 @@ int __init imx6sx_cpuidle_init(void) * except for power up sw2iso which need to be * larger than LDO ramp up time. */ - imx_gpc_set_arm_power_up_timing(0xf, 1); + imx_gpc_set_arm_power_up_timing(cpu_is_imx6sx() ? 0xf : 0x2, 1); imx_gpc_set_arm_power_down_timing(1, 1); return cpuidle_register(&imx6sx_cpuidle_driver, NULL);
Since commit 1e434b703248 ("ARM: imx: update the cpu power up timing setting on i.mx6sx") some characters loss is noticed on i.MX6ULL UART as reported by Christoph Niedermaier. The intention of such commit was to increase the SW2ISO field for i.MX6SX only, but since cpuidle-imx6sx is also used on i.MX6UL/i.MX6ULL this caused unintended side effects on other SoCs. Fix this problem by keeping the original SW2ISO value for i.MX6UL/i.MX6ULL and only increase SW2ISO in the i.MX6SX case. Cc: stable@vger.kernel.org Fixes: 1e434b703248 ("ARM: imx: update the cpu power up timing setting on i.mx6sx") Reported-by: Christoph Niedermaier <cniedermaier@dh-electronics.com> Signed-off-by: Fabio Estevam <festevam@gmail.com> Tested-by: Sébastien Szymanski <sebastien.szymanski@armadeus.com> Tested-by: Christoph Niedermaier <cniedermaier@dh-electronics.com> --- Changes since v1: - Simplify the code by using the C ternary operator so that the patch can be backported all the way to kernel 4.4 arch/arm/mach-imx/cpuidle-imx6sx.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)