Hung-Te Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31766 )
Change subject: Documentation: Explain FMAP and FMD ......................................................................
Patch Set 5:
But what does that actually say that isn't already said by the WP_RO region?
WP_RO is simply a recommendation for Chrome OS firmware layout. - We had firmware before WP_RO was introduced. - What if someday we want multiple sections being write-protected? - Will it be more descriptive to set ME section (when ME locked) as RO? - For people that wants to use Coreboot but not ChromeOS, they can use whatever names they like, and enable AREA_RO in their process.
Somehow I feel we should remove the logic of hard-coded 'WP_RO' name in CrOS factory, and always looking at AREA_RO flag to decide which section to apply WP. Unfortunately there's too much legacy that we can't drop it.
I think we should only have flags when someone has a concrete
In latest version I've added a note these are not used by any coreboot users today and hope that's enough.