drm/amdgpu: fix UVD suspend and resume for VI APU

User space passed the same handle before suspend and after resume,
so we have remove the session and handle destroy, and keep the
firmware untouched.

Signed-off-by: Leo Liu <leo.liu@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Cc: stable@vger.kernel.org

1 files changed, 12 insertions, 8 deletions

diff --git a/drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c b/drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c
index a7622efef506..7e9934fa4193 100644
--- a/drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c
@@ -214,10 +214,12 @@ static int uvd_v6_0_suspend(void *handle)
         int r;
         struct amdgpu_device *adev = (struct amdgpu_device *)handle;
 
-        r = amdgpu_uvd_suspend(adev);
-        if (r)
-                return r;
-
+        /* Skip this for APU for now */
+        if (!(adev->flags & AMD_IS_APU)) {
+                r = amdgpu_uvd_suspend(adev);
+                if (r)
+                        return r;
+        }
         r = uvd_v6_0_hw_fini(adev);
         if (r)
                 return r;
@@ -230,10 +232,12 @@ static int uvd_v6_0_resume(void *handle)
         int r;
         struct amdgpu_device *adev = (struct amdgpu_device *)handle;
 
-        r = amdgpu_uvd_resume(adev);
-        if (r)
-                return r;
-
+        /* Skip this for APU for now */
+        if (!(adev->flags & AMD_IS_APU)) {
+                r = amdgpu_uvd_resume(adev);
+                if (r)
+                        return r;
+        }
         r = uvd_v6_0_hw_init(adev);
         if (r)
                 return r;