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

[Frieder Schrempf]

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 at 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 at 1 {
>>>                         offset = <0x0>;
>>>                         filename = "flash.bin";
>>>                 };
>>>                 uboot: blob-ext at 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 at 1 {
>> +                        offset = <0x0>;
>> +                        filename = "flash.bin";
>> +                };
>> +                uboot: blob-ext at 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.