mbox series

[RESEND,v2,00/11] update bsp binaries for TI SoCs

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

Message

Bryan Brattlof May 2, 2024, 4:26 p.m. UTC 
Hello everyone!

My apologies to everyone I just spammed with the first v2. I dropped the 
mailing list (still setting up my b4 configs) :/

This series updates all the boot binaries (U-Boot, OP-TEE, TF-A, and 
Linux) to their latest versions to pull in all bug fixes and new 
features additions in these projects.

This also includes a patch I send out a while ago just to keep things 
organized. 

  https://lore.kernel.org/buildroot/20240402153328.1859642-1-bb@ti.com/

and finally some proof I didn't completely break something ;)

  https://paste.sr.ht/~bryanb/b666adef7da412de76efb5e4a68e68e0651f7928

Thanks for reviewing
~Bryan

Signed-off-by: Bryan Brattlof <bb@ti.com>
---
Changes in v2:
- most emails never made it to the mailing list
- updated commit messages to be more explicit about versions picked
- Link to v1: https://lore.kernel.org/r/20240430-optee-update-v1-0-6f73c4903ec3@ti.com

---
Bryan Brattlof (11):
      boot/optee-os: bump to release v4.2.0
      boot/uboot: bump to version v2024.04
      boot/ti-k3-r5-loader: bump to version v2024.04
      boot/ti-k3-boot-firmware: bump to version 09.02.00.009
      configs/ti_am62x_sk_defconfig: update TFA to v2.10
      configs/ti_am64x_sk_defconfig: update TFA to v2.10
      configs/ti_am62x_sk_defconfig: update U-Boot to v2024.04
      configs/ti_am64x_sk_defconfig: update U-Boot to v2024.04
      configs/ti_am62x_sk_defconfig: bump kernel to v6.8.8
      configs/ti_am64x_sk_defconfig: update kernel to v6.8.8
      DEVELOPERS: add myself to TI's packages and reference boards

 DEVELOPERS                                                       | 7 +++++++
 .../patches/arm-trusted-firmware/arm-trusted-firmware.hash       | 2 +-
 board/ti/am62x-sk/patches/linux/linux.hash                       | 2 +-
 .../patches/arm-trusted-firmware/arm-trusted-firmware.hash       | 2 +-
 board/ti/am64x-sk/patches/linux/linux.hash                       | 2 +-
 boot/optee-os/Config.in                                          | 4 ++--
 boot/optee-os/optee-os.hash                                      | 4 ++--
 boot/ti-k3-boot-firmware/ti-k3-boot-firmware.hash                | 2 +-
 boot/ti-k3-boot-firmware/ti-k3-boot-firmware.mk                  | 4 +---
 boot/ti-k3-r5-loader/Config.in                                   | 4 ++--
 boot/ti-k3-r5-loader/ti-k3-r5-loader.hash                        | 2 +-
 boot/ti-k3-r5-loader/ti-k3-r5-loader.mk                          | 1 +
 boot/uboot/Config.in                                             | 6 +++---
 boot/uboot/uboot.hash                                            | 2 +-
 configs/ti_am62x_sk_defconfig                                    | 9 ++++-----
 configs/ti_am64x_sk_defconfig                                    | 9 ++++-----
 16 files changed, 33 insertions(+), 29 deletions(-)
---
base-commit: 929bffbf4bf8fcb91bda446777f907f32c782656
change-id: 20240430-optee-update-b9c1bc4e9f29

Best regards,

Comments

Andreas Dannenberg May 6, 2024, 8:48 a.m. UTC | #1 
Hi Bryan,

On Thu, May 02, 2024 at 11:26:50AM -0500, Bryan Brattlof wrote:
> Hello everyone!
> 
> My apologies to everyone I just spammed with the first v2. I dropped the 
> mailing list (still setting up my b4 configs) :/
> 
> This series updates all the boot binaries (U-Boot, OP-TEE, TF-A, and 
> Linux) to their latest versions to pull in all bug fixes and new 
> features additions in these projects.
> 
> This also includes a patch I send out a while ago just to keep things 
> organized. 
> 
>   https://lore.kernel.org/buildroot/20240402153328.1859642-1-bb@ti.com/
> 
> and finally some proof I didn't completely break something ;)
> 
>   https://paste.sr.ht/~bryanb/b666adef7da412de76efb5e4a68e68e0651f7928

I tested the series applied on top of today's HEAD commit...

66ebbebdb4 ("package/stress-ng: fix uclibc-ng build")

...and was able to boot successfully on both SK-AM62B and SK-AM64B EVMs
(for record, both are HS-FS production-silicon EVMs). U-Boot, Kernel,
ATF, OP-TEE, and TIFS/DM FW versions are all as advertised. I also
tested networking to make sure it works. Boot logs look clean, except
the TISCK clock related issues that are also visible in the log you
shared...

<snip>
[    2.673148] ti-sci-clk 44043000.system-controller:clock-controller: is_prepared failed for dev=40, clk=9, ret=-19
[    2.683736] ti-sci-clk 44043000.system-controller:clock-controller: is_prepared failed for dev=39, clk=18, ret=-19
[    2.694116] ti-sci-clk 44043000.system-controller:clock-controller: is_prepared failed for dev=39, clk=17, ret=-19
[    2.704494] ti-sci-clk 44043000.system-controller:clock-controller: is_prepared failed for dev=39, clk=16, ret=-19
<snip>

As per earlier discussion I know it needs a patch [1] that's not yet available
in the Kernel tree we are using. But I think it would be nice if we could add
this a patch, this would help to clean up the boot logs qute a bit. A lot of
"failed" messages will be concerning to users.

For the series:

Tested-by: Andreas Dannenberg <dannenberg@ti.com>

[1] https://github.com/torvalds/linux/commit/ad3ac13c6ec318b43e769cc9ffde67528e58e555

> 
> Thanks for reviewing
> ~Bryan
> 
> Signed-off-by: Bryan Brattlof <bb@ti.com>
> ---
> Changes in v2:
> - most emails never made it to the mailing list
> - updated commit messages to be more explicit about versions picked
> - Link to v1: https://lore.kernel.org/r/20240430-optee-update-v1-0-6f73c4903ec3@ti.com
> 
> ---
> Bryan Brattlof (11):
>       boot/optee-os: bump to release v4.2.0
>       boot/uboot: bump to version v2024.04
>       boot/ti-k3-r5-loader: bump to version v2024.04
>       boot/ti-k3-boot-firmware: bump to version 09.02.00.009
>       configs/ti_am62x_sk_defconfig: update TFA to v2.10
>       configs/ti_am64x_sk_defconfig: update TFA to v2.10
>       configs/ti_am62x_sk_defconfig: update U-Boot to v2024.04
>       configs/ti_am64x_sk_defconfig: update U-Boot to v2024.04
>       configs/ti_am62x_sk_defconfig: bump kernel to v6.8.8
>       configs/ti_am64x_sk_defconfig: update kernel to v6.8.8
>       DEVELOPERS: add myself to TI's packages and reference boards
> 
>  DEVELOPERS                                                       | 7 +++++++
>  .../patches/arm-trusted-firmware/arm-trusted-firmware.hash       | 2 +-
>  board/ti/am62x-sk/patches/linux/linux.hash                       | 2 +-
>  .../patches/arm-trusted-firmware/arm-trusted-firmware.hash       | 2 +-
>  board/ti/am64x-sk/patches/linux/linux.hash                       | 2 +-
>  boot/optee-os/Config.in                                          | 4 ++--
>  boot/optee-os/optee-os.hash                                      | 4 ++--
>  boot/ti-k3-boot-firmware/ti-k3-boot-firmware.hash                | 2 +-
>  boot/ti-k3-boot-firmware/ti-k3-boot-firmware.mk                  | 4 +---
>  boot/ti-k3-r5-loader/Config.in                                   | 4 ++--
>  boot/ti-k3-r5-loader/ti-k3-r5-loader.hash                        | 2 +-
>  boot/ti-k3-r5-loader/ti-k3-r5-loader.mk                          | 1 +
>  boot/uboot/Config.in                                             | 6 +++---
>  boot/uboot/uboot.hash                                            | 2 +-
>  configs/ti_am62x_sk_defconfig                                    | 9 ++++-----
>  configs/ti_am64x_sk_defconfig                                    | 9 ++++-----
>  16 files changed, 33 insertions(+), 29 deletions(-)
> ---
> base-commit: 929bffbf4bf8fcb91bda446777f907f32c782656
> change-id: 20240430-optee-update-b9c1bc4e9f29
> 
> Best regards,
> -- 
> Bryan Brattlof <bb@ti.com>
>
Bryan Brattlof May 6, 2024, 2:42 p.m. UTC | #2 
Hi Andreas!

On May  6, 2024 thus sayeth Andreas Dannenberg via buildroot:
> Hi Bryan,
> 
> On Thu, May 02, 2024 at 11:26:50AM -0500, Bryan Brattlof wrote:
> > Hello everyone!
> > 
> > My apologies to everyone I just spammed with the first v2. I dropped the 
> > mailing list (still setting up my b4 configs) :/
> > 
> > This series updates all the boot binaries (U-Boot, OP-TEE, TF-A, and 
> > Linux) to their latest versions to pull in all bug fixes and new 
> > features additions in these projects.
> > 
> > This also includes a patch I send out a while ago just to keep things 
> > organized. 
> > 
> >   https://lore.kernel.org/buildroot/20240402153328.1859642-1-bb@ti.com/
> > 
> > and finally some proof I didn't completely break something ;)
> > 
> >   https://paste.sr.ht/~bryanb/b666adef7da412de76efb5e4a68e68e0651f7928
> 
> I tested the series applied on top of today's HEAD commit...
> 
> 66ebbebdb4 ("package/stress-ng: fix uclibc-ng build")
> 
> ...and was able to boot successfully on both SK-AM62B and SK-AM64B EVMs
> (for record, both are HS-FS production-silicon EVMs). U-Boot, Kernel,
> ATF, OP-TEE, and TIFS/DM FW versions are all as advertised. I also
> tested networking to make sure it works. Boot logs look clean, except
> the TISCK clock related issues that are also visible in the log you
> shared...
> 
> <snip>
> [    2.673148] ti-sci-clk 44043000.system-controller:clock-controller: is_prepared failed for dev=40, clk=9, ret=-19
> [    2.683736] ti-sci-clk 44043000.system-controller:clock-controller: is_prepared failed for dev=39, clk=18, ret=-19
> [    2.694116] ti-sci-clk 44043000.system-controller:clock-controller: is_prepared failed for dev=39, clk=17, ret=-19
> [    2.704494] ti-sci-clk 44043000.system-controller:clock-controller: is_prepared failed for dev=39, clk=16, ret=-19
> <snip>
> 
> As per earlier discussion I know it needs a patch [1] that's not yet available
> in the Kernel tree we are using. But I think it would be nice if we could add
> this a patch, this would help to clean up the boot logs qute a bit. A lot of
> "failed" messages will be concerning to users.
> 
> For the series:
> 
> Tested-by: Andreas Dannenberg <dannenberg@ti.com>
> 
> [1] https://github.com/torvalds/linux/commit/ad3ac13c6ec318b43e769cc9ffde67528e58e555
> 

That's not a bad idea! All K3 will need this patch so it makes sense to 
backport this into our board/ patches. I'll add that in in v2.

~Bryan

> > 
> > Thanks for reviewing
> > ~Bryan
> > 
> > Signed-off-by: Bryan Brattlof <bb@ti.com>
> > ---
> > Changes in v2:
> > - most emails never made it to the mailing list
> > - updated commit messages to be more explicit about versions picked
> > - Link to v1: https://lore.kernel.org/r/20240430-optee-update-v1-0-6f73c4903ec3@ti.com
> > 
> > ---
> > Bryan Brattlof (11):
> >       boot/optee-os: bump to release v4.2.0
> >       boot/uboot: bump to version v2024.04
> >       boot/ti-k3-r5-loader: bump to version v2024.04
> >       boot/ti-k3-boot-firmware: bump to version 09.02.00.009
> >       configs/ti_am62x_sk_defconfig: update TFA to v2.10
> >       configs/ti_am64x_sk_defconfig: update TFA to v2.10
> >       configs/ti_am62x_sk_defconfig: update U-Boot to v2024.04
> >       configs/ti_am64x_sk_defconfig: update U-Boot to v2024.04
> >       configs/ti_am62x_sk_defconfig: bump kernel to v6.8.8
> >       configs/ti_am64x_sk_defconfig: update kernel to v6.8.8
> >       DEVELOPERS: add myself to TI's packages and reference boards
> > 
> >  DEVELOPERS                                                       | 7 +++++++
> >  .../patches/arm-trusted-firmware/arm-trusted-firmware.hash       | 2 +-
> >  board/ti/am62x-sk/patches/linux/linux.hash                       | 2 +-
> >  .../patches/arm-trusted-firmware/arm-trusted-firmware.hash       | 2 +-
> >  board/ti/am64x-sk/patches/linux/linux.hash                       | 2 +-
> >  boot/optee-os/Config.in                                          | 4 ++--
> >  boot/optee-os/optee-os.hash                                      | 4 ++--
> >  boot/ti-k3-boot-firmware/ti-k3-boot-firmware.hash                | 2 +-
> >  boot/ti-k3-boot-firmware/ti-k3-boot-firmware.mk                  | 4 +---
> >  boot/ti-k3-r5-loader/Config.in                                   | 4 ++--
> >  boot/ti-k3-r5-loader/ti-k3-r5-loader.hash                        | 2 +-
> >  boot/ti-k3-r5-loader/ti-k3-r5-loader.mk                          | 1 +
> >  boot/uboot/Config.in                                             | 6 +++---
> >  boot/uboot/uboot.hash                                            | 2 +-
> >  configs/ti_am62x_sk_defconfig                                    | 9 ++++-----
> >  configs/ti_am64x_sk_defconfig                                    | 9 ++++-----
> >  16 files changed, 33 insertions(+), 29 deletions(-)
> > ---
> > base-commit: 929bffbf4bf8fcb91bda446777f907f32c782656
> > change-id: 20240430-optee-update-b9c1bc4e9f29
> > 
> > Best regards,
> > -- 
> > Bryan Brattlof <bb@ti.com>
> > 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot
Thomas Petazzoni May 6, 2024, 8:07 p.m. UTC | #3 
Hello Bryan,

On Thu, 2 May 2024 11:26:50 -0500
Bryan Brattlof via buildroot <buildroot@buildroot.org> wrote:

> Bryan Brattlof (11):
>       boot/optee-os: bump to release v4.2.0
>       boot/uboot: bump to version v2024.04
>       boot/ti-k3-r5-loader: bump to version v2024.04
>       boot/ti-k3-boot-firmware: bump to version 09.02.00.009
>       configs/ti_am62x_sk_defconfig: update TFA to v2.10
>       configs/ti_am64x_sk_defconfig: update TFA to v2.10
>       configs/ti_am62x_sk_defconfig: update U-Boot to v2024.04
>       configs/ti_am64x_sk_defconfig: update U-Boot to v2024.04
>       configs/ti_am62x_sk_defconfig: bump kernel to v6.8.8
>       configs/ti_am64x_sk_defconfig: update kernel to v6.8.8
>       DEVELOPERS: add myself to TI's packages and reference boards

Thanks, entire series applied! I'll reply individually to a few of the
patches with a few remarks/comments (not much).

Best regards,

Thomas