On Sun, Aug 7, 2016 at 7:23 AM, Simon Glass <s...@chromium.org> wrote:
> It is useful to have a basic sanity check for EFI loader support. Add a
> 'bootefi hello' command which loads HelloWord.efi and runs it under U-Boot.
>
> Signed-off-by: Simon Glass <s...@chromium.org>
> ---
>
>  arch/x86/lib/HelloWorld.efi | Bin 0 -> 11168 bytes
>  arch/x86/lib/Makefile       |   1 +
>  2 files changed, 1 insertion(+)
>  create mode 100644 arch/x86/lib/HelloWorld.efi

Should we rename this to HelloWorld32.efi, just like ARM?

>

[snip]

> diff --git a/arch/x86/lib/Makefile b/arch/x86/lib/Makefile
> index e46e7f1..e8172cb 100644
> --- a/arch/x86/lib/Makefile
> +++ b/arch/x86/lib/Makefile
> @@ -12,6 +12,7 @@ obj-$(CONFIG_CMD_BOOTM) += bootm.o
>  obj-y  += cmd_boot.o
>  obj-$(CONFIG_SEABIOS) += coreboot_table.o
>  obj-$(CONFIG_EFI) += efi/
> +obj-$(CONFIG_CMD_BOOTEFI_HELLO) += efi_hello_world.o

This should be HelloWorld.o, or if we do like ARM, HelloWorld32.o

>  obj-y  += e820.o
>  obj-y  += gcc.o
>  obj-y  += init_helpers.o
> --

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

Reply via email to