[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 5:

(2 comments)

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: 0xffff
> Core Voltage = MSR_PERF_STATUS[37:32] * (float) 1/(2^13). […]

current voltage = float([47:32] / (2^13)).
so it is 0xffff

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. […]

I remove this line and keep other changes. Not sure this is fine for other platform.