Re: [PATCH v3 2/9] arm: dts: imx8mp: Add common u-boot dtsi

Marcel Ziswiler
2021-08-18T08:18:08Z

Hi Fabio and Frieder

On Mon, 2021-08-16 at 12:10 -0300, Fabio Estevam wrote:
> Hi Frieder,
>
> On Mon, Aug 16, 2021 at 11:18 AM Frieder Schrempf
> <frieder.schrempf@kontron.de> wrote:
>
> > > If you prepare a patch I would prefer to have separate names for the SPL image and the final concatenated
> > > image. So we don't overwrite the SPL image and can still use it on its own if needed.
> > > But rather than choosing a new name for the complete image (as Heiko did above), I would like to rename
> > > the SPL image to something more meaningful like spl.bin first and then use flash.bin for the full image.
> > > What do you think?
>
> It seems we cannot rename the SPL to spl.bin as this would cause
> breakage to i.MX8MM boards that do not use binman, such as
> verdin-imx8mm_defconfig.

I am actually just about to send our conversion to using binman for this as well (;-p).

> > And it would also be nice if we could share the common parts of the binman config between the different
> > boards to avoid having this in each boards dtsi and to avoid having diverging configurations, where e. g.
> > flash.bin refers to different kinds of images.
>
> Agreed.

I am also all in for this idea. Should I still be sending my patch set converting Verdin iMX8M Mini separately
first? Maybe that would be the easiest and we could then transition them all to a common approach, not?

> Let me try to send a proposal for imx8mm-evk first. If it gets
> accepted, then I can try to make a common approach.

Yes, that's kinda what I was thinking. Thanks!

> Regards,
>
> Fabio Estevam

Cheers

Marcel