Change in flashrom[stable]: Fix erase block structure on Spansion S25FL128P - flashrom-gerrit

2 May 2017


      Timothy Pearson has posted comments on this change. ( https://review.coreboot.org/19531 )
Change subject: Fix erase block structure on Spansion S25FL128P
......................................................................
Patch Set 2:
...
I think what you have is a S25FL128P......0 the 0 marks 64KiB
sectors.
The exact part number printed on the Flash chip is:
Spansion FL128P1F
Is the "P1" a red herring?  If so, perhaps the Flashrom sources need a comment to help with this confusing aspect of the device naming.
-- 
To view, visit https://review.coreboot.org/19531
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4b4e3673f7019f38a3b6ab2f7002bbe56dc6de07
Gerrit-PatchSet: 2
Gerrit-Project: flashrom
Gerrit-Branch: stable
Gerrit-Owner: Timothy Pearson tpearson@raptorengineering.com
Gerrit-Reviewer: Martin Roth martinroth@google.com
Gerrit-Reviewer: Nico Huber nico.h@gmx.de
Gerrit-Reviewer: Paul Menzel paulepanter@users.sourceforge.net
Gerrit-Reviewer: Timothy Pearson tpearson@raptorengineering.com
Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org
Gerrit-HasComments: No