[U-Boot] [PATCH v4 0/4] Homogeneize semantics of CONFIG_SPL_MAX_SIZE

[Benoît Thébaudeau]

Hi Albert,

On Sunday, April 14, 2013 4:10:37 PM, Albert ARIBAUD wrote:
> On Fri, 12 Apr 2013 17:14:29 +0200, Albert ARIBAUD
> <albert.u.boot at aribaud.net> wrote:
> 
> > CONFIG_SPL_MAX_SIZE and CONFIG_SPL_BSS_MAX_SIZE did not have constant
> > semantics across all of U-boot. This patch series aims at fixing this by
> > splitting the maximum size into separate image (code + data + rodata +
> > linker list) size on the one hand, and BSS size on the other hand.
> > 
> > Changes in v4:
> > - reordered patches so that FOOTPRINT is introduced first,
> >   then individual boards are switched to using it
> > - rewrote README entries
> > - converted to CONFIG_SPL_MAX_FOOTPRINT
> > - limited SPL size to exactly 6 2K pages
> > - converted to CONFIG_SPL_MAX_FOOTPRINT
> > - converted to CONFIG_SPL_MAX_FOOTPRINT
> > 
> > Changes in v3:
> > - introduced CONFIG_SPL_MAX_FOOTPRINT
> > - fixed typo in BSS size test
> > 
> > Changes in v2:
> > - removed mmutable in SPL linker file
> > - brought back total size to 12K
> > - fixed commit summary typoes
> > - fixed spacing in commit summary
> > 
> > Albert ARIBAUD (4):
> >   ARM: fix CONFIG_SPL_MAX_SIZE semantics
> >   cam_enc_4xx: convert to CONFIG_SPL_MAX_FOOTPRINT
> >   da850evm, da850_am18xxevm: convert to CONFIG_SPL_MAX_FOOTPRINT
> >   smdk5250, snow: convert to CONFIG_SPL_MAX_FOOTPRINT
> > 
> >  README                                         |   18 ++++++++++++++++--
> >  arch/arm/cpu/u-boot-spl.lds                    |   24
> >  +++++++++++++-----------
> >  arch/arm/cpu/u-boot.lds                        |    4 ----
> >  board/ait/cam_enc_4xx/u-boot-spl.lds           |    2 +-
> >  board/davinci/da8xxevm/u-boot-spl-da850evm.lds |    2 +-
> >  board/samsung/smdk5250/smdk5250-uboot-spl.lds  |    2 +-
> >  include/configs/cam_enc_4xx.h                  |    2 +-
> >  include/configs/da850evm.h                     |    2 +-
> >  include/configs/exynos5250-dt.h                |    2 +-
> >  include/configs/tegra-common.h                 |    2 +-
> >  10 files changed, 36 insertions(+), 24 deletions(-)
> > 
> 
> Applied to u-boot-arm/master, with README corrected on the fly as
> suggested by Tom.

This on-the-fly correction is incomplete:
CONFIG_SPL_MAX_BSS_SIZE -> CONFIG_SPL_BSS_MAX_SIZE

Best regards,
Benoît