bf561-coreb: BKL unneeded for open()

According to Mike Frysinger:

	http://permalink.gmane.org/gmane.linux.kernel/699279

open() does not need the BKL, so take it back out.
This commit is contained in:
Jonathan Corbet 2008-06-30 16:45:15 -06:00
parent a238b790d5
commit 57debddaa1
1 changed files with 1 additions and 4 deletions

View File

@ -32,7 +32,6 @@
#include <linux/device.h> #include <linux/device.h>
#include <linux/ioport.h> #include <linux/ioport.h>
#include <linux/module.h> #include <linux/module.h>
#include <linux/smp_lock.h>
#include <linux/uaccess.h> #include <linux/uaccess.h>
#include <linux/fs.h> #include <linux/fs.h>
#include <asm/dma.h> #include <asm/dma.h>
@ -195,9 +194,9 @@ static loff_t coreb_lseek(struct file *file, loff_t offset, int origin)
return ret; return ret;
} }
/* No BKL needed here */
static int coreb_open(struct inode *inode, struct file *file) static int coreb_open(struct inode *inode, struct file *file)
{ {
lock_kernel();
spin_lock_irq(&coreb_lock); spin_lock_irq(&coreb_lock);
if (coreb_status & COREB_IS_OPEN) if (coreb_status & COREB_IS_OPEN)
@ -206,12 +205,10 @@ static int coreb_open(struct inode *inode, struct file *file)
coreb_status |= COREB_IS_OPEN; coreb_status |= COREB_IS_OPEN;
spin_unlock_irq(&coreb_lock); spin_unlock_irq(&coreb_lock);
unlock_kernel();
return 0; return 0;
out_busy: out_busy:
spin_unlock_irq(&coreb_lock); spin_unlock_irq(&coreb_lock);
unlock_kernel();
return -EBUSY; return -EBUSY;
} }