USB: BKL removal: ldusb
authorOliver Neukum <oliver@neukum.org>
Thu, 14 Jan 2010 15:12:27 +0000 (16:12 +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/ldusb.c

index 8de32df5978a0756f7b63135e4b67f793259f4b4..7c0bd13eccb2cf0e0ee216a89a901108c274556a 100644 (file)
@@ -33,7 +33,6 @@
 #include <linux/slab.h>
 #include <linux/module.h>
 #include <linux/mutex.h>
-#include <linux/smp_lock.h>
 
 #include <asm/uaccess.h>
 #include <linux/input.h>
@@ -297,14 +296,12 @@ static int ld_usb_open(struct inode *inode, struct file *file)
        int retval;
        struct usb_interface *interface;
 
-       lock_kernel();
        nonseekable_open(inode, file);
        subminor = iminor(inode);
 
        interface = usb_find_interface(&ld_usb_driver, subminor);
 
        if (!interface) {
-               unlock_kernel();
                err("%s - error, can't find device for minor %d\n",
                     __func__, subminor);
                return -ENODEV;
@@ -312,16 +309,12 @@ static int ld_usb_open(struct inode *inode, struct file *file)
 
        dev = usb_get_intfdata(interface);
 
-       if (!dev) {
-               unlock_kernel();
+       if (!dev)
                return -ENODEV;
-       }
 
        /* lock this device */
-       if (mutex_lock_interruptible(&dev->mutex)) {
-               unlock_kernel();
+       if (mutex_lock_interruptible(&dev->mutex))
                return -ERESTARTSYS;
-       }
 
        /* allow opening only once */
        if (dev->open_count) {
@@ -360,7 +353,6 @@ static int ld_usb_open(struct inode *inode, struct file *file)
 
 unlock_exit:
        mutex_unlock(&dev->mutex);
-       unlock_kernel();
 
        return retval;
 }