sravan/system76-coreboot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

soc/amd: introduce and use get_pstate_core_uvolts for SVI2 and SVI3

Browse Source 
Since SVI3 has the CPU voltage ID split into two parts, a serial voltage
ID version specific function is needed to get the raw core VID value.
This will allow making get_pstate_core_power common for all AMD CPUs in
a follow-up patch.

Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: I71ca88c38b307558905a26cce8be1e8ffc5fbed4
Reviewed-on: https://review.coreboot.org/c/coreboot/+/73996
Reviewed-by: Eric Lai <eric_lai@quanta.corp-partner.google.com>
Reviewed-by: Fred Reitberger <reitbergerfred@gmail.com>
Reviewed-by: Matt DeVillier <matt.devillier@amd.corp-partner.google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
Felix Held
2023-03-23 23:57:44 +01:00
parent 23a398e001
commit 2c9de49a97
8 changed files with 28 additions and 30 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/soc/amd/phoenix/acpi.c
View File

@@ -137,10 +137,10 @@ uint32_t get_pstate_core_freq(union pstate_msr pstate_reg)
uint32_t get_pstate_core_power(union pstate_msr pstate_reg)
{
uint32_t voltage_in_uvolts, core_vid, current_value_amps, current_divisor, power_in_mw;
uint32_t voltage_in_uvolts, current_value_amps, current_divisor, power_in_mw;
/* Core voltage ID */
core_vid = pstate_reg.cpu_vid_0_7 | pstate_reg.cpu_vid_8 << 8;
/* Get Voltage from core voltage ID */
voltage_in_uvolts = get_pstate_core_uvolts(pstate_reg);
/* Current value in amps */
current_value_amps = pstate_reg.idd_value;
@@ -148,9 +148,6 @@ uint32_t get_pstate_core_power(union pstate_msr pstate_reg)
/* Current divisor */
current_divisor = pstate_reg.idd_div;
/* Voltage */
voltage_in_uvolts = get_uvolts_from_vid(core_vid);
/* Power in mW */
power_in_mw = (voltage_in_uvolts) / 10 * current_value_amps;