[U-Boot] [PATCH 7/8] armv8: ls1046a: disable SATA ECC in DCSR

[Mingkai Hu]

> -----Original Message-----
> From: Shaohui Xie
> Sent: Monday, August 29, 2016 12:45 PM
> To: york sun <york.sun at nxp.com>; Qianyu Gong <qianyu.gong at nxp.com>;
> u-boot at lists.denx.de
> Cc: Prabhakar Kushwaha <prabhakar.kushwaha at nxp.com>; Mingkai Hu
> <mingkai.hu at nxp.com>; Zhiqiang Hou <zhiqiang.hou at nxp.com>; Wenbin
> Song <wenbin.song at nxp.com>
> Subject: RE: [PATCH 7/8] armv8: ls1046a: disable SATA ECC in DCSR
> 
> > -----Original Message-----
> > From: york sun
> > Sent: Saturday, August 27, 2016 12:08 AM
> > To: Qianyu Gong <qianyu.gong at nxp.com>; u-boot at lists.denx.de
> > Cc: Prabhakar Kushwaha <prabhakar.kushwaha at nxp.com>; Mingkai Hu
> > <mingkai.hu at nxp.com>; Shaohui Xie <shaohui.xie at nxp.com>; Zhiqiang
> Hou
> > <zhiqiang.hou at nxp.com>; Wenbin Song <wenbin.song at nxp.com>
> > Subject: Re: [PATCH 7/8] armv8: ls1046a: disable SATA ECC in DCSR
> >
> > On 08/26/2016 04:40 AM, Gong Qianyu wrote:
> > > From: Shaohui Xie <Shaohui.Xie at nxp.com>
> > >
> > > So to fix SATA CRC error.
> >
> > Does ECC cause the CRC error? Please explain more what you are doing
> > in this patch.
> [S.H] We only know disable ECC can fix CRC error which provided by
> validation team As a workaround, the root cause is not clear.
> 
> Maybe we can drop it for now, there might be a better solution for this issue
> in future, or the solution is still the same, but we know exactly what is going
> on.
> 
> Thanks,
> Shaohui

Maybe we keep this patch and add the description in the git log, and remove the
Disabling ECC when the root cause has found. At lease, disabling ECC can make 
the SATA working.

Thanks,
Mingkai