[U-Boot] [PATCH 00/48] Add support for running U-Boot as an EFI payload/application

[Simon Glass]

[trimming the list - wow a lot of people!]

Hi Bin,

On 23 July 2015 at 10:05, Bin Meng <bmeng.cn at gmail.com> wrote:
> Hi Simon,
>
> On Wed, Jul 22, 2015 at 11:48 PM, Simon Glass <sjg at chromium.org> wrote:
>> This series allows U-Boot to be built as an EFI payload so that U-Boot
>> can be started on almost any x86 platform that supports EFI. This is
>> implemented as a stub which EFI can load plus a payload that is copied
>> to RAM. The payload contains a normal U-Boot binary image and device tree.
>>
>> This allows U-Boot to run on platforms that have EFI support but are not
>> supported natively by U-Boot. It also allows testing and fiddling with the
>> board using U-Boots memory display and other commands.
>>
>> In addition, U-Boot can be built as an EFI application. This should work
>> regardless of the board type since it only relies on EFI services. However
>> only 32-bit EFI is supported in this case. Again it can be used to snoop
>> around the platform.
>>
>> A README provides further details of how this series operates.
>>
>
> This is really great work! Thank you.
>
> So far I've finished reviewing the first 33 patch in the series, which
> covers the u-boot efi application support. After I started to look at
> the efi payload support whening review patch#34, I came up with
> another solution[1] to create efi payload which is different from
> yours. My version does not touch ASFLAGS_REMOVE for u-boot-dtb.o hence
> I believe it can be easily portable to ARM.
>
> I've tested u-boot.efi on Intel Crown Bay and QEMU, and
> u-boot-payload.efi on QEMU 64-bit. They work like a charm. It's
> amazing work!
>
> I will continue reviewing the remaining patches tomorrow.
>
> [1]: http://patchwork.ozlabs.org/patch/499458/

Thanks very much! I'm pleased you think it is useful. The
implementation certainly has opportunities for improvement but I think
it's a good solid start.

Also thanks for your reviews, very much appreciated. I know it is a
lot of work to go through such a large series, but it will produce a
better result with fewer patches!

>
> [snip]
>
> Regards,
> Bin

Regards,
Simon