[coreboot-gerrit] [S] Change in coreboot[master]: soc/intel/meteorlake: Set max Pkg C-states to Auto

[Kapil Porwal (Code Review)]

Kapil Porwal has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/71653 )

Change subject: soc/intel/meteorlake: Set max Pkg C-states to Auto
......................................................................

soc/intel/meteorlake: Set max Pkg C-states to Auto

This patch configures max Pkg C-state to Auto which limits the max
C-state to deep C-state.

Port of commit af42906efa72 ("soc/intel/alderlake: Set max Pkg C-states
to Auto")

BUG=none
TEST=Boot to the OS on Google/Rex.

Snippet from FSP log:
[SPEW ]   PkgCStateLimit : 0xFF

Signed-off-by: Kapil Porwal 
Change-Id: Ic403ab83a594b04920d5cf600432939687a2598b
---
M src/soc/intel/meteorlake/chip.h
M src/soc/intel/meteorlake/fsp_params.c
2 files changed, 42 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/53/71653/1

diff --git a/src/soc/intel/meteorlake/chip.h b/src/soc/intel/meteorlake/chip.h
index 19b34a2..ad382e3 100644
--- a/src/soc/intel/meteorlake/chip.h
+++ b/src/soc/intel/meteorlake/chip.h
@@ -43,6 +43,25 @@
    DDI_ENABLE_HPD = 1 << 1,
 };

+/*
+ * The Max Pkg Cstate
+ * Values 0 - C0/C1, 1 - C2, 2 - C3, 3 - C6, 4 - C7, 5 - C7S, 6 - C8, 7 - C9, 8 - C10,
+ * 254 - CPU Default , 255 - Auto.
+ */
+enum pkgcstate_limit {
+	LIMIT_C0_C1		= 0,
+	LIMIT_C2		= 1,
+	LIMIT_C3		= 2,
+	LIMIT_C6		= 3,
+	LIMIT_C7		= 4,
+	LIMIT_C7S		= 5,
+	LIMIT_C8		= 6,
+	LIMIT_C9		= 7,
+	LIMIT_C10		= 8,
+	LIMIT_CPUDEFAULT	= 254,
+	LIMIT_AUTO		= 255,
+};
+
 /* Bit values for use in LpmStateEnableMask. */
 enum lpm_state_mask {
    LPM_S0i2_0 = BIT(0),
diff --git a/src/soc/intel/meteorlake/fsp_params.c b/src/soc/intel/meteorlake/fsp_params.c
index d40779a..50b4a76 100644
--- a/src/soc/intel/meteorlake/fsp_params.c
+++ b/src/soc/intel/meteorlake/fsp_params.c
@@ -423,6 +423,7 @@
    s_cfg->Hwp = 1;
    s_cfg->Cx = 1;
    s_cfg->PsOnEnable = 1;
+	s_cfg->PkgCStateLimit = LIMIT_AUTO;
    /* Enable the energy efficient turbo mode */
    s_cfg->EnergyEfficientTurbo = 1;
    s_cfg->PmcLpmS0ixSubStateEnableMask = get_supported_lpm_mask();