Module: Mesa Branch: main Commit: c902b9e1d91d5682e2cb1ccb374fd3d300ad68ba URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=c902b9e1d91d5682e2cb1ccb374fd3d300ad68ba
Author: Marek Olšák <marek.ol...@amd.com> Date: Fri Dec 1 02:55:29 2023 -0500 winsys/amdgpu: don't return a value from cs_add_buffer It's unused. The return value is only used with the radeon winsys. Reviewed-by: Yogesh Mohan Marimuthu <yogesh.mohanmarimu...@amd.com> Reviewed-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-pra...@amd.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/26547> --- src/gallium/winsys/amdgpu/drm/amdgpu_cs.c | 5 ++--- src/gallium/winsys/amdgpu/drm/amdgpu_cs.h | 1 - 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c b/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c index 545fbbe7631..2c876a80ed4 100644 --- a/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c +++ b/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c @@ -783,7 +783,7 @@ static unsigned amdgpu_cs_add_buffer(struct radeon_cmdbuf *rcs, */ if (bo == cs->last_added_bo && (usage & cs->last_added_bo_usage) == usage) - return cs->last_added_bo_index; + return 0; if (!(bo->base.usage & RADEON_FLAG_SPARSE)) { if (!bo->bo) { @@ -818,8 +818,7 @@ static unsigned amdgpu_cs_add_buffer(struct radeon_cmdbuf *rcs, } cs->last_added_bo = bo; - cs->last_added_bo_index = index; - return index; + return 0; } static bool amdgpu_ib_new_buffer(struct amdgpu_winsys *ws, diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_cs.h b/src/gallium/winsys/amdgpu/drm/amdgpu_cs.h index 8edfaa05b92..60439dc26b9 100644 --- a/src/gallium/winsys/amdgpu/drm/amdgpu_cs.h +++ b/src/gallium/winsys/amdgpu/drm/amdgpu_cs.h @@ -95,7 +95,6 @@ struct amdgpu_cs_context { int16_t *buffer_indices_hashlist; struct amdgpu_winsys_bo *last_added_bo; - unsigned last_added_bo_index; unsigned last_added_bo_usage; struct amdgpu_fence_list fence_dependencies;
