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

[Tim Chu (Code Review)]

Tim Chu 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/cpu/intel/common/voltag... 
File src/cpu/intel/common/voltage.c:

https://review.coreboot.org/c/coreboot/+/43904/4/src/cpu/intel/common/voltag... 
PS4, Line 12: 	return (rdmsr(MSR_PERF_STATUS).hi & 0xffff) * 10 / 8192;
A question about this function. It seems not all intel platforms support this field(such as CPX). Not sure it is fine to select this function in SOC_INTEL_COMMON_BLOCK_CPU.