On 03/05/2022 22:03, Fabio Estevam wrote:
> From: Fabio Estevam <feste...@denx.de>
>
> The generated flash.bin does not boot the imx8mn evk LPDDR4 variant
> as it misses the spl.bin description in binman.
The dtsi file #includes "imx8mn-ddr4-evk-u-boot.dtsi", which provides an
'spl' description. It also provides a 'imx-boot' description for a
working 'flash.bin' file. The issue here is the 'flash' description here
is overwriting the same file with its own broken definition. You're
removing it, so things work fine.
> Add its entry to fix the boot on the imx8mn evk LPDDR4 variant.
>
> Signed-off-by: Fabio Estevam <feste...@denx.de>
> ---
> arch/arm/dts/imx8mn-evk-u-boot.dtsi | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
Reviewed-by: Alper Nebi Yasak <alpernebiya...@gmail.com>
> diff --git a/arch/arm/dts/imx8mn-evk-u-boot.dtsi
> b/arch/arm/dts/imx8mn-evk-u-boot.dtsi
> index 3db46d4cbc..593cf06eb9 100644
> --- a/arch/arm/dts/imx8mn-evk-u-boot.dtsi
> +++ b/arch/arm/dts/imx8mn-evk-u-boot.dtsi
> @@ -58,7 +58,9 @@
> };
>
>
> - flash {
> + spl {
> + filename = "spl.bin";
> +
> mkimage {
> args = "-n spl/u-boot-spl.cfgout -T imx8mimage -e
> 0x912000";
>
You might want to copy 'imx-boot' from "imx8mn-ddr4-evk-u-boot.dtsi" for
completeness. But it doesn't really matter.
