Merge tag 'char-misc-4.19-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/gregk...
[muen/linux.git] / drivers / uio / uio.c
index d7fb49366e7849f5f37c6ba75c57636603b6ce20..70a7981b94b3fd854b0e4ef7460cd4eebce8ec9b 100644 (file)
@@ -1,3 +1,4 @@
+// SPDX-License-Identifier: GPL-2.0
 /*
  * drivers/uio/uio.c
  *
@@ -9,8 +10,6 @@
  * Userspace IO
  *
  * Base Functions
- *
- * Licensed under the GPLv2 only.
  */
 
 #include <linux/module.h>
@@ -622,6 +621,12 @@ static ssize_t uio_write(struct file *filep, const char __user *buf,
        ssize_t retval;
        s32 irq_on;
 
+       if (count != sizeof(s32))
+               return -EINVAL;
+
+       if (copy_from_user(&irq_on, buf, count))
+               return -EFAULT;
+
        mutex_lock(&idev->info_lock);
        if (!idev->info) {
                retval = -EINVAL;
@@ -633,21 +638,11 @@ static ssize_t uio_write(struct file *filep, const char __user *buf,
                goto out;
        }
 
-       if (count != sizeof(s32)) {
-               retval = -EINVAL;
-               goto out;
-       }
-
        if (!idev->info->irqcontrol) {
                retval = -ENOSYS;
                goto out;
        }
 
-       if (copy_from_user(&irq_on, buf, count)) {
-               retval = -EFAULT;
-               goto out;
-       }
-
        retval = idev->info->irqcontrol(idev->info, irq_on);
 
 out:
@@ -811,7 +806,7 @@ static int uio_mmap(struct file *filep, struct vm_area_struct *vma)
 
 out:
        mutex_unlock(&idev->info_lock);
-       return 0;
+       return ret;
 }
 
 static const struct file_operations uio_fops = {
@@ -955,8 +950,6 @@ int __uio_register_device(struct module *owner,
        if (ret)
                goto err_uio_dev_add_attributes;
 
-       info->uio_dev = idev;
-
        if (info->irq && (info->irq != UIO_IRQ_CUSTOM)) {
                /*
                 * Note that we deliberately don't use devm_request_irq
@@ -972,6 +965,7 @@ int __uio_register_device(struct module *owner,
                        goto err_request_irq;
        }
 
+       info->uio_dev = idev;
        return 0;
 
 err_request_irq: