Anastasia Klimchuk submitted this change.

View Change

Approvals: build bot (Jenkins): Verified Peter Marheine: Looks good to me, approved Matti Finder: Looks good to me, approved
sfdp: Update the message shown when SFDP-capable chip is detected

Testing:
flashrom -p dummy:emulate=MX25L6436 -c "SFDP-capable chip"

Change-Id: If1a480ae78f158cc4626e345149ea9025443f8a7
Signed-off-by: Anastasia Klimchuk <aklm@flashrom.org>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/85092
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Peter Marheine <pmarheine@chromium.org>
Reviewed-by: Matti Finder <matti.finder@gmail.com>
---
M flashrom.c
1 file changed, 7 insertions(+), 14 deletions(-)

diff --git a/flashrom.c b/flashrom.c
index 4ae9390..8daad74 100644
--- a/flashrom.c
+++ b/flashrom.c
@@ -1159,25 +1159,18 @@
* been found on this interface.
*/
if (startchip == 0 && flash->chip->model_id == SFDP_DEVICE_ID) {
- msg_cinfo("===\n"
- "SFDP has autodetected a flash chip which is "
- "not natively supported by flashrom yet.\n");
+ msg_cinfo("===\nSFDP has autodetected a flash chip.\n");
if (count_usable_erasers(flash) == 0)
msg_cinfo("The standard operations read and "
- "verify should work, but to support "
- "erase, write and all other "
- "possible features");
+ "verify should work, but support for "
+ "erase and write needs to be added manually.\n");
else
msg_cinfo("All standard operations (read, "
- "verify, erase and write) should "
- "work, but to support all possible "
- "features");
+ "verify, erase and write) should work.\n");

- msg_cinfo(" we need to add them manually.\n"
- "You can help us by mailing us the output of the following command to "
- "flashrom@flashrom.org:\n"
- "'flashrom -VV [plus the -p/--programmer parameter]'\n"
- "Thanks for your help!\n"
+ msg_cinfo("Additionally, flashrom supports RPMC commands via SFDP autodetection.\n"
+ "We may add support for more features via SFDP in future.\n"
+ "If you are interested, join us on the mailing list https://flashrom.org/contact.html#mailing-list-1\n"
"===\n");
}


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

Gerrit-MessageType: merged
Gerrit-Project: flashrom
Gerrit-Branch: main
Gerrit-Change-Id: If1a480ae78f158cc4626e345149ea9025443f8a7
Gerrit-Change-Number: 85092
Gerrit-PatchSet: 4
Gerrit-Owner: Anastasia Klimchuk <aklm@chromium.org>
Gerrit-Reviewer: Anastasia Klimchuk <aklm@chromium.org>
Gerrit-Reviewer: Matti Finder <matti.finder@gmail.com>
Gerrit-Reviewer: Peter Marheine <pmarheine@chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>