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

Albert ARIBAUD
2013-06-22T09:29:59Z

On Tue, 11 Jun 2013 12:00:19 +0200, Albert ARIBAUD
<albert.u.boot@aribaud.net> wrote:

> Hi Sascha,
>
> On Mon, 27 May 2013 11:42:26 +0200, Sascha Silbe
> <t-uboot@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@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.

Amicalement,
--
Albert.