summaryrefslogtreecommitdiffstats
path: root/drivers/gpu/drm
diff options
context:
space:
mode:
authorShashank Sharma <shashank.sharma@amd.com>2023-02-27 15:42:28 +0100
committerAlex Deucher <alexander.deucher@amd.com>2023-03-07 14:22:14 -0500
commitc07edf915ece65ce11a765bf4f9f0a1000bb7a33 (patch)
treee0ea5093272735ebbd1364e2c3b4c185417d8e50 /drivers/gpu/drm
parentb059cba51979b3431b75e0c6f18e9f75e427537c (diff)
downloadlinux-stable-c07edf915ece65ce11a765bf4f9f0a1000bb7a33.tar.gz
linux-stable-c07edf915ece65ce11a765bf4f9f0a1000bb7a33.tar.bz2
linux-stable-c07edf915ece65ce11a765bf4f9f0a1000bb7a33.zip
drm/amdgpu: fix return value check in kfd
This patch fixes a return value check in kfd doorbell handling. This function should return 0(error) only when the ida_simple_get returns < 0(error), return > 0 is a success case. Cc: Felix Kuehling <Felix.Kuehling@amd.com> Cc: Alex Deucher <alexander.deucher@amd.com> Fixes: 16f0013157bf ("drm/amdkfd: Allocate doorbells only when needed") Acked-by: Christian Koenig <chriatian.koenig@amd.com> Reviewed-by: Felix Kuehling <Felix.Kuehling@amd.com> Signed-off-by: Shashank Sharma <shashank.sharma@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm')
-rw-r--r--drivers/gpu/drm/amd/amdkfd/kfd_doorbell.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_doorbell.c b/drivers/gpu/drm/amd/amdkfd/kfd_doorbell.c
index cd4e61bf0493..3ac599f74fea 100644
--- a/drivers/gpu/drm/amd/amdkfd/kfd_doorbell.c
+++ b/drivers/gpu/drm/amd/amdkfd/kfd_doorbell.c
@@ -280,7 +280,7 @@ phys_addr_t kfd_get_process_doorbells(struct kfd_process_device *pdd)
if (!pdd->doorbell_index) {
int r = kfd_alloc_process_doorbells(pdd->dev,
&pdd->doorbell_index);
- if (r)
+ if (r < 0)
return 0;
}