Anastasia Klimchuk has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/85075?usp=email )
Change subject: flashchips: Skip "WP untested" message for SFDP-capable chip ......................................................................
flashchips: Skip "WP untested" message for SFDP-capable chip
This entry in the flashchips represent a "SFDP-capable chip" and it doesn't make sense to show the message "WP operation has status untested, please report this". The entry can cover any generic SDFP chip and what would you report?
Secondly, the entry "SFDP-capable chip" does not currently support WP operations anyway.
Going further, we will be working with SFDP way more, so this area needs to be gradually upgraded.
Change-Id: I7e945389895a8042df3aaae72bccf73405be8651 Signed-off-by: Anastasia Klimchuk aklm@flashrom.org --- M flashchips.c 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/75/85075/1
diff --git a/flashchips.c b/flashchips.c index 742e017..17b5fe8 100644 --- a/flashchips.c +++ b/flashchips.c @@ -23522,7 +23522,7 @@ /* We present our own "report this" text hence we do not */ /* want the default "This flash part has status UNTESTED..." */ /* text to be printed. */ - .tested = TEST_OK_PREW, + .tested = TEST_OK_PREWB, .probe = PROBE_SPI_SFDP, .block_erasers = {}, /* set by probing function */ .unlock = SPI_DISABLE_BLOCKPROTECT, /* is this safe? */