[U-Boot] [PATCH v3 00/12] U-Boot MMC SPL for OMAP4

Aneesh V
2011-07-18T13:38:37Z

Dear Wolfgang,

On Saturday 16 July 2011 06:23 PM, Aneesh V wrote:
> This series adds mmc SPL support for OMAP4. This is essentially
> an up-streaming effort for TI's x-loader for OMAP4 using the SPL
> framework
>
> This work partly draws upon previous work done for x-loader by:
> Santosh Shilimkar<santosh.shilimkar@ti.com>
> Rajendra Nayak<rnayak@ti.com>
> and many others
>
> This series depends on the following series for the new SPL framework:
> http://marc.info/?l=u-boot&m=131056990001719&w=2
>
> Features supported:
> * Clock init - DPLL locking, clock domains/modules enabling
> * SDRAM/DMM initialization
> * Mux initialization
> * MMC raw read support
> * MMC FAT read support
>
> Improvments over x-loader:
> * Code completely revamped
> * DPLL programming that works for all SYS_CLK frequencies
> * Automatic SDRAM identification and initialization
> * Board dependent parts(especially mux) deferred to u-boot. So, same
> SPL will work for Panda and SDP and most likely on any other board
> that comes along
> * Flexible about the payload. Any image with a mkimage header can be
> loaded.
> * The core SPL logic is generic and available for OMAP3/4. So, OMAP3
> platforms can easily extend this work to add SPL support for the
> respective boards.
>
> V2:
> * Some fixes in SDRAM init, clock init
> * Use of pre-calculated M& N values for DPLL locking instead
> run-time calculation
> * Change in make files to allow reuse of common rules
> between boards
> * Console initialization moved further earlier in the boot
> allowing traces from even clock initialization code
> * Reuse utility macros from a previous series
>
> V3:
> * Moved to the new SPL framework
> * struct based register access for clock init
> * Squashed together related patches as per Wolfgang's suggestions
> * Re-organization of code in sdram init for better readability
> * SPL loading the payload using mkimage header
> * Avoided using custom bit-field accessor macros
> * Several other minor changes(documented in respective patches)

I think I have addressed all the comments you and others had on the
previous series and adapted it to the new SPL framework.

There are just two comments for this series so far. I have to omit the
'SYS' from CONFIG_SYS_SPL_TEXT_BASE and a minor comment by Simon.

If I fix these two, is it ready to go. Do you have any other comments.
If not, shall I re-spin the next revision?

br,
Aneesh