Re: [U-Boot] [PATCH v2 00/16] sf: Update sf framework to support all sizes of flashes

2013-06-05 Thread Jagan Teki
Request for comments.

--
Thanks,
Jagan.

On Fri, May 31, 2013 at 6:22 PM, Jagannadha Sutradharudu Teki
 wrote:
> This series is v2 for the patch series sent few weeks back with a head
> "sf: Accessing > 16MBytes flashes in existing 3-byte addr mode."
>
> The current implementation in sf supports 3-byte address mode due
> to this up to 16MB amount of flash is able to access for those
> flashes which has an actual size of > 16MB.
>
> This series of patches is more detailed/meatured changes w.r.t the current
> sf framework in addition to changes related to support all sizes using
> bank/exnt register addr accessing support.
>
> With these new updates on sf framework, the flashes which has < 16MB
> are not effected as per as performance is concern and but the
> u-boot.bin size incrased ~600 bytes.
>
> sf update(for first 16MBytes), Changes before:
> U-Boot> sf update 0x100 0x0 0x100
> - N25Q256
>   16777216 bytes written, 0 bytes skipped in 199.72s, speed 86480 B/s
> - W25Q128BV
>   16777216 bytes written, 0 bytes skipped in 351.739s, speed 48913 B/s
> - S25FL256S_64K
>   16777216 bytes written, 0 bytes skipped in 65.659s, speed 262144 B/s
>
> sf update(for first 16MBytes), Changes after:
> U-Boot> sf update 0x100 0x0 0x100
> - N25Q256
>   16777216 bytes written, 0 bytes skipped in 198.953s, speed 86480 B/s
> - W25Q128BV
>   16777216 bytes written, 0 bytes skipped in 350.90s, speed 49200 B/s
> - S25FL256S_64K
>   16777216 bytes written, 0 bytes skipped in 66.521s, speed 262144 B/s
>
> The main aim of these changes is to not effect the current framework
> and at the same time to support the > 16Mbyte flashes, becuase of this
> I involved few flash vendor people in CC [thought that they may/mayn't
> be a mailing list members] to know their views.
>
> REQUEST FOR ALL SPI CODE CONTRIBUTORS/USERS, PLEASE TEST
> THESE CHANGES W.R.T YOUR HW IF POSSIBLE.
>
> Please let me know for any issues/concerns/questions.
>
> Thanks,
> Jagan.
>
> Jagannadha Sutradharudu Teki (16):
>   sf: Add bank address register writing support
>   sf: Add bank address register reading support
>   sf: Add extended addr write support for winbond|stmicro
>   sf: Add extended addr read support for winbond|stmicro
>   sf: read flash bank addr register at probe time
>   sf: Update sf to support all sizes of flashes
>   sf: Update sf read to support all sizes of flashes
>   sf: Use spi_flash_addr() in write call
>   sf: stmicro: Add support for N25Q512
>   sf: stmicro: Add support for N25Q512A
>   sf: stmicro: Add support for N25Q1024
>   sf: stmicro: Add support for N25Q1024A
>   sf: spansion: Add support for S25FL512S_256K
>   sf: Use spi_flash_read_common() in write status poll
>   sf: Remove spi_flash_cmd_poll_bit()
>   sf: Add Flag status register polling support
>
>  drivers/mtd/spi/spansion.c   |   7 ++
>  drivers/mtd/spi/spi_flash.c  | 192 
> +++
>  drivers/mtd/spi/spi_flash_internal.h |  21 +++-
>  drivers/mtd/spi/stmicro.c|  24 +
>  include/spi_flash.h  |   4 +
>  5 files changed, 203 insertions(+), 45 deletions(-)
>
> --
> 1.8.3
>
>
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v2 00/16] sf: Update sf framework to support all sizes of flashes

2013-05-31 Thread Jagannadha Sutradharudu Teki
This series is v2 for the patch series sent few weeks back with a head
"sf: Accessing > 16MBytes flashes in existing 3-byte addr mode."

The current implementation in sf supports 3-byte address mode due 
to this up to 16MB amount of flash is able to access for those 
flashes which has an actual size of > 16MB.

This series of patches is more detailed/meatured changes w.r.t the current
sf framework in addition to changes related to support all sizes using
bank/exnt register addr accessing support.

With these new updates on sf framework, the flashes which has < 16MB
are not effected as per as performance is concern and but the
u-boot.bin size incrased ~600 bytes.

sf update(for first 16MBytes), Changes before:
U-Boot> sf update 0x100 0x0 0x100
- N25Q256
  16777216 bytes written, 0 bytes skipped in 199.72s, speed 86480 B/s
- W25Q128BV
  16777216 bytes written, 0 bytes skipped in 351.739s, speed 48913 B/s
- S25FL256S_64K
  16777216 bytes written, 0 bytes skipped in 65.659s, speed 262144 B/s

sf update(for first 16MBytes), Changes after:
U-Boot> sf update 0x100 0x0 0x100
- N25Q256
  16777216 bytes written, 0 bytes skipped in 198.953s, speed 86480 B/s
- W25Q128BV
  16777216 bytes written, 0 bytes skipped in 350.90s, speed 49200 B/s
- S25FL256S_64K
  16777216 bytes written, 0 bytes skipped in 66.521s, speed 262144 B/s

The main aim of these changes is to not effect the current framework 
and at the same time to support the > 16Mbyte flashes, becuase of this 
I involved few flash vendor people in CC [thought that they may/mayn't 
be a mailing list members] to know their views.

REQUEST FOR ALL SPI CODE CONTRIBUTORS/USERS, PLEASE TEST
THESE CHANGES W.R.T YOUR HW IF POSSIBLE.

Please let me know for any issues/concerns/questions.

Thanks,
Jagan.

Jagannadha Sutradharudu Teki (16):
  sf: Add bank address register writing support
  sf: Add bank address register reading support
  sf: Add extended addr write support for winbond|stmicro
  sf: Add extended addr read support for winbond|stmicro
  sf: read flash bank addr register at probe time
  sf: Update sf to support all sizes of flashes
  sf: Update sf read to support all sizes of flashes
  sf: Use spi_flash_addr() in write call
  sf: stmicro: Add support for N25Q512
  sf: stmicro: Add support for N25Q512A
  sf: stmicro: Add support for N25Q1024
  sf: stmicro: Add support for N25Q1024A
  sf: spansion: Add support for S25FL512S_256K
  sf: Use spi_flash_read_common() in write status poll
  sf: Remove spi_flash_cmd_poll_bit()
  sf: Add Flag status register polling support

 drivers/mtd/spi/spansion.c   |   7 ++
 drivers/mtd/spi/spi_flash.c  | 192 +++
 drivers/mtd/spi/spi_flash_internal.h |  21 +++-
 drivers/mtd/spi/stmicro.c|  24 +
 include/spi_flash.h  |   4 +
 5 files changed, 203 insertions(+), 45 deletions(-)

-- 
1.8.3


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot