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@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.