[Buildroot] [PATCH v2 1/2] configs/qemu_arm_vexpress_tz: bump tf-a and u-boot versions

[Buildroot] [PATCH v2 1/2] configs/qemu_arm_vexpress_tz: bump tf-a and u-boot versions

[Etienne Carriere]

Bump TF-A to version v2.10 and U-Boot to version v2024.07 for
Qemu arm-vexpress/trustzone board config.

Signed-off-by: Etienne Carriere <etienne.carriere@foss.st.com>
---
 configs/qemu_arm_vexpress_tz_defconfig | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/configs/qemu_arm_vexpress_tz_defconfig b/configs/qemu_arm_vexpress_tz_defconfig
index 900bd17fc2..52b5e62e6a 100644
--- a/configs/qemu_arm_vexpress_tz_defconfig
+++ b/configs/qemu_arm_vexpress_tz_defconfig
@@ -33,7 +33,7 @@ BR2_LINUX_KERNEL_CONFIG_FRAGMENT_FILES="board/qemu/arm-vexpress-tz/linux.fragmen
 # TF-A for booting OP-TEE secure and uboot/linux non secure
 BR2_TARGET_ARM_TRUSTED_FIRMWARE=y
 BR2_TARGET_ARM_TRUSTED_FIRMWARE_CUSTOM_VERSION=y
-BR2_TARGET_ARM_TRUSTED_FIRMWARE_CUSTOM_VERSION_VALUE="v2.7"
+BR2_TARGET_ARM_TRUSTED_FIRMWARE_CUSTOM_VERSION_VALUE="v2.10"
 BR2_TARGET_ARM_TRUSTED_FIRMWARE_PLATFORM="qemu"
 BR2_TARGET_ARM_TRUSTED_FIRMWARE_FIP=y
 BR2_TARGET_ARM_TRUSTED_FIRMWARE_BL32_OPTEE=y
@@ -59,7 +59,7 @@ BR2_ROOTFS_POST_SCRIPT_ARGS="$(BR2_DEFCONFIG)"
 BR2_TARGET_UBOOT=y
 BR2_TARGET_UBOOT_BUILD_SYSTEM_KCONFIG=y
 BR2_TARGET_UBOOT_CUSTOM_VERSION=y
-BR2_TARGET_UBOOT_CUSTOM_VERSION_VALUE="2022.04"
+BR2_TARGET_UBOOT_CUSTOM_VERSION_VALUE="2024.07"
 BR2_TARGET_UBOOT_BOARD_DEFCONFIG="qemu_arm"
 BR2_TARGET_UBOOT_CONFIG_FRAGMENT_FILES="board/qemu/arm-vexpress-tz/u-boot.config"
 
-- 
2.25.1

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

[Buildroot] [PATCH v2 2/2] board/qemu/arm-vexpress-tz: fix typos in readme file

[Etienne Carriere]

Fix typos in QEMU arm-vexpress-tz readme file where 'i.e.' occurrences
should be replaced with 'e.g.'.

Signed-off-by: Etienne Carriere <etienne.carriere@foss.st.com>
---
 board/qemu/arm-vexpress-tz/readme.txt | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/board/qemu/arm-vexpress-tz/readme.txt b/board/qemu/arm-vexpress-tz/readme.txt
index 3e84fe6391..920777afd4 100644
--- a/board/qemu/arm-vexpress-tz/readme.txt
+++ b/board/qemu/arm-vexpress-tz/readme.txt
@@ -26,7 +26,7 @@ If you want to emulate more cores, use "-smp {1|2|3|4}" to select the
 number of cores.
 
 Note: "-netdev user,id=vmnic -device virtio-net-device,netdev=vmnic"
-brings network support that is used i.e. in OP-TEE regression tests.
+brings network support that is used e.g. in OP-TEE regression tests.
 
 
 -- Boot Details --
@@ -38,7 +38,7 @@ non-secure bootloader (BL33 stage).
 QEMU natively hosts and loads in RAM the QEMU ARM target device tree. OP-TEE
 reads and modifies its content according to OP-TEE configuration.
 
-Enable TF-A traces from LOG_LEVEL (I.e LOG_LEVEL=40) from
+Enable TF-A traces from LOG_LEVEL (e.g. LOG_LEVEL=40) from
 BR2_TARGET_ARM_TRUSTED_FIRMWARE_ADDITIONAL_VARIABLES.
 
 
