Commit 908fd7e5 authored by Ruchika Kharwar's avatar Ruchika Kharwar Committed by Greg Kroah-Hartman

serial: omap: Fix IRQ handling return value

Ensure the Interrupt handling routine return IRQ_HANDLED vs
IRQ_NONE.
Signed-off-by: default avatarRuchika Kharwar <ruchika@ti.com>
Signed-off-by: default avatarAlexander Savchenko <oleksandr.savchenko@ti.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent a0a490f9
...@@ -485,7 +485,6 @@ static irqreturn_t serial_omap_irq(int irq, void *dev_id) ...@@ -485,7 +485,6 @@ static irqreturn_t serial_omap_irq(int irq, void *dev_id)
struct uart_omap_port *up = dev_id; struct uart_omap_port *up = dev_id;
unsigned int iir, lsr; unsigned int iir, lsr;
unsigned int type; unsigned int type;
irqreturn_t ret = IRQ_NONE;
int max_count = 256; int max_count = 256;
spin_lock(&up->port.lock); spin_lock(&up->port.lock);
...@@ -496,7 +495,6 @@ static irqreturn_t serial_omap_irq(int irq, void *dev_id) ...@@ -496,7 +495,6 @@ static irqreturn_t serial_omap_irq(int irq, void *dev_id)
if (iir & UART_IIR_NO_INT) if (iir & UART_IIR_NO_INT)
break; break;
ret = IRQ_HANDLED;
lsr = serial_in(up, UART_LSR); lsr = serial_in(up, UART_LSR);
/* extract IRQ type from IIR register */ /* extract IRQ type from IIR register */
...@@ -535,7 +533,7 @@ static irqreturn_t serial_omap_irq(int irq, void *dev_id) ...@@ -535,7 +533,7 @@ static irqreturn_t serial_omap_irq(int irq, void *dev_id)
pm_runtime_put_autosuspend(up->dev); pm_runtime_put_autosuspend(up->dev);
up->port_activity = jiffies; up->port_activity = jiffies;
return ret; return IRQ_HANDLED;
} }
static unsigned int serial_omap_tx_empty(struct uart_port *port) static unsigned int serial_omap_tx_empty(struct uart_port *port)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment