diff options
author | Greg Kroah-Hartman <gregkh@google.com> | 2016-06-23 14:20:02 -0700 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@google.com> | 2016-06-23 14:20:02 -0700 |
commit | 19cdabcf0ba92ddd87bdb86f8e3ceaae2dd6f8bb (patch) | |
tree | e408991db47d3af8823442433780188b05771591 /drivers/staging/greybus/uart.c | |
parent | 1211915127c152a86e68ea35770c9e2524020d4f (diff) | |
download | op-kernel-dev-19cdabcf0ba92ddd87bdb86f8e3ceaae2dd6f8bb.zip op-kernel-dev-19cdabcf0ba92ddd87bdb86f8e3ceaae2dd6f8bb.tar.gz |
greybus: Revert "greybus: don't use spin_lock_irq()"
This reverts commit 469fbe5da0229edcb42aa08bef8e10feaa37e6d7.
It isn't correct in places.
Reported-by: Gjorgji Rosikopulos <rosikopulos_gjorgji@projectara.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
Diffstat (limited to 'drivers/staging/greybus/uart.c')
-rw-r--r-- | drivers/staging/greybus/uart.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/staging/greybus/uart.c b/drivers/staging/greybus/uart.c index 8089638..6260569 100644 --- a/drivers/staging/greybus/uart.c +++ b/drivers/staging/greybus/uart.c @@ -674,7 +674,6 @@ static int set_serial_info(struct gb_tty *gb_tty, static int wait_serial_change(struct gb_tty *gb_tty, unsigned long arg) { int retval = 0; - unsigned long flags; DECLARE_WAITQUEUE(wait, current); struct async_icount old; struct async_icount new; @@ -683,11 +682,11 @@ static int wait_serial_change(struct gb_tty *gb_tty, unsigned long arg) return -EINVAL; do { - spin_lock_irqsave(&gb_tty->read_lock, flags); + spin_lock_irq(&gb_tty->read_lock); old = gb_tty->oldcount; new = gb_tty->iocount; gb_tty->oldcount = new; - spin_unlock_irqrestore(&gb_tty->read_lock, flags); + spin_unlock_irq(&gb_tty->read_lock); if ((arg & TIOCM_DSR) && (old.dsr != new.dsr)) break; |