[U-Boot] [PATCH 0/6] ARM: DRA7: AM57xx: Enable SPL_DM

[Andrew F. Davis]

On 02/08/2017 07:07 AM, Lokesh Vutla wrote:
> 
> 
> On Wednesday 08 February 2017 06:26 PM, Tom Rini wrote:
>> On Wed, Feb 08, 2017 at 06:03:31PM +0530, Lokesh Vutla wrote:
>>
>>> - Enable SPL_DM on all DRA7 and AM57xx based platforms.
>>> - SPL uses a generic dts on all platforms with same defconfig.
>>> - After this series SPL size is increased by ~7KB.
>>
>> That's not a bad size growth, esp here where we can afford it.  Did you
> 
> Yeah, but I am worried about this size on am335x.
> 

I am also, I'm working to save about 30 bytes right now to include
another boot mode into HS SPL, +7Kb will simply not work. It looks like
you also get some of your saving from enabling simple_malloc, this is
already enabled for AM335x-HS, so the size increase will likely be
greater on the platform that needs it the most :(

>> experiment with going the board-data way?  Also it looks like we're
> 
> hmm.. IIUC, board-data is duplication of same data that is available in
> DT. Anyways all the unsed nodes in DT are removed for SPL case. So, I
> used this approach.
> 

It is a duplication, but it will save us having to append and maintain
yet another dts file. If the info is static across the platform, we
don't gain anything by encoding it in DT. Plus we can drop the
SPL_OF_CONTROL and save some space.

Andrew

>> going the same DT for U-Boot proper now, will this work long term?  Or
>> do you need to switch to having U-Boot itself be loaded from a FIT image
>> and given the correct board dtb?  Thanks!
> 
> The DT that is being introduced in this series will be used only by SPL.
> For U-Boot proper, a FIT image will be built using OF_LIST. I don't
> think we need to switch.
> 
> Thanks and regards,
> Lokesh
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>