[U-Boot] [PATCH v4 3/9] ls2080ardb: Reserve DP-DDR RAM

[Prabhakar Kushwaha]

Hi Alex,


> -----Original Message-----
> From: Alexander Graf [mailto:agraf at suse.de]
> Sent: Tuesday, June 21, 2016 4:37 AM
> To: u-boot at lists.denx.de
> Cc: york sun <york.sun at nxp.com>; Prabhakar Kushwaha
> <prabhakar.kushwaha at nxp.com>
> Subject: [PATCH v4 3/9] ls2080ardb: Reserve DP-DDR RAM
> 
> The DP-DDR shouldn't be exposed as conventional memory to an OS, so let's
> rather claim it's a reserved region in the EFI memory map
> 
> Signed-off-by: Alexander Graf <agraf at suse.de>
> ---
>  board/freescale/ls2080ardb/ls2080ardb.c | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/board/freescale/ls2080ardb/ls2080ardb.c
> b/board/freescale/ls2080ardb/ls2080ardb.c
> index d1475a5..493f66d 100644
> --- a/board/freescale/ls2080ardb/ls2080ardb.c
> +++ b/board/freescale/ls2080ardb/ls2080ardb.c
> @@ -16,6 +16,7 @@
>  #include <fsl_debug_server.h>
>  #include <fsl-mc/fsl_mc.h>
>  #include <environment.h>
> +#include <efi_loader.h>
>  #include <i2c.h>
>  #include <asm/arch/soc.h>
>  #include <fsl_sec.h>
> @@ -202,6 +203,14 @@ int misc_init_r(void)
>  	if (adjust_vdd(0))
>  		printf("Warning: Adjusting core voltage failed.\n");
> 
> +#ifdef CONFIG_EFI_LOADER
> +	if (soc_has_dp_ddr() && gd->bd->bi_dram[2].size) {
> +		efi_add_memory_map(gd->bd->bi_dram[2].start,
> +				   gd->bd->bi_dram[2].size >>
> EFI_PAGE_SHIFT,
> +				   EFI_RESERVED_MEMORY_TYPE, false);
> +	}
> +#endif

DP-DDR is not for ARM cores.  It is for other accelerators present in LS2080A.
I will suggest not to use DP-DDR for EFI memory.

--prabhakar