Anastasia Klimchuk has uploaded this change for review.

View Change

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? */

To view, visit change 85075. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: newchange
Gerrit-Project: flashrom
Gerrit-Branch: main
Gerrit-Change-Id: I7e945389895a8042df3aaae72bccf73405be8651
Gerrit-Change-Number: 85075
Gerrit-PatchSet: 1
Gerrit-Owner: Anastasia Klimchuk <aklm@chromium.org>