public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Tom Rini <trini@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 01/12] am33xx: Correct and expand comments on CONFIG_SPL_MAX_SIZE
Date: Tue, 20 Aug 2013 08:53:43 -0400	[thread overview]
Message-ID: <1377003234-16792-2-git-send-email-trini@ti.com> (raw)
In-Reply-To: <1377003234-16792-1-git-send-email-trini@ti.com>

We had been allowing the max size to be larger than actually allowed by
the ROM.  Expand the commentary here to explain why we set these
locations.

Signed-off-by: Tom Rini <trini@ti.com>
---
 arch/arm/include/asm/arch-am33xx/omap.h |    2 +-
 include/configs/ti_am335x_common.h      |   10 ++++++++--
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/arch/arm/include/asm/arch-am33xx/omap.h b/arch/arm/include/asm/arch-am33xx/omap.h
index 1f84311..2250721 100644
--- a/arch/arm/include/asm/arch-am33xx/omap.h
+++ b/arch/arm/include/asm/arch-am33xx/omap.h
@@ -18,7 +18,7 @@
 #ifdef CONFIG_AM33XX
 #define NON_SECURE_SRAM_START	0x402F0400
 #define NON_SECURE_SRAM_END	0x40310000
-#define SRAM_SCRATCH_SPACE_ADDR	0x4030C000
+#define SRAM_SCRATCH_SPACE_ADDR	0x4030B800
 #elif defined(CONFIG_TI81XX)
 #define NON_SECURE_SRAM_START	0x40300000
 #define NON_SECURE_SRAM_END	0x40320000
diff --git a/include/configs/ti_am335x_common.h b/include/configs/ti_am335x_common.h
index fd3ffab..0eae9c8 100644
--- a/include/configs/ti_am335x_common.h
+++ b/include/configs/ti_am335x_common.h
@@ -41,9 +41,15 @@
 #define CONFIG_BOOTP_SUBNETMASK
 #define CONFIG_NET_RETRY_COUNT         10
 
-/* SPL defines. */
+/*
+ * SPL related defines.  The Public RAM memory map the ROM defines the
+ * area between 0x402F0400 and 0x4030B800 as a download area and
+ * 0x4030B800 to 0x4030CE00 as a public stack area.  The ROM also
+ * supports X-MODEM loading via UART, and we leverage this and then use
+ * Y-MODEM to load u-boot.img, when booted over UART.
+ */
 #define CONFIG_SPL_TEXT_BASE		0x402F0400
-#define CONFIG_SPL_MAX_SIZE		(0x4030C000 - CONFIG_SPL_TEXT_BASE)
+#define CONFIG_SPL_MAX_SIZE		(0x4030B800 - CONFIG_SPL_TEXT_BASE)
 #define CONFIG_SPL_YMODEM_SUPPORT
 
 /*
-- 
1.7.9.5

  reply	other threads:[~2013-08-20 12:53 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-20 12:53 [U-Boot] [PATCH 00/12] am33xx/omap5: Improve docs for customization Tom Rini
2013-08-20 12:53 ` Tom Rini [this message]
2013-08-20 12:53 ` [U-Boot] [PATCH 02/12] TI:armv7: Move CONFIG_SPL_LIBDISK_SUPPORT to MMC section Tom Rini
2013-08-20 12:53 ` [U-Boot] [PATCH 03/12] omap5: Expand CONFIG_SPL_MAX_SIZE and comment upon SRAM_SCRATCH_SPACE_ADDR Tom Rini
2013-08-21  5:22   ` Sricharan R
2013-08-21 13:49     ` Tom Rini
2013-08-20 12:53 ` [U-Boot] [PATCH 04/12] TI:am335x: Better comment and organize the networking related options Tom Rini
2013-08-20 12:53 ` [U-Boot] [PATCH 05/12] am335x_evm: Add comment by SPL SPI support Tom Rini
2013-08-20 12:53 ` [U-Boot] [PATCH 06/12] am335x_evm: Regroup USB options Tom Rini
2013-08-20 12:53 ` [U-Boot] [PATCH 07/12] TI:armv7: Re-order slightly the generic CONFIG options, expand related comments Tom Rini
2013-08-20 12:53 ` [U-Boot] [PATCH 08/12] am335x_evm: Update README for customization Tom Rini
2013-08-20 12:53 ` [U-Boot] [PATCH 09/12] TI:am33xx: Move SPL YMODEM support to the per-board config Tom Rini
2013-08-20 12:53 ` [U-Boot] [PATCH 10/12] TI:omap5: Clarify comments about SPL and DDR timings in common config Tom Rini
2013-08-20 12:53 ` [U-Boot] [PATCH 11/12] omap5_uevm: Better comment why we have TCA642X and the reset time Tom Rini
2013-08-20 12:53 ` [U-Boot] [PATCH 12/12] dra7xx_evm: Re-order and comment the networking related config options Tom Rini
2013-08-28 18:25 ` [U-Boot] [PATCH 00/12] am33xx/omap5: Improve docs for customization Tom Rini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1377003234-16792-2-git-send-email-trini@ti.com \
    --to=trini@ti.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox