[coreboot-gerrit] Change in coreboot[master]: cpu/intel/common: Fill cpu voltage in SMBIOS tables

[Jonathan Zhang (Code Review)]

Jonathan Zhang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43904 )

Change subject: cpu/intel/common: Fill cpu voltage in SMBIOS tables
......................................................................

Patch Set 4:

(1 comment)

https://review.coreboot.org/c/coreboot/+/43904/4/src/soc/intel/common/block/... 
File src/soc/intel/common/block/cpu/Kconfig:

https://review.coreboot.org/c/coreboot/+/43904/4/src/soc/intel/common/block/... 
PS4, Line 3: 	select CPU_INTEL_COMMON_VOLTAGE
Along the same line as Tim commented in https://review.coreboot.org/c/coreboot/+/43904/4/src/cpu/intel/common/voltag..., CPX-SP does have CONFIG_SOC_INTEL_COMMON_BLOCK_CPU selected, but its MSR 198h does not have those fields (doc 604926), so it should not have CPU_INTEL_COMMON_VOLTAGE selected.