All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Weber <weber@corscience.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] devkit8000: Move CONFIG_SYS_TEXT_BASE out of bss
Date: Thu, 01 Dec 2011 09:47:13 +0100	[thread overview]
Message-ID: <4ED73F11.1080903@corscience.de> (raw)
In-Reply-To: <1322672741-27396-1-git-send-email-simonschwarzcor@gmail.com>

Hello Simon,

On 30.11.2011 18:05, Simon Schwarz wrote:
> This moves CONFIG_SYS_TEXT_BASE one MB after beginning of SD-RAM. Move
> CONFIG_SYS_SPL_MALLOC_START to have one MB of free space for the u-boot image.
> 
> CONFIG_SYS_TEXT_BASE was in the middle of the bss-section. This was the reason
> for the problems with MMC boot described here:
> http://article.gmane.org/gmane.comp.boot-loaders.u-boot/118711
> 
> Signed-off-by: Simon Schwarz <simonschwarzcor@gmail.com>
> Cc: s-paulraj at ti.com
> Cc: wd at denx.de
> Cc: trini at ti.com
> ---
>  include/configs/devkit8000.h |    5 +++--
>  1 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/include/configs/devkit8000.h b/include/configs/devkit8000.h
> index e1743dc..b29d564 100644
> --- a/include/configs/devkit8000.h
> +++ b/include/configs/devkit8000.h
> @@ -37,7 +37,8 @@
>  #define CONFIG_OMAP3430		1	/* which is in a 3430 */
>  #define CONFIG_OMAP3_DEVKIT8000	1	/* working with DevKit8000 */
>  
> -#define	CONFIG_SYS_TEXT_BASE	0x80008000
> +/* 1 MB into SD-RAM to leave place for .bss-section */
> +#define	CONFIG_SYS_TEXT_BASE	0x80100000
>  
>  #define CONFIG_SDRC	/* The chip has SDRC controller */
>  
> @@ -359,7 +360,7 @@
>  #define CONFIG_SYS_NAND_U_BOOT_OFFS	0x80000
>  #define CONFIG_SYS_NAND_U_BOOT_SIZE	0x200000
>  
> -#define CONFIG_SYS_SPL_MALLOC_START	0x80108000
> +#define CONFIG_SYS_SPL_MALLOC_START	0x80200000
>  #define CONFIG_SYS_SPL_MALLOC_SIZE	0x100000	/* 1 MB */
>  
>  #endif /* __CONFIG_H */

Tested-by: Thomas Weber <weber@corscience.de>

boots from NAND and from MMC now, on both Devkit8000 versions (128 MiB
and 256 MiB).


Thomas

  parent reply	other threads:[~2011-12-01  8:47 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-30 17:05 [U-Boot] [PATCH] devkit8000: Move CONFIG_SYS_TEXT_BASE out of bss Simon Schwarz
2011-11-30 17:12 ` Simon Schwarz
2011-11-30 17:20 ` Tom Rini
2011-12-01  7:49 ` Igor Grinberg
2011-12-01  9:07   ` Simon Schwarz
2011-12-01 10:03     ` Igor Grinberg
2011-12-01 10:16       ` Aneesh V
2011-12-01 11:04         ` Igor Grinberg
2011-12-01  8:47 ` Thomas Weber [this message]
2011-12-06  9:16 ` [U-Boot] [PATCH V2] " Simon Schwarz
2011-12-06 13:52   ` Thomas Weber

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=4ED73F11.1080903@corscience.de \
    --to=weber@corscience.de \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.