Attention is currently required from: Anastasia Klimchuk.
Nikolai Artemiev has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/69130 )
Change subject: tests: ensure chip erase operation is executed ......................................................................
Patch Set 3:
(3 comments)
Commit Message:
https://review.coreboot.org/c/flashrom/+/69130/comment/977d339b_1fa0484f PS1, Line 18: (hopefully)
"and is expected to fail"
I have rephrased this sentence a bit now.
https://review.coreboot.org/c/flashrom/+/69130/comment/7b6f389a_3162107a PS1, Line 18: exectued
exectued -> executed (typo)
Done
File include/flash.h:
https://review.coreboot.org/c/flashrom/+/69130/comment/545a7087_b748f374 PS1, Line 159: #define UNERASED_VALUE(flash)
Will this ever be needed outside of tests? […]
It's a useful counterpart to `ERASED_VALUE()` and it might be helpful for erase optimization code etc, so I think it makes sense to leave them both here.