On Wed, Jan 18, 2012 at 6:09 AM, Eric Nelson <eric.nel...@boundarydevices.com> wrote: > Signed-off-by: Eric Nelson <eric.nel...@boundarydevices.com> > --- > include/configs/mx6qsabrelite.h | 12 +++++++++++- > 1 files changed, 11 insertions(+), 1 deletions(-) > > diff --git a/include/configs/mx6qsabrelite.h b/include/configs/mx6qsabrelite.h > index 44b028a..160894c 100644 > --- a/include/configs/mx6qsabrelite.h > +++ b/include/configs/mx6qsabrelite.h > @@ -174,10 +174,20 @@ > /* FLASH and environment organization */ > #define CONFIG_SYS_NO_FLASH > > -#define CONFIG_ENV_OFFSET (6 * 64 * 1024) > #define CONFIG_ENV_SIZE (8 * 1024) > + > #define CONFIG_ENV_IS_IN_MMC > +/* #define CONFIG_ENV_IS_IN_SPI_FLASH */
>From the commit log, it says the default is in serial flash, but apparently in the code the env is default to MMC, which mismatch. please fix it. > + > +#if defined(CONFIG_ENV_IS_IN_MMC) > +#define CONFIG_ENV_OFFSET (6 * 64 * 1024) > #define CONFIG_SYS_MMC_ENV_DEV 0 > +#elif defined(CONFIG_ENV_IS_IN_SPI_FLASH) > +#define CONFIG_ENV_OFFSET (768 * 1024) > +#define CONFIG_ENV_SECT_SIZE (8 * 1024) > +#define CONFIG_ENV_SPI_CS 0x5300 I'm wondering how the CONFIG_ENV_SPI_CS could be 0x5300? Vague? > +#define CONFIG_ENV_SPI_MODE SPI_MODE_0 > +#endif > > #define CONFIG_OF_LIBFDT > > -- > 1.7.1 > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
