Re: [meta-ti] [dunfell/master][PATCH 6/7] conf/machine: am64xx: Switch to multi-certificate boot images

["Denys Dmytriyenko" <denis@denix.org>]

On Wed, Mar 31, 2021 at 06:35:00PM -0400, Denys Dmytriyenko wrote:
> On Wed, Mar 31, 2021 at 11:51:32AM -0500, Nishanth Menon wrote:
> > Switch to using all of multi-certificate boot images. This allows us
> > to create bootable images - in this case, we are missing tiboot3.bin
> > 
> > Signed-off-by: Nishanth Menon <nm@ti.com>
> > ---
> >  conf/machine/include/am64xx.inc | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> > 
> > diff --git a/conf/machine/include/am64xx.inc b/conf/machine/include/am64xx.inc
> > index 3173319df354..9d40b066a03a 100644
> > --- a/conf/machine/include/am64xx.inc
> > +++ b/conf/machine/include/am64xx.inc
> > @@ -10,7 +10,8 @@ KERNEL_DEVICETREE = " \
> >      ti/k3-am642-evm.dtb \
> >  "
> >  
> > -IMAGE_BOOT_FILES = "tispl.bin u-boot.img"
> > +# AM64 supports multi-certificate images, use the same
> > +IMAGE_BOOT_FILES = "${IMAGE_BOOT_FILES_MULTI_CERT}"
> >  do_image_wic[depends] += "trusted-firmware-a:do_deploy"
> >  do_image_tar[depends] += "trusted-firmware-a:do_deploy"
> 
> Same comment - TF-A is not needed here.

I guess this can be cleaned up separately. The main change is fine:

Reviewed-by: Denys Dmytriyenko <denys@konsulko.com>


> >  do_image_wic[mcdepends] = "mc::k3r5:ti-sci-fw:do_deploy"
> > -- 
> > 2.31.0

-- 
Regards,
Denys Dmytriyenko <denis@denix.org>
PGP: 0x420902729A92C964 - https://denix.org/0x420902729A92C964
Fingerprint: 25FC E4A5 8A72 2F69 1186  6D76 4209 0272 9A92 C964