[U-Boot] [PATCH v2 4/5] Fix env address for Sheevaplug board

[Prafulla Wadaskar]

> -----Original Message-----
> From: DrEagle [mailto:dreagle at doukki.net]
> Sent: 26 September 2012 11:06
> To: Prafulla Wadaskar
> Cc: DrEagle; u-boot at lists.denx.de; uboot at doukki.net
> Subject: Re: [U-Boot] [PATCH v2 4/5] Fix env address for Sheevaplug
> board
> 
> Le 24/09/2012 10:46, Prafulla Wadaskar a écrit :
> >
> >
> >> -----Original Message-----
> >> From: u-boot-bounces at lists.denx.de [mailto:u-boot-
> >> bounces at lists.denx.de] On Behalf Of DrEagle
> >> Sent: 22 September 2012 12:48
> >> To: DrEagle
> >> Cc: Lior Amsalem; u-boot at lists.denx.de; uboot at doukki.net
> >> Subject: [U-Boot] [PATCH v2 4/5] Fix env address for Sheevaplug
> board
> >>
> >>
> >> Fix env address for Sheevaplug board
> >> Move env address and offset from 0x60000 to 0x80000
> >>
> >> Signed-off-by: drEagle <dreagle at doukki.net>
> >> ---
> >>  include/configs/sheevaplug.h |    5 ++---
> >>  1 file changed, 2 insertions(+), 3 deletions(-)
> >>
> >> diff --git a/include/configs/sheevaplug.h
> >> b/include/configs/sheevaplug.h
> >> index 7d710bf..a10e825 100644
> >> --- a/include/configs/sheevaplug.h
> >> +++ b/include/configs/sheevaplug.h
> >> @@ -71,9 +71,8 @@
> >>   * it has to be rounded to sector size
> >>   */
> >>  #define CONFIG_ENV_SIZE			0x20000	/* 128k */
> >> -#define CONFIG_ENV_ADDR			0x60000
> >> -#define CONFIG_ENV_OFFSET		0x60000	/* env starts here */
> >> -
> >> +#define CONFIG_ENV_ADDR			0x80000
> >> +#define CONFIG_ENV_OFFSET		0x80000	/* env starts here */
> >
> > Why do you need this shift?
> 
> Because the size of the new u-boot binary overlap the actual env
> offset.
> If you reset the env or save env with a new u-boot, the code is
> corrupt, and noo boot anymore.
> 
> This shift fix this.

Then this will be valid for all the boards on which MMC support is being enabled, right?

In that case for sheevaplug, make this change in one patch in your next version (i.e. relocating ENV offset, and enabling MMC support)

Regards...
Prafulla . . .