Hi, Fabio Best Regards! Anson Huang
> -----Original Message----- > From: Fabio Estevam [mailto:[email protected]] > Sent: 2018年12月3日 20:03 > To: Anson Huang <[email protected]> > Cc: Shawn Guo <[email protected]>; Sascha Hauer > <[email protected]>; Sascha Hauer <[email protected]>; Fabio > Estevam <[email protected]>; Russell King - ARM Linux > <[email protected]>; moderated list:ARM/FREESCALE IMX / MXC ARM > ARCHITECTURE <[email protected]>; linux-kernel > <[email protected]>; dl-linux-imx <[email protected]> > Subject: Re: [PATCH] ARM: imx: update the cpu power up timing setting > > On Mon, Dec 3, 2018 at 9:42 AM Fabio Estevam <[email protected]> > wrote: > > > > On Mon, Dec 3, 2018 at 12:45 AM Anson Huang <[email protected]> > 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 <[email protected]> > > > > Reviewed-by: Fabio Estevam <[email protected]> > > 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.
