OK York, will send new patch..
> -----Original Message----- > From: york sun > Sent: Tuesday, February 28, 2017 9:56 PM > To: Suresh Gupta <suresh.gu...@nxp.com> > Cc: u-boot@lists.denx.de; Scott Wood <scott.w...@nxp.com>; Leo Li > <leoyang...@nxp.com>; Sriram Dash <sriram.d...@nxp.com>; Rajesh Bhagat > <rajesh.bha...@nxp.com> > Subject: Re: [PATCH v3 4/8] armv8: Add workaround for USB erratum A-009007 > > On 02/28/2017 02:52 AM, Suresh Gupta wrote: > > > > > >> -----Original Message----- > >> From: york sun > >> Sent: Friday, February 24, 2017 10:31 PM > >> To: Suresh Gupta <suresh.gu...@nxp.com> > >> Cc: u-boot@lists.denx.de; Scott Wood <scott.w...@nxp.com>; Leo Li > >> <leoyang...@nxp.com>; Sriram Dash <sriram.d...@nxp.com>; Rajesh > >> Bhagat <rajesh.bha...@nxp.com> > >> Subject: Re: [PATCH v3 4/8] armv8: Add workaround for USB erratum > >> A-009007 > >> > >> On 02/23/2017 11:19 PM, Suresh Gupta wrote: > >>> Hi York, > >>> > >>> It is not good idea to change the values of all macro at this time > >>> as the code > >> tested on different platforms. > >> > >> I am not talking about any value change. You are using writew. Why > >> not using > >> out_be16 as you thought? > > > > For now all values in macro (like USB_PHY_RX_EQ_VAL_2) are swapped and > > if I want to use out_be16, then I need to change values of all macros, > > which intern require testing on all platform. > > That's the reason, I don't want to make such changes and break the > > working USB > > > > Suresh, > > This erratum only applies to LS1043A, LS1046A, LS2080A. It wouldn't be too > much trouble to verify all of them. I'd rather we do it right at the first > place than > coming back to fix it. Are you in a rush to get this patch out? > > Another thing, please drop defined(CONFIG_ARCH_LS2085A) for all the patches. > We only use CONFIG_ARCH_LS2080A. > > York _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
