Jason Glenesk has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/76091?usp=email )
Change subject: mb/google/skyrim/variants/baseboard:Update system_configuration ......................................................................
mb/google/skyrim/variants/baseboard:Update system_configuration
Update system_configuration to 3 for 15W. Specification "FT6 Infrastructure Roadmap"#57316" incorrectly lists system config index should be 4 for 15W. Setting to 4 will cause an additional call to the SMU that is not needed. Both SMU and FSP interpret configs > 3 as 3.
Change-Id: I1f3e305c48801b4e499de56d06c0dcd3eeacc626 Signed-off-by: Jason Glenesk jason.glenesk@amd.com --- M src/mainboard/google/skyrim/variants/baseboard/devicetree.cb 1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/91/76091/1
diff --git a/src/mainboard/google/skyrim/variants/baseboard/devicetree.cb b/src/mainboard/google/skyrim/variants/baseboard/devicetree.cb index b605e1c..f24dc59 100644 --- a/src/mainboard/google/skyrim/variants/baseboard/devicetree.cb +++ b/src/mainboard/google/skyrim/variants/baseboard/devicetree.cb @@ -262,8 +262,8 @@ end
# DPTC: Refer the spec "FT6 Infrastructure Roadmap"#57316 - # Set system_configuration to 4 for 15W - register "system_configuration" = "4" + # Set system_configuration to 3 for 15W + register "system_configuration" = "3" # Normal register "slow_ppt_limit_mW" = "25000" register "fast_ppt_limit_mW" = "30000"