[U-Boot] imx6dl_mamoj broken

[Stefano Babic]

Hi Robert,

On 17/09/19 01:37, Robert Hancock wrote:
> 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.

Your patch is "in" - it is applied to u-boot-imx. However, it cannot
help because the current limitation is set for Solo / DL. With your
patch, SPL on Dual / Quad can be larger.

> If the platform doesn't
> need to support the "lite" chip versions with smaller OCRAM size,

The SOC is a DL, 128KB

> then
> the available size for the SPL can be increased.

It cannot

Best regards,
Stefano Babic


-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================