Hi Bin, On 9 August 2016 at 00:50, Bin Meng <bmeng...@gmail.com> wrote: > Hi Simon, > > 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/arm/lib/HelloWorld32.efi | Bin 0 -> 11712 bytes >> arch/arm/lib/Makefile | 6 ++++++ >> cmd/Kconfig | 10 ++++++++++ >> cmd/bootefi.c | 26 ++++++++++++++++++++------ >> include/asm-generic/sections.h | 2 ++ >> scripts/Makefile.lib | 19 +++++++++++++++++++ >> 6 files changed, 57 insertions(+), 6 deletions(-) >> create mode 100644 arch/arm/lib/HelloWorld32.efi >> > > [snip] > >> diff --git a/arch/arm/lib/Makefile b/arch/arm/lib/Makefile >> index a8d1557..0f3ea0c 100644 >> --- a/arch/arm/lib/Makefile >> +++ b/arch/arm/lib/Makefile >> @@ -29,6 +29,12 @@ obj-$(CONFIG_OF_LIBFDT) += bootm-fdt.o >> obj-$(CONFIG_CMD_BOOTM) += bootm.o >> obj-$(CONFIG_CMD_BOOTM) += zimage.o >> obj-$(CONFIG_SYS_L2_PL310) += cache-pl310.o >> +ifdef CONFIG_ARM64 >> +# This option does not work for arm64, as there is no binary. > > If so, can we just remove this for arm64?
Actually I was hoping that Alexander might have a suitable arm64 HelloWorld.efi lying around. When I tried building UEFI for arm64, for some reason it did not create it. > >> +obj-$(CONFIG_CMD_BOOTEFI_HELLO) += HelloWorld64.o >> +else >> +obj-$(CONFIG_CMD_BOOTEFI_HELLO) += HelloWorld32.o >> +endif >> obj-$(CONFIG_USE_ARCH_MEMSET) += memset.o >> obj-$(CONFIG_USE_ARCH_MEMCPY) += memcpy.o >> else [...] Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot