Ben Kao has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/60254 )
Change subject: soc/intel/jasperlake: Disable C1 C-state auto demotion ......................................................................
soc/intel/jasperlake: Disable C1 C-state auto demotion
Disable C1 C-state auto demotion to deduce SOC package power.
When enabled, CPU will conditionally demote C3/C6/C7 requests to C1 based on uncore auto-demote information.
TEST=test_that --board=dedede [IP] power_VideoCall
VideoCall Enable C1 auto demotion Disable C1 auto demotion package-0_pwr_avg 3.92 3.15
Signed-off-by: Ben Kao ben.kao@intel.com Change-Id: I82565c734b28c60a1eee18ac079f5deaa6939d1e --- M src/soc/intel/jasperlake/fsp_params.c 1 file changed, 3 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/54/60254/1
diff --git a/src/soc/intel/jasperlake/fsp_params.c b/src/soc/intel/jasperlake/fsp_params.c index 085148f7..c8545dd 100644 --- a/src/soc/intel/jasperlake/fsp_params.c +++ b/src/soc/intel/jasperlake/fsp_params.c @@ -209,6 +209,9 @@ params->PchFivrExtVnnRailEnabledStates = 0; }
+ /* Disable C1 C-state Demotion */ + params->C1StateAutoDemotion = 0; + /* Override/Fill FSP Silicon Param for mainboard */ mainboard_silicon_init_params(params); }