Hi York, >-----Original Message----- >From: U-Boot [mailto:u-boot-boun...@lists.denx.de] On Behalf Of york sun >Sent: Monday, October 24, 2016 9:45 PM >To: Prabhakar Kushwaha <prabhakar.kushw...@nxp.com>; Pratiyush Srivastava ><pratiyush.srivast...@nxp.com>; u- >b...@lists.denx.de; Mingkai Hu <mingkai...@nxp.com> >Cc: Hou Zhiqiang <zhiqiang....@freescale.com> >Subject: Re: [U-Boot] [PATCH] armv8/ls1043a: Add the OCRAM initialization > >On 10/23/2016 06:59 AM, Prabhakar Kushwaha wrote: >> Hi York, >> >> >>> -----Original Message----- >>> From: york sun >>> Sent: Saturday, October 22, 2016 1:39 AM >>> To: Prabhakar Kushwaha <prabhakar.kushw...@nxp.com>; Pratiyush >>> Srivastava <pratiyush.srivast...@nxp.com>; u-boot@lists.denx.de; >>> Mingkai Hu <mingkai...@nxp.com> >>> Cc: Hou Zhiqiang <zhiqiang....@freescale.com> >>> Subject: Re: [PATCH] armv8/ls1043a: Add the OCRAM initialization >>> >>> On 10/16/2016 10:35 PM, Prabhakar Kushwaha wrote: >>>> Hi Mingkai, >>>> >>>>> -----Original Message----- >>>>> From: Pratiyush Srivastava [mailto:pratiyush.srivast...@nxp.com] >>>>> Sent: Wednesday, October 12, 2016 5:46 PM >>>>> To: u-boot@lists.denx.de >>>>> Cc: york sun <york....@nxp.com>; Prabhakar Kushwaha >>>>> <prabhakar.kushw...@nxp.com>; Pratiyush Srivastava >>>>> <pratiyush.srivast...@nxp.com>; Hou Zhiqiang >>> <zhiqiang....@freescale.com> >>>>> Subject: [PATCH] armv8/ls1043a: Add the OCRAM initialization >>>>> >>>>> Clear the content to zero and the ECC error bit of OCRAM1/2. >>>>> >>>>> The OCRAM must be initialized to ZERO by the unit of 8-Byte before >>>>> accessing it, or else it will generate ECC error. And the IBR has >>>>> accessed the OCRAM before this initialization, so the ECC error >>>>> status bit should to be cleared. >>>>> >>>>> Signed-off-by: Pratiyush Srivastava <pratiyush.srivast...@nxp.com> >>>>> Signed-off-by: Hou Zhiqiang <zhiqiang....@freescale.com> >>>>> Signed-off-by: Prabhakar Kushwaha <prabhakar.kushw...@nxp.com> >>>>> --- >>>> >>>> This requirement is for both ls1043 and ls1088a. was this patch >>>> taken care >>> during ls1043a upstreaming >>>> If not, how it is being taken care for ls1043a. Same approach can >>>> be used for >>> ls1088a >>>> >>> >>> I wonder why we don't see ECC errors before this patch. We have >>> LS1043A boots on NAND, SD. >>> >> >> OCRAM has a requirement of initializing before first time "read". >> If user reads OCRAM before **initializing**; ECC error will come. (u-boot >> is not handling this error for now). >> >> I can only guess the reason of not seeing this error as OCRAM never read >> before any write. >> Even in case of Stack, data is first written and then read. >> > >Is there a case you want to read from OCRAM before writing anything to it? Why >don't we need to do so for SPL or LSCH3?
This issue will be seen ONLY in secure boot. It was reproduced on LS1043A also. Regards Calvin _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot