public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Albert ARIBAUD <albert.u.boot@aribaud.net>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] OpenRD: relocate environment to 640kB
Date: Sat, 22 Jun 2013 11:29:59 +0200	[thread overview]
Message-ID: <20130622112959.7395cf62@lilith> (raw)
In-Reply-To: <20130611120019.182fa315@lilith>

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.

  reply	other threads:[~2013-06-22  9:29 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-27  9:42 [U-Boot] [PATCH] OpenRD: relocate environment to 640kB Sascha Silbe
2013-06-11 10:00 ` Albert ARIBAUD
2013-06-22  9:29   ` Albert ARIBAUD [this message]
2013-06-22 20:57     ` Tom Rini
2013-06-25  9:42   ` Sascha Silbe
2013-06-27  9:41     ` Albert ARIBAUD
2013-07-11 16:15       ` Tom Rini
2013-07-11 17:40         ` Albert ARIBAUD
2013-07-11 18:08           ` Tom Rini
2013-07-15  9:23       ` Sascha Silbe
2013-07-15 11:55         ` Albert ARIBAUD
2013-07-15 12:19         ` Tom Rini
2013-07-29  7:24           ` Albert ARIBAUD
2013-08-11 14:49             ` Sascha Silbe
2013-08-11 14:40           ` [U-Boot] [PATCH 0/3] openrd: Check U-Boot size at build time Sascha Silbe
2013-08-11 14:40             ` [U-Boot] [PATCH 1/3] README: document CONFIG_BOARD_SIZE_LIMIT Sascha Silbe
2014-08-14 14:08               ` Anatolij Gustschin
2013-08-11 14:40             ` [U-Boot] [PATCH 2/3] Makefile: check native boot image sizes against CONFIG_BOARD_SIZE_LIMIT Sascha Silbe
2014-08-14 14:10               ` Anatolij Gustschin
2013-08-11 14:40             ` [U-Boot] [PATCH 3/3] openrd: fail build if U-Boot would overlap with environment in flash Sascha Silbe
2014-08-14 14:09               ` Anatolij Gustschin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20130622112959.7395cf62@lilith \
    --to=albert.u.boot@aribaud.net \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox