Attention is currently required from: Nikolai Artemiev.
Hello build bot (Jenkins), Edward O'Callaghan, Angel Pons, Anastasia Klimchuk,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/flashrom/+/69130
to look at the new patch set (#2).
Change subject: tests: ensure chip erase operation is executed ......................................................................
tests: ensure chip erase operation is executed
The `full_chip_erase_with_wp_dummyflasher_test_success` test case checks that erasing a write-protected region of a dummyflasher chip fails.
However erase optimization may cause the erase operation to be skipped if the flash contents are already erased, so the erase operation appears to succeed and the test case fails.
Writing a non-erased value to the chip ensures that an erase operation will be executed and (hopefully) fail due to write protection.
BUG=b:237620197 BRANCH=none TEST=ninja test
Change-Id: Ia00444dcd2ad96c64832a13201efbd064cd7302d Signed-off-by: Nikolai Artemiev nartemiev@google.com --- M include/flash.h M tests/chip_wp.c 2 files changed, 35 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/30/69130/2