<div dir="ltr"><div><div>This review covers the GigaDevice (except GD25Q10) to Numonyx (partial, list below) chips that were added.<br></div>Here's a couple of general notes:<br>Please add page programming eraseblocks<br>
<br>WREN & WRSR flags only get added when the chip actually has the command.<br><br></div><div><div class="gmail_extra"><br></div><div class="gmail_extra">Notes on specific chips:<br></div><div class="gmail_extra">GigaDevice GD25Q512-<br>
.voltage    = {2700, 3600}, <br>My datasheet (v1.1 March 13 2012, GD25Q10/512) says 2.4-3.6V<br>
<br></div><div class="gmail_extra">Numonyx M45PE10-<br>+        .feature_bits    = FEATURE_WRSR_WREN, <br></div><div class="gmail_extra">The datasheet I have (Rev 7, April 2008) doesn't have a Write Status Register operation- only a Read Status Register.  <br>
<br></div><div class="gmail_extra">Numonyx M45PE20-<br>+        .feature_bits    = FEATURE_WRSR_WREN,<br>The datasheet I have (Rev 6, May 2008) doesn't have a Write Status Register operation- only a Read Status Register. <br>
<br>Numonyx M45PE40-<br>
+        .feature_bits    = FEATURE_WRSR_WREN,<br>
The datasheet I have (Rev 9, May 2008) doesn't have a Write Status Register operation- only a Read Status Register. <br><br>Numonyx M45PE80-<br>+        .feature_bits    = FEATURE_WRSR_WREN,<br>The datasheet I have (Rev 6, May 2008) doesn't have a Write Status Register operation- only a Read Status Register.<br>
<br>Numonyx M45PE16-<br>+        .feature_bits    = FEATURE_WRSR_WREN,<br>The datasheet I have (Rev 8, May 2008) doesn't have a Write Status Register operation- only a Read Status Register.<br><br></div><div class="gmail_extra">
I'll cover the rest of the Numonyx chips from the patch in a follow-up review.  <br></div><div class="gmail_extra">
<br><br><br>
</div></div></div>