Attention is currently required from: Raul Rangel, Martin Roth, Furquan Shaikh, Marshall Dawson, Angel Pons, Andrey Petrov, Patrick Rudolph, Felix Held. Hello build bot (Jenkins), Nico Huber, Raul Rangel, Marshall Dawson, Paul Menzel, Andrey Petrov, Patrick Rudolph,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/51575
to look at the new patch set (#3).
Change subject: cpu/x86/Kconfig.debug_cpu: drop HAVE_DISPLAY_MTRRS option ......................................................................
cpu/x86/Kconfig.debug_cpu: drop HAVE_DISPLAY_MTRRS option
Since all x86 CPUs in tree have MTRR support, there is no need to guard the DISPLAY_MTRRS option with HAVE_DISPLAY_MTRRS. Also all x86 CPUs/SoCs have a display_mtrrs call at least somewhere in their code, so selecting the DISPLAY_MTRRS option will always have an effect. All SoCs that don't select RESET_VECTOR_IN_RAM have the postcar stage where it gets called. The two AMD SoCs that select RESET_VECTOR_IN_RAM use the FSP2 driver which contains plenty of display_mtrrs calls.
Signed-off-by: Felix Held felix-coreboot@felixheld.de Change-Id: I2894689ce58e7404d9d5a894f3c288bc4016ea19 --- M src/cpu/intel/haswell/Kconfig M src/cpu/x86/Kconfig.debug_cpu M src/soc/intel/common/Kconfig.common 3 files changed, 0 insertions(+), 6 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/75/51575/3