Anastasia Klimchuk has submitted this change. ( https://review.coreboot.org/c/flashrom/+/85092?usp=email )
Change subject: sfdp: Update the message shown when SFDP-capable chip is detected ......................................................................
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(-)
Approvals: build bot (Jenkins): Verified Peter Marheine: Looks good to me, approved Matti Finder: Looks good to me, approved
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%5Cn" "===\n"); }