Julius Werner has posted comments on this change. ( https://review.coreboot.org/29684 )
Change subject: soc/intel/common: Bring DISPLAY_MTRRS into the light ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/#/c/29684/2/src/Kconfig File src/Kconfig:
https://review.coreboot.org/#/c/29684/2/src/Kconfig@708 PS2, Line 708: depends on HAVE_DISPLAY_MTRRS Should we maybe consider building a better way for arch/soc/mainboard to add options here? This is heavily x86 specific, as are several others already in here... it's going to get pretty full (especially if you need a dummy HAVE_... option for everything as well).
In Linux they solve this by having a separate Kconfig.debug file next to the normal Kconfig in every subdirectory that offers debug options. We could do the same.
-
Julius Werner (Code Review)