16 Jun
2020
16 Jun
'20
7:22 a.m.
Simon Buhrow has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/40855 )
Change subject: flashchips: Add W25Q256.W ......................................................................
Patch Set 9:
Patch Set 8: Code-Review-1
This looks good to me. Just one request - Can you move the entry in flashchips.c down so it's with the other W25Q256 entries?
Otherwise I'll move my W25Q256JW_DTR entry up so it's closer to where yours is.
Done
--
To view, visit https://review.coreboot.org/c/flashrom/+/40855
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I3608945a38b5ed42b8b2b9b2b388cdea97141d7b
Gerrit-Change-Number: 40855
Gerrit-PatchSet: 9
Gerrit-Owner: Simon Buhrow
Gerrit-Reviewer: David Hendricks david.hendricks@gmail.com
Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org
Gerrit-CC: Alex Thiessen alex.thiessen.de+coreboot@gmail.com
Gerrit-CC: Angel Pons th3fanbus@gmail.com
Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net
Gerrit-Comment-Date: Tue, 16 Jun 2020 07:22:34 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment