[U-Boot] [PATCH 00/17]: ARM: SPL: Make more generic, merge DaVinci and OMAP

Stefan Roese
2012-08-16T06:30:24Z

Hi Tom,

On 08/15/2012 11:30 PM, Tom Rini wrote:
> The following patch series merges the davinci and omap-common SPL
> frameworks into a single framework, CONFIG_SPL_FRAMEWORK along with a
> few small cleanups to the code to make it a little smaller and more
> flexible. The end result is that davinci can now opt in on
> SPL-boots-Linux by just setting the right defines and other platforms
> would just need to adapt to this framework. In my testing, davinci
> gains between 300 and 700 bytes for this (not enabling the SPL OS
> feature) and omap4/5/etc lose just a little bit (from the printf->puts
> changes). I've tested this on omap3_beagle (xM and classic) and
> omap4_panda. I don't have any davinci platforms that were previously
> using SPL so I can't boot-test those changes but since everyone sets
> CONFIG_SPL_MAX_SIZE, we're OK in that department.
>
> To make this series easier to test I've placed it on
> http://github.com/trini/u-boot WIP/spl-improvements
>
> I've looked a little into re-reducing the size and the biggest problem I
> see is that SPI a lot of informational prints that we don't need,
> strictly speaking, but are nice in a normal U-Boot context. I'm unsure
> of the best way to quiet these as I don't like the idea of sprinkling
> #ifndef CONFIG_SPL_BUILD in random places.

Making SPL more generic is great. So moving these files out of the
omap-common directory is the way to go. But your approach moves them to
arch/arm/lib, restricting it only to ARM. Wouldn't it be possible to
move it into an even more common location, so that other archs may also
benefit from it?

I'm asking, since I'm currently working on an PowerPC based board which
shall also use the SPL OS booting feature (Falcon boot).

Are the files ARM specific (I'm just starting looking into them right now)?

Thanks,
Stefan