USB: BKL removal: vstusb
authorOliver Neukum <oliver@neukum.org>
Thu, 14 Jan 2010 15:13:24 +0000 (16:13 +0100)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 2 Mar 2010 22:54:31 +0000 (14:54 -0800)
BKL was not needed at all. Removed without replacement.

Signed-off-by: Oliver Neukum <oliver@neukum.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/usb/misc/vstusb.c

index b787b25..874c81b 100644 (file)
@@ -27,7 +27,6 @@
 #include <linux/slab.h>
 #include <linux/module.h>
 #include <linux/mutex.h>
-#include <linux/smp_lock.h>
 #include <linux/uaccess.h>
 #include <linux/usb.h>
 
@@ -104,23 +103,19 @@ static int vstusb_open(struct inode *inode, struct file *file)
        struct vstusb_device *vstdev;
        struct usb_interface *interface;
 
-       lock_kernel();
        interface = usb_find_interface(&vstusb_driver, iminor(inode));
 
        if (!interface) {
                printk(KERN_ERR KBUILD_MODNAME
                       ": %s - error, can't find device for minor %d\n",
                       __func__, iminor(inode));
-               unlock_kernel();
                return -ENODEV;
        }
 
        vstdev = usb_get_intfdata(interface);
 
-       if (!vstdev) {
-               unlock_kernel();
+       if (!vstdev)
                return -ENODEV;
-       }
 
        /* lock this device */
        mutex_lock(&vstdev->lock);
@@ -128,7 +123,6 @@ static int vstusb_open(struct inode *inode, struct file *file)
        /* can only open one time */
        if ((!vstdev->present) || (vstdev->isopen)) {
                mutex_unlock(&vstdev->lock);
-               unlock_kernel();
                return -EBUSY;
        }
 
@@ -143,7 +137,6 @@ static int vstusb_open(struct inode *inode, struct file *file)
        dev_dbg(&vstdev->usb_dev->dev, "%s: opened\n", __func__);
 
        mutex_unlock(&vstdev->lock);
-       unlock_kernel();
 
        return 0;
 }