2 May
2017
2 May
'17
10:10 p.m.
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