From mboxrd@z Thu Jan  1 00:00:00 1970
From: Bo Shen <voice.shen@atmel.com>
Date: Tue, 1 Jul 2014 11:52:49 +0800
Subject: [U-Boot] [PATCH 2/2] ARM: atmel: sama5d3xek: enable NOR flash
 support
In-Reply-To: <010c01cf9450$848818c0$8d984a40$@pont@sdcsystems.com>
References: <1403252161-20526-1-git-send-email-voice.shen@atmel.com>
	<1403252161-20526-3-git-send-email-voice.shen@atmel.com>
	<00ef01cf9449$6378fa70$2a6aef50$@pont@sdcsystems.com>
	<53B13583.7040705@atmel.com>
	<010c01cf9450$848818c0$8d984a40$@pont@sdcsystems.com>
Message-ID: <53B23091.4090305@atmel.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

Hi Andy,

On 06/30/2014 06:46 PM, Andy Pont wrote:
> Hi Bo,
>
>>>> +#ifdef CONFIG_CMD_FLASH
>>>> +#define CONFIG_SYS_FLASH_CFI
>>>> +#define CONFIG_FLASH_CFI_DRIVER
>>>> +#define CONFIG_SYS_FLASH_BASE		0x10000000
>>>> +#define CONFIG_SYS_MAX_FLASH_SECT	0x20000
>
> Is this the correct value for CONFIG_SYS_MAX_FLASH_SECT?  Shouldn't it be
> 127 (if we ignore the top boot block) or 131 if they are included?

oops, I fill this with the sector size.
Thanks point it out. Here, I think we should fill with 131.

> Regards,
>
> Andy.
>

Best Regards,
Bo Shen