From: Wolfgang Denk <wd@denx.de>
Date: Tue, 25 Mar 2008 22:59:25 +0100
Subject: [U-Boot-Users] [PATCH] [new uImage] Fix build issue on ARM
In-Reply-To: Your message of "Wed, 05 Mar 2008 10:55:12 CST."
	<Pine.LNX.4.64.0803051055010.17721@blarg.am.freescale.net>
Message-ID: <20080325215925.459EC248BF@gemini.denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

In message <Pine.LNX.4.64.0803051055010.17721@blarg.am.freescale.net> you wrote:
> ARM platforms don't have a bd->bi_memsize so use bd->bi_dram[0].size instead.
> 
> Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
> ---
> 
> Let's see if this fixes the issue on ARM.
> 
>  common/image.c |    8 +++++++-
>  1 files changed, 7 insertions(+), 1 deletions(-)
> 
> diff --git a/common/image.c b/common/image.c
> index 9e446fa..4e0e96d 100644
> --- a/common/image.c
> +++ b/common/image.c
> @@ -140,8 +140,10 @@ ulong getenv_bootm_low(void)
>  		return tmp;
>  	}
> 
> -#ifdef CFG_SDRAM_BASE
> +#if defined(CFG_SDRAM_BASE)
>  	return CFG_SDRAM_BASE;
> +#elif defined(CONFIG_ARM)
> +	return gd->bd->bi_dram[0].start;

But this part of the patch is not mentioned in the description, and I
don't think it's necessary (nor necessarily correct).


Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
There's another way to survive.  Mutual trust -- and help.
	-- Kirk, "Day of the Dove", stardate unknown