drm/amd/pm: drop unneeded dpm features disablement for SMU 13.0.4/11
authorTim Huang <tim.huang@amd.com>
Sat, 21 Jan 2023 02:35:04 +0000 (10:35 +0800)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 31 Jan 2023 19:03:55 +0000 (14:03 -0500)
PMFW will handle the features disablement properly for gpu reset case,
driver involvement may cause some unexpected issues.

Cc: stable@vger.kernel.org # 6.1
Signed-off-by: Tim Huang <tim.huang@amd.com>
Reviewed-by: Yifan Zhang <yifan1.zhang@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c

index ec52830..8bae3fe 100644 (file)
@@ -1498,6 +1498,20 @@ static int smu_disable_dpms(struct smu_context *smu)
        }
 
        /*
+        * For SMU 13.0.4/11, PMFW will handle the features disablement properly
+        * for gpu reset case. Driver involvement is unnecessary.
+        */
+       if (amdgpu_in_reset(adev)) {
+               switch (adev->ip_versions[MP1_HWIP][0]) {
+               case IP_VERSION(13, 0, 4):
+               case IP_VERSION(13, 0, 11):
+                       return 0;
+               default:
+                       break;
+               }
+       }
+
+       /*
         * For gpu reset, runpm and hibernation through BACO,
         * BACO feature has to be kept enabled.
         */