From: Alex Deucher <[email protected]>

For S0ix, the RLC is not powered down. Rework the Van Gogh logic to
skip powering it down and skip part of post-init.

Cc: Robert Beckett <[email protected]>
Fixes: 8c4e9105b2a8 ("drm/amdgpu: optimize RLC powerdown notification on 
Vangogh")
Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/4659
Signed-off-by: Alex Deucher <[email protected]>
Tested-by: Antheas Kapenekakis <[email protected]>
Signed-off-by: Antheas Kapenekakis <[email protected]>
Co-developed-by: Mario Limonciello <[email protected]>
Signed-off-by: Mario Limonciello <[email protected]>
---
 drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c        | 6 ++++++
 drivers/gpu/drm/amd/pm/swsmu/smu11/vangogh_ppt.c | 6 ++++++
 2 files changed, 12 insertions(+)

diff --git a/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c 
b/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c
index 10d42267085b..5bee02f0ba86 100644
--- a/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c
+++ b/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c
@@ -2054,6 +2054,12 @@ static int smu_disable_dpms(struct smu_context *smu)
            smu->is_apu && (amdgpu_in_reset(adev) || adev->in_s0ix))
                return 0;
 
+       /* vangogh s0ix */
+       if ((amdgpu_ip_version(adev, MP1_HWIP, 0) == IP_VERSION(11, 5, 0) ||
+            amdgpu_ip_version(adev, MP1_HWIP, 0) == IP_VERSION(11, 5, 2)) &&
+           adev->in_s0ix)
+               return 0;
+
        /*
         * For gpu reset, runpm and hibernation through BACO,
         * BACO feature has to be kept enabled.
diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu11/vangogh_ppt.c 
b/drivers/gpu/drm/amd/pm/swsmu/smu11/vangogh_ppt.c
index 3dc1919b8c64..e67fd972fc28 100644
--- a/drivers/gpu/drm/amd/pm/swsmu/smu11/vangogh_ppt.c
+++ b/drivers/gpu/drm/amd/pm/swsmu/smu11/vangogh_ppt.c
@@ -2201,6 +2201,9 @@ static int vangogh_notify_rlc_state(struct smu_context 
*smu, bool en)
        struct amdgpu_device *adev = smu->adev;
        int ret = 0;
 
+       if (adev->in_s0ix)
+               return 0;
+
        if (adev->pm.fw_version >= 0x43f1700 && !en)
                ret = smu_cmn_send_smc_msg_with_param(smu, 
SMU_MSG_RlcPowerNotify,
                                                      RLC_STATUS_OFF, NULL);
@@ -2236,6 +2239,9 @@ static int vangogh_post_smu_init(struct smu_context *smu)
        uint32_t total_cu = adev->gfx.config.max_cu_per_sh *
                adev->gfx.config.max_sh_per_se * 
adev->gfx.config.max_shader_engines;
 
+       if (adev->in_s0ix)
+               return 0;
+
        /* allow message will be sent after enable message on Vangogh*/
        if (smu_cmn_feature_is_enabled(smu, SMU_FEATURE_DPM_GFXCLK_BIT) &&
                        (adev->pg_flags & AMD_PG_SUPPORT_GFX_PG)) {
-- 
2.49.0

Reply via email to