[U-Boot] [PATCH] driver/ddr/fsl: Force enabling parity for A-009803

York Sun york.sun at nxp.com
Fri May 20 17:29:13 CEST 2016


On 05/20/2016 01:15 AM, Shengzhou Liu wrote:
> Default address parity is disabled for DDR4 UDIMM for considing
> performance, it needs to enable parity for A-009803 workaround.
> 
> Signed-off-by: Shengzhou Liu <Shengzhou.Liu at nxp.com>
> ---
>  drivers/ddr/fsl/options.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/ddr/fsl/options.c b/drivers/ddr/fsl/options.c
> index d0075ff..dcef0bb 100644
> --- a/drivers/ddr/fsl/options.c
> +++ b/drivers/ddr/fsl/options.c
> @@ -1016,6 +1016,9 @@ unsigned int populate_memctl_options(const common_timing_params_t *common_dimm,
>  		}
>  	}
>  
> +#ifdef CONFIG_SYS_FSL_ERRATUM_A009803
> +	popts->ap_en = 1;
> +#endif
>  	/*
>  	 * BSTTOPRE precharge interval
>  	 *
> 

Shengzhou,

It looks backward. You should detect if the condition is right before applying
the workaround, not by forcing the condition.

York



More information about the U-Boot mailing list