Hi Tom

On 04/05/23 11:10, Manorit Chawdhry wrote:
On 09:57-20230503, Tom Rini wrote:
On Wed, May 03, 2023 at 12:57:13PM +0530, Neha Malcom Francis wrote:
Hi Tom

On 27/04/23 04:07, Tom Rini wrote:
On Fri, Apr 21, 2023 at 06:01:44PM +0530, 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)

So, next up is fixing this in CI. After taking Andrew's patch to fix the
typedef issue, and after my patches to ensure we can get
pyyaml/jsonschema for python, there's problems still:


Thanks for checking this! Couple things:

Over at https://source.denx.de/u-boot/u-boot/-/jobs/617966:
binman: Filename 'spl/dts/k3-am68-sk-base-board.dtb' not found in input
path (.,/builds/u-boot/u-boot,board/ti/j721s2,arch/arm/dts)
(cwd='/tmp/.bm-work/j721s2_hs_evm_a72')

1. This is dependent on the patch merging J721S2 HS and GP configs [1].
However it has been reverted on -next, seen in the same thread.

OK.  I'm not sure the priority order here. I would like to see this
series get in first, and get everything else rebased on top of it.

--

Hi Tom, I aligned with Neha on the order which will be easier for us
both in terms of handling both the series,

  1. J721S2 and J7200 HS defconfig merge
   ( 
https://lore.kernel.org/r/20230405-j721s2-hs-evm-upstream-v2-0-c0f10a410...@ti.com
 )
  2. Binman can go after that
  3. J721E HS defconfig patches
   ( 
https://lore.kernel.org/u-boot/20230324-j721e-upstream-hs-v6-0-5aa43a481...@ti.com
 )
   Will re-roll once binman is merged

Thanks and regards,
Manorit

Tom



Bringing this back after yesterday's discussion. J7 HS is a definite regression and I think this should be taken into 2023.07 and not wait for binman in 2023.10. Rebasing binman on top of the J7 HS fix patches is very minimal effort.

--
Thanking You
Neha Malcom Francis

Reply via email to