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

[Albert ARIBAUD <albert.u.boot@aribaud.net>]

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,
-- 
Albert.