From mboxrd@z Thu Jan 1 00:00:00 1970 From: Albert ARIBAUD Date: Sat, 22 Jun 2013 11:29:59 +0200 Subject: [U-Boot] [PATCH] OpenRD: relocate environment to 640kB In-Reply-To: <20130611120019.182fa315@lilith> References: <1369647746-18120-1-git-send-email-t-uboot@infra-silbe.de> <20130611120019.182fa315@lilith> Message-ID: <20130622112959.7395cf62@lilith> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Tue, 11 Jun 2013 12:00:19 +0200, Albert ARIBAUD wrote: > Hi Sascha, > > On Mon, 27 May 2013 11:42:26 +0200, Sascha Silbe > 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 > > --- > > 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.