[U-Boot] [PATCH v2 1/4] arm: socfpga: cyclone5-socdk: Enabling U-Boot environment in QSPI

[Marek Vasut]

On Saturday, December 12, 2015 at 01:47:38 AM, 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 at altera.com>
> Cc: Dinh Nguyen <dinguyen at opensource.altera.com>
> Cc: Dinh Nguyen <dinh.linux at gmail.com>
> Cc: Pavel Machek <pavel at denx.de>
> Cc: Marek Vasut <marex at denx.de>
> Cc: Stefan Roese <sr at denx.de>
> ---
> Changes for v2
> - remove the undef
> ---
>  include/configs/socfpga_cyclone5_socdk.h | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/include/configs/socfpga_cyclone5_socdk.h
> b/include/configs/socfpga_cyclone5_socdk.h index 67bb35f..340b2c7 100644
> --- a/include/configs/socfpga_cyclone5_socdk.h
> +++ b/include/configs/socfpga_cyclone5_socdk.h
> @@ -58,9 +58,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

The offset of the env is therefore 4 MiB, right ? This does not match with
the other patch:

[PATCH 1/6] arm: socfpga: cyclone5-socdk: Enabling mtd partitioning layout

Please be more careful next time. Also, it might make sense to enable
redundant environment in QSPI NOR.

Best regards,
Marek Vasut