Frans Hendriks has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33484 )
Change subject: mainboard/facebook/fbg1701: Add FMAP file ......................................................................
Patch Set 2:
(2 comments)
https://review.coreboot.org/#/c/33484/1/src/mainboard/facebook/fbg1701/Kconf... File src/mainboard/facebook/fbg1701/Kconfig:
https://review.coreboot.org/#/c/33484/1/src/mainboard/facebook/fbg1701/Kconf... PS1, Line 57: 0x00800000
Will change it to 0x5F700 which is CBFS size with 4K alignment for MRC cache.
Done
https://review.coreboot.org/#/c/33484/1/src/mainboard/facebook/fbg1701/fmap.... File src/mainboard/facebook/fbg1701/fmap.fmd:
https://review.coreboot.org/#/c/33484/1/src/mainboard/facebook/fbg1701/fmap.... PS1, Line 17: RW_MRC_CACHE@0x200 0x8000
Expect the alignment is related to SPI layout/routines. […]
Done