Hi Alex,

> -----Original Message-----
> From: Alexander Graf [mailto:ag...@suse.de]
> Sent: Tuesday, June 21, 2016 4:37 AM
> To: u-boot@lists.denx.de
> Cc: york sun <york....@nxp.com>; Prabhakar Kushwaha
> <prabhakar.kushw...@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 <ag...@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

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to