Re: [PATCH v2 8/8] imx: imx8: apalis: switch to binman

oliver.graute@kococonnector.com
2022-11-10T16:35:35Z

On 09/11/22, Marcel Ziswiler wrote:
> Hi Oliver
>
> Thanks for working on this.
>
> On Fri, 2022-11-04 at 16:03 +0100, Oliver Graute wrote:
> > Switch to use binman to pack images
> >
> > Signed-off-by: Oliver Graute <oliver.graute@kococonnector.com>
> > ---
> > Changes for v2
> >  - use common imx8qm-u-boot.dtsi
> >  - guard SPL nodes with CONFIG_SPL
>
> I don't think it is that simple. Even after guarding them SPL nodes the flash.bin one still references them,
> not? And yes, so far we never used SPL for them 8/8X as that honestly does not really make much sense given the
> SCFW does the actual RAM initialisation. Anyway, I am not very clear on how/what exactly that all means now.

On imx8qm-rom7720 we use the SCFW for RAM initalisation too but still use
SPL.
>
> Another topic is that flash.bin is now just the SPL whereas previously u-boot-dtb.imx was the entire thing. But
> again, that could have been related to us not using SPL.

yes, thats right, if I guard the SPL subnode in imx-boot node I get a 0 byte
flash.bin.

At the moment I don't know the best way here. I would be grateful for
some hints here.

> >  CONFIG_FIT=y +CONFIG_FIT_EXTERNAL_OFFSET=0x3000
>
> Also not quite sure what exactly that is now.

its introduced by commit 3814fcba12323b9f30f39ee5455f3f9a7e955c64

Best regards,

Oliver