staging: android: ion: Remove empty ion_ioctl_dir() function
authorAndrew F. Davis <afd@ti.com>
Fri, 11 Jan 2019 18:05:11 +0000 (12:05 -0600)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 18 Jan 2019 09:42:27 +0000 (10:42 +0100)
This function is empty of real function and can be replaced with
_IOC_DIR().

Signed-off-by: Andrew F. Davis <afd@ti.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/android/ion/ion-ioctl.c

index a8d3cc412fb9013aabd42f74dd9995b718ea841f..b366f97a5728249523d71370bc8e77bec96976fa 100644 (file)
@@ -31,23 +31,11 @@ static int validate_ioctl_arg(unsigned int cmd, union ion_ioctl_arg *arg)
        return 0;
 }
 
-/* fix up the cases where the ioctl direction bits are incorrect */
-static unsigned int ion_ioctl_dir(unsigned int cmd)
-{
-       switch (cmd) {
-       default:
-               return _IOC_DIR(cmd);
-       }
-}
-
 long ion_ioctl(struct file *filp, unsigned int cmd, unsigned long arg)
 {
        int ret = 0;
-       unsigned int dir;
        union ion_ioctl_arg data;
 
-       dir = ion_ioctl_dir(cmd);
-
        if (_IOC_SIZE(cmd) > sizeof(data))
                return -EINVAL;
 
@@ -65,7 +53,7 @@ long ion_ioctl(struct file *filp, unsigned int cmd, unsigned long arg)
                return ret;
        }
 
-       if (!(dir & _IOC_WRITE))
+       if (!(_IOC_DIR(cmd) & _IOC_WRITE))
                memset(&data, 0, sizeof(data));
 
        switch (cmd) {
@@ -90,7 +78,7 @@ long ion_ioctl(struct file *filp, unsigned int cmd, unsigned long arg)
                return -ENOTTY;
        }
 
-       if (dir & _IOC_READ) {
+       if (_IOC_DIR(cmd) & _IOC_READ) {
                if (copy_to_user((void __user *)arg, &data, _IOC_SIZE(cmd)))
                        return -EFAULT;
        }