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:
(2 comments)
https://review.coreboot.org/#/c/31766/3/Documentation/lib/flashmap.md File Documentation/lib/flashmap.md:
https://review.coreboot.org/#/c/31766/3/Documentation/lib/flashmap.md@14 PS3, Line 14: to make everything that : has strict layout requirements (e.g. must be aligned to erase blocks or : something else) should be
Sorry, I think you copied the grammar mistakes out of my comment. […]
Done
https://review.coreboot.org/#/c/31766/3/Documentation/lib/flashmap.md@45 PS3, Line 45: programmed (or after some security policy is enabled).
Can we clarify that these first three are deprecated and (should) currently not (be) used by anyone? […]
I'm not sure if they are deprecated. For example, all the WP_RO sections in chromeos.fmd may include FMAP_AREA_RO (and yes it's nested). It's probably just because we didn't have good tools before.
COMPRESSED - it may be still helpful to help people realizing "hey, you should not try a raw dump or execution for this section".
+dhendrix to see how we should revise the descriptions above.