[PATCH v1 5/7] verdin-imx8mm: switch to use binman to pack images

[Heiko Thiery]

 Hi Marcel,

Am Mo., 23. Aug. 2021 um 12:02 Uhr schrieb Marcel Ziswiler
<marcel.ziswiler at toradex.com>:
>
> Hi Heiko
>
> On Mon, 2021-08-23 at 08:37 +0200, Heiko Thiery wrote:
>
> > ...
> > > diff --git a/board/toradex/verdin-imx8mm/imximage.cfg b/board/toradex/verdin-imx8mm/imximage.cfg
> > > index b8b25ff4201..9bd60e39fb5 100644
> > > --- a/board/toradex/verdin-imx8mm/imximage.cfg
> > > +++ b/board/toradex/verdin-imx8mm/imximage.cfg
> > > @@ -1,16 +1,9 @@
> > >  /* SPDX-License-Identifier: GPL-2.0+ */
> > >  /*
> > > - * Copyright 2020 Toradex
> > > + * Copyright 2021 Toradex
> > >   */
> > >
> > >  #define __ASSEMBLY__
> > >
> > > -FIT
> > >  BOOT_FROM      emmc_fastboot
> > > -LOADER         spl/u-boot-spl-ddr.bin  0x7E1000
> > > -SECOND_LOADER  u-boot.itb              0x40200000 0x60000
> > > -
> > > -DDR_FW lpddr4_pmu_train_1d_imem.bin
> > > -DDR_FW lpddr4_pmu_train_1d_dmem.bin
> > > -DDR_FW lpddr4_pmu_train_2d_imem.bin
> > > -DDR_FW lpddr4_pmu_train_2d_dmem.bin
> > > +LOADER         mkimage.spl.mkimage     0x7E1000
> >
> > I think you should use "spl.bin" here and not the
> > "mkimage.spl.mkimage". As far as I understand the mkimage files are
> > only binman temporary files.
>
> Unfortunately, that does not seem to work:
>
> binman: Error 1 running 'mkimage -d ./mkimage.spl.mkimage -n spl/u-boot-spl.cfgout -T imx8mimage -e 0x7e1000
> ./mkimage-out.spl.mkimage': spl.bin: Can't open: No such file or directory
>
> Thinking about it, I guess, it is a chicken/egg problem. As it is that step of binman which actually creates
> spl.bin from this mkimage.spl.mkimage intermediate artifact, not?

Sorry for the confusion. It was my fault to say spl.bin is required in
the imximage.cfg. The right file should be "u-boot-spl-ddr.bin". This
is the input for the mkimage. Isn't it?

-- 
Heiko