[coreboot-gerrit] [XS] Change in coreboot[master]: soc/intel/meteorlake: Set Energy Perf Bias appropriate default value.

[Erin Park (Code Review)]

Erin Park has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/75907?usp=email )

Change subject: soc/intel/meteorlake: Set Energy Perf Bias appropriate default value.
......................................................................

soc/intel/meteorlake: Set Energy Perf Bias appropriate default value.

Based on meteorlake PnP analysis, ENERGY_POLICY_BALANCE_POWERSAVE is
recommended in order to achieve best PnP user experience.

Change-Id: Ib716584f516a538251c022e15b3193359b794366
---
M src/include/cpu/x86/msr.h
M src/soc/intel/meteorlake/cpu.c
2 files changed, 3 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/07/75907/1

diff --git a/src/include/cpu/x86/msr.h b/src/include/cpu/x86/msr.h
index d369972..e398bea 100644
--- a/src/include/cpu/x86/msr.h
+++ b/src/include/cpu/x86/msr.h
@@ -51,7 +51,9 @@
 #define  SPEED_STEP_ENABLE_BIT		(1 << 16)
 #define IA32_ENERGY_PERF_BIAS		0x1b0
 #define  ENERGY_POLICY_PERFORMANCE	0
+#define  ENERGY_POLICY_BALANCE_PERFORMANCE    4
 #define  ENERGY_POLICY_NORMAL		6
+#define  ENERGY_POLICY_BALANCE_POWERSAVE    8
 #define  ENERGY_POLICY_POWERSAVE	15
 #define  ENERGY_POLICY_MASK		0xf
 #define IA32_PACKAGE_THERM_INTERRUPT	0x1b2
diff --git a/src/soc/intel/meteorlake/cpu.c b/src/soc/intel/meteorlake/cpu.c
index 5177c4d..ae082f0 100644
--- a/src/soc/intel/meteorlake/cpu.c
+++ b/src/soc/intel/meteorlake/cpu.c
@@ -123,7 +123,7 @@
    configure_dca_cap();

/* Set energy policy */
-	set_energy_perf_bias(ENERGY_POLICY_NORMAL);
+	set_energy_perf_bias(ENERGY_POLICY_BALANCE_POWERSAVE);

const config_t *conf = config_of_soc();
    /* Set energy-performance preference */