drm/panfrost: Use drm_sched_job_add_syncobj_dependency()
authorMaíra Canal <mcanal@igalia.com>
Thu, 9 Feb 2023 12:44:47 +0000 (09:44 -0300)
committerMaíra Canal <mairacanal@riseup.net>
Fri, 24 Feb 2023 20:20:14 +0000 (17:20 -0300)
As panfrost_copy_in_sync() performs the same steps as
drm_sched_job_add_syncobj_dependency(), replace the open-coded
implementation in Panfrost in order to simply use the DRM function.

Reviewed-by: Alyssa Rosenzweig <alyssa@collabora.com>
Signed-off-by: Maíra Canal <mcanal@igalia.com>
Reviewed-by: Steven Price <steven.price@arm.com>
Signed-off-by: Maíra Canal <mairacanal@riseup.net>
Link: https://patchwork.freedesktop.org/patch/msgid/20230209124447.467867-5-mcanal@igalia.com
drivers/gpu/drm/panfrost/panfrost_drv.c

index abb0dad..f49096f 100644 (file)
@@ -220,15 +220,8 @@ panfrost_copy_in_sync(struct drm_device *dev,
        }
 
        for (i = 0; i < in_fence_count; i++) {
-               struct dma_fence *fence;
-
-               ret = drm_syncobj_find_fence(file_priv, handles[i], 0, 0,
-                                            &fence);
-               if (ret)
-                       goto fail;
-
-               ret = drm_sched_job_add_dependency(&job->base, fence);
-
+               ret = drm_sched_job_add_syncobj_dependency(&job->base, file_priv,
+                                                          handles[i], 0);
                if (ret)
                        goto fail;
        }