]> www.infradead.org Git - nvme.git/commitdiff
drm/amdgpu: Drop 'fence' check in 'to_amdgpu_amdkfd_fence()'
authorSrinivasan Shanmugam <srinivasan.shanmugam@amd.com>
Wed, 27 Dec 2023 07:24:44 +0000 (12:54 +0530)
committerAlex Deucher <alexander.deucher@amd.com>
Fri, 5 Jan 2024 21:10:43 +0000 (16:10 -0500)
Return value of container_of(...) can't be null, so null check is not
required for 'fence'. Hence drop its NULL check.

Fixes the below:
drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_fence.c:93 to_amdgpu_amdkfd_fence() warn: can 'fence' even be NULL?

Cc: Felix Kuehling <Felix.Kuehling@amd.com>
Cc: Christian König <christian.koenig@amd.com>
Cc: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam@amd.com>
Reviewed-by: Felix Kuehling <felix.kuehling@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_fence.c

index 469785d337911a093cba2620cfd6e7fd77d207bd..1ef758ac5076ef9a56ba5191ac38964897564912 100644 (file)
@@ -90,7 +90,7 @@ struct amdgpu_amdkfd_fence *to_amdgpu_amdkfd_fence(struct dma_fence *f)
                return NULL;
 
        fence = container_of(f, struct amdgpu_amdkfd_fence, base);
-       if (fence && f->ops == &amdkfd_fence_ops)
+       if (f->ops == &amdkfd_fence_ops)
                return fence;
 
        return NULL;