Okay, so I applied the same changes to the W25x16 and got good results. It looks like the same applies to the W25x{16,32,64} products in addition to the W25x{10,20,40,80}. Please update the patch to include the additional chips.<div>

<br></div><div>W25x{16,32,64} datasheet: <a href="http://www.winbond-usa.com/products/Nexflash/pdfs/datasheets/W25X16_32_64h.pdf">http://www.winbond-usa.com/products/Nexflash/pdfs/datasheets/W25X16_32_64h.pdf</a><br><br>
<div class="gmail_quote">
On Thu, Feb 11, 2010 at 3:56 PM, David Hendricks <span dir="ltr"><<a href="mailto:dhendrix@google.com">dhendrix@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

D'oh! I just realized that this patch doesn't affect the x16. I really ought to read the patch more carefully...<div><div></div><div class="h5"><br><br><div class="gmail_quote">On Thu, Feb 11, 2010 at 3:50 PM, David Hendricks <span dir="ltr"><<a href="mailto:dhendrix@google.com" target="_blank">dhendrix@google.com</a>></span> wrote:<br>



<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="gmail_quote"><div>On Sun, Jan 31, 2010 at 6:31 PM, Sean Nelson <span dir="ltr"><<a href="mailto:audiohacked@gmail.com" target="_blank">audiohacked@gmail.com</a>></span> wrote:<br>



<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Fix erase blocks for Winbond W25X* SPI chips. The Winbond W25X10 and related chips only have 4k and 64k blocks and only accept erase commands: 20h, d8h, and c7h.<br>
<br>
Signed-off-by: Sean Nelson <<a href="mailto:audiohacked@gmail.com" target="_blank">audiohacked@gmail.com</a>><br></blockquote><div><br></div></div>Tested on a W25x16 flash chip using an NM10 southbridge. Verbose flashrom -E log attached. Following an erase with a read, the image produced turns up all 0xFF bytes. The warnings reported seem benign, and probably have more to do with the NM10 than the flash chip...<div>




<br></div><div>Acked by: David Hendricks <<a href="mailto:dhendrix@google.com" target="_blank">dhendrix@google.com</a>></div><div><br></div></div>-- <br>David Hendricks (dhendrix)<br>Systems Software Engineer, Google Inc.<br>




</div>
</blockquote></div><br><br clear="all"><br>-- <br>David Hendricks (dhendrix)<br>Systems Software Engineer, Google Inc.<br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>David Hendricks (dhendrix)<br>Systems Software Engineer, Google Inc.<br>
</div>