Frans Hendriks has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40040 )
Change subject: mb/facebook/monolith: Add fmd files for 6MB BIOS area ......................................................................
Patch Set 1:
(4 comments)
https://review.coreboot.org/c/coreboot/+/40040/1/src/mainboard/facebook/mono... File src/mainboard/facebook/monolith/vboot-ro_6mb.fmd:
https://review.coreboot.org/c/coreboot/+/40040/1/src/mainboard/facebook/mono... PS1, Line 1: FLASH 16M { Correct indent.
https://review.coreboot.org/c/coreboot/+/40040/1/src/mainboard/facebook/mono... PS1, Line 6: UNUSED_3@0x700000 0x300000 Sequential Numbering? Use UNUSED_2
https://review.coreboot.org/c/coreboot/+/40040/1/src/mainboard/facebook/mono... File src/mainboard/facebook/monolith/vboot-rw_6mb.fmd:
https://review.coreboot.org/c/coreboot/+/40040/1/src/mainboard/facebook/mono... PS1, Line 1: FLASH 16M { Correct indent here also.
https://review.coreboot.org/c/coreboot/+/40040/1/src/mainboard/facebook/mono... PS1, Line 6: UNUSED_3@0x700000 0x300000 sequencial number: UNUSED_2