[U-Boot] [PATCH] OpenRD: relocate environment to 640kB

[Tom Rini]

On Sat, Jun 22, 2013 at 11:29:59AM +0200, Albert ARIBAUD wrote:

> On Tue, 11 Jun 2013 12:00:19 +0200, Albert ARIBAUD
> <albert.u.boot at aribaud.net> wrote:
> 
> > Hi Sascha,
> > 
> > On Mon, 27 May 2013 11:42:26 +0200, Sascha Silbe
> > <t-uboot at infra-silbe.de> wrote:
> > 
> > > U-Boot has become slightly larger than where the environment sector
> > > was previously located. Saving the environment would brick the device.
> > > 
> > > Relocate the environment to where it is with the stock (i.e. Marvell
> > > USP) U-Boot version. That should give plenty of room for U-Boot to
> > > grow, including local customisations.
> > > 
> > > This obviously breaks compatibility with previous mainline versions of
> > > U-Boot. Users will need to back up the environment before an update
> > > and restore it afterwards, or manually copy it to the new address
> > > before the update.
> > > 
> > > Signed-off-by: Sascha Silbe <t-uboot at infra-silbe.de>
> > > ---
> > >  include/configs/openrd.h | 4 ++--
> > >  1 file changed, 2 insertions(+), 2 deletions(-)
> > > 
> > > diff --git a/include/configs/openrd.h b/include/configs/openrd.h
> > > index 53bafe1..ea7f9aa 100644
> > > --- a/include/configs/openrd.h
> > > +++ b/include/configs/openrd.h
> > > @@ -90,8 +90,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			0xa0000
> > > +#define CONFIG_ENV_OFFSET		0xa0000	/* env starts here */
> > >  
> > >  /*
> > >   * Default environment variables
> > 
> > CC:ing Tom.
> > 
> > This patch is for 2013.10, not 2013.07, but I prefer raising the issue
> > as early as possible.
> > 
> > If there is no way to make things smoother, then I think the 2013.10
> > release notes should contain a red, blinking, paragraph about this. I
> > would *hate* it if people were not warned and given a method to port
> > their current environment setting over.
> > 
> > Possibly even, the 2013.07 could have a warning about the change to
> > come, so that people have a better chance yet to prepare for the change.
> > 
> > Amicalement,
> 
> Ping.

Oh right, agree your idea here.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20130622/9653d35d/attachment.pgp>