commit 9c17665aaac5890df2926230ae32f3cb608044d8
author Martin Roth <martinroth@chromium.org> Fri Apr 23 12:24:35 2021 -0600
committer Martin Roth <martinroth@google.com> Mon Apr 26 21:44:47 2021 +0000
tree 41d5fa78fff7d1cc5d7d3ea8a68b94ef87690a19
parent 029d997b6eb0c784b844bf554dec5c33286b6507

soc/amd/cezanne: Update STAPM vars with units

Like the Picasso platform, it's very useful to have units on these
variables.

BUG=b:185209734
TEST=Build & Boot

Signed-off-by: Martin Roth <martinroth@chromium.org>
Change-Id: I592c807c5e9a2c17b1c5959e56a01237352c5204
Reviewed-on: https://review.coreboot.org/c/coreboot/+/52649
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Eric Peers <epeers@google.com>
Reviewed-by: Raul Rangel <rrangel@chromium.org>

src/soc/amd/cezanne/fsp_m_params.c

diff --git a/src/soc/amd/cezanne/fsp_m_params.c b/src/soc/amd/cezanne/fsp_m_params.c
index 964ec18..2528086 100644
--- a/src/soc/amd/cezanne/fsp_m_params.c
+++ b/src/soc/amd/cezanne/fsp_m_params.c
@@ -93,11 +93,11 @@
 
 	/* all following fields being 0 is a valid config */
 	mcfg->stapm_boost = config->stapm_boost;
-	mcfg->stapm_time_constant = config->stapm_time_constant;
+	mcfg->stapm_time_constant = config->stapm_time_constant_s;
 	mcfg->apu_only_sppt_limit = config->apu_only_sppt_limit;
-	mcfg->sustained_power_limit = config->sustained_power_limit;
-	mcfg->fast_ppt_limit = config->fast_ppt_limit;
-	mcfg->slow_ppt_limit = config->slow_ppt_limit;
+	mcfg->sustained_power_limit = config->sustained_power_limit_mW;
+	mcfg->fast_ppt_limit = config->fast_ppt_limit_mW;
+	mcfg->slow_ppt_limit = config->slow_ppt_limit_mW;
 	mcfg->slow_ppt_time_constant = config->slow_ppt_time_constant_s;
 	mcfg->thermctl_limit = config->thermctl_limit_degreeC;