Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47988 )
Change subject: soc/intel/common/dmi: Add DMI driver support ......................................................................
Patch Set 9:
(1 comment)
https://review.coreboot.org/c/coreboot/+/47988/9/src/soc/intel/common/block/... File src/soc/intel/common/block/dmi/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/47988/9/src/soc/intel/common/block/... PS9, Line 6: smm-$(CONFIG_SOC_INTEL_COMMON_BLOCK_DMI) += dmi.c
Done
I guess I would rather have SMM opt-in to code rather than opt-out