drm/compat: more dummy implementations

drm_noop really doesn't do much, and who cares about the permission checks.
So let's delete some code.

Acked-by: Maxime Ripard <mripard@kernel.org>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210222100608.400730-1-daniel.vetter@ffwll.ch

diff --git a/drivers/gpu/drm/drm_ioc32.c b/drivers/gpu/drm/drm_ioc32.c
index dc734d4828a179441a41e445962c82779fbf9c1a..33390f02f5eb1664b60cd02bf99deecc0bce0f1f 100644 (file)
--- a/drivers/gpu/drm/drm_ioc32.c
+++ b/drivers/gpu/drm/drm_ioc32.c
@@ -302,12 +302,8 @@ static int compat_drm_getstats(struct file *file, unsigned int cmd,
                               unsigned long arg)
 {
        drm_stats32_t __user *argp = (void __user *)arg;
-       int err;
-
-       err = drm_ioctl_kernel(file, drm_noop, NULL, 0);
-       if (err)
-               return err;
 
+       /* getstats is defunct, just clear */
        if (clear_user(argp, sizeof(drm_stats32_t)))
                return -EFAULT;
        return 0;
@@ -820,13 +816,8 @@ typedef struct drm_update_draw32 {
 static int compat_drm_update_draw(struct file *file, unsigned int cmd,
                                  unsigned long arg)
 {
-       drm_update_draw32_t update32;
-
-       if (copy_from_user(&update32, (void __user *)arg, sizeof(update32)))
-               return -EFAULT;
-
-       return drm_ioctl_kernel(file, drm_noop, NULL,
-                               DRM_AUTH|DRM_MASTER|DRM_ROOT_ONLY);
+       /* update_draw is defunct */
+       return 0;
 }
 #endif