[U-Boot] [PATCH v2 3/5] ARM: dts: OMAP5+: Add u-boot specific dtsi

Lokesh Vutla lokeshvutla at ti.com
Fri Feb 10 14:32:03 UTC 2017



On Friday 10 February 2017 05:42 PM, Tom Rini wrote:
> On Fri, Feb 10, 2017 at 05:22:12PM +0530, Lokesh Vutla wrote:
>>
>>
>> On Friday 10 February 2017 09:00 AM, Tom Rini wrote:
>>> On Thu, Feb 09, 2017 at 09:30:10AM +0530, Lokesh Vutla wrote:
>>>
>>>> Add u-boot specific dtsi so that this will be
>>>> included automatically while building dts.
>>>>
>>>> Signed-off-by: Lokesh Vutla <lokeshvutla at ti.com>
>>>
>>> Reviewed-by: Tom Rini <trini at konsulko.com>
>>
>> As I said earlier, I see a problem with this approach. We want this
>> omap5-u-boot.dtsi only
>> for SPL. But Makefile.lib tries to append this for each dtb in proper
>> U-Boot as well. As we might not need any of these in U-Boot before
>> relocation, the behavior is unexpected(sometimes U-boot does not comes
>> up ). I would like to go ahead with V1 of this series, what do you say?
> 
> All of the other platforms are able to have pre-reloc on nodes without
> issue, can you describe what's going on a bit more?  Simon, any ideas?
> Thanks!
> 

So, when more nodes were added with pre-reloc, dm_scan_fdt() is failing
with NOMEM. Then I realized that  SYS_MALLOC_F_LEN is 0x400 which is
very less. Increased the length to 0x2000 and everything working fine.
Sorry for the noise :)
Will be posting an updated series soon.

Thanks and regards,
Lokesh


More information about the U-Boot mailing list