Attention is currently required from: Evan Benn, Hsuan-ting Chen.

Brian Norris uploaded patch set #4 to this change.

View Change

The following approvals got outdated and were removed: Verified-1 by build bot (Jenkins)

flashrom_tester: Correct "WP screw" message

flashrom_tester prints hints on how to modify hardware write protect
state as follows:

...
> connect the battery (and/or open the WP screw)
...
> disconnect the battery (and/or open the WP screw)
...

The first advice should be "[...] close the WP screw".

TEST=`flashrom_tester --flashrom_binary=$(which flashrom) \
internal Erase_and_Write Fail_to_verify`

Change-Id: I45f06db51e92e68bf724b13bdf5b31bba511d270
Signed-off-by: Brian Norris <briannorris@chromium.org>
---
M README.rst
M util/flashrom_tester/src/utils.rs
2 files changed, 4 insertions(+), 2 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/83/82083/4

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

Gerrit-Project: flashrom
Gerrit-Branch: main
Gerrit-Change-Id: I45f06db51e92e68bf724b13bdf5b31bba511d270
Gerrit-Change-Number: 82083
Gerrit-PatchSet: 4
Gerrit-Owner: Brian Norris <briannorris@chromium.org>
Gerrit-Reviewer: Evan Benn <evanbenn@gmail.com>
Gerrit-Reviewer: Hsuan-ting Chen <roccochen@google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Attention: Evan Benn <evanbenn@gmail.com>
Gerrit-Attention: Hsuan-ting Chen <roccochen@google.com>
Gerrit-MessageType: newpatchset