[U-Boot] [PATCH v3 0/19] ARM: SPL: Make more generic, merge DaVinci and OMAP

Christian Riesch
2012-08-23T12:33:01Z

Hi Tom,

On Thu, Aug 23, 2012 at 1:19 AM, Tom Rini <trini@ti.com> wrote:
> Hey all,
>
> 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
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54303 and I'm sending
> another email to discuss that.
>
> --
> Tom
>
> Changes in v2:
> - Add
> - Add CONFIG_SYS_SPL_MALLOC_START / SIZE defines from Christian Riesch
> - Add arch/arm/cpu/arm926ejs/davinci/config.mk to build u-boot.ais
> - Fix checkpatch warning (u32* boot_params_ptr -> u32 *boot_params_ptr)
> - Place files into common/spl (due to gcc bug #54303) and include/spl.h
> - Reorder all of the patches that used to follow this to precede.
>
> Changes in v3:
> CONFIG_SPL_FRAMEWORK
> perform early init. Fixes the cannot reset problem.

Tested it on the LogicPD AM1808 experimenter's kit, works fine now :-)
Thanks!
Christian

> - Fix checkpatch.pl warning
> - Fix thinko, Nand -> SPI (spotted by Prabhakar Lad)
> - Make u-boot.ais use u-boot.img not u-boot.bin, now that it uses
> - Make use of board_init_f being a weak function now so that we can
> - Rename to README.da850, add more direct recovery method (Christian Riesch)
>