---------- Forwarded message ----------
From: Николай Николаев <evrinoma@gmail.com>
Date: Thu, May 16, 2013 at 7:03 AM
Subject: Re: [flashrom] Added chipsets
To: Steven Zakulec <spzakulec@gmail.com>


Hi Steven

"Notes on specific chips: GigaDevice GD25Q512- .voltage    = {2700, 3600}, My datasheet (v1.1 March 13 2012, GD25Q10/512) says 2.4-3.6V"
I used last datasheet from web site gigadevice - datasheet (v1.1 March 13 2012, GD25Q10/512) says 2.7-3.6V
but this document says 2.4 
http://www.gigadevice.com/UserFiles/File/datasheet0320/GD25Q10&512_Rev1.1.pdf

we can easy write 2.4 - 3.6 V

Thanks you right i made a mull with M25 and M45 chips. I send a new patch

2013/5/16 Steven Zakulec <spzakulec@gmail.com>
This review covers the GigaDevice (except GD25Q10) to Numonyx (partial, list below) chips that were added.
Here's a couple of general notes:
Please add page programming eraseblocks

WREN & WRSR flags only get added when the chip actually has the command.


Numonyx M45PE10-
+        .feature_bits    = FEATURE_WRSR_WREN,
The datasheet I have (Rev 7, April 2008) doesn't have a Write Status Register operation- only a Read Status Register. 

Numonyx M45PE20-
+        .feature_bits    = FEATURE_WRSR_WREN,
The datasheet I have (Rev 6, May 2008) doesn't have a Write Status Register operation- only a Read Status Register.

Numonyx M45PE40-
+        .feature_bits    = FEATURE_WRSR_WREN,
The datasheet I have (Rev 9, May 2008) doesn't have a Write Status Register operation- only a Read Status Register.

Numonyx M45PE80-
+        .feature_bits    = FEATURE_WRSR_WREN,
The datasheet I have (Rev 6, May 2008) doesn't have a Write Status Register operation- only a Read Status Register.

Numonyx M45PE16-
+        .feature_bits    = FEATURE_WRSR_WREN,
The datasheet I have (Rev 8, May 2008) doesn't have a Write Status Register operation- only a Read Status Register.

I'll cover the rest of the Numonyx chips from the patch in a follow-up review. 






--
With best regards Nikolay Nikolaev
С Уважением Николаев Николай