drm/amdgpu: fix modprobe failure for uvd_4/5/6
authorHawking Zhang <Hawking.Zhang@amd.com>
Mon, 24 Jun 2019 11:26:00 +0000 (19:26 +0800)
committerAlex Deucher <alexander.deucher@amd.com>
Mon, 24 Jun 2019 14:29:59 +0000 (09:29 -0500)
For uvd_4/5/6, amdgpu driver will only power on them when
there are jobs assigned to decode/enc rings.uvd_4/5/6 dpm was broken
since amdgpu_dpm_set_powergating_by_smu only covers gfx block.

The change would add more IP block support in amdgpu_dpm_set_powergating_by_smu
For GFX/UVD/VCN/VCE, if the new SMU driver is supported, invoke new
power gate helper function smu_dpm_set_power_gate, otherwise, fallback to
legacy powerplay helper function pp_set_powergating_by_smu. For other IP blocks
always invoke legacy powerplay helper function.

Signed-off-by: Hawking Zhang <Hawking.Zhang@amd.com>
Reviewed-by: Tianci Yin <tianci.yin@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_dpm.c
drivers/gpu/drm/amd/powerplay/amdgpu_smu.c

index 6c2ec8123178aed0854aa90c43657ae290ef180a..f082b2547b831084b2d0319ba3ea9cfb485ac848 100644 (file)
@@ -946,12 +946,21 @@ int amdgpu_dpm_set_powergating_by_smu(struct amdgpu_device *adev, uint32_t block
 
        switch (block_type) {
        case AMD_IP_BLOCK_TYPE_GFX:
+       case AMD_IP_BLOCK_TYPE_UVD:
+       case AMD_IP_BLOCK_TYPE_VCN:
+       case AMD_IP_BLOCK_TYPE_VCE:
                if (swsmu)
-                       ret = smu_gfx_off_control(&adev->smu, gate);
+                       ret = smu_dpm_set_power_gate(&adev->smu, block_type, gate);
                else
                        ret = ((adev)->powerplay.pp_funcs->set_powergating_by_smu(
                                (adev)->powerplay.pp_handle, block_type, gate));
                break;
+       case AMD_IP_BLOCK_TYPE_GMC:
+       case AMD_IP_BLOCK_TYPE_ACP:
+       case AMD_IP_BLOCK_TYPE_SDMA:
+               ret = ((adev)->powerplay.pp_funcs->set_powergating_by_smu(
+                               (adev)->powerplay.pp_handle, block_type, gate));
+               break;
        default:
                break;
        }
index 8bf42f80f25361ee7b45aba148826dd63696a6ef..5aa0594fb53f2d6e8a6962048de617e9e8eec303 100644 (file)
@@ -233,6 +233,9 @@ int smu_dpm_set_power_gate(struct smu_context *smu, uint32_t block_type,
        case AMD_IP_BLOCK_TYPE_VCE:
                ret = smu_dpm_set_vce_enable(smu, gate);
                break;
+       case AMD_IP_BLOCK_TYPE_GFX:
+               ret = smu_gfx_off_control(smu, gate);
+               break;
        default:
                break;
        }