Re: [PATCH v2] imx8mm_venice: switch to use binman to pack images

[Tim Harvey <tharvey@gateworks.com>]

On Thu, Oct 7, 2021 at 10:12 AM Marcel Ziswiler <marcel@ziswiler.com> wrote:
>
> Hi Tim
>
> On Thu, 2021-10-07 at 08:52 -0700, Tim Harvey wrote:
> > )On Wed, Oct 6, 2021 at 2:44 PM Marcel Ziswiler <marcel@ziswiler.com> wrote:
> > >
> > > On Wed, 2021-10-06 at 13:17 -0700, Tim Harvey wrote:
> > > > Use binman to pack images.
> > > >
> > > > Note that imx8mm_venice supports several boards via multiple DTB's thus
> > > > in the fit node we must use:
> > > > - fit,fdt-list = "of-list"
> > > > - fdt-SEQ
> > > > - config-SEQ
> > > >
> > > > Signed-off-by: Tim Harvey <tharvey@gateworks.com>
> > >
> > > Reviewed-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> > >
> > > > ---
> > > > v2:
> > > >  - rebase on origin/mater
> > >
> > > You probably meant master (;-p).
> > >
> > > Rest look legit.
> >
> > Marcel,
> >
> > Thanks for the review. I will submit a v3 as I forgot to include a few
> > other patches required for this that I had in v1 (using a common
> > u-boot.dtsi file)
> >
> > I am seeing 'WARNING 'u-boot-spl-ddr.bin' not found, resulting binary
> > is not-functional' until I do a 'make' a second time so there is some
> > sort of race condition with the binman blobs.
>
> Yes, remember, this still needs Peng's Makefile fix [1] as otherwise, it may not quite generate all binman
> artefacts in the right order as discussed here [2].
>
> [1] https://marc.info/?l=u-boot&m=162908373904742
> [2] https://marc.info/?l=u-boot&m=162945614207220
>
> Cheers
>
> Marcel

Marvell,

Indeed that patch resolves the issue - thanks for reminding me that
hasn't made it in yet!

Best regards,

Tim