From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jagan Teki <jagannadh.teki@gmail.com>
Date: Thu, 11 Dec 2014 12:56:16 +0530
Subject: [U-Boot] [PATCH v3 0/9] sf: Update flash params for supported
 read commands and sector size
In-Reply-To: <CAEUhbmWqw8aL=RAVG8G1zvxnSFWSZDD1Xj=rJuAXYCLSJ0ifjA@mail.gmail.com>
References: <1418215892-17617-1-git-send-email-bmeng.cn@gmail.com>
	<CAD6G_RQSvnyXeUTHbs7rgwY7+oTnRZOE=woHgta2Zi5LdcxRig@mail.gmail.com>
	<CAEUhbmWqw8aL=RAVG8G1zvxnSFWSZDD1Xj=rJuAXYCLSJ0ifjA@mail.gmail.com>
Message-ID: <CAD6G_RTApFhzG63gF0zbLvSq2GgKqj582JOo0tQrW=JuN5b80g@mail.gmail.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 Bin,

On 11 December 2014 at 08:34, Bin Meng <bmeng.cn@gmail.com> wrote:
> Hi Jagan,
>
> On Thu, Dec 11, 2014 at 2:41 AM, Jagan Teki <jagannadh.teki@gmail.com> wrote:
>> Hi Bin,
>>
>> On 10 December 2014 at 18:21, Bin Meng <bmeng.cn@gmail.com> wrote:
>>> This series update SPI flash supported read commands per datasheet
>>> in the flash params table, and change flash sector size to 4KiB as
>>> long as flash supports sector erase (20h) command, to ensure
>>> 'sf erase offset +len' work on 4KiB boundary instead of 64KiB when
>>> given SECT_4K.
>>>
>>> Changes in v3:
>>> - Rebase with Jagan's patch series @ http://patchwork.ozlabs.org/patch/419154/
>>>
>>>
>>> Bin Meng (9):
>>>   sf: Update SST flash params
>>>   sf: Update Atmel flash params
>>>   sf: Update EON flash params
>>>   sf: Update GigaDevice flash params
>>>   sf: Update Macronix flash params
>>>   sf: Update Spansion flash params
>>>   sf: Update Micron flash params
>>>   sf: Update Winbond flash params
>>>   sf: Give proper spacing between flash table params
>>
>> Thanks for the updates - have you verified these changes?
>
> I verified some, but not all of these flash parts. The update is based
> on flash datasheet, so if something is broken, eg before this series
> the flash advertises only READ_NORM and after my series it is changed
> to READ_FULL, and let's say QUAD_IO_FAST is not working, it is very
> likely that the SPI controller driver has some bugs when supporting
> QUAD_IO_FAST.

Since these updates were tested before, I will skip these for this PR.
Will test all the rest (except these) and send the PR soon.

Let me know your inputs?

>
>>>
>>>  drivers/mtd/spi/sf_internal.h |   5 ++
>>>  drivers/mtd/spi/sf_params.c   | 185 ++++++++++++++++++++++--------------------
>>>  drivers/mtd/spi/sf_probe.c    |   4 +
>>>  3 files changed, 104 insertions(+), 90 deletions(-)


thanks!
-- 
Jagan.