Patrick Rudolph has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/86604?usp=email )
Change subject: soc/intel/xeon_sp/spr: Use default turbo ratio ......................................................................
soc/intel/xeon_sp/spr: Use default turbo ratio
Allow a board to use the default turbo ratio by not specifying turbo_ratio_limit and turbo_ratio_limit_cores in the devicetree.cb.
FIXES: Intel PTAT tool no longer complains about 0Mhz turbo frequency.
Change-Id: Ib8fbc78997fc7f8e6c80b2029d63b70f6117542e Signed-off-by: Patrick Rudolph patrick.rudolph@9elements.com --- M src/soc/intel/xeon_sp/spr/cpu.c 1 file changed, 10 insertions(+), 6 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/04/86604/1
diff --git a/src/soc/intel/xeon_sp/spr/cpu.c b/src/soc/intel/xeon_sp/spr/cpu.c index febc72b..383bf5f 100644 --- a/src/soc/intel/xeon_sp/spr/cpu.c +++ b/src/soc/intel/xeon_sp/spr/cpu.c @@ -95,14 +95,18 @@ wrmsr(MSR_VR_CURRENT_CONFIG, msr);
/* Set Turbo Ratio Limits */ - msr.lo = chip_config->turbo_ratio_limit & 0xffffffff; - msr.hi = (chip_config->turbo_ratio_limit >> 32) & 0xffffffff; - wrmsr(MSR_TURBO_RATIO_LIMIT, msr); + if (chip_config->turbo_ratio_limit) { + msr.lo = chip_config->turbo_ratio_limit & 0xffffffff; + msr.hi = (chip_config->turbo_ratio_limit >> 32) & 0xffffffff; + wrmsr(MSR_TURBO_RATIO_LIMIT, msr); + }
/* Set Turbo Ratio Limit Cores */ - msr.lo = chip_config->turbo_ratio_limit_cores & 0xffffffff; - msr.hi = (chip_config->turbo_ratio_limit_cores >> 32) & 0xffffffff; - wrmsr(MSR_TURBO_RATIO_LIMIT_CORES, msr); + if (chip_config->turbo_ratio_limit_cores) { + msr.lo = chip_config->turbo_ratio_limit_cores & 0xffffffff; + msr.hi = (chip_config->turbo_ratio_limit_cores >> 32) & 0xffffffff; + wrmsr(MSR_TURBO_RATIO_LIMIT_CORES, msr); + }
/* Set energy policy */ msr = rdmsr(MSR_ENERGY_PERF_BIAS_CONFIG);