Attention is currently required from: Kaiyen Chang, Patrick Rudolph. Hello build bot (Jenkins), Kaiyen Chang, Angel Pons, Patrick Rudolph,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/52029
to look at the new patch set (#4).
Change subject: soc/intel/jasperlake: Update CpuRatio settings ......................................................................
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_params.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 Signed-off-by: Kaiyen Chang kaiyen.chang@intel.corp-partner.google.com --- 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/4