[U-Boot] [PATCH v6 0/6] LS2080ARDB: Enable EFI boot support

[Alexander Graf]

On 16/11/2016 22:57, york sun wrote:
> On 11/15/2016 02:20 PM, Alexander Graf wrote:
>>
>>
>> On 15/11/2016 23:11, york sun wrote:
>>>
>>> On 11/15/2016 12:36 PM, Alexander Graf wrote:
>>>> Hi York,
>>>>
>>>> Do you want to take that patch set?
>>>>
>>>
>>> Yes, I can. I saw it assigned to you in patchwork so I didn't act.
>>
>> I think since it's really hardware enablement for your platform, I'd
>> prefer if you pull them all in.
>>
>>
>
> Alex,
>
> I got compiling warnings
>
> 19: ls2080: Exit dpaa only right before exiting U-Boot
> 20: efi_loader: AArch64: Run EFI payloads in EL2 if U-Boot runs in EL3
> 21: ls2080ardb: Reserve DP-DDR RAM
>     aarch64:  +   ls2080ardb_nand
> +(ls2080ardb_nand)    efi_add_memory_map(gd->bd->bi_dram[2].start,
> +(ls2080ardb_nand)    ^
> w+(ls2080ardb_nand) ../board/freescale/ls2080ardb/ls2080ardb.c: In
> function 'misc_init_r':
> w+(ls2080ardb_nand) ../board/freescale/ls2080ardb/ls2080ardb.c:207:3:
> warning: implicit declaration of function 'efi_add_memory_map'
> [-Wimplicit-function-declaration]

Let's start with the first one :).

Is this doing an SPL build? The header has a check for

#if defined(CONFIG_EFI_LOADER) && !defined(CONFIG_SPL_BUILD)

while the using code is only conditional on

#ifdef CONFIG_EFI_LOADER


Alex