summaryrefslogtreecommitdiffstats
path: root/src/soc/amd/picasso/root_complex.c
diff options
context:
space:
mode:
authorFelix Held <felix-coreboot@felixheld.de>2021-05-07 19:02:45 +0200
committerFelix Held <felix-coreboot@felixheld.de>2021-05-08 17:56:21 +0000
commitef51157e52cf27a2173a72d6faed19893bb0911f (patch)
treec036ff66c2b54ef37b0b32fd29766f3b45199199 /src/soc/amd/picasso/root_complex.c
parent95f1bb852525808a7b514745ebe3d19b6d2df7c4 (diff)
downloadcoreboot-ef51157e52cf27a2173a72d6faed19893bb0911f.tar.gz
coreboot-ef51157e52cf27a2173a72d6faed19893bb0911f.tar.bz2
coreboot-ef51157e52cf27a2173a72d6faed19893bb0911f.zip
soc/amd/picasso/root_complex: move DPTC_TOTAL_UPDATE_PARAMS out of enum
The other enum entries are control IDs for the ALIB_FUNCTION_DYNAMIC_POWER_THERMAL_CONFIG ALIB function while DPTC_TOTAL_UPDATE_PARAMS is the total number of configuration settings that will get passed as parameter in the ALIB call, so it shouldn't be part of that enum. TEST=Timeless build for Mandolin results in identical binary. Signed-off-by: Felix Held <felix-coreboot@felixheld.de> Change-Id: I0cb9e9d2ba579a74d916011b4ead71cc86d69a24 Reviewed-on: https://review.coreboot.org/c/coreboot/+/53916 Reviewed-by: Raul Rangel <rrangel@chromium.org> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/soc/amd/picasso/root_complex.c')
-rw-r--r--src/soc/amd/picasso/root_complex.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/soc/amd/picasso/root_complex.c b/src/soc/amd/picasso/root_complex.c
index 47b3a43cd320..932ca766b2d4 100644
--- a/src/soc/amd/picasso/root_complex.c
+++ b/src/soc/amd/picasso/root_complex.c
@@ -23,9 +23,10 @@ enum {
SUSTAINED_POWER_LIMIT_PARAM_ID = 0x5,
FAST_PPT_LIMIT_PARAM_ID = 0x6,
SLOW_PPT_LIMIT_PARAM_ID = 0x7,
- DPTC_TOTAL_UPDATE_PARAMS = 4,
};
+#define DPTC_TOTAL_UPDATE_PARAMS 4
+
struct dptc_param {
uint8_t id;
uint32_t value;