[flashrom] [PATCH] Add support for SST26VF064B
Stefan Tauner
stefan.tauner at alumni.tuwien.ac.at
Sun May 4 22:06:09 CEST 2014
On Mon, 31 Mar 2014 22:18:48 -0700
Wei Hu <wei at aristanetworks.com> wrote:
> Datasheet available at
> http://ww1.microchip.com/downloads/en/DeviceDoc/20005119D.pdf
Hi,
thank you for your patch! I have reviewed it and I think you got a few
things wrong:
- Most importantly, I think the block erase sizes are non-uniform
(for opcode 0xD8). See note 11 of Table 5.1 and Section 3.0 etc.
- Also, the ULBPR requires a WREN opcode sent before it. OTOH your
spi_disable_blockprotect_generic() calls seems useless (all bits of
the status register are read-only).
- The spi_prettyprint_status_register_sst25 function is completely
wrong for these chips.
- The OTP memory, and other advanced features were not documented at
all.
- Debatably the name of the chip should include the optional A that
indicates the disable default state of the WP/Hold feature of this
chip variant.
I have attached my counterproposal that also adds support for the
SST26VF032B(A). Please take a look and tell me if you disagree with
anything I have changed, thanks.
--
Kind regards/Mit freundlichen Grüßen, Stefan Tauner
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Add-support-for-SST26VF032B-A-and-SST26VF064B-A.patch
Type: text/x-patch
Size: 5145 bytes
Desc: not available
URL: <http://www.flashrom.org/pipermail/flashrom/attachments/20140504/19c761cd/attachment.patch>
More information about the flashrom
mailing list