Hi Peng,
On Mon, May 11, 2020 at 6:30 AM Peng Fan <[email protected]> wrote:
>
> From: Sherry Sun <[email protected]>
>
> Add inline ECC support for lpddr4 on imx8mp-evk. And add a config which
> can enable/disable inline ECC feature for lpddr4 on imx8mp-evk board.
Please elaborate more on this inline ECC feature: when and why does it
need to be selected, for example.
> --- a/board/freescale/imx8mp_evk/lpddr4_timing.c
> +++ b/board/freescale/imx8mp_evk/lpddr4_timing.c
> @@ -14,6 +14,9 @@ struct dram_cfg_param ddr_ddrc_cfg[] = {
> { 0x3d400020, 0x323 },
> { 0x3d400024, 0x1e84800 },
> { 0x3d400064, 0x7a0118 },
> +#ifdef CONFIG_IMX8M_DRAM_INLINE_ECC
I see no user for this symbol at the moment.
It is usually better to introduce the symbol when there is a real user for it.
