Re: [meta-ti][dunfell][PATCH] conf: machine: am64xx-hs-evm: Include required boot binaries in wic image

["Aparna M" <a-m1@ti.com>]

On 19:17-20220301, Denys Dmytriyenko wrote:
Hi Denys,

Yes, the wic image was tested on the HS platform and the boot works.
This should give you an idea about the boot flow :
https://software-dl.ti.com/processor-sdk-linux/esd/AM65X/08_01_00_01/exports/docs/linux/Foundational_Components/U-Boot/UG-General-Info.html#boot-flow
Boot images are x509 cert signed and the fitImage and DTBs are present 
in /boot in the rootfs partition of the wic image. Also, yes, I
believe the other K3 HS platforms need similar fixes.

> Thanks, Aparna.
> 
> Was this tested on the actual HS platform?
> Can you describe the boot flow briefly?
> Where does fitImage with the kernel and DTBs go? I don't see it in wic.
> Also, what about other K3 HS platforms? Should they be updated similarly?
> Thanks.
> 
> 
> On Wed, Mar 02, 2022 at 05:38:57AM +0530, Aparna M wrote:
> > The wic image generated for HS platforms do not contain all required
> > boot files. This happens because the IMAGE_BOOT_FILES variable that
> > contains boot binaries to be included in the wic image is not set
> > correctly. Populate this variable to include all boot binaries required.
> > 
> > All the HS platforms generate the u-boot binary as u-boot.img_HS but the
> > bootloader expects the binary to be named as u-boot.img. Use UBOOT_IMAGE
> > variable to include the binary as u-boot.img in the wic image.
> > 
> > Signed-off-by: Aparna M <a-m1@ti.com>
> > ---
> >  conf/machine/am64xx-hs-evm.conf | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> > 
> > diff --git a/conf/machine/am64xx-hs-evm.conf b/conf/machine/am64xx-hs-evm.conf
> > index 44e81463..960f63cf 100644
> > --- a/conf/machine/am64xx-hs-evm.conf
> > +++ b/conf/machine/am64xx-hs-evm.conf
> > @@ -17,6 +17,7 @@ UBOOT_DTBO_OFFSET =      "0x00010000"
> >  SPL_BINARY = "tispl.bin_HS"
> >  SPL_BINARYNAME = "tispl.bin"
> >  UBOOT_BINARY = "u-boot.img_HS"
> > -IMAGE_BOOT_FILES = "${UBOOT_BINARY}"
> > +UBOOT_IMAGE = "u-boot.img"
> > +IMAGE_BOOT_FILES = "tiboot3.bin ${SPL_BINARYNAME} ${UBOOT_IMAGE}"
> >  
> >  TI_SECURE_DEV_PKG = "${TI_SECURE_DEV_PKG_K3}"
> > -- 
> > 2.17.1
> > 
> 
> -- 
> Regards,
> Denys Dmytriyenko <denis@denix.org>
> PGP: 0x420902729A92C964 - https://denix.org/0x420902729A92C964
> Fingerprint: 25FC E4A5 8A72 2F69 1186  6D76 4209 0272 9A92 C964