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

Marcel Ziswiler marcel at ziswiler.com
Thu Oct 7 19:12:30 CEST 2021


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 at 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 at gateworks.com>
> > 
> > Reviewed-by: Marcel Ziswiler <marcel.ziswiler at 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


More information about the U-Boot mailing list