[U-Boot] [PATCH] M28EVK: Fix build if CONFIG_CMD_NAND not selected

Marek Vasut marek.vasut at gmail.com
Tue Jan 10 12:46:01 CET 2012


> Hi Marek,
> 
> On Sun, Jan 1, 2012 at 2:28 AM, Marek Vasut <marek.vasut at gmail.com> wrote:
> > Signed-off-by: Marek Vasut <marek.vasut at gmail.com>
> > Cc: Wolfgang Denk <wd at denx.de>
> > Cc: Stefano Babic <sbabic at denx.de>
> > ---
> >  include/configs/m28evk.h |    4 +++-
> >  1 files changed, 3 insertions(+), 1 deletions(-)
> > 
> > diff --git a/include/configs/m28evk.h b/include/configs/m28evk.h
> > index 39c841f..dcea878 100644
> > --- a/include/configs/m28evk.h
> > +++ b/include/configs/m28evk.h
> > @@ -140,6 +140,7 @@
> >  /*
> >  * NAND
> >  */
> > +#define        CONFIG_ENV_SIZE                 (16 * 1024)
> >  #ifdef CONFIG_CMD_NAND
> >  #define        CONFIG_NAND_MXS
> >  #define CONFIG_APBH_DMA
> > @@ -150,7 +151,6 @@
> > 
> >  /* Environment is in NAND */
> >  #define        CONFIG_ENV_IS_IN_NAND
> > -#define        CONFIG_ENV_SIZE                 (16 * 1024)
> >  #define        CONFIG_ENV_SIZE_REDUND          CONFIG_ENV_SIZE
> >  #define        CONFIG_ENV_SECT_SIZE            (128 * 1024)
> >  #define        CONFIG_ENV_RANGE                (512 * 1024)
> > @@ -173,6 +173,8 @@
> >                "512k(redundant-environment),"  \
> >                "4m(kernel),"                   \
> >                "-(filesystem)"
> > +#else
> > +#define        CONFIG_ENV_IS_NOWHERE
> 
> Couldn't you use CONFIG_ENV_IS_IN_MMC instead of CONFIG_ENV_IS_NOWHERE
> here?

We decided to not have env elsewhere until env-in-multiple-locations patch is in 
place.

M
> 
> Regards,
> 
> Fabio Estevam


More information about the U-Boot mailing list