Hi Tom, On Thu, Aug 23, 2012 at 1:19 AM, Tom Rini <tr...@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) > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot