On 16.08.21 16:09, Frieder Schrempf wrote:
Hi Fabio,
On 16.08.21 15:56, Fabio Estevam wrote:
Hi Heiko,
On Mon, Aug 16, 2021 at 7:27 AM Heiko Schocher hs@denx.de wrote:
I am not a binman expert, but may you try to add in your u-boot.dtsi in your "binman" node at the end:
imx-boot { filename = "imx-boot.bin"; pad-byte = <0x00>; spl: blob-ext@1 { offset = <0x0>; filename = "flash.bin"; }; uboot: blob-ext@2 { offset = <0x59000>; filename = "u-boot.itb"; };This helped!
I had to change the offset and now I can flash only flash.bin and it boots fine:
--- a/arch/arm/dts/imx8mm-evk-u-boot.dtsi +++ b/arch/arm/dts/imx8mm-evk-u-boot.dtsi @@ -217,4 +217,18 @@ }; }; };
imx-boot {filename = "flash.bin";pad-byte = <0x00>;spl: blob-ext@1 {offset = <0x0>;filename = "flash.bin";};uboot: blob-ext@2 {offset = <0x57c00>;filename = "u-boot.itb";};};};
I will submit the patch shortly.
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?
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.