Patch is:
Reviewed-by: Qiang Yu <yuq825@xxxxxxxxx>
On Sat, Feb 25, 2023 at 5:41 AM Maíra Canal <mcanal@xxxxxxxxxx> wrote:
>
> As lima_gem_add_deps() performs the same steps as
> drm_sched_job_add_syncobj_dependency(), replace the open-coded
> implementation in Lima in order to simply use the DRM function.
>
> Signed-off-by: Maíra Canal <mcanal@xxxxxxxxxx>
> ---
> drivers/gpu/drm/lima/lima_gem.c | 12 ++----------
> 1 file changed, 2 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/gpu/drm/lima/lima_gem.c b/drivers/gpu/drm/lima/lima_gem.c
> index 0f1ca0b0db49..10252dc11a22 100644
> --- a/drivers/gpu/drm/lima/lima_gem.c
> +++ b/drivers/gpu/drm/lima/lima_gem.c
> @@ -277,21 +277,13 @@ static int lima_gem_add_deps(struct drm_file *file, struct lima_submit *submit)
> int i, err;
>
> for (i = 0; i < ARRAY_SIZE(submit->in_sync); i++) {
> - struct dma_fence *fence = NULL;
> -
> if (!submit->in_sync[i])
> continue;
>
> - err = drm_syncobj_find_fence(file, submit->in_sync[i],
> - 0, 0, &fence);
> + err = drm_sched_job_add_syncobj_dependency(&submit->task->base, file,
> + submit->in_sync[i], 0);
> if (err)
> return err;
> -
> - err = drm_sched_job_add_dependency(&submit->task->base, fence);
> - if (err) {
> - dma_fence_put(fence);
> - return err;
> - }
> }
>
> return 0;
> --
> 2.39.2
>
