Attention is currently required from: Anastasia Klimchuk, Chinmay Lonkar.
Martin L Roth has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/64259?usp=email )
Change subject: flashchips: Add voltage data for several chips ......................................................................
Patch Set 4:
(3 comments)
File flashchips.c:
https://review.coreboot.org/c/flashrom/+/64259/comment/9fdb16d6_81442d32 : PS4, Line 8451: {2700, 3300},
https://html.datasheetq.com/pdf-html/123108/Intel/31page/28F008S3.html […]
Will update in follow-on patch.
https://review.coreboot.org/c/flashrom/+/64259/comment/8ba06fab_65fd7bf0 : PS4, Line 8480: .voltage = {2700, 3600}, For this, we probably just want to use 5v for external programmers so we can use the same voltage for read, write, and erase.
https://www.datasheetq.com/28F400BV-doc-Intel
5V or 12V Program/Erase 2.7V, 3.3V or 5V Read Operation
Will update in follow-on patch
https://review.coreboot.org/c/flashrom/+/64259/comment/81b00109_e13aaaa0 : PS4, Line 8509: .voltage = {2700, 3600}, I think this version should probably be 5V as well:
https://www.datasheetq.com/28F400CV-T-doc-Intel
5V or 12V Program/Erase 2.7V, 3.3V or 5V Read Operation
Will update in follow-on patch.