Attention is currently required from: Angel Pons.
Edward O'Callaghan has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/69196 )
Change subject: layout: Factor out flash_region structure from romentry ......................................................................
Patch Set 2:
(2 comments)
Patchset:
PS2:
What benefit does this provide?
That's a good question since this patch on its own doesn't have any extra context.
I'll leave this space for Nik to fill in and flesh out as it's his project but just wanted to get him started here. The tl;dr is to get the Intel CSME co-processor to work properly in the writeprotect/layout frameworks.
This patch almost stands on its own merit but just needs some more context. Over to you Nik possibly with a topic and some follow up patches?
File include/layout.h:
https://review.coreboot.org/c/flashrom/+/69196/comment/e959c942_88deb8ca PS2, Line 42: bool read_prot; : bool write_prot; @Nik, when you hijack this patch maybe drop these in this patch and add in a subsequent patch when you need them.