Skip to content

ARM: SPL: Make more generic, merge DaVinci and OMAP

Tom Rini u-boot 8 versions – New

Get series with b4
b4 am 20120927200432.GL14678@bill-the-cat

Versions newest first · linked by author and title

  1. v7 Latest posting 2/20 of 29 patches 0 repliesNew
  2. v7 posting 1/229 patches 0 repliesSuperseded
  3. v629 patches 0 repliesSuperseded
Earlier versions · 5
VersionPostedPatchesRepliesStatus
v525 patches0 repliesSuperseded
v420 patches0 repliesSuperseded
v319 patches0 repliesSuperseded
v218 patches0 repliesSuperseded
v10 of 17 patches0 repliesSuperseded

Patches in v7 posting 2/2 No applies recorded yet

None of the 29 patches the cover letter announces are in this archive.

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v7 yet.

Read the v7 thread

Cover letter [U-Boot] [PATCH v7 0/29] ARM: SPL: Make more generic, merge DaVinci and OMAP
Quoted 31 lines · Tom Rini

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.

v6 is tested on:
- AM335x EVM, Beaglebone, Beagleboard rev C (NAND and MMC), Beagleboard xM, OMAP3EVM (MMC) Devkit8000 (MMC + SPL OS goto U-Boot instead, no kernel so no SPL Linux test), Pandaboard, Pandaboard ES, LogicPD AM1808 EVM

Quoted 6 lines

v7 is tested on:
- Beaglebone, Beagleboard rev C (NAND and MMC), Beagleboard xM.

I did more limited testing this time due to the scope of the changes.

'reset' and cold power-on tested in various combinations.

With the one change I have mentioned already, I have now pushed this. I figure going this direcion will hopefully make the merging for Albert and others simpiler.

Signature · 8 lines
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20120927/abefb4d6/attachment.pgp>

Read the cover letter in its thread