diff mbox

[U-Boot,v3,05/28] efi: Support building a u-boot-app.efi executable

Message ID 1438713246-1887-6-git-send-email-sjg@chromium.org
State Accepted
Delegated to: Simon Glass
Headers show

Commit Message

Simon Glass Aug. 4, 2015, 6:33 p.m. UTC
Add support for building U-Boot as an EFI application with a .efi suffix.
This can be loaded by EFI provided that EFI has the same bit width (32-
or 64-bit) as U-Boot. This unfortunate limitation is imposed by EFI.

Signed-off-by: Simon Glass <sjg@chromium.org>
---

Changes in v3:
- Move the rename to u-boot-app.efi into this patch
- Update the patch subject

Changes in v2:
- Rename CONFIG_ARCH_EFI to CONFIG_EFI_APP

 Makefile | 5 +++++
 1 file changed, 5 insertions(+)

Comments

Bin Meng Aug. 5, 2015, 7:42 a.m. UTC | #1
On Wed, Aug 5, 2015 at 2:33 AM, Simon Glass <sjg@chromium.org> wrote:
> Add support for building U-Boot as an EFI application with a .efi suffix.
> This can be loaded by EFI provided that EFI has the same bit width (32-
> or 64-bit) as U-Boot. This unfortunate limitation is imposed by EFI.
>
> Signed-off-by: Simon Glass <sjg@chromium.org>
> ---
>
> Changes in v3:
> - Move the rename to u-boot-app.efi into this patch
> - Update the patch subject
>
> Changes in v2:
> - Rename CONFIG_ARCH_EFI to CONFIG_EFI_APP
>
>  Makefile | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/Makefile b/Makefile
> index 1b03357..620c18f 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -754,6 +754,7 @@ ifneq ($(CONFIG_SPL_TARGET),)
>  ALL-$(CONFIG_SPL) += $(CONFIG_SPL_TARGET:"%"=%)
>  endif
>  ALL-$(CONFIG_REMAKE_ELF) += u-boot.elf
> +ALL-$(CONFIG_EFI_APP) += u-boot-app.efi
>
>  ifneq ($(BUILD_ROM),)
>  ALL-$(CONFIG_X86_RESET_VECTOR) += u-boot.rom
> @@ -1082,6 +1083,10 @@ u-boot-dtb-tegra.bin: u-boot-nodtb-tegra.bin dts/dt.dtb FORCE
>  endif
>  endif
>
> +OBJCOPYFLAGS_u-boot-app.efi := $(OBJCOPYFLAGS_EFI)
> +u-boot-app.efi: u-boot FORCE
> +       $(call if_changed,zobjcopy)
> +
>  u-boot-img.bin: spl/u-boot-spl.bin u-boot.img FORCE
>         $(call if_changed,cat)
>
> --

Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
Simon Glass Aug. 5, 2015, 6:02 p.m. UTC | #2
On 5 August 2015 at 01:42, Bin Meng <bmeng.cn@gmail.com> wrote:
> On Wed, Aug 5, 2015 at 2:33 AM, Simon Glass <sjg@chromium.org> wrote:
>> Add support for building U-Boot as an EFI application with a .efi suffix.
>> This can be loaded by EFI provided that EFI has the same bit width (32-
>> or 64-bit) as U-Boot. This unfortunate limitation is imposed by EFI.
>>
>> Signed-off-by: Simon Glass <sjg@chromium.org>
>> ---
>>
>> Changes in v3:
>> - Move the rename to u-boot-app.efi into this patch
>> - Update the patch subject
>>
>> Changes in v2:
>> - Rename CONFIG_ARCH_EFI to CONFIG_EFI_APP
>>
>>  Makefile | 5 +++++
>>  1 file changed, 5 insertions(+)
>>
>> diff --git a/Makefile b/Makefile
>> index 1b03357..620c18f 100644
>> --- a/Makefile
>> +++ b/Makefile
>> @@ -754,6 +754,7 @@ ifneq ($(CONFIG_SPL_TARGET),)
>>  ALL-$(CONFIG_SPL) += $(CONFIG_SPL_TARGET:"%"=%)
>>  endif
>>  ALL-$(CONFIG_REMAKE_ELF) += u-boot.elf
>> +ALL-$(CONFIG_EFI_APP) += u-boot-app.efi
>>
>>  ifneq ($(BUILD_ROM),)
>>  ALL-$(CONFIG_X86_RESET_VECTOR) += u-boot.rom
>> @@ -1082,6 +1083,10 @@ u-boot-dtb-tegra.bin: u-boot-nodtb-tegra.bin dts/dt.dtb FORCE
>>  endif
>>  endif
>>
>> +OBJCOPYFLAGS_u-boot-app.efi := $(OBJCOPYFLAGS_EFI)
>> +u-boot-app.efi: u-boot FORCE
>> +       $(call if_changed,zobjcopy)
>> +
>>  u-boot-img.bin: spl/u-boot-spl.bin u-boot.img FORCE
>>         $(call if_changed,cat)
>>
>> --
>
> Reviewed-by: Bin Meng <bmeng.cn@gmail.com>

Applied to u-boot-x86.
diff mbox

Patch

diff --git a/Makefile b/Makefile
index 1b03357..620c18f 100644
--- a/Makefile
+++ b/Makefile
@@ -754,6 +754,7 @@  ifneq ($(CONFIG_SPL_TARGET),)
 ALL-$(CONFIG_SPL) += $(CONFIG_SPL_TARGET:"%"=%)
 endif
 ALL-$(CONFIG_REMAKE_ELF) += u-boot.elf
+ALL-$(CONFIG_EFI_APP) += u-boot-app.efi
 
 ifneq ($(BUILD_ROM),)
 ALL-$(CONFIG_X86_RESET_VECTOR) += u-boot.rom
@@ -1082,6 +1083,10 @@  u-boot-dtb-tegra.bin: u-boot-nodtb-tegra.bin dts/dt.dtb FORCE
 endif
 endif
 
+OBJCOPYFLAGS_u-boot-app.efi := $(OBJCOPYFLAGS_EFI)
+u-boot-app.efi: u-boot FORCE
+	$(call if_changed,zobjcopy)
+
 u-boot-img.bin: spl/u-boot-spl.bin u-boot.img FORCE
 	$(call if_changed,cat)