* [U-Boot] OMAP4: Broken SPL generation
@ 2011-08-05 18:54 Dirk Behme
2011-08-07 9:50 ` Albert ARIBAUD
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Dirk Behme @ 2011-08-05 18:54 UTC (permalink / raw)
To: u-boot
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
[1]
http://git.denx.de/cgi-bin/gitweb.cgi?p=u-boot.git;a=commit;h=8cf686e19b6922a2a6b401af188dfb83414c3c04
[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
===================================================================
--- u-boot.git.orig/include/configs/omap4_panda.h
+++ u-boot.git/include/configs/omap4_panda.h
@@ -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
Index: u-boot.git/include/configs/omap4_sdp4430.h
===================================================================
--- u-boot.git.orig/include/configs/omap4_sdp4430.h
+++ u-boot.git/include/configs/omap4_sdp4430.h
@@ -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
^ permalink raw reply [flat|nested] 5+ messages in thread* [U-Boot] OMAP4: Broken SPL generation
2011-08-05 18:54 [U-Boot] OMAP4: Broken SPL generation Dirk Behme
@ 2011-08-07 9:50 ` Albert ARIBAUD
2011-08-08 5:56 ` Aneesh V
2011-08-08 6:05 ` [U-Boot] [PATCH] omap4: increase SRAM budget to fix build error Aneesh V
2 siblings, 0 replies; 5+ messages in thread
From: Albert ARIBAUD @ 2011-08-07 9:50 UTC (permalink / raw)
To: u-boot
Hi Dirk,
Le 05/08/2011 20:54, Dirk Behme a ?crit :
> 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,
--
Albert.
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] OMAP4: Broken SPL generation
2011-08-05 18:54 [U-Boot] OMAP4: Broken SPL generation Dirk Behme
2011-08-07 9:50 ` Albert ARIBAUD
@ 2011-08-08 5:56 ` Aneesh V
2011-08-08 6:05 ` [U-Boot] [PATCH] omap4: increase SRAM budget to fix build error Aneesh V
2 siblings, 0 replies; 5+ messages in thread
From: Aneesh V @ 2011-08-08 5:56 UTC (permalink / raw)
To: u-boot
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
On Saturday 06 August 2011 12:24 AM, Dirk Behme wrote:
>
> 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
>
> [1]
> http://git.denx.de/cgi-bin/gitweb.cgi?p=u-boot.git;a=commit;h=8cf686e19b6922a2a6b401af188dfb83414c3c04
>
>
> [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
> ===================================================================
> --- u-boot.git.orig/include/configs/omap4_panda.h
> +++ u-boot.git/include/configs/omap4_panda.h
> @@ -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
> Index: u-boot.git/include/configs/omap4_sdp4430.h
> ===================================================================
> --- u-boot.git.orig/include/configs/omap4_sdp4430.h
> +++ u-boot.git/include/configs/omap4_sdp4430.h
> @@ -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
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH] omap4: increase SRAM budget to fix build error
2011-08-05 18:54 [U-Boot] OMAP4: Broken SPL generation Dirk Behme
2011-08-07 9:50 ` Albert ARIBAUD
2011-08-08 5:56 ` Aneesh V
@ 2011-08-08 6:05 ` Aneesh V
2011-08-08 14:51 ` Dirk Behme
2 siblings, 1 reply; 5+ messages in thread
From: Aneesh V @ 2011-08-08 6:05 UTC (permalink / raw)
To: u-boot
Signed-off-by: Aneesh V <aneesh@ti.com>
Cc: Dirk Behme <dirk.behme@googlemail.com>
Cc: Sandeep Paulraj <s-paulraj@ti.com>
---
include/configs/omap4_panda.h | 2 +-
include/configs/omap4_sdp4430.h | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/configs/omap4_panda.h b/include/configs/omap4_panda.h
index e313231..814f15c 100644
--- a/include/configs/omap4_panda.h
+++ b/include/configs/omap4_panda.h
@@ -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
diff --git a/include/configs/omap4_sdp4430.h b/include/configs/omap4_sdp4430.h
index 5b3110c..1a5d7a6 100644
--- a/include/configs/omap4_sdp4430.h
+++ b/include/configs/omap4_sdp4430.h
@@ -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
--
1.7.0.4
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH] omap4: increase SRAM budget to fix build error
2011-08-08 6:05 ` [U-Boot] [PATCH] omap4: increase SRAM budget to fix build error Aneesh V
@ 2011-08-08 14:51 ` Dirk Behme
0 siblings, 0 replies; 5+ messages in thread
From: Dirk Behme @ 2011-08-08 14:51 UTC (permalink / raw)
To: u-boot
On 08.08.2011 08:05, Aneesh V wrote:
> Signed-off-by: Aneesh V<aneesh@ti.com>
> Cc: Dirk Behme<dirk.behme@googlemail.com>
> Cc: Sandeep Paulraj<s-paulraj@ti.com>
Acked-by: Dirk Behme<dirk.behme@googlemail.com>
> ---
> include/configs/omap4_panda.h | 2 +-
> include/configs/omap4_sdp4430.h | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/include/configs/omap4_panda.h b/include/configs/omap4_panda.h
> index e313231..814f15c 100644
> --- a/include/configs/omap4_panda.h
> +++ b/include/configs/omap4_panda.h
> @@ -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
> diff --git a/include/configs/omap4_sdp4430.h b/include/configs/omap4_sdp4430.h
> index 5b3110c..1a5d7a6 100644
> --- a/include/configs/omap4_sdp4430.h
> +++ b/include/configs/omap4_sdp4430.h
> @@ -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
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2011-08-08 14:51 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-08-05 18:54 [U-Boot] OMAP4: Broken SPL generation Dirk Behme
2011-08-07 9:50 ` Albert ARIBAUD
2011-08-08 5:56 ` Aneesh V
2011-08-08 6:05 ` [U-Boot] [PATCH] omap4: increase SRAM budget to fix build error Aneesh V
2011-08-08 14:51 ` Dirk Behme
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox