drm/amdgpu: drop redundant cancel_delayed_work_sync call
authorEvan Quan <evan.quan@amd.com>
Thu, 19 Aug 2021 04:14:20 +0000 (12:14 +0800)
committerAlex Deucher <alexander.deucher@amd.com>
Wed, 25 Aug 2021 22:15:10 +0000 (18:15 -0400)
As those _sw_fini() APIs follow just after _suspend() APIs.
And the cancel_delayed_work_sync was already called in latter.

Signed-off-by: Evan Quan <evan.quan@amd.com>
Reviewed-by: Guchun Chen <guchun.chen@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_jpeg.c
drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c
drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c
drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c

index 8996cb4ed57a87f568249b0daec623a998045d30..9342aa23ebd260b8f699b78156cd8bb8425a7fe9 100644 (file)
@@ -47,8 +47,6 @@ int amdgpu_jpeg_sw_fini(struct amdgpu_device *adev)
 {
        int i;
 
-       cancel_delayed_work_sync(&adev->jpeg.idle_work);
-
        for (i = 0; i < adev->jpeg.num_jpeg_inst; ++i) {
                if (adev->jpeg.harvest_config & (1 << i))
                        continue;
index 0f576f294d8a5f5c7c1b3af67cce4c94dd1817b7..d451c359606aec349cb5ab7263b1c7a244156857 100644 (file)
@@ -326,7 +326,6 @@ int amdgpu_uvd_sw_fini(struct amdgpu_device *adev)
 {
        int i, j;
 
-       cancel_delayed_work_sync(&adev->uvd.idle_work);
        drm_sched_entity_destroy(&adev->uvd.entity);
 
        for (j = 0; j < adev->uvd.num_uvd_inst; ++j) {
index 1ae7f824adc7ac4992c80d5fbf602fd6b69ddeca..8e8dee9fac9f6b77b31c9c70a5d2a0c91d15efa2 100644 (file)
@@ -218,7 +218,6 @@ int amdgpu_vce_sw_fini(struct amdgpu_device *adev)
        if (adev->vce.vcpu_bo == NULL)
                return 0;
 
-       cancel_delayed_work_sync(&adev->vce.idle_work);
        drm_sched_entity_destroy(&adev->vce.entity);
 
        amdgpu_bo_free_kernel(&adev->vce.vcpu_bo, &adev->vce.gpu_addr,
index 6780df0fb265545e7d43b73857d866c47594e6d6..008a308a4ecaf0a68dd350d440c68ad0fd3e222b 100644 (file)
@@ -258,8 +258,6 @@ int amdgpu_vcn_sw_fini(struct amdgpu_device *adev)
 {
        int i, j;
 
-       cancel_delayed_work_sync(&adev->vcn.idle_work);
-
        for (j = 0; j < adev->vcn.num_vcn_inst; ++j) {
                if (adev->vcn.harvest_config & (1 << j))
                        continue;