[U-Boot] [PATCH 0/13] ARM: UniPhier: enable CONFIG_SPL_DM with some clean-ups

[Simon Glass]

Hi Masahiro,

On 24 March 2015 at 22:08, Masahiro Yamada
<yamada.masahiro at socionext.com> wrote:
> Hi Simon,
>
>
> 2015-03-24 7:47 GMT+09:00 Simon Glass <sjg at chromium.org>:
>> Hi Masahiro,
>>
>> On 23 March 2015 at 09:19, Masahiro Yamada
>> <yamada.masahiro at socionext.com> wrote:
>>> 2015-03-23 0:07 GMT+09:00 Masahiro Yamada <yamada.masahiro at socionext.com>:
>>>>
>>>>
>>>>
>>>> Masahiro Yamada (13):
>>>>   ARM: UniPhier: include PH1-LD4 Makefile from PH1-sLD8
>>>>   ARM: UniPhier: move platform devices to SPL
>>>>   ARM: UniPhier: move UART pin settings to SPL
>>>>   ARM: UniPhier: enable CONFIG_PANIC_HANG
>>>>   ARM: UniPhier: enable Driver Model and UART on SPL
>>>>   ARM: UniPhier: use CONFIG_SPL_STACK to define SPL stack pointer
>>>>   ARM: UniPhier: add CONFIG_SPL_MAX_FOOTPRINT
>>>>   ARM: UniPhier: move init stack area just below TEXT_BASE
>>>>   ARM: UniPhier: add empty lowlevel_init to U-boot proper
>>>>   ARM: UniPhier: fix typos in comments
>>>>   ARM: UniPhier: optimize kicking secondary CPUs code
>>>>   ARM: UniPhier: disable L2 cache by lowlevel_init of U-Boot proper
>>>>   ARM: UniPhier: remove unnecessary ifdef conditional
>>>
>>>
>>> Applied to u-boot-uniphier/master.
>>
>> Great! Also you could look at my series here:
>>
>> http://lists.denx.de/pipermail/u-boot/2015-February/206581.html
>>
>> The device tree method is not good, I think we need a tool that chops
>> the device tree down automatically for SPL. But otherwise, the support
>> is there for device tree in SPL.
>
> Amazing!
> It is very nice that OF_CONTROL can be available with such small
> memory footprint.
>
> I can use 64KB for SPL on my boards, so it will surely work for me.
> I will check it out!
>
> Thanks for your hard work all the time!

Thanks! Yes it has worked out well. It is still overhead but it is
tolerable overhead for most modern platforms, I think.

I look forward to hearing what you find.

Re the tool, I think we need something like:

fdtsubset -a serial0 -a /gpio <in.dtb> <out.dtb>

which looks up the alias 'serial0' and the /gpio node and creates a
device tree subset containing just enough to allow that to work. I did
a more general-purpose 'fdtgrep' tool along those lines a long time
ago but it was not accepted to dtc.

Regards,
Simon