From: York Sun <york.sun@nxp.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 3/3] driver/ddr/fsl: Add workaround for erratum A-009801
Date: Tue, 24 May 2016 10:24:40 -0700 [thread overview]
Message-ID: <57448E58.3010107@nxp.com> (raw)
In-Reply-To: <1458107423-9841-3-git-send-email-Shengzhou.Liu@nxp.com>
On 03/15/2016 10:59 PM, Shengzhou Liu wrote:
> The initial training for the DDRC may provide results that are not
> optimized. The workaround provides better read timing margins.
>
> Signed-off-by: Shengzhou Liu <Shengzhou.Liu@nxp.com>
> ---
> arch/arm/include/asm/arch-fsl-layerscape/config.h | 1 +
> drivers/ddr/fsl/fsl_ddr_gen4.c | 7 +++++++
> include/fsl_ddr_sdram.h | 4 ++++
> 3 files changed, 12 insertions(+)
>
> diff --git a/arch/arm/include/asm/arch-fsl-layerscape/config.h b/arch/arm/include/asm/arch-fsl-layerscape/config.h
> index 6ec7e50..ba06465 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_A009801
> #define CONFIG_SYS_FSL_ERRATUM_A009803
> #define CONFIG_SYS_FSL_ERRATUM_A009942
>
> diff --git a/drivers/ddr/fsl/fsl_ddr_gen4.c b/drivers/ddr/fsl/fsl_ddr_gen4.c
> index 7cdb700..1dc0631 100644
> --- a/drivers/ddr/fsl/fsl_ddr_gen4.c
> +++ b/drivers/ddr/fsl/fsl_ddr_gen4.c
> @@ -251,6 +251,13 @@ void fsl_ddr_set_memctl_regs(const fsl_ddr_cfg_regs_t *regs,
> }
> #endif
>
> +#ifdef CONFIG_SYS_FSL_ERRATUM_A009801
> + temp32 = ddr_in32(&ddr->debug[25]);
> + temp32 &= ~DDR_CAS_TO_PRE_SUB_MASK;
> + temp32 |= 9 << DDR_CAS_TO_PRE_SUB_SHIFT;
> + ddr_out32(&ddr->debug[25], temp32);
> +#endif
> +
Shengzhou,
Please examine workaround for A008511. This workaround has been included, but
wasn't named as A009801. You can move out that code and put under A009801 if like.
York
next prev parent reply other threads:[~2016-05-24 17:24 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-03-16 5:50 [U-Boot] [PATCH 1/3] armv8/ls1043: Add workaround for DDR erratum A-008850 Shengzhou Liu
2016-03-16 5:50 ` [U-Boot] [PATCH 2/3] drivers/ddr/fsl: update workaround for erratum A-008511 Shengzhou Liu
2016-05-24 16:58 ` York Sun
2016-05-24 17:06 ` York Sun
2016-03-16 5:50 ` [U-Boot] [PATCH 3/3] driver/ddr/fsl: Add workaround for erratum A-009801 Shengzhou Liu
2016-05-24 17:24 ` York Sun [this message]
2016-03-16 15:19 ` [U-Boot] [PATCH 1/3] armv8/ls1043: Add workaround for DDR erratum A-008850 york sun
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=57448E58.3010107@nxp.com \
--to=york.sun@nxp.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox