Hi York, Thanks a lot for your comments!
> -----Original Message----- > From: york sun > Sent: 2016年12月14日 0:59 > To: Z.Q. Hou <zhiqiang....@nxp.com>; u-boot@lists.denx.de; Prabhakar > Kushwaha <prabhakar.kushw...@nxp.com>; Mingkai Hu > <mingkai...@nxp.com>; Pratiyush Srivastava <pratiyush.srivast...@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....@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....@nxp.com> > > Signed-off-by: Pratiyush Srivastava <pratiyush.srivast...@nxp.com> > > Signed-off-by: Prabhakar Kushwaha <prabhakar.kushw...@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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot