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

[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

>  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