[U-Boot] [PATCH v3 0/9] sf: Update flash params for supported read commands and sector size

[Jagan Teki]

Hi Bin,

On 11 December 2014 at 08:34, Bin Meng <bmeng.cn at gmail.com> wrote:
> Hi Jagan,
>
> On Thu, Dec 11, 2014 at 2:41 AM, Jagan Teki <jagannadh.teki at gmail.com> wrote:
>> Hi Bin,
>>
>> On 10 December 2014 at 18:21, Bin Meng <bmeng.cn at 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.