Message ID | 20240319122344.3107639-3-javad.rahimipetroudi@mind.be |
---|---|
State | Accepted |
Headers | show |
Series | [v4,1/3] configs/avenger96_defconfig: bump U-Boot version to 2024.01 | expand |
On 19/03/2024 13:23, Javad Rahimipetroudi wrote: > This patch upgrades the ATF version to lts-v2.8.16. Please note that > due to DTS chages from commit 51e223058fe70b311542178f1865514745fa7874 > ("feat(stm32mp15-fdts): add Avenger96 board with STM32MP157A DHCOR SoM") > The ATF additional build variable also modified to use the new DTS file. > > Furthermore, 'E=0' flag also removed from ATF additional build > variable. As it was added by > commit deb8d71c9285eb60bc3d28e8abcf7208f78980a7 to avoid TFA build > fail because of '-Werror' flag. However, from version v2.6 or later, > it is not required anymore. > > The ATF patch > "board/arrow/avenger96/patches/arm-trusted-firmware/ > 0001-stm32mp157a-avenger96.dts-enable-hash-device-to-unbr.patch" also > has been removed. As it was not required due to using Dhcore DTS > file for the ATF build. > > Signed-off-by: Javad Rahimipetroudi <javad.rahimipetroudi@mind.be> > --- > Changes v1 -> v2: > - Upgrade ATF version to lts-2.8.16 (suggested by Arnout) > - Remove ATF patch (suggested by Peter) > - Improve commit document > - Modify commit title > [snip] > @@ -29,13 +28,13 @@ BR2_TARGET_ROOTFS_EXT2_SIZE="120M" > # Bootloaders > BR2_TARGET_ARM_TRUSTED_FIRMWARE=y > BR2_TARGET_ARM_TRUSTED_FIRMWARE_CUSTOM_VERSION=y > -BR2_TARGET_ARM_TRUSTED_FIRMWARE_CUSTOM_VERSION_VALUE="v2.6" > +BR2_TARGET_ARM_TRUSTED_FIRMWARE_CUSTOM_VERSION_VALUE="lts-v2.8.16" I changed this from custom version to BR2_TARGET_ARM_TRUSTED_FIRMWARE_LATEST_LTS_2_8_VERSION=y Note that that implies that it was downgraded to lts-v2.8.13, because that is the version we currently have. So it would be nice if you could do an update of the arm-trusted-firmware LTS version. Applied to master, thanks. Regards, Arnout > BR2_TARGET_ARM_TRUSTED_FIRMWARE_PLATFORM="stm32mp1" > BR2_TARGET_ARM_TRUSTED_FIRMWARE_FIP=y > BR2_TARGET_ARM_TRUSTED_FIRMWARE_BL31=y > BR2_TARGET_ARM_TRUSTED_FIRMWARE_UBOOT_AS_BL33=y > BR2_TARGET_ARM_TRUSTED_FIRMWARE_UBOOT_BL33_IMAGE="u-boot-nodtb.bin" > -BR2_TARGET_ARM_TRUSTED_FIRMWARE_ADDITIONAL_VARIABLES="STM32MP_SDMMC=1 AARCH32_SP=sp_min DTB_FILE_NAME=stm32mp157a-avenger96.dtb E=0 BL33_CFG=$(BINARIES_DIR)/u-boot.dtb" > +BR2_TARGET_ARM_TRUSTED_FIRMWARE_ADDITIONAL_VARIABLES="STM32MP_SDMMC=1 AARCH32_SP=sp_min DTB_FILE_NAME=stm32mp157a-dhcor-avenger96.dtb BL33_CFG=$(BINARIES_DIR)/u-boot.dtb" > BR2_TARGET_ARM_TRUSTED_FIRMWARE_IMAGES="fip.bin *.stm32" > BR2_TARGET_ARM_TRUSTED_FIRMWARE_NEEDS_DTC=y > BR2_TARGET_UBOOT=y
diff --git a/board/arrow/avenger96/patches/arm-trusted-firmware/0001-stm32mp157a-avenger96.dts-enable-hash-device-to-unbr.patch b/board/arrow/avenger96/patches/arm-trusted-firmware/0001-stm32mp157a-avenger96.dts-enable-hash-device-to-unbr.patch deleted file mode 100644 index 8cec0af3b1..0000000000 --- a/board/arrow/avenger96/patches/arm-trusted-firmware/0001-stm32mp157a-avenger96.dts-enable-hash-device-to-unbr.patch +++ /dev/null @@ -1,51 +0,0 @@ -From 336dc301e02d64507447f82020ce7a349797bef3 Mon Sep 17 00:00:00 2001 -From: Peter Korsgaard <peter@korsgaard.com> -Date: Sun, 5 Nov 2023 14:59:16 +0100 -Subject: [PATCH] stm32mp157a-avenger96.dts: enable hash device to unbreak boot - issue - -The avenger96 board was forgotten when authentication support was added with -commit 4bdb1a7a6a1325343 (stm32mp1: add authentication support for -stm32image), causing a panic when stm32mp_init_auth() is called, so fix it -similar to how it was done for the STM32MP157C-ED1 board with: - -commit b37b52ef8bc05bfd8dcca992d4ba84cd7c5d23bb -Author: Yann Gautier <yann.gautier@st.com> -Date: Tue Oct 13 18:05:06 2020 +0200 - - fdts: add missing hash node in STM32MP157C-ED1 board DT - - Without this node, the board fails to boot and panics in the function - stm32mp_init_auth(). - - Change-Id: Ia54924410dac2a8c94dd6e45d7e93977fe7d87e2 - Signed-off-by: Yann Gautier <yann.gautier@st.com> - -Upstream: N/A - Upstream reworked authentication to skip it for MP157A - variant since v2.7, see "feat(st): disable authentication based on - part_number" - (https://github.com/ARM-software/arm-trusted-firmware/commit/49abdfd8cececb91a4bc7e7b29a30c09dce461c7) - -Signed-off-by: Peter Korsgaard <peter@korsgaard.com> ---- - fdts/stm32mp157a-avenger96.dts | 4 ++++ - 1 file changed, 4 insertions(+) - -diff --git a/fdts/stm32mp157a-avenger96.dts b/fdts/stm32mp157a-avenger96.dts -index b967736e4..76edecb83 100644 ---- a/fdts/stm32mp157a-avenger96.dts -+++ b/fdts/stm32mp157a-avenger96.dts -@@ -271,6 +271,10 @@ - }; - }; - -+&hash1 { -+ status = "okay"; -+}; -+ - &rng1 { - status = "okay"; - }; --- -2.39.2 - diff --git a/configs/avenger96_defconfig b/configs/avenger96_defconfig index 218955fc02..f73b654dc8 100644 --- a/configs/avenger96_defconfig +++ b/configs/avenger96_defconfig @@ -6,7 +6,6 @@ BR2_cortex_a7=y BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_6_6=y # System configuration -BR2_GLOBAL_PATCH_DIR="board/arrow/avenger96/patches" BR2_ROOTFS_OVERLAY="board/arrow/avenger96/overlay/" BR2_ROOTFS_POST_IMAGE_SCRIPT="board/stmicroelectronics/common/stm32mp157/post-image.sh" @@ -29,13 +28,13 @@ BR2_TARGET_ROOTFS_EXT2_SIZE="120M" # Bootloaders BR2_TARGET_ARM_TRUSTED_FIRMWARE=y BR2_TARGET_ARM_TRUSTED_FIRMWARE_CUSTOM_VERSION=y -BR2_TARGET_ARM_TRUSTED_FIRMWARE_CUSTOM_VERSION_VALUE="v2.6" +BR2_TARGET_ARM_TRUSTED_FIRMWARE_CUSTOM_VERSION_VALUE="lts-v2.8.16" BR2_TARGET_ARM_TRUSTED_FIRMWARE_PLATFORM="stm32mp1" BR2_TARGET_ARM_TRUSTED_FIRMWARE_FIP=y BR2_TARGET_ARM_TRUSTED_FIRMWARE_BL31=y BR2_TARGET_ARM_TRUSTED_FIRMWARE_UBOOT_AS_BL33=y BR2_TARGET_ARM_TRUSTED_FIRMWARE_UBOOT_BL33_IMAGE="u-boot-nodtb.bin" -BR2_TARGET_ARM_TRUSTED_FIRMWARE_ADDITIONAL_VARIABLES="STM32MP_SDMMC=1 AARCH32_SP=sp_min DTB_FILE_NAME=stm32mp157a-avenger96.dtb E=0 BL33_CFG=$(BINARIES_DIR)/u-boot.dtb" +BR2_TARGET_ARM_TRUSTED_FIRMWARE_ADDITIONAL_VARIABLES="STM32MP_SDMMC=1 AARCH32_SP=sp_min DTB_FILE_NAME=stm32mp157a-dhcor-avenger96.dtb BL33_CFG=$(BINARIES_DIR)/u-boot.dtb" BR2_TARGET_ARM_TRUSTED_FIRMWARE_IMAGES="fip.bin *.stm32" BR2_TARGET_ARM_TRUSTED_FIRMWARE_NEEDS_DTC=y BR2_TARGET_UBOOT=y
This patch upgrades the ATF version to lts-v2.8.16. Please note that due to DTS chages from commit 51e223058fe70b311542178f1865514745fa7874 ("feat(stm32mp15-fdts): add Avenger96 board with STM32MP157A DHCOR SoM") The ATF additional build variable also modified to use the new DTS file. Furthermore, 'E=0' flag also removed from ATF additional build variable. As it was added by commit deb8d71c9285eb60bc3d28e8abcf7208f78980a7 to avoid TFA build fail because of '-Werror' flag. However, from version v2.6 or later, it is not required anymore. The ATF patch "board/arrow/avenger96/patches/arm-trusted-firmware/ 0001-stm32mp157a-avenger96.dts-enable-hash-device-to-unbr.patch" also has been removed. As it was not required due to using Dhcore DTS file for the ATF build. Signed-off-by: Javad Rahimipetroudi <javad.rahimipetroudi@mind.be> --- Changes v1 -> v2: - Upgrade ATF version to lts-2.8.16 (suggested by Arnout) - Remove ATF patch (suggested by Peter) - Improve commit document - Modify commit title --- ...ger96.dts-enable-hash-device-to-unbr.patch | 51 ------------------- configs/avenger96_defconfig | 5 +- 2 files changed, 2 insertions(+), 54 deletions(-) delete mode 100644 board/arrow/avenger96/patches/arm-trusted-firmware/0001-stm32mp157a-avenger96.dts-enable-hash-device-to-unbr.patch