From 6934fe29fb1115fff35e02da9dc94bbd9441059b Mon Sep 17 00:00:00 2001
From: Kent Russell <kent.russell@amd.com>
Date: Wed, 30 Mar 2022 09:27:41 -0400
Subject: drm/amdgpu: Fix unique_id references for Sienna Cichlid
Git-commit: 5e9c445172fde2ee761af91ad302e41041de6be8
Patch-mainline: v5.19-rc1
References: jsc#PED-1166 jsc#PED-1168 jsc#PED-1170 jsc#PED-1218 jsc#PED-1220 jsc#PED-1222 jsc#PED-1223 jsc#PED-1225
Since unique_id is only supported in PMFW 0x3A5300 and higher, we will
only be able to use it inside Smu_Metrics_V3_t, which requires PMFW
0x3A4900 and higher. Remove the unique_id/serial_number references from
the v1 and v2 tables to avoid any confusion, and return 0 if metrics_v1
or metrics_v2 are used to try to get the unique_id/serial_number.
Signed-off-by: Kent Russell <kent.russell@amd.com>
Reviewed-by: Lijo Lazar <lijo.lazar@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Acked-by: Patrik Jakobsson <pjakobsson@suse.de>
---
.../swsmu/inc/pmfw_if/smu11_driver_if_sienna_cichlid.h | 6 ------
.../gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c | 10 ++++------
2 files changed, 4 insertions(+), 12 deletions(-)
diff --git a/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu11_driver_if_sienna_cichlid.h b/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu11_driver_if_sienna_cichlid.h
index 5831145646e6..08f0bb2af5d2 100644
--- a/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu11_driver_if_sienna_cichlid.h
+++ b/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu11_driver_if_sienna_cichlid.h
@@ -1420,9 +1420,6 @@ typedef struct {
uint8_t PcieWidth ;
uint16_t AverageGfxclkFrequencyTarget;
- uint32_t PublicSerialNumLower32;
- uint32_t PublicSerialNumUpper32;
-
uint16_t Padding16_2;
} SmuMetrics_t;
@@ -1480,9 +1477,6 @@ typedef struct {
uint8_t PcieWidth ;
uint16_t AverageGfxclkFrequencyTarget;
- uint32_t PublicSerialNumLower32;
- uint32_t PublicSerialNumUpper32;
-
uint16_t Padding16_2;
} SmuMetrics_V2_t;
diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c b/drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c
index b2f3d80e5945..ab3e9d8b831e 100644
--- a/drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c
+++ b/drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c
@@ -716,14 +716,12 @@ static int sienna_cichlid_get_smu_metrics_data(struct smu_context *smu,
use_metrics_v2 ? metrics_v2->CurrFanSpeed : metrics->CurrFanSpeed;
break;
case METRICS_UNIQUE_ID_UPPER32:
- *value = use_metrics_v3 ? metrics_v3->PublicSerialNumUpper32 :
- use_metrics_v2 ? metrics_v2->PublicSerialNumUpper32 :
- metrics->PublicSerialNumUpper32;
+ /* Only supported in 0x3A5300+, metrics_v3 requires 0x3A4900+ */
+ *value = use_metrics_v3 ? metrics_v3->PublicSerialNumUpper32 : 0;
break;
case METRICS_UNIQUE_ID_LOWER32:
- *value = use_metrics_v3 ? metrics_v3->PublicSerialNumLower32 :
- use_metrics_v2 ? metrics_v2->PublicSerialNumLower32 :
- metrics->PublicSerialNumLower32;
+ /* Only supported in 0x3A5300+, metrics_v3 requires 0x3A4900+ */
+ *value = use_metrics_v3 ? metrics_v3->PublicSerialNumLower32 : 0;
break;
default:
*value = UINT_MAX;
--
2.38.1