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

[Z.Q. Hou]

Hi York,

Thanks a lot for your comments!

> -----Original Message-----
> From: york sun
> Sent: 2016年12月14日 0:59
> To: Z.Q. Hou <zhiqiang.hou at nxp.com>; u-boot at lists.denx.de; Prabhakar
> Kushwaha <prabhakar.kushwaha at nxp.com>; Mingkai Hu
> <mingkai.hu at nxp.com>; Pratiyush Srivastava <pratiyush.srivastava at nxp.com>
> Subject: Re: [PATCH] armv8/fsl_lsch2: Add the OCRAM initialization
> 
> On 12/13/2016 01:45 AM, Zhiqiang Hou wrote:
> > From: Hou Zhiqiang <Zhiqiang.Hou at nxp.com>
> >
> > 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: Hou Zhiqiang <Zhiqiang.Hou at nxp.com>
> > Signed-off-by: Pratiyush Srivastava <pratiyush.srivastava at nxp.com>
> > Signed-off-by: Prabhakar Kushwaha <prabhakar.kushwaha at nxp.com>
> > ---
> >  arch/arm/cpu/armv8/fsl-layerscape/lowlevel.S      | 37
> +++++++++++++++++++++++
> >  arch/arm/include/asm/arch-fsl-layerscape/config.h |  3 ++
> >  2 files changed, 40 insertions(+)
> >
> 
> Zhiqiang,
> 
> This looks like an updated version of previous patch "armv8/ls1043a: Add the
> OCRAM initialization". Was there any effort to include LSCH3?

Yes, I wrote that patch for ls1043a bringup, and re-write it for LSCH2.
I don't know if LSCH3 need this operation, is there any update about verifying this issue on LSCH3?

Thanks,
Zhiqiang