Attention is currently required from: Matti Finder, Nikolai Artemiev, Sergii Dmytruk, Stefan Reinauer.
Anastasia Klimchuk has posted comments on this change by Anastasia Klimchuk. ( https://review.coreboot.org/c/flashrom/+/85075?usp=email )
Change subject: flashchips: Skip "WP untested" message for SFDP-capable chip ......................................................................
Patch Set 3:
(2 comments)
Patchset:
PS2:
I wonder if `chip->tested. […]
1) Ignored in the meaning, not showing the message "flash has status untested for WP"? 2) Or ignored in the meaning, if it's set to OK, don't believe it because it can't be?
I think the first one, the message still useful: people see it and sometimes wonder and sometimes add support for WP. Apart from the very edge cases, like "generic" chip entries, I would keep the message (for now... but things can change).
The second option: I don't think there are such entries in flashchips at the moment. It has always been that the contributor who adds support also marks as tested.
Commit Message:
https://review.coreboot.org/c/flashrom/+/85075/comment/958ec65c_daeea200?usp... : PS2, Line 12: SDFP chip and what would you report?
`SFDP`
Thank you! And I found the same typo in the commit message of the next patch too!