Attention is currently required from: Anastasia Klimchuk, Nikolai Artemiev, Stefan Reinauer.
Sergii Dmytruk has posted comments on this change by Anastasia Klimchuk. ( https://review.coreboot.org/c/flashrom/+/83132?usp=email )
Change subject: flashchips: Set default wp test status for chips with custom config ......................................................................
Patch Set 1:
(2 comments)
File flashchips.c:
https://review.coreboot.org/c/flashrom/+/83132/comment/47f27c43_8f4b055e?usp... : PS1, Line 2651: .tested = {.probe = NT, .read = NT, .erase = NT, .write = BAD, .wp = NT}, The chip seems to have only sector-based write protection and no block protection. "B" was chosen for block protection and sector protection is not implemented, so maybe this should be `NA`.
https://review.coreboot.org/c/flashrom/+/83132/comment/1aac5416_5d07c594?usp... : PS1, Line 3720: .tested = {.probe = OK, .read = OK, .erase = BAD, .write = BAD, .wp = NT}, It's [this chip](https://pdf1.alldatasheet.com/datasheet-pdf/view/57384/CATALYST/CAT28F512.ht...), right? I see no write-protection in there.