diff mbox

[U-Boot] ARM: tegra: expand all SPL sizes to be consistent

Message ID 1442961372-5473-1-git-send-email-swarren@wwwdotorg.org
State Accepted
Delegated to: Tom Warren
Headers show

Commit Message

Stephen Warren Sept. 22, 2015, 10:36 p.m. UTC 
From: Stephen Warren <swarren@nvidia.com>

The size allocation for SPL is increased in all cases to match the
already-expanded value used on Tegra124. This is both for general
consistency, and because the seaboard build trips over the limit already
when using one of the ARM compilers packaged with 14.04. For the record,
when building Seaboard:

arm-linux-gnueabi- SPL is too big by 0x36 bytes
arm-linux-gnueabihf- SPL fits by 0x2a bytes
arm-none-eabi- SPL fits by 0xa bytes

(Those figures are from builds with the expanded SPL size allocation,
relative to the non-expanded SPL size limit; they're better by about
6 bytes in the more constrained build.)

Fixes: ba521994229c ("tegra124: Expand SPL space by 8KB")
Signed-off-by: Stephen Warren <swarren@nvidia.com>
---
 include/configs/tegra114-common.h | 2 +-
 include/configs/tegra20-common.h  | 2 +-
 include/configs/tegra210-common.h | 2 +-
 include/configs/tegra30-common.h  | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

Comments

Stephen Warren Sept. 23, 2015, 5:27 p.m. UTC | #1 
On 09/22/2015 04:36 PM, Stephen Warren wrote:
> From: Stephen Warren <swarren@nvidia.com>
>
> The size allocation for SPL is increased in all cases to match the
> already-expanded value used on Tegra124. This is both for general
> consistency, and because the seaboard build trips over the limit already
> when using one of the ARM compilers packaged with 14.04. For the record,
> when building Seaboard:
>
> arm-linux-gnueabi- SPL is too big by 0x36 bytes
> arm-linux-gnueabihf- SPL fits by 0x2a bytes
> arm-none-eabi- SPL fits by 0xa bytes
>
> (Those figures are from builds with the expanded SPL size allocation,
> relative to the non-expanded SPL size limit; they're better by about
> 6 bytes in the more constrained build.)

Oops, hold off on this one. I increased TEXT_BASE to 0x..111000 rather 
than 0x..110000 which would be consistent with the T124 value. I'll respin.
diff mbox

Patch

diff --git a/include/configs/tegra114-common.h b/include/configs/tegra114-common.h
index 252e607d73f4..33746cb90986 100644
--- a/include/configs/tegra114-common.h
+++ b/include/configs/tegra114-common.h
@@ -34,7 +34,7 @@ 
 /*-----------------------------------------------------------------------
  * Physical Memory Map
  */
-#define CONFIG_SYS_TEXT_BASE	0x8010E000
+#define CONFIG_SYS_TEXT_BASE	0x80111000
 
 /*
  * Memory layout for where various images get loaded by boot scripts:
diff --git a/include/configs/tegra20-common.h b/include/configs/tegra20-common.h
index 0841f33bfc9e..45b0a1a61427 100644
--- a/include/configs/tegra20-common.h
+++ b/include/configs/tegra20-common.h
@@ -32,7 +32,7 @@ 
 /*-----------------------------------------------------------------------
  * Physical Memory Map
  */
-#define CONFIG_SYS_TEXT_BASE	0x0010E000
+#define CONFIG_SYS_TEXT_BASE	0x00111000
 
 /*
  * Memory layout for where various images get loaded by boot scripts:
diff --git a/include/configs/tegra210-common.h b/include/configs/tegra210-common.h
index e6c815212d7b..aee26c14c272 100644
--- a/include/configs/tegra210-common.h
+++ b/include/configs/tegra210-common.h
@@ -26,7 +26,7 @@ 
 /*-----------------------------------------------------------------------
  * Physical Memory Map
  */
-#define CONFIG_SYS_TEXT_BASE	0x8010E000
+#define CONFIG_SYS_TEXT_BASE	0x80111000
 
 /* Generic Interrupt Controller */
 #define CONFIG_GICV2
diff --git a/include/configs/tegra30-common.h b/include/configs/tegra30-common.h
index 3e8e3c1e5bd9..b6952fdb1eee 100644
--- a/include/configs/tegra30-common.h
+++ b/include/configs/tegra30-common.h
@@ -31,7 +31,7 @@ 
 /*-----------------------------------------------------------------------
  * Physical Memory Map
  */
-#define CONFIG_SYS_TEXT_BASE	0x8010E000
+#define CONFIG_SYS_TEXT_BASE	0x80111000
 
 /*
  * Memory layout for where various images get loaded by boot scripts: