From mboxrd@z Thu Jan  1 00:00:00 1970
From: Chin Liang See <clsee@altera.com>
Date: Sun, 13 Dec 2015 08:16:35 +0800
Subject: [U-Boot] [PATCH v2 4/4] arm: socfpga: socrates: Enabling U-Boot
 environment in QSPI
In-Reply-To: <20151212225047.GB2332@amd>
References: <1449881261-3297-1-git-send-email-clsee@altera.com>
 <1449881261-3297-4-git-send-email-clsee@altera.com>
 <20151212225047.GB2332@amd>
Message-ID: <1449965795.2601.34.camel@altera.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Sat, 2015-12-12 at 23:50 +0100, Pavel Machek wrote:
> On Sat 2015-12-12 08:47:41, Chin Liang See wrote:
> > Enabling the support of storing U-Boot environment
> > within serial NOR flash. By default, its still
> > store into SDMMC
> > 
> > Signed-off-by: Chin Liang See <clsee@altera.com>
> > Cc: Dinh Nguyen <dinguyen@opensource.altera.com>
> > Cc: Dinh Nguyen <dinh.linux@gmail.com>
> > Cc: Pavel Machek <pavel@denx.de>
> > Cc: Marek Vasut <marex@denx.de>
> > Cc: Stefan Roese <sr@denx.de>
> 
> 
> 
> > --- a/include/configs/socfpga_socrates.h
> > +++ b/include/configs/socfpga_socrates.h
> > @@ -54,9 +54,15 @@
> >  
> >  #endif
> >  
> > +/* U-Boot environment */
> >  #define CONFIG_ENV_IS_IN_MMC
> > +#ifdef CONFIG_ENV_IS_IN_MMC
> >  #define CONFIG_SYS_MMC_ENV_DEV		0	/* device 0
> > */
> >  #define CONFIG_ENV_OFFSET		512	/* just after
> > the MBR */
> > +#elif CONFIG_ENV_IS_IN_SPI_FLASH
> > +#define CONFIG_ENV_SECT_SIZE		(64 * 1024)
> > +#define CONFIG_ENV_OFFSET		0x400000
> > +#endif
> 
> This configuration is again shared with all the boards that have NOR,
> right? So time to create include/configs/socfpga_spi_nor.h or
> something like that?
> 

I can consolidate too into socfpga_common.h

Thanks
Chin Liang

> Thanks,								
> 	Pavel