[PATCH] arm: dts: imx8mp: Generate single bootable binary

[Teresa Remmet]

Am Mittwoch, den 25.08.2021, 12:56 -0300 schrieb Fabio Estevam:
> Hi Teresa,
> 
> On Wed, Aug 25, 2021 at 12:25 PM Teresa Remmet <t.remmet at phytec.de>
> wrote:
> > binman conversion made flashing flash.bin
> > and u-boot.itb necessary. Update binman config
> > to create a single flash.bin image again.
> > 
> > This updates imx8mp_evk and phyCORE-i.MX8MP as they share the
> > same binman config.
> > 
> > Updated also imx8mp_evk documentation.
> > 
> > Tested on phyCORE-i.MX8MP.
> 
> Thanks for taking care of this.
> 
> > Signed-off-by: Teresa Remmet <t.remmet at phytec.de>
> > ---
> >  arch/arm/dts/imx8mp-u-boot.dtsi                 | 17
> > ++++++++++++++++-
> >  .../imx8mp_evk/imximage-8mp-lpddr4.cfg          |  2 +-
> >  board/phytec/phycore_imx8mp/imximage-8mp-sd.cfg |  2 +-
> >  doc/board/nxp/imx8mp_evk.rst                    |  1 -
> >  4 files changed, 18 insertions(+), 4 deletions(-)
> > 
> > diff --git a/arch/arm/dts/imx8mp-u-boot.dtsi b/arch/arm/dts/imx8mp-
> > u-boot.dtsi
> > index a6ede2ed6e30..ace7b2051e7f 100644
> > --- a/arch/arm/dts/imx8mp-u-boot.dtsi
> > +++ b/arch/arm/dts/imx8mp-u-boot.dtsi
> > @@ -82,7 +82,7 @@
> >                 };
> >         };
> > 
> > -       flash {
> > +       spl {
> 
> To make this more standard, could you please also pass 'filename =
> "spl.bin" here too?
> 
> Please check:
> https://lists.denx.de/pipermail/u-boot/2021-August/458754.html

Oh, I missed the latest version of your patch. I will update this
accordingly. Thanks.

Teresa

> 
> >  BOOT_FROM      sd
> > -LOADER         mkimage.flash.mkimage   0x920000
> > +LOADER         mkimage.spl.mkimage     0x920000
> 
> Please use "LOADER u-boot-spl-ddr.bin  0x920000" here instead.
> 
> > diff --git a/board/phytec/phycore_imx8mp/imximage-8mp-sd.cfg
> > b/board/phytec/phycore_imx8mp/imximage-8mp-sd.cfg
> > index b2920b4908ed..2be18de674ba 100644
> > --- a/board/phytec/phycore_imx8mp/imximage-8mp-sd.cfg
> > +++ b/board/phytec/phycore_imx8mp/imximage-8mp-sd.cfg
> > @@ -7,4 +7,4 @@
> > 
> >  ROM_VERSION    v2
> >  BOOT_FROM      sd
> > -LOADER         mkimage.flash.mkimage   0x920000
> > +LOADER         mkimage.spl.mkimage     0x920000
> 
> Ditto.
> 
> With these changes, you can add:
> 
> Reviewed-by: Fabio Estevam <festevam at gmail.com>
> 
> Thanks