> -----Original Message-----
> From: U-Boot <u-boot-boun...@lists.denx.de> On Behalf Of Zhiqiang Hou
> Sent: Tuesday, March 24, 2020 1:42 PM
> To: u-boot@lists.denx.de; Priyanka Jain <priyanka.j...@nxp.com>; Biwen Li
> <biwen...@nxp.com>
> Cc: Z.q. Hou <zhiqiang....@nxp.com>
> Subject: [PATCHv2 2/9] arm64: fsl-layerscape: Assign addr to resv_ram if
> enabled RESV_RAM config
>
> From: Hou Zhiqiang <zhiqiang....@nxp.com>
>
> The initialization of gd->arch.resv_ram pointer should depend on if the
> RESV_RAM config is enabled.
>
> Signed-off-by: Hou Zhiqiang <zhiqiang....@nxp.com>
> ---
> V2:
> - No change.
>
> arch/arm/cpu/armv8/fsl-layerscape/cpu.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm/cpu/armv8/fsl-layerscape/cpu.c
> b/arch/arm/cpu/armv8/fsl-layerscape/cpu.c
> index b443894453..1b7729c046 100644
> --- a/arch/arm/cpu/armv8/fsl-layerscape/cpu.c
> +++ b/arch/arm/cpu/armv8/fsl-layerscape/cpu.c
> @@ -1379,7 +1379,7 @@ static int tfa_dram_init_banksize(void)
> if (i > 0)
> ret = 0;
>
> -#if defined(CONFIG_FSL_MC_ENET) && !defined(CONFIG_SPL_BUILD)
> +#if defined(CONFIG_RESV_RAM) && !defined(CONFIG_SPL_BUILD)
> /* Assign memory for MC */
> #ifdef CONFIG_SYS_DDR_BLOCK3_BASE
> if (gd->bd->bi_dram[2].size >=
> @@ -1402,7 +1402,7 @@ static int tfa_dram_init_banksize(void)
> board_reserve_ram_top(gd->bd-
> >bi_dram[0].size);
> }
> }
> -#endif /* CONFIG_FSL_MC_ENET */
> +#endif /* CONFIG_RESV_RAM */
>
> return ret;
> }
> @@ -1465,7 +1465,7 @@ int dram_init_banksize(void)
> }
> #endif /* CONFIG_SYS_MEM_RESERVE_SECURE */
>
> -#if defined(CONFIG_FSL_MC_ENET) && !defined(CONFIG_SPL_BUILD)
> +#if defined(CONFIG_RESV_RAM) && !defined(CONFIG_SPL_BUILD)
> /* Assign memory for MC */
> #ifdef CONFIG_SYS_DDR_BLOCK3_BASE
> if (gd->bd->bi_dram[2].size >=
> @@ -1488,7 +1488,7 @@ int dram_init_banksize(void)
> board_reserve_ram_top(gd->bd-
> >bi_dram[0].size);
> }
> }
> -#endif /* CONFIG_FSL_MC_ENET */
> +#endif /* CONFIG_RESV_RAM */
>
CONFIG_FSL_MC_ENET will select the CONFIG_RESV_RAM. What benefit we have with
this change ?
because we are reserving memory for MC in the following code, so earlier check
(with CONFIG_FSL_MC_ENET) looks good to be.
> #ifdef CONFIG_SYS_DP_DDR_BASE_PHY
> #ifdef CONFIG_SYS_DDR_BLOCK3_BASE
> --
> 2.17.1
