Wim Vervoorn 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 2:
(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.
Done
https://review.coreboot.org/c/coreboot/+/40040/1/src/mainboard/facebook/mono... PS1, Line 6: UNUSED_3@0x700000 0x300000
Sequential Numbering? Use UNUSED_2
Done
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.
Done
https://review.coreboot.org/c/coreboot/+/40040/1/src/mainboard/facebook/mono... PS1, Line 6: UNUSED_3@0x700000 0x300000
sequencial number: UNUSED_2
Done