drm/amd/powerplay: error out when force clock level under auto dpm mode V2

Forcing clock level is supported under manual dpm mode only. Error out
when trying to set under manual mode. Instead of doing nothing and
reporting success.

V2: update for mclk/pcie clock level settings also

Signed-off-by: Evan Quan <evan.quan@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

1 files changed, 12 insertions, 3 deletions

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c
index 94055a485e01..59cc678de8c1 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c
@@ -704,7 +704,10 @@ static ssize_t amdgpu_set_pp_dpm_sclk(struct device *dev,
                 return ret;
 
         if (adev->powerplay.pp_funcs->force_clock_level)
-                amdgpu_dpm_force_clock_level(adev, PP_SCLK, mask);
+                ret = amdgpu_dpm_force_clock_level(adev, PP_SCLK, mask);
+
+        if (ret)
+                return -EINVAL;
 
         return count;
 }
@@ -737,7 +740,10 @@ static ssize_t amdgpu_set_pp_dpm_mclk(struct device *dev,
                 return ret;
 
         if (adev->powerplay.pp_funcs->force_clock_level)
-                amdgpu_dpm_force_clock_level(adev, PP_MCLK, mask);
+                ret = amdgpu_dpm_force_clock_level(adev, PP_MCLK, mask);
+
+        if (ret)
+                return -EINVAL;
 
         return count;
 }
@@ -770,7 +776,10 @@ static ssize_t amdgpu_set_pp_dpm_pcie(struct device *dev,
                 return ret;
 
         if (adev->powerplay.pp_funcs->force_clock_level)
-                amdgpu_dpm_force_clock_level(adev, PP_PCIE, mask);
+                ret = amdgpu_dpm_force_clock_level(adev, PP_PCIE, mask);
+
+        if (ret)
+                return -EINVAL;
 
         return count;
 }