Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31766 )
Change subject: Documentation: Explain FMAP and FMD ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/#/c/31766/1/Documentation/flashmap.md File Documentation/flashmap.md:
https://review.coreboot.org/#/c/31766/1/Documentation/flashmap.md@17 PS1, Line 17: non-coreboot data. I think some of this may have been written before CBFS was fully integrated into FMAP and could probably use a rewrite (if you have the time). It would be better to have less focus on history and more on the current state: the FMAP partitions the image into clearly delimited sections and some of those sections may be CBFSes that can hold arbitrary-length files (at least one, the default CBFS, called COREBOOT). General guidance is to make everything that has strict layout requirements (e.g. must be aligned to erase blocks or something else) should be its own FMAP section, and everything else should normally go into CBFS.