[Public]

Reviewed-by: Lijo Lazar <lijo.la...@amd.com>

-----Original Message-----
From: Sundararaju, Sathishkumar <sathishkumar.sundarar...@amd.com> 
Sent: Thursday, June 10, 2021 10:04 AM
To: amd-gfx@lists.freedesktop.org
Cc: Deucher, Alexander <alexander.deuc...@amd.com>; Lazar, Lijo 
<lijo.la...@amd.com>; Sharma, Shashank <shashank.sha...@amd.com>; Sundararaju, 
Sathishkumar <sathishkumar.sundarar...@amd.com>
Subject: [PATCH] drm/amd/pm: support ss metrics read on renoir

add support to read smart shift apu and dgpu power share on renoir.

Signed-off-by: Sathishkumar S <sathishkumar.sundarar...@amd.com>
---
 .../gpu/drm/amd/pm/swsmu/smu12/renoir_ppt.c   | 34 +++++++++++++++++++
 1 file changed, 34 insertions(+)

diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu12/renoir_ppt.c 
b/drivers/gpu/drm/amd/pm/swsmu/smu12/renoir_ppt.c
index ccfcfcf18e4d..9a9c24a6ec35 100644
--- a/drivers/gpu/drm/amd/pm/swsmu/smu12/renoir_ppt.c
+++ b/drivers/gpu/drm/amd/pm/swsmu/smu12/renoir_ppt.c
@@ -1180,6 +1180,28 @@ static int renoir_get_smu_metrics_data(struct 
smu_context *smu,
        case METRICS_VOLTAGE_VDDSOC:
                *value = metrics->Voltage[1];
                break;
+       case METRICS_SS_APU_SHARE:
+               /* return the percentage of APU power with respect to APU's 
power limit.
+                * percentage is reported, this isn't boost value. Smartshift 
power
+                * boost/shift is only when the percentage is more than 100.
+                */
+               if (metrics->StapmOriginalLimit > 0)
+                       *value =  (metrics->ApuPower * 100) / 
metrics->StapmOriginalLimit;
+               else
+                       *value = 0;
+               break;
+       case METRICS_SS_DGPU_SHARE:
+               /* return the percentage of dGPU power with respect to dGPU's 
power limit.
+                * percentage is reported, this isn't boost value. Smartshift 
power
+                * boost/shift is only when the percentage is more than 100.
+                */
+               if ((metrics->dGpuPower > 0) &&
+                   (metrics->StapmCurrentLimit > metrics->StapmOriginalLimit))
+                       *value = (metrics->dGpuPower * 100) /
+                                 (metrics->StapmCurrentLimit - 
metrics->StapmOriginalLimit);
+               else
+                       *value = 0;
+               break;
        default:
                *value = UINT_MAX;
                break;
@@ -1251,6 +1273,18 @@ static int renoir_read_sensor(struct smu_context *smu,
                                                  (uint32_t *)data);
                *size = 4;
                break;
+       case AMDGPU_PP_SENSOR_SS_APU_SHARE:
+               ret = renoir_get_smu_metrics_data(smu,
+                                                 METRICS_SS_APU_SHARE,
+                                                 (uint32_t *)data);
+               *size = 4;
+               break;
+       case AMDGPU_PP_SENSOR_SS_DGPU_SHARE:
+               ret = renoir_get_smu_metrics_data(smu,
+                                                 METRICS_SS_DGPU_SHARE,
+                                                 (uint32_t *)data);
+               *size = 4;
+               break;
        default:
                ret = -EOPNOTSUPP;
                break;
--
2.17.1
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to