From mboxrd@z Thu Jan 1 00:00:00 1970 From: York Sun Date: Wed, 25 May 2016 09:42:06 -0700 Subject: [U-Boot] [PATCH 2/2 v2] driver/ddr/fsl: Add workaround for erratum A-009803 In-Reply-To: <1457602617-968-2-git-send-email-Shengzhou.Liu@nxp.com> References: <1457602617-968-1-git-send-email-Shengzhou.Liu@nxp.com> <1457602617-968-2-git-send-email-Shengzhou.Liu@nxp.com> Message-ID: <5745D5DE.5020808@nxp.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 03/10/2016 01:46 AM, Shengzhou Liu wrote: > During initial DDR training, false parity errors may be detected. > This patch adds workaround to fix the erratum. > Tested on LS2085QDS and LS2080RDB. > > Signed-off-by: Shengzhou Liu > --- > v2: Integrated York's comments. > > arch/arm/include/asm/arch-fsl-layerscape/config.h | 1 + > drivers/ddr/fsl/fsl_ddr_gen4.c | 44 ++++++++++++++++++++--- > 2 files changed, 40 insertions(+), 5 deletions(-) > > diff --git a/arch/arm/include/asm/arch-fsl-layerscape/config.h b/arch/arm/include/asm/arch-fsl-layerscape/config.h > index 0ef7c9d..b0c112b 100644 > --- a/arch/arm/include/asm/arch-fsl-layerscape/config.h > +++ b/arch/arm/include/asm/arch-fsl-layerscape/config.h > @@ -119,6 +119,7 @@ > #define CONFIG_SYS_FSL_ERRATUM_A008751 > #define CONFIG_SYS_FSL_ERRATUM_A009635 > #define CONFIG_SYS_FSL_ERRATUM_A009663 > +#define CONFIG_SYS_FSL_ERRATUM_A009803 > #define CONFIG_SYS_FSL_ERRATUM_A009942 > > /* ARM A57 CORE ERRATA */ > diff --git a/drivers/ddr/fsl/fsl_ddr_gen4.c b/drivers/ddr/fsl/fsl_ddr_gen4.c > index 6f76980..608810d 100644 > --- a/drivers/ddr/fsl/fsl_ddr_gen4.c > +++ b/drivers/ddr/fsl/fsl_ddr_gen4.c > @@ -12,7 +12,8 @@ > #include > #include > > -#ifdef CONFIG_SYS_FSL_ERRATUM_A008511 > +#if defined(CONFIG_SYS_FSL_ERRATUM_A008511) | \ > + defined(CONFIG_SYS_FSL_ERRATUM_A009803) Shengzhou, I missed this error. It should be logic or "||", not bit or "|" here. Please fix it with your latest patch for 9803. York