[U-Boot] [PATCH v2 00/12] spi: sf: ICH SPI driver fix and flash params update

[Jagan Teki]

On 1 November 2014 14:23, Bin Meng <bmeng.cn at gmail.com> wrote:
> This series fix several bugs in current ICH SPI driver as well as
> adding byte program support for the SST25* flash.
>
> Flash params are updated to explicitly list supported read commands
> and change flash sector size to 4KiB as long as flash supports
> sector erase (20h) command.
>
> Changes for v2:
>   - Rebased to u-boot-spi/mater.
>   - Reviewed and updated the params of all currently supported flash
>     parts per their datasheets.
>   - Corrected AT25DF321 JEDEC ID.
>   - Corrected Atmel bulk erase command to 50h instead of D8h.
>   - Added AT25DF321A, W25X10, W25X20, W25X80 params.
>
>
> Bin Meng (12):
>   spi/ich.c: Fix a bug of reading from a non-64 bytes aligned address
>   spi/ich.c: Set the rx operation mode for ich 7
>   spi: sf: Support byte program for sst spi flash
>   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

I think you combined two or more changes(unrelated) in a common patches and
Added Bulk erase support in e_cmd_rd of sf_params ie quite not correct.

Please fix those and send me one more.

Mean while I will look at your scenario like you're controller only supports AS,
As I said before as AS of AF both are similar way of transferring
except the dummy
bits passing from the driver, try to see the fix on driver point of of
instead of digging
common sf stuff.

>
>  drivers/mtd/spi/sf_internal.h |  11 ++-
>  drivers/mtd/spi/sf_ops.c      |  31 +++++++
>  drivers/mtd/spi/sf_params.c   | 186 ++++++++++++++++++++++--------------------
>  drivers/mtd/spi/sf_probe.c    |  12 ++-
>  drivers/spi/ich.c             |  26 +++---
>  include/spi.h                 |   1 +
>  6 files changed, 159 insertions(+), 108 deletions(-)
>
> --
> 1.8.2.1
>

thanks!
-- 
Jagan.