drm/amdgpu:No action when VCN PG state is unchanged

When VCN PG state is unchanged, it is unnecessary to reset power
gate state

Signed-off-by: James Zhu <James.Zhu@amd.com>
Acked-by: Alex Deucher <alexander.deucher@amd.com>
Acked-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

1 files changed, 10 insertions, 2 deletions

diff --git a/drivers/gpu/drm/amd/amdgpu/vcn_v1_0.c b/drivers/gpu/drm/amd/amdgpu/vcn_v1_0.c
index 2664bb2c47c3..2cde0b4046db 100644
--- a/drivers/gpu/drm/amd/amdgpu/vcn_v1_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/vcn_v1_0.c
@@ -1633,12 +1633,20 @@ static int vcn_v1_0_set_powergating_state(void *handle,
          * revisit this when there is a cleaner line between
          * the smc and the hw blocks
          */
+        int ret;
         struct amdgpu_device *adev = (struct amdgpu_device *)handle;
 
+        if(state == adev->vcn.cur_state)
+                return 0;
+
         if (state == AMD_PG_STATE_GATE)
-                return vcn_v1_0_stop(adev);
+                ret = vcn_v1_0_stop(adev);
         else
-                return vcn_v1_0_start(adev);
+                ret = vcn_v1_0_start(adev);
+
+        if(!ret)
+                adev->vcn.cur_state = state;
+        return ret;
 }
 
 static const struct amd_ip_funcs vcn_v1_0_ip_funcs = {