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

[Alexander Graf]

> Am 22.06.2016 um 04:51 schrieb Prabhakar Kushwaha <prabhakar.kushwaha at nxp.com>:
> 
> 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.

That's exactly what this patch does :). It tells marks the DP-DDR as reserved in the memory map.

Alex