1. no need to set cg as use hw dynamic cg.
2. when uvd idle, stop uvd. encode, start uvd.
3. if pg feature enabled, power on/down uvd by smu.
4. drm/amdgpu: dpm do not set uvd pg status.

Signed-off-by: Rex Zhu <rex....@amd.com>
Reviewed-by: Alex Deucher <alexander.deuc...@amd.com>
---
 drivers/gpu/drm/amd/amdgpu/kv_dpm.c | 29 ++++++++---------------------
 1 file changed, 8 insertions(+), 21 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/kv_dpm.c 
b/drivers/gpu/drm/amd/amdgpu/kv_dpm.c
index 8785ca5..8159a2d 100644
--- a/drivers/gpu/drm/amd/amdgpu/kv_dpm.c
+++ b/drivers/gpu/drm/amd/amdgpu/kv_dpm.c
@@ -1688,38 +1688,25 @@ static void kv_dpm_powergate_uvd(struct amdgpu_device 
*adev, bool gate)
        struct kv_power_info *pi = kv_get_pi(adev);
        int ret;
 
-       if (pi->uvd_power_gated == gate)
-               return;
-
        pi->uvd_power_gated = gate;
 
        if (gate) {
-               if (pi->caps_uvd_pg) {
-                       /* disable clockgating so we can properly shut down the 
block */
-                       ret = amdgpu_set_clockgating_state(adev, 
AMD_IP_BLOCK_TYPE_UVD,
-                                                           
AMD_CG_STATE_UNGATE);
-                       /* shutdown the UVD block */
-                       ret = amdgpu_set_powergating_state(adev, 
AMD_IP_BLOCK_TYPE_UVD,
-                                                           AMD_PG_STATE_GATE);
-                       /* XXX: check for errors */
-               }
+               /* stop the UVD block */
+               ret = amdgpu_set_powergating_state(adev, AMD_IP_BLOCK_TYPE_UVD,
+                                                       AMD_PG_STATE_GATE);
                kv_update_uvd_dpm(adev, gate);
                if (pi->caps_uvd_pg)
                        /* power off the UVD block */
                        amdgpu_kv_notify_message_to_smu(adev, 
PPSMC_MSG_UVDPowerOFF);
        } else {
-               if (pi->caps_uvd_pg) {
+               if (pi->caps_uvd_pg)
                        /* power on the UVD block */
                        amdgpu_kv_notify_message_to_smu(adev, 
PPSMC_MSG_UVDPowerON);
                        /* re-init the UVD block */
-                       ret = amdgpu_set_powergating_state(adev, 
AMD_IP_BLOCK_TYPE_UVD,
-                                                           
AMD_PG_STATE_UNGATE);
-                       /* enable clockgating. hw will dynamically gate/ungate 
clocks on the fly */
-                       ret = amdgpu_set_clockgating_state(adev, 
AMD_IP_BLOCK_TYPE_UVD,
-                                                           AMD_CG_STATE_GATE);
-                       /* XXX: check for errors */
-               }
                kv_update_uvd_dpm(adev, gate);
+
+               ret = amdgpu_set_powergating_state(adev, AMD_IP_BLOCK_TYPE_UVD,
+                                                       AMD_PG_STATE_UNGATE);
        }
 }
 
@@ -3010,7 +2997,7 @@ static int kv_dpm_late_init(void *handle)
        kv_dpm_powergate_acp(adev, true);
        kv_dpm_powergate_samu(adev, true);
        kv_dpm_powergate_vce(adev, true);
-       kv_dpm_powergate_uvd(adev, true);
+
        return 0;
 }
 
-- 
1.9.1

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

Reply via email to