It's not needed for BACO.

Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>
---
 drivers/gpu/drm/amd/powerplay/hwmgr/vega12_hwmgr.c | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/vega12_hwmgr.c 
b/drivers/gpu/drm/amd/powerplay/hwmgr/vega12_hwmgr.c
index 3d3504411b63..939c53a51f71 100644
--- a/drivers/gpu/drm/amd/powerplay/hwmgr/vega12_hwmgr.c
+++ b/drivers/gpu/drm/amd/powerplay/hwmgr/vega12_hwmgr.c
@@ -2460,6 +2460,7 @@ static int vega12_disable_dpm_tasks(struct pp_hwmgr 
*hwmgr)
 static int vega12_power_off_asic(struct pp_hwmgr *hwmgr)
 {
        struct vega12_hwmgr *data = (struct vega12_hwmgr *)(hwmgr->backend);
+       struct amdgpu_device *adev = hwmgr->adev;
        int result;
 
        result = vega12_disable_dpm_tasks(hwmgr);
@@ -2468,10 +2469,13 @@ static int vega12_power_off_asic(struct pp_hwmgr *hwmgr)
                        );
        data->water_marks_bitmap &= ~(WaterMarksLoaded);
 
-       PP_ASSERT_WITH_CODE((result = smum_send_msg_to_smc(hwmgr,
-                       PPSMC_MSG_PrepareMp1ForUnload)) == 0,
-                       "[PrepareMp1ForUnload] Failed!",
-                       return result);
+       if (amdgpu_asic_reset_method(adev) == AMD_RESET_METHOD_MODE1) {
+               PP_ASSERT_WITH_CODE((result =
+                                    smum_send_msg_to_smc(hwmgr,
+                                                         
PPSMC_MSG_PrepareMp1ForUnload)) == 0,
+                                   "[PrepareMp1ForUnload] Failed!",
+                                   return result);
+       }
 
        return result;
 }
-- 
2.20.1

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to