[U-Boot] [PATCH] Env. vars fix for lite5200

[Wolfgang Denk]

Dear Karim Yaghmour,

In message <cce4b6d6c15fe8f412e82dcbacc5d56c at mail.gmail.com> you wrote:
> 
> I still have a few older lite5200 boards around and when I recently tried
> to upgrade U-Boot I ran into weird problems where "saveenv" would fry the
> bootloader. Attached is a fix. It seems that somewhere along the way the
> image size grew larger than 256K and the address for the env vars
> overlapped with the code.
...
> diff -urN u-boot-2010.12/include/configs/IceCube.h u-boot-2010.12-fix-lite5200//include/configs/IceCube.h
> --- u-boot-2010.12/include/configs/IceCube.h	2010-12-22 14:22:14.000000000 -0500
> +++ u-boot-2010.12-fix-lite5200//include/configs/IceCube.h	2011-02-23 16:46:35.751137042 -0500

Proper commit message and Signed-off-by: line missing.

Also, patches should be made using "git format-patch". If you really
have to do it by hand, then please make sure that they can be applied
with "git am", i. e. at least make sure that "patch -p1 works", and
that your attached patch has the full information including From:,
Date: and Subject.

> @@ -234,7 +234,7 @@
>  #define CONFIG_ENV_ADDR		(CONFIG_SYS_FLASH_BASE + 0x00040000 + 0x00800000)
>  #endif
>  #if defined(CONFIG_SYS_LOWBOOT16)
> -#define CONFIG_ENV_ADDR		(CONFIG_SYS_FLASH_BASE + 0x00040000)
> +#define CONFIG_ENV_ADDR		(CONFIG_SYS_FLASH_BASE + 0x00050000)
>  #endif
>  #endif	/* CONFIG_SYS_LOWBOOT */
>  #endif /* CONFIG_LITE5200B */

Hm... This should also be fixed for the highboot case, then.

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
Quotation, n. The act of repeating erroneously the words of  another.
The  words  erroneously  repeated.
                            - Ambrose Bierce _The Devil's Dictionary_