[U-Boot] [PATCH v2 5/5] move erratum a008336 and a008514 to soc specific file

[Yao Yuan]

On 12/05/2015 01:12 AM, York Sun wrote:
> On 12/04/2015 01:37 AM, Yuan Yao wrote:
> > As the errata A008336 and A008514 do not apply to all LS series SoCs
> > (such as LS1021A, LS1043A) we move them to an soc specific file
> >
> > Signed-off-by: Yuan Yao <yao.yuan at freescale.com>
> > ---
> > Changed in v2:
> > 	Update the patch commit message.
> > ---
> >
> >  arch/arm/cpu/armv8/fsl-layerscape/soc.c | 37
> +++++++++++++++++++++++++++++++++
> >  drivers/ddr/fsl/fsl_ddr_gen4.c          | 34 ------------------------------
> >  2 files changed, 37 insertions(+), 34 deletions(-)
> >
> > diff --git a/arch/arm/cpu/armv8/fsl-layerscape/soc.c
> > b/arch/arm/cpu/armv8/fsl-layerscape/soc.c
> > index 8896b70..738b113 100644
> > --- a/arch/arm/cpu/armv8/fsl-layerscape/soc.c
> > +++ b/arch/arm/cpu/armv8/fsl-layerscape/soc.c
> > @@ -14,6 +14,41 @@
> >  DECLARE_GLOBAL_DATA_PTR;
> >
> >  #if defined(CONFIG_LS2080A) || defined(CONFIG_LS2085A)
> > +/*
> > + * This erratum requires setting a value to eddrtqcr1 to
> > + * optimal the DDR performance.
> > + */
> > +static void erratum_a008336(void)
> > +{
> > +	u32 *eddrtqcr1;
> > +
> > +#ifdef CONFIG_SYS_FSL_ERRATUM_A008336 #ifdef
> > +CONFIG_SYS_FSL_DCSR_DDR_ADDR
> > +	eddrtqcr1 = (void *)CONFIG_SYS_FSL_DCSR_DDR_ADDR + 0x800;
> > +	out_le32(eddrtqcr1, 0x63b30002);
> > +#endif
> > +#ifdef CONFIG_SYS_FSL_DCSR_DDR2_ADDR
> > +	eddrtqcr1 = (void *)CONFIG_SYS_FSL_DCSR_DDR2_ADDR + 0x800;
> > +	out_le32(eddrtqcr1, 0x63b30002);
> > +#endif
> > +#endif
> > +}
> > +
> > +/*
> > + * This erratum requires a register write before being Memory
> > + * controller 3 being enabled.
> > + */
> > +static void erratum_a008514(void)
> > +{
> > +	u32 *eddrtqcr1;
> > +
> > +#ifdef CONFIG_SYS_FSL_ERRATUM_A008514 #ifdef
> > +CONFIG_SYS_FSL_DCSR_DDR2_ADDR
> 
> I believe this is a typo. It should be CONFIG_SYS_FSL_DCSR_DDR3_ADDR.
> 
Thanks  york,
Yes, it's a typo. It should be CONFIG_SYS_FSL_DCSR_DDR3_ADDR. 
Sorry for the mistake. I will update it soon.