[AMD Official Use Only - General]

This patch is
Reviewed-by: Yifan Zhang <yifan1.zh...@amd.com>

-----Original Message-----
From: Jesse Zhang <jesse.zh...@amd.com>
Sent: Wednesday, August 23, 2023 3:16 PM
To: amd-gfx@lists.freedesktop.org
Cc: Deucher, Alexander <alexander.deuc...@amd.com>; Zhang, Yifan 
<yifan1.zh...@amd.com>; Zhang, Jesse(Jie) <jesse.zh...@amd.com>
Subject: [PATCH] drm/amdgpu/pm: Remove the duplicate dpm status check

Since the smu firmware has fixed the issue that described in the commit 
"60d61f4ed6ead".
So we only need keep dpm status check in the funciton - 
smu_v13_0_5_set_soft_freq_limited_range.

Signed-off-by: Jesse Zhang <jesse.zh...@amd.com>
---
 drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_5_ppt.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_5_ppt.c 
b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_5_ppt.c
index c6e7c2115a26..0dce672ac1b9 100644
--- a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_5_ppt.c
+++ b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_5_ppt.c
@@ -1060,7 +1060,7 @@ static int smu_v13_0_5_set_performance_level(struct 
smu_context *smu,
                return -EINVAL;
        }

-       if (sclk_min && sclk_max && smu_v13_0_5_clk_dpm_is_enabled(smu, 
SMU_SCLK)) {
+       if (sclk_min && sclk_max) {
                ret = smu_v13_0_5_set_soft_freq_limited_range(smu,
                                                            SMU_SCLK,
                                                            sclk_min,
--
2.25.1

Reply via email to