Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33453 )
Change subject: drivers/fsp1_0: select CACHE_MRC_SETTINGS if MRC_CACHE_FMAP ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/#/c/33453/1/src/drivers/intel/fsp1_0/Kconfig File src/drivers/intel/fsp1_0/Kconfig:
https://review.coreboot.org/#/c/33453/1/src/drivers/intel/fsp1_0/Kconfig@a94 PS1, Line 94:
rephrase as 'Your FMAP must contain a region named "RW_MRC_CACHE"'?
Sounds good.