drm: omapdrm: Remove legacy buffer synchronization support

The omapdrm driver uses a custom API to synchronize with the SGX GPU.
This is unusable as such in the mainline kernel as the API is only
partially implemented and requires additional out-of-tree patches.
Furthermore, as no SGX driver is available in the mainline kernel, the
API can't be considered as a stable mainline API.

Now that the driver supports synchronization through fences, remove
legacy buffer synchronization support. The two userspace ioctls are
turned into no-ops to avoid breaking userspace and will be removed in
the future.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>

1 files changed, 4 insertions, 49 deletions

diff --git a/drivers/gpu/drm/omapdrm/omap_drv.c b/drivers/gpu/drm/omapdrm/omap_drv.c
index b4ef3025e3e3..e2e935813cc4 100644
--- a/drivers/gpu/drm/omapdrm/omap_drv.c
+++ b/drivers/gpu/drm/omapdrm/omap_drv.c
@@ -365,53 +365,6 @@ static int ioctl_gem_new(struct drm_device *dev, void *data,
                                    &args->handle);
 }
 
-static int ioctl_gem_cpu_prep(struct drm_device *dev, void *data,
-                struct drm_file *file_priv)
-{
-        struct drm_omap_gem_cpu_prep *args = data;
-        struct drm_gem_object *obj;
-        int ret;
-
-        VERB("%p:%p: handle=%d, op=%x", dev, file_priv, args->handle, args->op);
-
-        obj = drm_gem_object_lookup(file_priv, args->handle);
-        if (!obj)
-                return -ENOENT;
-
-        ret = omap_gem_op_sync(obj, args->op);
-
-        if (!ret)
-                ret = omap_gem_op_start(obj, args->op);
-
-        drm_gem_object_unreference_unlocked(obj);
-
-        return ret;
-}
-
-static int ioctl_gem_cpu_fini(struct drm_device *dev, void *data,
-                struct drm_file *file_priv)
-{
-        struct drm_omap_gem_cpu_fini *args = data;
-        struct drm_gem_object *obj;
-        int ret;
-
-        VERB("%p:%p: handle=%d", dev, file_priv, args->handle);
-
-        obj = drm_gem_object_lookup(file_priv, args->handle);
-        if (!obj)
-                return -ENOENT;
-
-        /* XXX flushy, flushy */
-        ret = 0;
-
-        if (!ret)
-                ret = omap_gem_op_finish(obj, args->op);
-
-        drm_gem_object_unreference_unlocked(obj);
-
-        return ret;
-}
-
 static int ioctl_gem_info(struct drm_device *dev, void *data,
                 struct drm_file *file_priv)
 {
@@ -440,9 +393,11 @@ static const struct drm_ioctl_desc ioctls[DRM_COMMAND_END - DRM_COMMAND_BASE] =
                           DRM_AUTH | DRM_MASTER | DRM_ROOT_ONLY),
         DRM_IOCTL_DEF_DRV(OMAP_GEM_NEW, ioctl_gem_new,
                           DRM_AUTH | DRM_RENDER_ALLOW),
-        DRM_IOCTL_DEF_DRV(OMAP_GEM_CPU_PREP, ioctl_gem_cpu_prep,
+        /* Deprecated, to be removed. */
+        DRM_IOCTL_DEF_DRV(OMAP_GEM_CPU_PREP, drm_noop,
                           DRM_AUTH | DRM_RENDER_ALLOW),
-        DRM_IOCTL_DEF_DRV(OMAP_GEM_CPU_FINI, ioctl_gem_cpu_fini,
+        /* Deprecated, to be removed. */
+        DRM_IOCTL_DEF_DRV(OMAP_GEM_CPU_FINI, drm_noop,
                           DRM_AUTH | DRM_RENDER_ALLOW),
         DRM_IOCTL_DEF_DRV(OMAP_GEM_INFO, ioctl_gem_info,
                           DRM_AUTH | DRM_RENDER_ALLOW),