On 1 November 2014 14:23, Bin Meng bmeng.cn@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!