All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATH v5 00/14] sf: Update sf framework to support all sizes of flashes
@ 2013-06-21 14:42 Jagannadha Sutradharudu Teki
  2013-06-23 18:04 ` Jagan Teki
  0 siblings, 1 reply; 3+ messages in thread
From: Jagannadha Sutradharudu Teki @ 2013-06-21 14:42 UTC (permalink / raw)
  To: u-boot

This is a v5 patch series with some modifications on previous series
with same head "sf: Update sf framework to support all sizes of flashes"

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 ~460 bytes.

sf update(for first 16MBytes), Changes before:
U-Boot> sf update 0x1000000 0x0 0x1000000
- 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 0x1000000 0x0 0x1000000
- 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 (14):
  sf: Add bank address register writing support
  sf: Discover the bank addr commands
  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: Add bank addr code in CONFIG_SPI_FLASH_BAR
  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_64K
  sf: Remove spi_flash_cmd_poll_bit()
  sf: Add flag status register polling support

 README                               |   5 +
 drivers/mtd/spi/spansion.c           |   7 ++
 drivers/mtd/spi/spi_flash.c          | 186 +++++++++++++++++++++++++++++------
 drivers/mtd/spi/spi_flash_internal.h |  29 +++++-
 drivers/mtd/spi/stmicro.c            |  28 ++++++
 include/spi_flash.h                  |  10 ++
 6 files changed, 232 insertions(+), 33 deletions(-)

-- 
1.8.3

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [U-Boot] [PATH v5 00/14] sf: Update sf framework to support all sizes of flashes
  2013-06-21 14:42 [U-Boot] [PATH v5 00/14] sf: Update sf framework to support all sizes of flashes Jagannadha Sutradharudu Teki
@ 2013-06-23 18:04 ` Jagan Teki
  2013-06-25  0:58   ` Simon Glass
  0 siblings, 1 reply; 3+ messages in thread
From: Jagan Teki @ 2013-06-23 18:04 UTC (permalink / raw)
  To: u-boot

On 21-06-2013 20:12, Jagannadha Sutradharudu Teki wrote:
> This is a v5 patch series with some modifications on previous series
> with same head "sf: Update sf framework to support all sizes of flashes"
>
> 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 ~460 bytes.
>
> sf update(for first 16MBytes), Changes before:
> U-Boot> sf update 0x1000000 0x0 0x1000000
> - 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 0x1000000 0x0 0x1000000
> - 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 (14):
>    sf: Add bank address register writing support
>    sf: Discover the bank addr commands
>    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: Add bank addr code in CONFIG_SPI_FLASH_BAR
>    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_64K
>    sf: Remove spi_flash_cmd_poll_bit()
>    sf: Add flag status register polling support
>
>   README                               |   5 +
>   drivers/mtd/spi/spansion.c           |   7 ++
>   drivers/mtd/spi/spi_flash.c          | 186 +++++++++++++++++++++++++++++------
>   drivers/mtd/spi/spi_flash_internal.h |  29 +++++-
>   drivers/mtd/spi/stmicro.c            |  28 ++++++
>   include/spi_flash.h                  |  10 ++
>   6 files changed, 232 insertions(+), 33 deletions(-)
>

Applied to u-boot-spi/master

--
Thanks,
Jagan.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [U-Boot] [PATH v5 00/14] sf: Update sf framework to support all sizes of flashes
  2013-06-23 18:04 ` Jagan Teki
@ 2013-06-25  0:58   ` Simon Glass
  0 siblings, 0 replies; 3+ messages in thread
From: Simon Glass @ 2013-06-25  0:58 UTC (permalink / raw)
  To: u-boot

Hi Jagan,


On Sun, Jun 23, 2013 at 11:04 AM, Jagan Teki <jagannadh.teki@gmail.com>wrote:

> On 21-06-2013 20:12, Jagannadha Sutradharudu Teki wrote:
>
>> This is a v5 patch series with some modifications on previous series
>> with same head "sf: Update sf framework to support all sizes of flashes"
>>
>> 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 ~460 bytes.
>>
>> sf update(for first 16MBytes), Changes before:
>> U-Boot> sf update 0x1000000 0x0 0x1000000
>> - 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 0x1000000 0x0 0x1000000
>> - 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 (14):
>>    sf: Add bank address register writing support
>>    sf: Discover the bank addr commands
>>    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: Add bank addr code in CONFIG_SPI_FLASH_BAR
>>    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_64K
>>    sf: Remove spi_flash_cmd_poll_bit()
>>    sf: Add flag status register polling support
>>
>>   README                               |   5 +
>>   drivers/mtd/spi/spansion.c           |   7 ++
>>   drivers/mtd/spi/spi_flash.c          | 186
>> +++++++++++++++++++++++++++++-**-----
>>   drivers/mtd/spi/spi_flash_**internal.h |  29 +++++-
>>   drivers/mtd/spi/stmicro.c            |  28 ++++++
>>   include/spi_flash.h                  |  10 ++
>>   6 files changed, 232 insertions(+), 33 deletions(-)
>>
>>
> Applied to u-boot-spi/master
>

I took a look at both series and they look good to me. Thanks for
addressing the comments.

I also sent out a sandbox series today that you may have seen.

Regards,
Simon


>
> --
> Thanks,
> Jagan.
> ______________________________**_________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/**listinfo/u-boot<http://lists.denx.de/mailman/listinfo/u-boot>
>

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-06-25  0:58 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-06-21 14:42 [U-Boot] [PATH v5 00/14] sf: Update sf framework to support all sizes of flashes Jagannadha Sutradharudu Teki
2013-06-23 18:04 ` Jagan Teki
2013-06-25  0:58   ` Simon Glass

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.