Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45889 )
Change subject: mb/google/volteer: Expand WP_RO region to 8MB in fmap ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/45889/2/src/mainboard/google/voltee... File src/mainboard/google/volteer/chromeos.fmd:
https://review.coreboot.org/c/coreboot/+/45889/2/src/mainboard/google/voltee... PS2, Line 9: # SPI flash only the top 16MiB actually gets memory mapped. +Furquan for details, but I believe you cannot do this due to the issue described in this comment. RW_SECTIONs need to stay below 16MB.
Would it be an option to reduce the RW_SECTION sizes by 2MB each instead?
-
Julius Werner (Code Review)