OMAP4: Broken SPL generation
Building the SPL for omap4_sdp4430 and omap4_panda in recent mainline is broken due to a resulting image size > 32k:
Configuring for omap4_sdp4430 board...
arm-none-linux-gnueabi-ld: u-boot-spl section .rodata will not fit in region .sram
arm-none-linux-gnueabi-ld: region .sram overflowed by 612 bytes
Configuring for omap4_panda board...
arm-none-linux-gnueabi-ld: u-boot-spl section .rodata will not fit in region .sram
arm-none-linux-gnueabi-ld: region .sram overflowed by 620 bytes
This seems to come from the the patch "omap: add MMC and FAT support to SPL" [1] adding a lot of code to the SPL.
Does anybody have a quick solution for this fixing the build of these boards for the -rc cycle? Else I'm about to send a patch like [2] disabling the SPL build until a patch to reduce the image size is available.
Best regards
Dirk
[2]
include/configs/omap4_panda.h | 2 +- include/configs/omap4_sdp4430.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)
include/configs/omap4_panda.h +1 -1
@@ -243,7 +243,7 @@#endif /* Defines for SPL */ -#define CONFIG_SPL +//#define CONFIG_SPL #define CONFIG_SPL_TEXT_BASE 0x40304350#define CONFIG_SPL_MAX_SIZE 0x8000/* 32 K */ #define CONFIG_SPL_STACK LOW_LEVEL_SRAM_STACK
include/configs/omap4_sdp4430.h +1 -1
@@ -249,7 +249,7 @@#endif /* Defines for SPL */ -#define CONFIG_SPL +//#define CONFIG_SPL #define CONFIG_SPL_TEXT_BASE 0x40304350#define CONFIG_SPL_MAX_SIZE 0x8000/* 32 K */ #define CONFIG_SPL_STACK LOW_LEVEL_SRAM_STACK
Hi Dirk,
Le 05/08/2011 20:54, Dirk Behme a ?crit :
Quoted 10 lines
Does anybody have a quick solution for this fixing the build of these boards for the -rc cycle? Else I'm about to send a patch like [2] disabling the SPL build until a patch to reduce the image size is available.
[2]
include/configs/omap4_panda.h | 2 +- include/configs/omap4_sdp4430.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)[...]
Just a side note: that's the second time someone quotes a patch or part of patch in a mail not intended to be a patch submission, only patchwork mistakenly takes it as a proper patch submission and either creates an unwanted patch entry or even worse, breaks the discussion on an original patch
Maybe a good habit to keep in mind is to only quote the true diff part of the patch or to actually quote it (with "> " prepended).
Amicalement,
Signature · 1 line
Albert.
Hi Dirk,
I can fix it. We have more SRAM available in 4430. I can increase the SRAM budget. I will send a patch shortly.
br,
Aneesh
Quoted 62 lines · Dirk Behme
Building the SPL for
omap4_sdp4430andomap4_pandain recent mainline is broken due to a resulting image size > 32k:Configuring for
omap4_sdp4430board...
arm-none-linux-gnueabi-ld: u-boot-spl section .rodata will not fit in region .sram
arm-none-linux-gnueabi-ld: region .sram overflowed by 612 bytesConfiguring for
omap4_pandaboard...
arm-none-linux-gnueabi-ld: u-boot-spl section .rodata will not fit in region .sram
arm-none-linux-gnueabi-ld: region .sram overflowed by 620 bytesThis seems to come from the the patch "omap: add MMC and FAT support to SPL" [1] adding a lot of code to the SPL.
Does anybody have a quick solution for this fixing the build of these boards for the -rc cycle? Else I'm about to send a patch like [2] disabling the SPL build until a patch to reduce the image size is available.
Best regards
Dirk
[2]
include/configs/omap4_panda.h| 2 +-include/configs/omap4_sdp4430.h| 2 +-2 files changed, 2 insertions(+), 2 deletions(-)Index:
u-boot.git/include/configs/omap4_panda.h
===================================================================include/configs/omap4_panda.h +1
-1@@ -243,7 +243,7 @@#endif /* Defines for SPL */ -#define CONFIG_SPL +//#define CONFIG_SPL #define CONFIG_SPL_TEXT_BASE 0x40304350#define CONFIG_SPL_MAX_SIZE 0x8000/* 32 K */ #define CONFIG_SPL_STACK LOW_LEVEL_SRAM_STACKIndex:
u-boot.git/include/configs/omap4_sdp4430.h
===================================================================include/configs/omap4_sdp4430.h +1
-1@@ -249,7 +249,7 @@#endif /* Defines for SPL */ -#define CONFIG_SPL +//#define CONFIG_SPL #define CONFIG_SPL_TEXT_BASE 0x40304350#define CONFIG_SPL_MAX_SIZE 0x8000/* 32 K */ #define CONFIG_SPL_STACK LOW_LEVEL_SRAM_STACK
PATCH 1/1 omap4: increase SRAM budget to fix build error
In reply to Dirk Behme--- include/configs/omap4_panda.h | 2 +- include/configs/omap4_sdp4430.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)
include/configs/omap4_panda.h +1 -1
@@ -245,7 +245,7 @@/* Defines for SPL */ #define CONFIG_SPL #define CONFIG_SPL_TEXT_BASE 0x40304350 -#define CONFIG_SPL_MAX_SIZE 0x8000/* 32 K */ +#define CONFIG_SPL_MAX_SIZE (38 * 1024)#define CONFIG_SPL_STACK LOW_LEVEL_SRAM_STACK#define CONFIG_SPL_BSS_START_ADDR 0x80000000
include/configs/omap4_sdp4430.h +1 -1
@@ -251,7 +251,7 @@/* Defines for SPL */ #define CONFIG_SPL #define CONFIG_SPL_TEXT_BASE 0x40304350 -#define CONFIG_SPL_MAX_SIZE 0x8000/* 32 K */ +#define CONFIG_SPL_MAX_SIZE (38 * 1024)#define CONFIG_SPL_STACK LOW_LEVEL_SRAM_STACK#define CONFIG_SPL_BSS_START_ADDR 0x80000000
Signature · 1 line
1.7.0.4
PATCH 1/1 omap4: increase SRAM budget to fix build error
Quoted 3 lines · Aneesh V
Signed-off-by Aneesh V<[email protected]>Cc Dirk Behme<[email protected]>Cc Sandeep Paulraj<[email protected]>
Quoted 31 lines
--- include/configs/omap4_panda.h | 2 +- include/configs/omap4_sdp4430.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)include/configs/omap4_panda.h +1
-1@@ -245,7 +245,7 @@/* Defines for SPL */ #define CONFIG_SPL #define CONFIG_SPL_TEXT_BASE 0x40304350 -#define CONFIG_SPL_MAX_SIZE 0x8000/* 32 K */ +#define CONFIG_SPL_MAX_SIZE (38 * 1024)#define CONFIG_SPL_STACK LOW_LEVEL_SRAM_STACK#define CONFIG_SPL_BSS_START_ADDR 0x80000000include/configs/omap4_sdp4430.h +1
-1@@ -251,7 +251,7 @@/* Defines for SPL */ #define CONFIG_SPL #define CONFIG_SPL_TEXT_BASE 0x40304350 -#define CONFIG_SPL_MAX_SIZE 0x8000/* 32 K */ +#define CONFIG_SPL_MAX_SIZE (38 * 1024)#define CONFIG_SPL_STACK LOW_LEVEL_SRAM_STACK#define CONFIG_SPL_BSS_START_ADDR 0x80000000