[U-Boot] [PATCH] armv8/ls1043a: Add the OCRAM initialization

[Calvin Johnson]

Hi York,

>-----Original Message-----
>From: york sun
>Sent: Wednesday, October 26, 2016 10:09 PM
>To: Calvin Johnson <calvin.johnson at nxp.com>; Prabhakar Kushwaha <prabhakar.kushwaha at nxp.com>; Pratiyush
>Srivastava <pratiyush.srivastava at nxp.com>; u-boot at lists.denx.de; Mingkai Hu <mingkai.hu at nxp.com>
>Cc: Hou Zhiqiang <Zhiqiang.Hou at freescale.com>
>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