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

Benoît Thébaudeau
2013-04-14T14:15:59Z

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@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