[v8,00/23] Add support for AM62x-SK HS-FS devices
mbox series

Message ID 20240301133620.3547430-1-dario.binacchi@amarulasolutions.com
Headers show
Series
  • Add support for AM62x-SK HS-FS devices
Related show

Message

Dario Binacchi March 1, 2024, 1:35 p.m. UTC
The SK-AM62B-P1 is the newest version of SK-AM62 which includes
high-security field-securable (HS-FS) silicon to optionally customize keys
and encryption for security applications.
This requires enabling the BR2_TARGET_TI_K3_IMAGE_GEN_SECTYPE_HS_FS option,
which, however, was buggy. This series fixes the errors and bumps the Linux
kernel and U-Boot required for such a new device.

Furthermore, this version complete the series by updating the
ti_am64x_sk_defconfig too and remove ti-k3-image-gen, as requested by
Romain Naour:
https://patchwork.ozlabs.org/project/buildroot/patch/20240224205654.1546744-17-dario.binacchi@amarulasolutions.com/
I have successfully executed compilation tests for all possible combinations of platform and security type:
am62x -> GP, HS, HS-FS
am64x -> GP, HS, HS-FS

The AM62x HS-FS configuration has also been tested on the SK-AM62B-P1 board.

Dario Binacchi (22):
  configs/ti_am62x_sk_defconfig: fix BR2_TARGET_OPTEE_OS_PLATFORM
    setting
  configs/ti_am64x_sk_defconfig: fix BR2_TARGET_OPTEE_OS_PLATFORM
    setting
  configs/ti_am62x_sk_defconfig: explicitly set the ti-k3-r5-loader
    version
  configs/ti_am64x_sk_defconfig: explicitly set the ti-k3-r5-loader
    version
  boot/ti-k3-r5-loader: bump to version 2024.01
  package/python-attrs: add host variant
  package/python-rpds-py: add host variant
  package/python-referencing: add host variant
  package/python-jsonschema-specifications: add host variant
  package/python-jsonschema: add host variant
  boot/ti-k3-r5-loader: set binman environment
  boot/uboot: add BR2_TARGET_UBOOT_USE_BINMAN option
  package/ti-k3: move TI_K3_{SECTYPE,SOC} out of the image-gen scope
  package/ti-k3: disable HS-FS for AM65 SOC
  boot/ti-k3/ti-k3-r5-loader: install tiboot3.bin
  configs/ti_am62x_sk_defconfig: bump U-Boot version to 2024.01
  configs/ti_am62x_sk_defconfig: bump Linux version to 6.6.18
  board/ti/am62x-sk: generalize post-build.sh
  board/ti/am62x-sk: move post-{build,image}.sh to board/ti/common/am6xx
  package/ti-k3: fix BR2_PACKAGE_TI_K3_SOC setting for am64x SOC
  configs/ti_am64x_sk_defconfig: bump U-Boot version to 2024.01
  configs/ti_am64x_sk_defconfig: bump Linux version to 6.6.18

Romain Naour (1):
  boot/ti-k3-image-gen: remove package

 Config.in.legacy                              |  7 ++
 DEVELOPERS                                    |  2 -
 board/ti/am62x-sk/genimage.cfg                |  7 +-
 board/ti/am64x-sk/genimage.cfg                |  7 +-
 board/ti/common/am6xx/post-build.sh           | 58 ++++++++++++++
 board/ti/common/am6xx/post-image.sh           | 11 +++
 boot/Config.in                                |  1 -
 boot/ti-k3-image-gen/Config.in                | 77 -------------------
 boot/ti-k3-image-gen/ti-k3-image-gen.hash     |  3 -
 boot/ti-k3-image-gen/ti-k3-image-gen.mk       | 54 -------------
 boot/ti-k3-r5-loader/Config.in                |  5 +-
 boot/ti-k3-r5-loader/ti-k3-r5-loader.hash     |  2 +-
 boot/ti-k3-r5-loader/ti-k3-r5-loader.mk       | 12 ++-
 boot/uboot/Config.in                          | 23 +++---
 boot/uboot/uboot.mk                           | 12 ++-
 configs/ti_am62x_sk_defconfig                 | 22 +++---
 configs/ti_am64x_sk_defconfig                 | 22 +++---
 package/Config.in                             |  1 +
 package/python-attrs/python-attrs.mk          |  6 ++
 .../python-jsonschema-specifications.mk       | 10 +++
 .../python-jsonschema/python-jsonschema.mk    | 14 ++++
 .../python-referencing/python-referencing.mk  | 11 +++
 package/python-rpds-py/python-rpds-py.mk      |  1 +
 package/ti-k3/Config.in                       | 61 +++++++++++++++
 package/ti-k3/ti-k3.mk                        |  7 ++
 25 files changed, 263 insertions(+), 173 deletions(-)
 create mode 100755 board/ti/common/am6xx/post-build.sh
 create mode 100755 board/ti/common/am6xx/post-image.sh
 delete mode 100644 boot/ti-k3-image-gen/Config.in
 delete mode 100644 boot/ti-k3-image-gen/ti-k3-image-gen.hash
 delete mode 100644 boot/ti-k3-image-gen/ti-k3-image-gen.mk
 create mode 100644 package/ti-k3/Config.in
 create mode 100644 package/ti-k3/ti-k3.mk

Comments

'Krzysztof Kozlowski' via Amarula Linux March 2, 2024, 1:22 a.m. UTC | #1
Hi Dario,

On Fri, Mar 01, 2024 at 02:35:57PM +0100, Dario Binacchi wrote:
> The SK-AM62B-P1 is the newest version of SK-AM62 which includes
> high-security field-securable (HS-FS) silicon to optionally customize keys
> and encryption for security applications.
> This requires enabling the BR2_TARGET_TI_K3_IMAGE_GEN_SECTYPE_HS_FS option,
> which, however, was buggy. This series fixes the errors and bumps the Linux
> kernel and U-Boot required for such a new device.
> 
> Furthermore, this version complete the series by updating the
> ti_am64x_sk_defconfig too and remove ti-k3-image-gen, as requested by
> Romain Naour:
> https://patchwork.ozlabs.org/project/buildroot/patch/20240224205654.1546744-17-dario.binacchi@amarulasolutions.com/
> I have successfully executed compilation tests for all possible combinations of platform and security type:
> am62x -> GP, HS, HS-FS
> am64x -> GP, HS, HS-FS

Thank you for the changes and the updated submission. I'll build and
boot-test your v8 on all 4 hardware variants of the above platforms
and report back before the end of this weekend.

Also yesterday I was able to get graphics to work on AM62x with a few
patches based on top of your v7 series, using the Imagination Rogue
Power VR GPU driver and Mesa, and was able to run (accelerated) Weston.
Will pobably submit an RFC of this next week.

--
Andreas Dannenberg
Texas Instruments Inc


> 
> The AM62x HS-FS configuration has also been tested on the SK-AM62B-P1 board.
> 
> Dario Binacchi (22):
>   configs/ti_am62x_sk_defconfig: fix BR2_TARGET_OPTEE_OS_PLATFORM
>     setting
>   configs/ti_am64x_sk_defconfig: fix BR2_TARGET_OPTEE_OS_PLATFORM
>     setting
>   configs/ti_am62x_sk_defconfig: explicitly set the ti-k3-r5-loader
>     version
>   configs/ti_am64x_sk_defconfig: explicitly set the ti-k3-r5-loader
>     version
>   boot/ti-k3-r5-loader: bump to version 2024.01
>   package/python-attrs: add host variant
>   package/python-rpds-py: add host variant
>   package/python-referencing: add host variant
>   package/python-jsonschema-specifications: add host variant
>   package/python-jsonschema: add host variant
>   boot/ti-k3-r5-loader: set binman environment
>   boot/uboot: add BR2_TARGET_UBOOT_USE_BINMAN option
>   package/ti-k3: move TI_K3_{SECTYPE,SOC} out of the image-gen scope
>   package/ti-k3: disable HS-FS for AM65 SOC
>   boot/ti-k3/ti-k3-r5-loader: install tiboot3.bin
>   configs/ti_am62x_sk_defconfig: bump U-Boot version to 2024.01
>   configs/ti_am62x_sk_defconfig: bump Linux version to 6.6.18
>   board/ti/am62x-sk: generalize post-build.sh
>   board/ti/am62x-sk: move post-{build,image}.sh to board/ti/common/am6xx
>   package/ti-k3: fix BR2_PACKAGE_TI_K3_SOC setting for am64x SOC
>   configs/ti_am64x_sk_defconfig: bump U-Boot version to 2024.01
>   configs/ti_am64x_sk_defconfig: bump Linux version to 6.6.18
> 
> Romain Naour (1):
>   boot/ti-k3-image-gen: remove package
> 
>  Config.in.legacy                              |  7 ++
>  DEVELOPERS                                    |  2 -
>  board/ti/am62x-sk/genimage.cfg                |  7 +-
>  board/ti/am64x-sk/genimage.cfg                |  7 +-
>  board/ti/common/am6xx/post-build.sh           | 58 ++++++++++++++
>  board/ti/common/am6xx/post-image.sh           | 11 +++
>  boot/Config.in                                |  1 -
>  boot/ti-k3-image-gen/Config.in                | 77 -------------------
>  boot/ti-k3-image-gen/ti-k3-image-gen.hash     |  3 -
>  boot/ti-k3-image-gen/ti-k3-image-gen.mk       | 54 -------------
>  boot/ti-k3-r5-loader/Config.in                |  5 +-
>  boot/ti-k3-r5-loader/ti-k3-r5-loader.hash     |  2 +-
>  boot/ti-k3-r5-loader/ti-k3-r5-loader.mk       | 12 ++-
>  boot/uboot/Config.in                          | 23 +++---
>  boot/uboot/uboot.mk                           | 12 ++-
>  configs/ti_am62x_sk_defconfig                 | 22 +++---
>  configs/ti_am64x_sk_defconfig                 | 22 +++---
>  package/Config.in                             |  1 +
>  package/python-attrs/python-attrs.mk          |  6 ++
>  .../python-jsonschema-specifications.mk       | 10 +++
>  .../python-jsonschema/python-jsonschema.mk    | 14 ++++
>  .../python-referencing/python-referencing.mk  | 11 +++
>  package/python-rpds-py/python-rpds-py.mk      |  1 +
>  package/ti-k3/Config.in                       | 61 +++++++++++++++
>  package/ti-k3/ti-k3.mk                        |  7 ++
>  25 files changed, 263 insertions(+), 173 deletions(-)
>  create mode 100755 board/ti/common/am6xx/post-build.sh
>  create mode 100755 board/ti/common/am6xx/post-image.sh
>  delete mode 100644 boot/ti-k3-image-gen/Config.in
>  delete mode 100644 boot/ti-k3-image-gen/ti-k3-image-gen.hash
>  delete mode 100644 boot/ti-k3-image-gen/ti-k3-image-gen.mk
>  create mode 100644 package/ti-k3/Config.in
>  create mode 100644 package/ti-k3/ti-k3.mk
> 
> -- 
> 2.43.0
>
'Krzysztof Kozlowski' via Amarula Linux March 2, 2024, 7:11 a.m. UTC | #2
Hi Dario,

On Fri, Mar 01, 2024 at 07:22:53PM -0600, Andreas Dannenberg via buildroot wrote:
> Hi Dario,
> 
> On Fri, Mar 01, 2024 at 02:35:57PM +0100, Dario Binacchi wrote:
> > The SK-AM62B-P1 is the newest version of SK-AM62 which includes
> > high-security field-securable (HS-FS) silicon to optionally customize keys
> > and encryption for security applications.
> > This requires enabling the BR2_TARGET_TI_K3_IMAGE_GEN_SECTYPE_HS_FS option,
> > which, however, was buggy. This series fixes the errors and bumps the Linux
> > kernel and U-Boot required for such a new device.
> > 
> > Furthermore, this version complete the series by updating the
> > ti_am64x_sk_defconfig too and remove ti-k3-image-gen, as requested by
> > Romain Naour:
> > https://patchwork.ozlabs.org/project/buildroot/patch/20240224205654.1546744-17-dario.binacchi@amarulasolutions.com/
> > I have successfully executed compilation tests for all possible combinations of platform and security type:
> > am62x -> GP, HS, HS-FS
> > am64x -> GP, HS, HS-FS
> 
> Thank you for the changes and the updated submission. I'll build and
> boot-test your v8 on all 4 hardware variants of the above platforms
> and report back before the end of this weekend.

I just finished build and boot testing all 4 device/security
combinations without any issues:

1) AM62 GP
2) AM62 HS-FS
3) AM64 GP
4) AM64 HS-FS

For this I applied your v8 series on top of...
75a3562324 ("support/scripts/pkg-stats: sort CVEs in HTML output")
...and performed 4 individual completely clean builds.

Then, I boot tested all 4 SD card images on the respective HW platforms
(4 different SK boards). The boot logs of each case look pretty clean
and about as expected and I've archived them here [1] if somebody is
curious. While the boot logs show there's further work needed (e.g.,
updating TIFS/DM device firmwares to more current versions, enabling/
fixing power off functionality) all what I saw should be done as future
work and not as part of this series. I think it does a great job making
those device families accessible and enabling said further development.

So for the series:

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

[1] https://gist.github.com/adannenb-ti/35feca65e7d86a7e062a36c48efddc63




> Also yesterday I was able to get graphics to work on AM62x with a few
> patches based on top of your v7 series, using the Imagination Rogue
> Power VR GPU driver and Mesa, and was able to run (accelerated) Weston.
> Will pobably submit an RFC of this next week.
> 
> --
> Andreas Dannenberg
> Texas Instruments Inc
> 
> 
> > 
> > The AM62x HS-FS configuration has also been tested on the SK-AM62B-P1 board.
> > 
> > Dario Binacchi (22):
> >   configs/ti_am62x_sk_defconfig: fix BR2_TARGET_OPTEE_OS_PLATFORM
> >     setting
> >   configs/ti_am64x_sk_defconfig: fix BR2_TARGET_OPTEE_OS_PLATFORM
> >     setting
> >   configs/ti_am62x_sk_defconfig: explicitly set the ti-k3-r5-loader
> >     version
> >   configs/ti_am64x_sk_defconfig: explicitly set the ti-k3-r5-loader
> >     version
> >   boot/ti-k3-r5-loader: bump to version 2024.01
> >   package/python-attrs: add host variant
> >   package/python-rpds-py: add host variant
> >   package/python-referencing: add host variant
> >   package/python-jsonschema-specifications: add host variant
> >   package/python-jsonschema: add host variant
> >   boot/ti-k3-r5-loader: set binman environment
> >   boot/uboot: add BR2_TARGET_UBOOT_USE_BINMAN option
> >   package/ti-k3: move TI_K3_{SECTYPE,SOC} out of the image-gen scope
> >   package/ti-k3: disable HS-FS for AM65 SOC
> >   boot/ti-k3/ti-k3-r5-loader: install tiboot3.bin
> >   configs/ti_am62x_sk_defconfig: bump U-Boot version to 2024.01
> >   configs/ti_am62x_sk_defconfig: bump Linux version to 6.6.18
> >   board/ti/am62x-sk: generalize post-build.sh
> >   board/ti/am62x-sk: move post-{build,image}.sh to board/ti/common/am6xx
> >   package/ti-k3: fix BR2_PACKAGE_TI_K3_SOC setting for am64x SOC
> >   configs/ti_am64x_sk_defconfig: bump U-Boot version to 2024.01
> >   configs/ti_am64x_sk_defconfig: bump Linux version to 6.6.18
> > 
> > Romain Naour (1):
> >   boot/ti-k3-image-gen: remove package
> > 
> >  Config.in.legacy                              |  7 ++
> >  DEVELOPERS                                    |  2 -
> >  board/ti/am62x-sk/genimage.cfg                |  7 +-
> >  board/ti/am64x-sk/genimage.cfg                |  7 +-
> >  board/ti/common/am6xx/post-build.sh           | 58 ++++++++++++++
> >  board/ti/common/am6xx/post-image.sh           | 11 +++
> >  boot/Config.in                                |  1 -
> >  boot/ti-k3-image-gen/Config.in                | 77 -------------------
> >  boot/ti-k3-image-gen/ti-k3-image-gen.hash     |  3 -
> >  boot/ti-k3-image-gen/ti-k3-image-gen.mk       | 54 -------------
> >  boot/ti-k3-r5-loader/Config.in                |  5 +-
> >  boot/ti-k3-r5-loader/ti-k3-r5-loader.hash     |  2 +-
> >  boot/ti-k3-r5-loader/ti-k3-r5-loader.mk       | 12 ++-
> >  boot/uboot/Config.in                          | 23 +++---
> >  boot/uboot/uboot.mk                           | 12 ++-
> >  configs/ti_am62x_sk_defconfig                 | 22 +++---
> >  configs/ti_am64x_sk_defconfig                 | 22 +++---
> >  package/Config.in                             |  1 +
> >  package/python-attrs/python-attrs.mk          |  6 ++
> >  .../python-jsonschema-specifications.mk       | 10 +++
> >  .../python-jsonschema/python-jsonschema.mk    | 14 ++++
> >  .../python-referencing/python-referencing.mk  | 11 +++
> >  package/python-rpds-py/python-rpds-py.mk      |  1 +
> >  package/ti-k3/Config.in                       | 61 +++++++++++++++
> >  package/ti-k3/ti-k3.mk                        |  7 ++
> >  25 files changed, 263 insertions(+), 173 deletions(-)
> >  create mode 100755 board/ti/common/am6xx/post-build.sh
> >  create mode 100755 board/ti/common/am6xx/post-image.sh
> >  delete mode 100644 boot/ti-k3-image-gen/Config.in
> >  delete mode 100644 boot/ti-k3-image-gen/ti-k3-image-gen.hash
> >  delete mode 100644 boot/ti-k3-image-gen/ti-k3-image-gen.mk
> >  create mode 100644 package/ti-k3/Config.in
> >  create mode 100644 package/ti-k3/ti-k3.mk
> > 
> > -- 
> > 2.43.0
> > 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot
Dario Binacchi March 2, 2024, 10:47 a.m. UTC | #3
Hi Andreas,

On Sat, Mar 2, 2024 at 8:11 AM Andreas Dannenberg <dannenberg@ti.com> wrote:
>
> Hi Dario,
>
> On Fri, Mar 01, 2024 at 07:22:53PM -0600, Andreas Dannenberg via buildroot wrote:
> > Hi Dario,
> >
> > On Fri, Mar 01, 2024 at 02:35:57PM +0100, Dario Binacchi wrote:
> > > The SK-AM62B-P1 is the newest version of SK-AM62 which includes
> > > high-security field-securable (HS-FS) silicon to optionally customize keys
> > > and encryption for security applications.
> > > This requires enabling the BR2_TARGET_TI_K3_IMAGE_GEN_SECTYPE_HS_FS option,
> > > which, however, was buggy. This series fixes the errors and bumps the Linux
> > > kernel and U-Boot required for such a new device.
> > >
> > > Furthermore, this version complete the series by updating the
> > > ti_am64x_sk_defconfig too and remove ti-k3-image-gen, as requested by
> > > Romain Naour:
> > > https://patchwork.ozlabs.org/project/buildroot/patch/20240224205654.1546744-17-dario.binacchi@amarulasolutions.com/
> > > I have successfully executed compilation tests for all possible combinations of platform and security type:
> > > am62x -> GP, HS, HS-FS
> > > am64x -> GP, HS, HS-FS
> >
> > Thank you for the changes and the updated submission. I'll build and
> > boot-test your v8 on all 4 hardware variants of the above platforms
> > and report back before the end of this weekend.
>
> I just finished build and boot testing all 4 device/security
> combinations without any issues:
>
> 1) AM62 GP
> 2) AM62 HS-FS
> 3) AM64 GP
> 4) AM64 HS-FS
>
> For this I applied your v8 series on top of...
> 75a3562324 ("support/scripts/pkg-stats: sort CVEs in HTML output")
> ...and performed 4 individual completely clean builds.
>
> Then, I boot tested all 4 SD card images on the respective HW platforms
> (4 different SK boards). The boot logs of each case look pretty clean
> and about as expected and I've archived them here [1] if somebody is
> curious. While the boot logs show there's further work needed (e.g.,
> updating TIFS/DM device firmwares to more current versions, enabling/
> fixing power off functionality) all what I saw should be done as future
> work and not as part of this series. I think it does a great job making
> those device families accessible and enabling said further development.
>
> So for the series:
>
> Tested-by: Andreas Dannenberg <dannenberg@ti.com>

Nice!
Thanks a lot for your tests!

Regards,
Dario

>
> [1] https://gist.github.com/adannenb-ti/35feca65e7d86a7e062a36c48efddc63
>
>
>
>
> > Also yesterday I was able to get graphics to work on AM62x with a few
> > patches based on top of your v7 series, using the Imagination Rogue
> > Power VR GPU driver and Mesa, and was able to run (accelerated) Weston.
> > Will pobably submit an RFC of this next week.
> >
> > --
> > Andreas Dannenberg
> > Texas Instruments Inc
> >
> >
> > >
> > > The AM62x HS-FS configuration has also been tested on the SK-AM62B-P1 board.
> > >
> > > Dario Binacchi (22):
> > >   configs/ti_am62x_sk_defconfig: fix BR2_TARGET_OPTEE_OS_PLATFORM
> > >     setting
> > >   configs/ti_am64x_sk_defconfig: fix BR2_TARGET_OPTEE_OS_PLATFORM
> > >     setting
> > >   configs/ti_am62x_sk_defconfig: explicitly set the ti-k3-r5-loader
> > >     version
> > >   configs/ti_am64x_sk_defconfig: explicitly set the ti-k3-r5-loader
> > >     version
> > >   boot/ti-k3-r5-loader: bump to version 2024.01
> > >   package/python-attrs: add host variant
> > >   package/python-rpds-py: add host variant
> > >   package/python-referencing: add host variant
> > >   package/python-jsonschema-specifications: add host variant
> > >   package/python-jsonschema: add host variant
> > >   boot/ti-k3-r5-loader: set binman environment
> > >   boot/uboot: add BR2_TARGET_UBOOT_USE_BINMAN option
> > >   package/ti-k3: move TI_K3_{SECTYPE,SOC} out of the image-gen scope
> > >   package/ti-k3: disable HS-FS for AM65 SOC
> > >   boot/ti-k3/ti-k3-r5-loader: install tiboot3.bin
> > >   configs/ti_am62x_sk_defconfig: bump U-Boot version to 2024.01
> > >   configs/ti_am62x_sk_defconfig: bump Linux version to 6.6.18
> > >   board/ti/am62x-sk: generalize post-build.sh
> > >   board/ti/am62x-sk: move post-{build,image}.sh to board/ti/common/am6xx
> > >   package/ti-k3: fix BR2_PACKAGE_TI_K3_SOC setting for am64x SOC
> > >   configs/ti_am64x_sk_defconfig: bump U-Boot version to 2024.01
> > >   configs/ti_am64x_sk_defconfig: bump Linux version to 6.6.18
> > >
> > > Romain Naour (1):
> > >   boot/ti-k3-image-gen: remove package
> > >
> > >  Config.in.legacy                              |  7 ++
> > >  DEVELOPERS                                    |  2 -
> > >  board/ti/am62x-sk/genimage.cfg                |  7 +-
> > >  board/ti/am64x-sk/genimage.cfg                |  7 +-
> > >  board/ti/common/am6xx/post-build.sh           | 58 ++++++++++++++
> > >  board/ti/common/am6xx/post-image.sh           | 11 +++
> > >  boot/Config.in                                |  1 -
> > >  boot/ti-k3-image-gen/Config.in                | 77 -------------------
> > >  boot/ti-k3-image-gen/ti-k3-image-gen.hash     |  3 -
> > >  boot/ti-k3-image-gen/ti-k3-image-gen.mk       | 54 -------------
> > >  boot/ti-k3-r5-loader/Config.in                |  5 +-
> > >  boot/ti-k3-r5-loader/ti-k3-r5-loader.hash     |  2 +-
> > >  boot/ti-k3-r5-loader/ti-k3-r5-loader.mk       | 12 ++-
> > >  boot/uboot/Config.in                          | 23 +++---
> > >  boot/uboot/uboot.mk                           | 12 ++-
> > >  configs/ti_am62x_sk_defconfig                 | 22 +++---
> > >  configs/ti_am64x_sk_defconfig                 | 22 +++---
> > >  package/Config.in                             |  1 +
> > >  package/python-attrs/python-attrs.mk          |  6 ++
> > >  .../python-jsonschema-specifications.mk       | 10 +++
> > >  .../python-jsonschema/python-jsonschema.mk    | 14 ++++
> > >  .../python-referencing/python-referencing.mk  | 11 +++
> > >  package/python-rpds-py/python-rpds-py.mk      |  1 +
> > >  package/ti-k3/Config.in                       | 61 +++++++++++++++
> > >  package/ti-k3/ti-k3.mk                        |  7 ++
> > >  25 files changed, 263 insertions(+), 173 deletions(-)
> > >  create mode 100755 board/ti/common/am6xx/post-build.sh
> > >  create mode 100755 board/ti/common/am6xx/post-image.sh
> > >  delete mode 100644 boot/ti-k3-image-gen/Config.in
> > >  delete mode 100644 boot/ti-k3-image-gen/ti-k3-image-gen.hash
> > >  delete mode 100644 boot/ti-k3-image-gen/ti-k3-image-gen.mk
> > >  create mode 100644 package/ti-k3/Config.in
> > >  create mode 100644 package/ti-k3/ti-k3.mk
> > >
> > > --
> > > 2.43.0
> > >
> > _______________________________________________
> > buildroot mailing list
> > buildroot@buildroot.org
> > https://lists.buildroot.org/mailman/listinfo/buildroot
Michael Trimarchi March 7, 2024, 7:48 a.m. UTC | #4
Hi Romain

On Sat, Mar 2, 2024 at 8:11 AM Andreas Dannenberg <dannenberg@ti.com> wrote:
>
> Hi Dario,
>
> On Fri, Mar 01, 2024 at 07:22:53PM -0600, Andreas Dannenberg via buildroot wrote:
> > Hi Dario,
> >
> > On Fri, Mar 01, 2024 at 02:35:57PM +0100, Dario Binacchi wrote:
> > > The SK-AM62B-P1 is the newest version of SK-AM62 which includes
> > > high-security field-securable (HS-FS) silicon to optionally customize keys
> > > and encryption for security applications.
> > > This requires enabling the BR2_TARGET_TI_K3_IMAGE_GEN_SECTYPE_HS_FS option,
> > > which, however, was buggy. This series fixes the errors and bumps the Linux
> > > kernel and U-Boot required for such a new device.
> > >
> > > Furthermore, this version complete the series by updating the
> > > ti_am64x_sk_defconfig too and remove ti-k3-image-gen, as requested by
> > > Romain Naour:
> > > https://patchwork.ozlabs.org/project/buildroot/patch/20240224205654.1546744-17-dario.binacchi@amarulasolutions.com/
> > > I have successfully executed compilation tests for all possible combinations of platform and security type:
> > > am62x -> GP, HS, HS-FS
> > > am64x -> GP, HS, HS-FS
> >
> > Thank you for the changes and the updated submission. I'll build and
> > boot-test your v8 on all 4 hardware variants of the above platforms
> > and report back before the end of this weekend.
>
> I just finished build and boot testing all 4 device/security
> combinations without any issues:
>
> 1) AM62 GP
> 2) AM62 HS-FS
> 3) AM64 GP
> 4) AM64 HS-FS
>
> For this I applied your v8 series on top of...
> 75a3562324 ("support/scripts/pkg-stats: sort CVEs in HTML output")
> ...and performed 4 individual completely clean builds.
>
> Then, I boot tested all 4 SD card images on the respective HW platforms
> (4 different SK boards). The boot logs of each case look pretty clean
> and about as expected and I've archived them here [1] if somebody is
> curious. While the boot logs show there's further work needed (e.g.,
> updating TIFS/DM device firmwares to more current versions, enabling/
> fixing power off functionality) all what I saw should be done as future
> work and not as part of this series. I think it does a great job making
> those device families accessible and enabling said further development.
>
> So for the series:
>
> Tested-by: Andreas Dannenberg <dannenberg@ti.com>
>
> [1] https://gist.github.com/adannenb-ti/35feca65e7d86a7e062a36c48efddc63
>
>

Can we merge it up?
Tested on my AM62X industrial product

Tested-by: Michael Trimarchi <michael@amarulasolutions.com>
>
>
> > Also yesterday I was able to get graphics to work on AM62x with a few
> > patches based on top of your v7 series, using the Imagination Rogue
> > Power VR GPU driver and Mesa, and was able to run (accelerated) Weston.
> > Will pobably submit an RFC of this next week.
> >
> > --
> > Andreas Dannenberg
> > Texas Instruments Inc
> >
> >
> > >
> > > The AM62x HS-FS configuration has also been tested on the SK-AM62B-P1 board.
> > >
> > > Dario Binacchi (22):
> > >   configs/ti_am62x_sk_defconfig: fix BR2_TARGET_OPTEE_OS_PLATFORM
> > >     setting
> > >   configs/ti_am64x_sk_defconfig: fix BR2_TARGET_OPTEE_OS_PLATFORM
> > >     setting
> > >   configs/ti_am62x_sk_defconfig: explicitly set the ti-k3-r5-loader
> > >     version
> > >   configs/ti_am64x_sk_defconfig: explicitly set the ti-k3-r5-loader
> > >     version
> > >   boot/ti-k3-r5-loader: bump to version 2024.01
> > >   package/python-attrs: add host variant
> > >   package/python-rpds-py: add host variant
> > >   package/python-referencing: add host variant
> > >   package/python-jsonschema-specifications: add host variant
> > >   package/python-jsonschema: add host variant
> > >   boot/ti-k3-r5-loader: set binman environment
> > >   boot/uboot: add BR2_TARGET_UBOOT_USE_BINMAN option
> > >   package/ti-k3: move TI_K3_{SECTYPE,SOC} out of the image-gen scope
> > >   package/ti-k3: disable HS-FS for AM65 SOC
> > >   boot/ti-k3/ti-k3-r5-loader: install tiboot3.bin
> > >   configs/ti_am62x_sk_defconfig: bump U-Boot version to 2024.01
> > >   configs/ti_am62x_sk_defconfig: bump Linux version to 6.6.18
> > >   board/ti/am62x-sk: generalize post-build.sh
> > >   board/ti/am62x-sk: move post-{build,image}.sh to board/ti/common/am6xx
> > >   package/ti-k3: fix BR2_PACKAGE_TI_K3_SOC setting for am64x SOC
> > >   configs/ti_am64x_sk_defconfig: bump U-Boot version to 2024.01
> > >   configs/ti_am64x_sk_defconfig: bump Linux version to 6.6.18
> > >
> > > Romain Naour (1):
> > >   boot/ti-k3-image-gen: remove package
> > >
> > >  Config.in.legacy                              |  7 ++
> > >  DEVELOPERS                                    |  2 -
> > >  board/ti/am62x-sk/genimage.cfg                |  7 +-
> > >  board/ti/am64x-sk/genimage.cfg                |  7 +-
> > >  board/ti/common/am6xx/post-build.sh           | 58 ++++++++++++++
> > >  board/ti/common/am6xx/post-image.sh           | 11 +++
> > >  boot/Config.in                                |  1 -
> > >  boot/ti-k3-image-gen/Config.in                | 77 -------------------
> > >  boot/ti-k3-image-gen/ti-k3-image-gen.hash     |  3 -
> > >  boot/ti-k3-image-gen/ti-k3-image-gen.mk       | 54 -------------
> > >  boot/ti-k3-r5-loader/Config.in                |  5 +-
> > >  boot/ti-k3-r5-loader/ti-k3-r5-loader.hash     |  2 +-
> > >  boot/ti-k3-r5-loader/ti-k3-r5-loader.mk       | 12 ++-
> > >  boot/uboot/Config.in                          | 23 +++---
> > >  boot/uboot/uboot.mk                           | 12 ++-
> > >  configs/ti_am62x_sk_defconfig                 | 22 +++---
> > >  configs/ti_am64x_sk_defconfig                 | 22 +++---
> > >  package/Config.in                             |  1 +
> > >  package/python-attrs/python-attrs.mk          |  6 ++
> > >  .../python-jsonschema-specifications.mk       | 10 +++
> > >  .../python-jsonschema/python-jsonschema.mk    | 14 ++++
> > >  .../python-referencing/python-referencing.mk  | 11 +++
> > >  package/python-rpds-py/python-rpds-py.mk      |  1 +
> > >  package/ti-k3/Config.in                       | 61 +++++++++++++++
> > >  package/ti-k3/ti-k3.mk                        |  7 ++
> > >  25 files changed, 263 insertions(+), 173 deletions(-)
> > >  create mode 100755 board/ti/common/am6xx/post-build.sh
> > >  create mode 100755 board/ti/common/am6xx/post-image.sh
> > >  delete mode 100644 boot/ti-k3-image-gen/Config.in
> > >  delete mode 100644 boot/ti-k3-image-gen/ti-k3-image-gen.hash
> > >  delete mode 100644 boot/ti-k3-image-gen/ti-k3-image-gen.mk
> > >  create mode 100644 package/ti-k3/Config.in
> > >  create mode 100644 package/ti-k3/ti-k3.mk
> > >
> > > --
> > > 2.43.0
> > >
> > _______________________________________________
> > buildroot mailing list
> > buildroot@buildroot.org
> > https://lists.buildroot.org/mailman/listinfo/buildroot