From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stefan Roese <sr@denx.de>
Date: Fri, 20 Mar 2015 11:07:00 +0100
Subject: [U-Boot] [PATCH 10/10] arm: mvebu: Add Armada A38x
 DB-88F6820-GP board support
In-Reply-To: <550A151A.1040303@elecsyscorp.com>
References: <1426665251-23868-1-git-send-email-sr@denx.de>
 <1426665251-23868-11-git-send-email-sr@denx.de>
 <550A151A.1040303@elecsyscorp.com>
Message-ID: <550BF144.6010202@denx.de>
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

Hi Kevin,

On 19.03.2015 01:15, Kevin Smith wrote:
> On 03/18/2015 02:54 AM, Stefan Roese wrote:
>> +/* Environment in SPI NOR flash */
>> +#define CONFIG_ENV_IS_IN_SPI_FLASH
>> +#define CONFIG_ENV_OFFSET		(1 << 20) /* 1MiB in */
>> +#define CONFIG_ENV_SIZE			(64 << 10) /* 64KiB */
>> +#define CONFIG_ENV_SECT_SIZE		(64 << 10) /* 64KiB sectors */
>>
> The M25P128 SPI flash chip on the DB-88F6820-GP board has 1024-page
> sectors, so these will need to be 256KiB instead of 64.  Otherwise the
> SF driver complains when trying to "saveenv".

Thanks for noticing this. I didn't write to the env yet in the new 
U-Boot. I'll incorporate this change in the next version.

Thanks,
Stefan