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 <feste...@denx.de>
>
> Reviewed-by: Frieder Schrempf <frieder.schre...@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@1 {
>> + offset = <0x0>;
>> + filename = "spl.bin";
>> + };
>> +
>> + uboot: blob-ext@2 {
>> + offset = <0x57c00>;
>> + filename = "u-boot.itb";
>> + };
>> + };
>> };
>>
