I'm actually not entirely sure why the W25Q80 and W25Q32 were marked as "OK" for erase but not write. In any case, I've successfully read/erased/written all three of these Winbond chips using nm10 and amd870 based mainboards.
I see no reason why the W25Q16 would not work as well, but I don't have one to try so I left it out of this patch.
Signed-off-by: David Hendricks dhendrix@google.com
* David Hendricks dhendrix@google.com [101015 05:27]:
I'm actually not entirely sure why the W25Q80 and W25Q32 were marked as "OK" for erase but not write. In any case, I've successfully read/erased/written all three of these Winbond chips using nm10 and amd870 based mainboards.
I see no reason why the W25Q16 would not work as well, but I don't have one to try so I left it out of this patch.
Signed-off-by: David Hendricks dhendrix@google.com
Acked-by: Stefan Reinauer stepan@coreboot.org
-- David Hendricks (dhendrix) Systems Software Engineer, Google Inc.
Index: flashrom-foo/flashchips.c
--- flashrom-foo.orig/flashchips.c 2010-10-14 20:18:44.000000000 -0700 +++ flashrom-foo/flashchips.c 2010-10-14 20:19:04.000000000 -0700 @@ -7009,7 +7009,7 @@ .total_size = 1024, .page_size = 256, .feature_bits = FEATURE_WRSR_WREN,
.tested = TEST_OK_PRE,
.probe = probe_spi_rdid, .probe_timing = TIMING_ZERO, .block_erasers =.tested = TEST_OK_PREW,
@@ -7081,7 +7081,7 @@ .total_size = 4096, .page_size = 256, .feature_bits = FEATURE_WRSR_WREN,
.tested = TEST_OK_PRE,
.probe = probe_spi_rdid, .probe_timing = TIMING_ZERO, .block_erasers =.tested = TEST_OK_PREW,
@@ -7117,7 +7117,7 @@ .total_size = 8192, .page_size = 256, .feature_bits = FEATURE_WRSR_WREN,
.tested = TEST_OK_PR,
.probe = probe_spi_rdid, .probe_timing = TIMING_ZERO, .block_erasers =.tested = TEST_OK_PREW,
On Fri, 15 Oct 2010 02:27:23 -0000 David Hendricks dhendrix@google.com wrote:
I'm actually not entirely sure why the W25Q80 and W25Q32 were marked as "OK" for erase but not write. In any case, I've successfully read/erased/written all three of these Winbond chips using nm10 and amd870 based mainboards.
I see no reason why the W25Q16 would not work as well, but I don't have one to try so I left it out of this patch.
Signed-off-by: David Hendricks dhendrix@google.com Acked-by: Stefan Reinauer stepan@coreboot.org
applied in r1303, thanks