[coreboot-gerrit] [S] Change in coreboot[master]: driver/{intel,mrc_cache}: Have MRC cache ver. based on FSP-M/MRC ver

[Arthur Heymans (Code Review)]

Attention is currently required from: Andrey Petrov, Kapil Porwal, Ronak Kanabar, Subrata Banik, Tarun Tuli.

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/75919?usp=email )

Change subject: driver/{intel,mrc_cache}: Have MRC cache ver. based on FSP-M/MRC ver
......................................................................

Patch Set 3: Code-Review-1

(1 comment)

File src/drivers/mrc_cache/Kconfig:

https://review.coreboot.org/c/coreboot/+/75919/comment/a2ef171e_a6da22a1 :
PS3, Line 14: 	help
This choice is user visible on ALL users of CACHE_MRC_SETTINGS, however not all users use FSP and the choice is not even meaningfull on all FSP platforms.

Is a user visible option even meaningful? The FSP either exposes extended header or it does not. The user should not have to figure this out. The platform should select the right option.