[U-Boot] [PATCH 27/31] M28: Save environment in NAND

[Marek Vasut]

On Friday, September 16, 2011 09:15:12 PM Scott Wood wrote:
> On 09/08/2011 03:42 PM, Marek Vasut wrote:
> > Signed-off-by: Marek Vasut <marek.vasut at gmail.com>
> > Cc: Stefano Babic <sbabic at denx.de>
> > Cc: Wolfgang Denk <wd at denx.de>
> > Cc: Detlev Zundel <dzu at denx.de>
> > ---
> > 
> >  include/configs/m28evk.h |   11 +++++++++--
> >  1 files changed, 9 insertions(+), 2 deletions(-)
> > 
> > diff --git a/include/configs/m28evk.h b/include/configs/m28evk.h
> > index 0d4a0a3..c120c63 100644
> > --- a/include/configs/m28evk.h
> > +++ b/include/configs/m28evk.h
> > @@ -131,6 +131,15 @@
> > 
> >  #define	CONFIG_SYS_NAND_5_ADDR_CYCLE
> >  #define	NAND_MAX_CHIPS			8
> > 
> > +/* Environment is in NAND */
> > +#define	CONFIG_ENV_IS_IN_NAND		1
> > +#define	CONFIG_ENV_SECT_SIZE		(128 * 1024)
> > +#define	CONFIG_ENV_SIZE			(16 * 1024)
> > +#define	CONFIG_ENV_SIZE_REDUND		CONFIG_ENV_SIZE
> > +#define	CONFIG_ENV_OFFSET		0x300000
> > +#define	CONFIG_ENV_OFFSET_REDUND	\
> > +		(CONFIG_ENV_OFFSET + CONFIG_ENV_SECT_SIZE)
> > +
> > 
> >  #define	CONFIG_CMD_UBI
> >  #define	CONFIG_CMD_UBIFS
> >  #define	CONFIG_CMD_MTDPARTS
> > 
> > @@ -230,6 +239,4 @@
> > 
> >  #define	CONFIG_LOADADDR		0x42000000
> >  #define	CONFIG_SYS_LOAD_ADDR	CONFIG_LOADADDR
> > 
> > -#define	CONFIG_ENV_IS_NOWHERE	1
> > -
> > 
> >  #endif /* __M28_H__ */
> 
> Consider using CONFIG_ENV_RANGE to allow for bad blocks.

How does this actually play with CONFIG_MTDPARTS ?

For you see -- I now have
CONFIG_MTDPARTS "...128k(environment), 128k(redundant-environment)..."
and CONFIG_ENV_SECT_SIZE (128 * 1024).

So what exactly am I supposed to set into CONFIG_ENV_RANGE and do I have to 
modify the CONFIG_MTDPARTS?

Thanks, cheers!
> 
> -Scott