[coreboot-gerrit] Change in coreboot[master]: drivers/intel/fsp2_0/Kconfig: select HAVE_DISPLAY_MTRRS

[Nico Huber (Code Review)]

Attention is currently required from: Raul Rangel, Martin Roth, Furquan Shaikh, Marshall Dawson, Angel Pons, Andrey Petrov, Patrick Rudolph, Felix Held.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/51575 )

Change subject: drivers/intel/fsp2_0/Kconfig: select HAVE_DISPLAY_MTRRS
......................................................................

Patch Set 2:

(1 comment)

Patchset:

PS2:
> I remember doing patches for this in the past, which got stalled.

There are currently three things done by the two Kconfigs:

1. Show the user option only for platforms where display_mtrrs() gets called.
2. Show the user option only for platforms that are known to be supported by display_mtrrs().
3. Decide if display_mtrrs() is going to do something.

The first can be ignored if we can show that it always gets called.
Likewise, 2. is not an issue if we can show that it supports all
coreboot-supported x86 CPUs. If we decide that the code should just
always be active if supported (e.g. at level DEBUG or SPEW), we can
merge the two Kconfigs into one.