Attention is currently required from: Edward O'Callaghan, Angel Pons.
Nikolai Artemiev has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/70128 )
Change subject: flashrom: Skip read/write/erase/verify ops on inaccessable flash regions ......................................................................
Patch Set 9:
(1 comment)
Commit Message:
https://review.coreboot.org/c/flashrom/+/70128/comment/c09b07a8_f360fc06 PS6, Line 11: platforms with active an CSME coprocessor.
Currently, the way to use flashrom to erase/write parts of a flash chip when some other regions are […]
I agree, silently skipping regions could easily go unnoticed by users and probably isn't what most users would expect to happen in that case.
I'm working on some patches to integrate this better with layout logic, but they're incremental changes over this implementation.
I think we should disable region skipping by default (i.e. error out if there are any protected regions) and make it opt-in feature via a flashrom flag, similar to `force-i-want-a-brick`.