From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?utf-8?Q?Beno=C3=AEt_Th=C3=A9baudeau?= Date: Sun, 14 Apr 2013 16:15:59 +0200 (CEST) Subject: [U-Boot] [PATCH v4 0/4] Homogeneize semantics of CONFIG_SPL_MAX_SIZE In-Reply-To: <20130414161037.2360b56c@lilith> References: <1365767743-23585-1-git-send-email-albert.u.boot@aribaud.net> <1365779673-17354-1-git-send-email-albert.u.boot@aribaud.net> <20130414161037.2360b56c@lilith> Message-ID: <1006753505.1729225.1365948959280.JavaMail.root@advansee.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 > 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