[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;
> What does it return? Does it crash?

It won't crash. It returns 6, and finally get 0.6V in smbios table.