diff mbox series

[RESEND,v2,05/11] configs/ti_am62x_sk_defconfig: update TFA to v2.10

Message ID 20240430-optee-update-v2-5-5bdc2f940db1@ti.com
State Accepted
Headers show
Series update bsp binaries for TI SoCs | expand

Commit Message

Bryan Brattlof May 2, 2024, 4:26 p.m. UTC 
Update the TFA firmware to the latest LTS tag for v2.10 release of
arm-trusted-firmware

Signed-off-by: Bryan Brattlof <bb@ti.com>
---
 .../ti/am62x-sk/patches/arm-trusted-firmware/arm-trusted-firmware.hash  | 2 +-
 configs/ti_am62x_sk_defconfig                                           | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

Comments

Thomas Petazzoni May 6, 2024, 8:10 p.m. UTC | #1 
On Thu, 2 May 2024 11:26:55 -0500
Bryan Brattlof via buildroot <buildroot@buildroot.org> wrote:

> Update the TFA firmware to the latest LTS tag for v2.10 release of
> arm-trusted-firmware
> 
> Signed-off-by: Bryan Brattlof <bb@ti.com>
> ---
>  .../ti/am62x-sk/patches/arm-trusted-firmware/arm-trusted-firmware.hash  | 2 +-
>  configs/ti_am62x_sk_defconfig                                           | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)

The update of TF-A/U-Boot/Linux in one defconfig could be done in one
single commit, so you could have done:

  configs/ti_am62x_sk: update Linux, U-Boot, TF-A
  configs/ti_am64x_sk: update Linux, U-Boot, TF-A

This is generally considered a good enough split.

Thomas
Bryan Brattlof May 8, 2024, 12:05 p.m. UTC | #2 
On May  6, 2024 thus sayeth Thomas Petazzoni via buildroot:
> On Thu, 2 May 2024 11:26:55 -0500
> Bryan Brattlof via buildroot <buildroot@buildroot.org> wrote:
> 
> > Update the TFA firmware to the latest LTS tag for v2.10 release of
> > arm-trusted-firmware
> > 
> > Signed-off-by: Bryan Brattlof <bb@ti.com>
> > ---
> >  .../ti/am62x-sk/patches/arm-trusted-firmware/arm-trusted-firmware.hash  | 2 +-
> >  configs/ti_am62x_sk_defconfig                                           | 2 +-
> >  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> The update of TF-A/U-Boot/Linux in one defconfig could be done in one
> single commit, so you could have done:
> 
>   configs/ti_am62x_sk: update Linux, U-Boot, TF-A
>   configs/ti_am64x_sk: update Linux, U-Boot, TF-A
> 
> This is generally considered a good enough split.
> 

Ah that's a good balance. Thank you

~Bryan
diff mbox series

Patch

diff --git a/board/ti/am62x-sk/patches/arm-trusted-firmware/arm-trusted-firmware.hash b/board/ti/am62x-sk/patches/arm-trusted-firmware/arm-trusted-firmware.hash
index 74fe35c9bc733..8f60864fbf67f 100644
--- a/board/ti/am62x-sk/patches/arm-trusted-firmware/arm-trusted-firmware.hash
+++ b/board/ti/am62x-sk/patches/arm-trusted-firmware/arm-trusted-firmware.hash
@@ -1,2 +1,2 @@ 
 # Locally computed:
-sha256  327c65b1bc231608a7a808b068b00c1a22310e9fc86158813cd10a9711d5725e  arm-trusted-firmware-v2.7.tar.gz
+sha256  ba215404fe9db26e5f2cef3fdce17b7c8ed344a2a1d592dd01a5f1c5e72cfdbd  arm-trusted-firmware-lts-v2.10.4.tar.gz
diff --git a/configs/ti_am62x_sk_defconfig b/configs/ti_am62x_sk_defconfig
index 405016818438a..e169b6834d89c 100644
--- a/configs/ti_am62x_sk_defconfig
+++ b/configs/ti_am62x_sk_defconfig
@@ -19,7 +19,7 @@  BR2_TARGET_ROOTFS_EXT2_4=y
 BR2_TARGET_ROOTFS_EXT2_SIZE="256M"
 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="lts-v2.10.4"
 BR2_TARGET_ARM_TRUSTED_FIRMWARE_PLATFORM="k3"
 BR2_TARGET_ARM_TRUSTED_FIRMWARE_TARGET_BOARD="lite"
 BR2_TARGET_ARM_TRUSTED_FIRMWARE_BL32_OPTEE=y