Attention is currently required from: Matt DeVillier. Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58184 )
Change subject: mb/google/sarien: Add default fmap for non-ChromeOS builds ......................................................................
Patch Set 2: Code-Review+1
(2 comments)
File src/mainboard/google/sarien/default.fmd:
PS2: nit: It seems that there are no gaps and that they all follow one another. So I would just use the region sizes (like you did with RP_VPD or FMAP).
https://review.coreboot.org/c/coreboot/+/58184/comment/9a9be1f1_13fd9b1c PS2, Line 7: SI_PDR(PRESERVE)@0x3fc000 0x4000 What is this region for?