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.