[U-Boot] imx6dl_mamoj broken

[Robert Hancock]

On 2019-09-16 3:08 p.m., Stefano Babic wrote:
> Hi Anatolji,
> 
> On 16/09/19 22:20, Anatolij Gustschin wrote:
>> Hi Jagan,
>>
>> On Mon, 16 Sep 2019 13:36:03 +0530
>> Jagan Teki jagan at amarulasolutions.com wrote:
>> ...
>>>>        arm:  +   imx6dl_mamoj
>>>> +spl/u-boot-spl.bin exceeds file size limit:
>>>> +  limit:  53248 bytes
>>>> +  actual: 58597 bytes
>>>> +  excess: 5349 bytes
>>>>
>>>> +make[1]: *** [spl/u-boot-spl.bin] Error 1
>>>> +make[1]: *** Deleting file `spl/u-boot-spl.bin'
>>>> +make: *** [sub-make] Error 2
>>>
>>> Any idea which commit make this blow-out. we didn't do anything with
>>> this board recently. I guess some default CLK framework or so?
>>
>> This check fails since the recent changes in SPL size limit check,
>> see commit 25d38f9b34be (imx6: spl: Reduce SPL limit size in case
>> CONFIG_SECURE_BOOT is enabled).
>>
>> i.MX6DL has 128KiB OCRAM, probably we could increase default
>> SPL_SIZE_LIMIT config in common/spl/Kconfig for i.MX6DL?
> 
> Well, the current size was computed for the Solo /DL - Dual / Quad has
> twice OCRAM. According to chapter "8.4 Internal ROM / RAM map", for Solo
> /Dl there is 68KB available for user (address 0x907000--0x917FFF). I do
> not remember the details, but then we have to reserve space for gd,
> stack, etc. This leads to the actual value.

The patch I posted a while ago ("[PATCH v2] ARM: imx: Support larger SPL 
size on IMX6DQ") may be helpful for this case. If the platform doesn't 
need to support the "lite" chip versions with smaller OCRAM size, then 
the available size for the SPL can be increased.

-- 
Robert Hancock
Senior Software Developer
SED Systems, a division of Calian Ltd.
Email: hancock at sedsystems.ca