[PATCH] imx8mm-evk: Generate a single bootable flash.bin again

[Frieder Schrempf]

Hi Fabio,

On 18.08.21 16:07, Frieder Schrempf wrote:
> On 18.08.21 14:19, Fabio Estevam wrote:
>> After the conversion to binman in commit 8996e6b7c6a1 ("imx8mm_evk: switch
>> to use binman to pack images"), it is necessary to flash both flash.bin and
>> u-boot.itb to get a bootable system. Prior to this commit, only flash.bin
>> was needed. 
>>
>> Such new requirement breaks existing distro mechanisms to generate the
>> final binary because the extra u-boot.itb is now required.
>>
>> Generate a final flash.bin that can be used again as a single
>> bootable binary to keep the original behavior.
>>
>> After this change the SPL binary is called spl.bin, which is a more
>> descriptive name for its purpose, and can still be used standalone
>> (for example, for secure boot purposes).
>>
>> Signed-off-by: Fabio Estevam <festevam at denx.de>
> 
> Reviewed-by: Frieder Schrempf <frieder.schrempf at kontron.de>

I tried to adapt this for my own board, but I needed to change the following in the imximage.cfg for the build to pass. Did you test this?

-LOADER         mkimage.flash.mkimage   0x7E1000
+LOADER         mkimage.spl.mkimage     0x7E1000

Best regards
Frieder

>> ---
>>  arch/arm/dts/imx8mm-evk-u-boot.dtsi | 17 ++++++++++++++++-
>>  1 file changed, 16 insertions(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/dts/imx8mm-evk-u-boot.dtsi b/arch/arm/dts/imx8mm-evk-u-boot.dtsi
>> index f200afac9f..453fe1d259 100644
>> --- a/arch/arm/dts/imx8mm-evk-u-boot.dtsi
>> +++ b/arch/arm/dts/imx8mm-evk-u-boot.dtsi
>> @@ -150,7 +150,7 @@
>>  	};
>>  
>>  
>> -       flash {
>> +	spl {
>>  		mkimage {
>>  			args = "-n spl/u-boot-spl.cfgout -T imx8mimage -e 0x7e1000";
>>  
>> @@ -217,4 +217,19 @@
>>  			};
>>  		};
>>  	};
>> +
>> +	imx-boot {
>> +		filename = "flash.bin";
>> +		pad-byte = <0x00>;
>> +
>> +		spl: blob-ext at 1 {
>> +			offset = <0x0>;
>> +			filename = "spl.bin";
>> +		};
>> +
>> +		uboot: blob-ext at 2 {
>> +			offset = <0x57c00>;
>> +			filename = "u-boot.itb";
>> +		};
>> +	};
>>  };
>>