Attention is currently required from: Evan Benn.
Nikolai Artemiev has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/67720 )
Change subject: writeprotect: Add some debug logging if wp_verify fails ......................................................................
Patch Set 1: Code-Review+1
(2 comments)
File writeprotect.c:
https://review.coreboot.org/c/flashrom/+/67720/comment/38dc7cdf_97c847c8 PS1, Line 166: return FLASHROM_WP_ERR_READ_FAILED;
This does slightly change the behaviour where before an early failure would be a VERIFY_FAILED but n […]
The error code change seems fine.
https://review.coreboot.org/c/flashrom/+/67720/comment/3d3c8405_b285b0ba PS1, Line 168: wp_verify nit: change to `%s:` and add `__func__` as a format arg
ditto in debug call below