Re: [PATCH v3 2/9] arm: dts: imx8mp: Add common u-boot dtsi

Frieder Schrempf
2021-08-16T14:09:36Z

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?

Thanks
Frieder