Attention is currently required from: Nikolai Artemiev, Sergii Dmytruk, Stefan Reinauer.
Anastasia Klimchuk 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 2:
(2 comments)
File flashchips.c:
https://review.coreboot.org/c/flashrom/+/83132/comment/7b1368f8_049b1985?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. […]
Thank you so much for catching! I put NA here (and also in the other chip).
https://review.coreboot.org/c/flashrom/+/83132/comment/0f952d8b_947270fb?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. […]
Done