Attention is currently required from: Nico Huber, Angel Pons, Nikolai Artemiev.
Sergii Dmytruk has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/66216 )
Change subject: flashchips.c: enable WP for to 7 entries of MX chips
......................................................................
Patch Set 6:
(1 comment)
Commit Message:
https://review.coreboot.org/c/flashrom/+/66216/comment/9cdac1dd_035fc6ec
PS6, Line 15: MX25L6436E/MX25L6445E/MX25L6465E/MX25L6473E/MX25L6473F was split into:
Do you know if SFDP can be used to tell apart these chips?
All 5 of them seem to have different SFDP (either JEDEC or vendor part), so it should be doable at least by comparing against expected values (I just haven't seen flashrom code using SFDP for identification). Features like `OTP TB bit in CONFIG1/STATUS2 (0x15 opcode)` might not be derivable from SFDP.
--
To view, visit
https://review.coreboot.org/c/flashrom/+/66216
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Ib3db9d39ffacd3e9e44de92c6cfb6c3ecc8615bd
Gerrit-Change-Number: 66216
Gerrit-PatchSet: 6
Gerrit-Owner: Sergii Dmytruk
sergii.dmytruk@3mdeb.com
Gerrit-Reviewer: Angel Pons
th3fanbus@gmail.com
Gerrit-Reviewer: Maciej Pijanowski
maciej.pijanowski@3mdeb.com
Gerrit-Reviewer: Nico Huber
nico.h@gmx.de
Gerrit-Reviewer: Nikolai Artemiev
nartemiev@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Attention: Nico Huber
nico.h@gmx.de
Gerrit-Attention: Angel Pons
th3fanbus@gmail.com
Gerrit-Attention: Nikolai Artemiev
nartemiev@google.com
Gerrit-Comment-Date: Thu, 06 Oct 2022 21:55:19 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Angel Pons
th3fanbus@gmail.com
Gerrit-MessageType: comment