Attention is currently required from: Kaiyen Chang, Patrick Rudolph.

Kaiyen Chang uploaded patch set #2 to this change.

View Change

soc/intel/jasperlake: Update CpuRatio settings

An extra reset is observed on JSL platform while the flex ratio in CPU
Straps is set to a non-zero value. The root cause is that the config of
CpuRatio on JSL is fixed to zero in fsp_param.c. While FSP finds the
value of CpuRatio from m_cfg is inconsistent with the value from CPU
straps, it will invoke a system reset to apply the value from m_cfg.

This change adds a mechanism to set CpuRatio to allowed maximum
processor non-turbo ratio (from CPU straps) when cpu_ratio_override is
not set, or overriding CpuRatio with the value of cpu_ratio_override
when cpu_ratio_override is set to a non-zero value instead of just
fixing CpuRatio to zero.

BUG=b:181588337
BRANCH=None
TEST=Boot JSL platform with a non-zero flex ratio set in CPU Straps and
     observe there is no extra reset in meminit while cpu_ratio_override
     is not set.

Change-Id: I20b9d5620b8e394201e82185eb28b67d6702b2d5
---
M src/soc/intel/jasperlake/romstage/fsp_params.c
1 file changed, 12 insertions(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/29/52029/2

To view, visit change 52029. To unsubscribe, or for help writing mail filters, visit settings.