@@ -53,7 +53,7 @@ serial interface.
 The OP-TEE OS uses the QEMU second serial interface.
 
 To get the OP-TEE OS traces, append a second -serial argument after
--serial stdio in the QEMU command line. I.e, the following enables 2 serial
+-serial stdio in the QEMU command line. E.g., the following enables 2 serial
 consoles over telnet connections:
 
   cd output/images && ../host/bin/qemu-system-arm \
-- 
2.25.1

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH v2 1/2] configs/qemu_arm_vexpress_tz: bump tf-a and u-boot versions

[Thomas Petazzoni via buildroot]

On Tue, 23 Jul 2024 18:30:24 +0200
Etienne Carriere <etienne.carriere@foss.st.com> wrote:

> Bump TF-A to version v2.10 and U-Boot to version v2024.07 for
> Qemu arm-vexpress/trustzone board config.
> 
> Signed-off-by: Etienne Carriere <etienne.carriere@foss.st.com>
> ---
>  configs/qemu_arm_vexpress_tz_defconfig | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Thanks for the patch. However, are you sure you tested it? Indeed this
defconfig has BR2_DOWNLOAD_FORCE_CHECK_HASHES=y, which means all hashes
are checked. So when you bump components, it cannot be done without
updating hashes in board/qemu/patches/. Please note that those hash
files are shared by all qemu defconfigs. Could you check this?

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH v2 2/2] board/qemu/arm-vexpress-tz: fix typos in readme file

[Thomas Petazzoni via buildroot]

On Tue, 23 Jul 2024 18:30:25 +0200
Etienne Carriere <etienne.carriere@foss.st.com> wrote:

> Fix typos in QEMU arm-vexpress-tz readme file where 'i.e.' occurrences
> should be replaced with 'e.g.'.
> 
> Signed-off-by: Etienne Carriere <etienne.carriere@foss.st.com>
> ---
>  board/qemu/arm-vexpress-tz/readme.txt | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH v2 1/2] configs/qemu_arm_vexpress_tz: bump tf-a and u-boot versions

[Etienne CARRIERE - foss]

Hello Thomas,

On Tuesday, July 23, 2024, Thomas Petazzoni wrote:
> On Tue, 23 Jul 2024 18:30:24 +0200
> Etienne Carriere <etienne.carriere@foss.st.com> wrote:
> 
> > Bump TF-A to version v2.10 and U-Boot to version v2024.07 for
> > Qemu arm-vexpress/trustzone board config.
> >
> > Signed-off-by: Etienne Carriere <etienne.carriere@foss.st.com>
> > ---
> >  configs/qemu_arm_vexpress_tz_defconfig | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> Thanks for the patch. However, are you sure you tested it? Indeed this
> defconfig has BR2_DOWNLOAD_FORCE_CHECK_HASHES=y, which means all hashes
> are checked. So when you bump components, it cannot be done without
> updating hashes in board/qemu/patches/. Please note that those hash
> files are shared by all qemu defconfigs. Could you check this?

Sorry, indeed I realize i forgot to re-parse the defconfig after applying the change.
Bad test, my apologies.

It happens that building with these U-Boot / TF-A versions works ok.
No hash failure on U-Boot or TF-A source tree blobs but actually, the source tarball hashes for these versions are defined in relative  boot/xxx/xxx.hash.

That said, booting this platform with U-Boot 2024.07 fails. I did not expect that.
I think I'll upgrade at least to TF-A v2.10 (there's an LTS on that tag) and will postpone U-Boot version bump to later.

Regards,
Etienne

> 
> Thomas
> --
> Thomas Petazzoni, CTO, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com
> 
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH v2 2/2] board/qemu/arm-vexpress-tz: fix typos in readme file

[Peter Korsgaard]

>>>>> "Etienne" == Etienne Carriere <etienne.carriere@foss.st.com> writes:

 > Fix typos in QEMU arm-vexpress-tz readme file where 'i.e.' occurrences
 > should be replaced with 'e.g.'.

 > Signed-off-by: Etienne Carriere <etienne.carriere@foss.st.com>

Committed to 2024.02.x and 2024.05.x, thanks.

-- 
Bye, Peter Korsgaard
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot