Re: [PATCH v7 00/23] Migration to using binman for bootloader

[Neha Malcom Francis]

Hi Jerome

On 14/07/23 19:37, Jerome Forissier wrote:

On 7/14/23 15:50, Neha Malcom Francis wrote:

This series aims to eliminate the use of additional custom repositories
such as k3-image-gen (K3 Image Generation) repo and core-secdev-k3 (K3
Security Development Tools) that was plumbed into the U-Boot build flow
to generate boot images for TI K3 platform devices. And instead, we move
towards using binman that aligns better with the community standard build
flow.

This series uses binman for all K3 platforms supported on U-Boot currently;
both HS (High Security, both SE and FS) and GP (General Purpose) devices.

Background on using k3-image-gen:
        * TI K3 devices require a SYSFW (System Firmware) image consisting
        of a signed system firmware image and board configuration binaries,
        this is needed to bring up system firmware during U-Boot R5 SPL
        startup.
        * Board configuration data contain board-specific information
        such as resource management, power management and security.

Background on using core-secdev-k3:
        * Contains resources to sign x509 certificates for HS devices

Series intends to use binman to take over the packaging and signing for
the R5 bootloader images tiboot3.bin (and sysfw.itb, for non-combined
boot flow) instead of k3-image-gen.

Series also packages the A72/A53 bootloader images (tispl.bin and
u-boot.img) using ATF, OPTEE and DM (Device Manager)

Changes in v7:
        - corrected Texas Instruments copyright year
        - k3-am65-iot2050 image fit@0x180000 filename retained as
          tispl.bin

Changes in v6:
        - addressed whitespace warnings
        - added testcase for overwriting symlink functionality
        - %s/Arm Trusted Firmware/Trusted Firmware-A
        - %s/tee-pager_v2.bin/tee-raw.bin

I can still find quite a few occurrences of tee-pager_v2.bin in the patch
set (and in v7 too). Did you forget to change?

Another note applicable to several patches in v7:
'description = "OPTEE"' should be
'description = "OP-TEE"' (the official spelling has a hyphen).

Thanks for catching these, it was an oversight. Will change it.

--
Thanking You
Neha Malcom Francis