RE: [PATCH] ARM: imx: update the cpu power up timing setting

From: Anson Huang
Date: Mon Dec 03 2018 - 22:19:32 EST


Hi, Fabio

Best Regards!
Anson Huang

> -----Original Message-----
> From: Fabio Estevam [mailto:festevam@xxxxxxxxx]
> Sent: 2018å12æ3æ 20:03
> To: Anson Huang <anson.huang@xxxxxxx>
> Cc: Shawn Guo <shawnguo@xxxxxxxxxx>; Sascha Hauer
> <s.hauer@xxxxxxxxxxxxxx>; Sascha Hauer <kernel@xxxxxxxxxxxxxx>; Fabio
> Estevam <fabio.estevam@xxxxxxx>; Russell King - ARM Linux
> <linux@xxxxxxxxxxxxxxx>; moderated list:ARM/FREESCALE IMX / MXC ARM
> ARCHITECTURE <linux-arm-kernel@xxxxxxxxxxxxxxxxxxx>; linux-kernel
> <linux-kernel@xxxxxxxxxxxxxxx>; dl-linux-imx <linux-imx@xxxxxxx>
> Subject: Re: [PATCH] ARM: imx: update the cpu power up timing setting
>
> On Mon, Dec 3, 2018 at 9:42 AM Fabio Estevam <festevam@xxxxxxxxx>
> wrote:
> >
> > On Mon, Dec 3, 2018 at 12:45 AM Anson Huang <anson.huang@xxxxxxx>
> wrote:
> > >
> > > The sw2iso count should cover ARM LDO ramp-up time, the MAX ARM LDO
> > > ramp-up time may be up to more than 100us on some boards, this patch
> > > sets sw2iso to 0xf
> > > (~384us) which is the reset value, and it is much more safe to cover
> > > different boards, since we have observed that some customer boards
> > > failed with current setting of 0x2.
> > >
> > > Fixes: 05136f0897b5 ("ARM: imx: support arm power off in cpuidle for
> > > i.mx6sx")
> > > Signed-off-by: Anson Huang <Anson.Huang@xxxxxxx>
> >
> > Reviewed-by: Fabio Estevam <festevam@xxxxxxxxx>
>
> Minor nit that maybe Shawn can fix while applying it: Subject should contain
> imx6sx as this change only affect this SoC.

I sent out the V2 patch to easy Shawn's work. Thanks.

Anson.