From mboxrd@z Thu Jan 1 00:00:00 1970 From: Calvin Johnson Date: Thu, 27 Oct 2016 09:47:03 +0000 Subject: [U-Boot] [PATCH] armv8/ls1043a: Add the OCRAM initialization In-Reply-To: References: <1476274533-17995-1-git-send-email-pratiyush.srivastava@nxp.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi York, >-----Original Message----- >From: york sun >Sent: Wednesday, October 26, 2016 10:09 PM >To: Calvin Johnson ; Prabhakar Kushwaha ; Pratiyush >Srivastava ; u-boot at lists.denx.de; Mingkai Hu >Cc: Hou Zhiqiang >Subject: Re: [PATCH] armv8/ls1043a: Add the OCRAM initialization > >On 10/24/2016 09:30 PM, Calvin Johnson wrote: > >>>>> 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. >> > >How about LSCH3? We have LS2080A secure boot. I don't know about LS2080A. Prabhakar or Ruchika(copied) may be able to comment on this. Regards Calvin