From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Sat, 17 Sep 2011 10:20:04 +0200 Subject: [U-Boot] [PATCH 27/31] M28: Save environment in NAND In-Reply-To: <4E73A040.5020202@freescale.com> References: <1315514579-19215-1-git-send-email-marek.vasut@gmail.com> <1315514579-19215-28-git-send-email-marek.vasut@gmail.com> <4E73A040.5020202@freescale.com> Message-ID: <201109171020.04821.marek.vasut@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 > > Cc: Stefano Babic > > Cc: Wolfgang Denk > > Cc: Detlev Zundel > > --- > > > > 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