-----Original Message----- From: Shaohui Xie Sent: Monday, August 29, 2016 12:45 PM To: york sun york.sun@nxp.com; Qianyu Gong qianyu.gong@nxp.com; u-boot@lists.denx.de Cc: Prabhakar Kushwaha prabhakar.kushwaha@nxp.com; Mingkai Hu mingkai.hu@nxp.com; Zhiqiang Hou zhiqiang.hou@nxp.com; Wenbin Song wenbin.song@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@nxp.com; u-boot@lists.denx.de Cc: Prabhakar Kushwaha prabhakar.kushwaha@nxp.com; Mingkai Hu mingkai.hu@nxp.com; Shaohui Xie shaohui.xie@nxp.com; Zhiqiang
Hou
zhiqiang.hou@nxp.com; Wenbin Song wenbin.song@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@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