> -----Original Message----- > From: Shaohui Xie > Sent: Monday, August 29, 2016 12:45 PM > To: york sun <york....@nxp.com>; Qianyu Gong <qianyu.g...@nxp.com>; > u-boot@lists.denx.de > Cc: Prabhakar Kushwaha <prabhakar.kushw...@nxp.com>; Mingkai Hu > <mingkai...@nxp.com>; Zhiqiang Hou <zhiqiang....@nxp.com>; Wenbin > Song <wenbin.s...@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.g...@nxp.com>; u-boot@lists.denx.de > > Cc: Prabhakar Kushwaha <prabhakar.kushw...@nxp.com>; Mingkai Hu > > <mingkai...@nxp.com>; Shaohui Xie <shaohui....@nxp.com>; Zhiqiang > Hou > > <zhiqiang....@nxp.com>; Wenbin Song <wenbin.s...@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....@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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
