[Buildroot] [PATCH v2] uboot: add support for bundling ATF BL31 in ELF format into U-Boot
Baruch Siach
baruch at tkos.co.il
Sun Nov 11 16:56:17 UTC 2018
Hi Michał,
Michał Łyszczek writes:
> Rockchip's rk3328 encapsulates the BL31 part of ATP but it needs
> image in ELF format. This is extension to
> 2c4809e6088660dc71de9de8cee0bba312aafbe4
>
> This commit introduces a BR2_TARGET_UBOOT_NEEDS_ATF_BL31_ELF variable
> to achieve this.
>
> Signed-off-by: Michał Łyszczek <michal.lyszczek at bofc.pl>
> Tested-by: Michał Łyszczek <michal.lyszczek at bofc.pl>
>
> ---
> Changes v1 -> v2:
> - fix wrong prefix BR_ to BR2_
> ---
> boot/arm-trusted-firmware/arm-trusted-firmware.mk | 7 +++++++
> boot/uboot/Config.in | 7 +++++++
> boot/uboot/uboot.mk | 4 ++++
> 3 files changed, 18 insertions(+)
>
> diff --git a/boot/arm-trusted-firmware/arm-trusted-firmware.mk b/boot/arm-trusted-firmware/arm-trusted-firmware.mk
> index 23f493653d..8f0f745ac0 100644
> --- a/boot/arm-trusted-firmware/arm-trusted-firmware.mk
> +++ b/boot/arm-trusted-firmware/arm-trusted-firmware.mk
> @@ -94,6 +94,12 @@ ARM_TRUSTED_FIRMWARE_MAKE_OPTS += RESET_TO_BL31=1
> ARM_TRUSTED_FIRMWARE_DEPENDENCIES += host-uboot-tools
> endif
>
> +ifeq ($(BR2_TARGET_UBOOT_NEEDS_ATF_BL31_ELF),y)
> +define ARM_TRUSTED_FIRMWARE_BL31_UBOOT_INSTALL_ELF
> + cp -dpf $(ARM_TRUSTED_FIRMWARE_IMG_DIR)/bl31/bl31.elf $(BINARIES_DIR)/
> +endef
> +endif
Why not add this 'ifeq' into ARM_TRUSTED_FIRMWARE_BL31_UBOOT_INSTALL ...
> +
> define ARM_TRUSTED_FIRMWARE_BUILD_CMDS
> $(ARM_TRUSTED_FIRMWARE_BUILD_FIPTOOL)
> $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) $(ARM_TRUSTED_FIRMWARE_MAKE_OPTS) \
> @@ -104,6 +110,7 @@ endef
> define ARM_TRUSTED_FIRMWARE_INSTALL_IMAGES_CMDS
> cp -dpf $(ARM_TRUSTED_FIRMWARE_IMG_DIR)/*.bin $(BINARIES_DIR)/
> $(ARM_TRUSTED_FIRMWARE_BL31_UBOOT_INSTALL)
> + $(ARM_TRUSTED_FIRMWARE_BL31_UBOOT_INSTALL_ELF)
... and drop this part.
> endef
>
> # Configuration check
> diff --git a/boot/uboot/Config.in b/boot/uboot/Config.in
> index ac6f8bc8c1..7f2f7945ac 100644
> --- a/boot/uboot/Config.in
> +++ b/boot/uboot/Config.in
> @@ -179,6 +179,13 @@ config BR2_TARGET_UBOOT_NEEDS_ATF_BL31
> variable pointing to ATF's BL31 binary, is passed during the
> Buildroot build.
>
> +config BR2_TARGET_UBOOT_NEEDS_ATF_BL31_ELF
> + bool "U-Boot needs bl31.elf"
> + depends on BR2_TARGET_UBOOT_NEEDS_ATF_BL31
> + help
> + Uses bl31.elf instead of bl31.bin image. This is needed by some
> + platforms (like rk3328) to generate valid uboot.itb with ATF
> +
> menu "U-Boot binary format"
>
> config BR2_TARGET_UBOOT_FORMAT_AIS
> diff --git a/boot/uboot/uboot.mk b/boot/uboot/uboot.mk
> index cbdfee6ac3..06efd7b052 100644
> --- a/boot/uboot/uboot.mk
> +++ b/boot/uboot/uboot.mk
> @@ -137,8 +137,12 @@ UBOOT_MAKE_OPTS += \
>
> ifeq ($(BR2_TARGET_UBOOT_NEEDS_ATF_BL31),y)
> UBOOT_DEPENDENCIES += arm-trusted-firmware
> +ifeq ($(BR2_TARGET_UBOOT_NEEDS_ATF_BL31_ELF),y)
> +UBOOT_MAKE_OPTS += BL31=$(BINARIES_DIR)/bl31.elf
> +else
> UBOOT_MAKE_OPTS += BL31=$(BINARIES_DIR)/bl31.bin
> endif
> +endif
>
> ifeq ($(BR2_TARGET_UBOOT_NEEDS_DTC),y)
> UBOOT_DEPENDENCIES += host-dtc
baruch
--
http://baruch.siach.name/blog/ ~. .~ Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
- baruch at tkos.co.il - tel: +972.52.368.4656, http://www.tkos.co.il -
More information about the buildroot
mailing list