Commit aa5beaf6 authored by Sergei Shtylyov's avatar Sergei Shtylyov Committed by Wolfram Sang

i2c: rcar: check for no IRQ in rcar_i2c_irq()

Check if the ICMSR register (masked with the ICMIER register) evaluates to 0  in
the driver's interrupt handler and return IRQ_NONE in that case, like many other
drivers do.
Signed-off-by: default avatarSergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Signed-off-by: default avatarWolfram Sang <wsa@the-dreams.de>
parent 7c7117ff
...@@ -363,6 +363,7 @@ static int rcar_i2c_irq_recv(struct rcar_i2c_priv *priv, u32 msr) ...@@ -363,6 +363,7 @@ static int rcar_i2c_irq_recv(struct rcar_i2c_priv *priv, u32 msr)
static irqreturn_t rcar_i2c_irq(int irq, void *ptr) static irqreturn_t rcar_i2c_irq(int irq, void *ptr)
{ {
struct rcar_i2c_priv *priv = ptr; struct rcar_i2c_priv *priv = ptr;
irqreturn_t result = IRQ_HANDLED;
u32 msr; u32 msr;
/*-------------- spin lock -----------------*/ /*-------------- spin lock -----------------*/
...@@ -372,6 +373,10 @@ static irqreturn_t rcar_i2c_irq(int irq, void *ptr) ...@@ -372,6 +373,10 @@ static irqreturn_t rcar_i2c_irq(int irq, void *ptr)
/* Only handle interrupts that are currently enabled */ /* Only handle interrupts that are currently enabled */
msr &= rcar_i2c_read(priv, ICMIER); msr &= rcar_i2c_read(priv, ICMIER);
if (!msr) {
result = IRQ_NONE;
goto exit;
}
/* Arbitration lost */ /* Arbitration lost */
if (msr & MAL) { if (msr & MAL) {
...@@ -406,10 +411,11 @@ static irqreturn_t rcar_i2c_irq(int irq, void *ptr) ...@@ -406,10 +411,11 @@ static irqreturn_t rcar_i2c_irq(int irq, void *ptr)
wake_up(&priv->wait); wake_up(&priv->wait);
} }
exit:
spin_unlock(&priv->lock); spin_unlock(&priv->lock);
/*-------------- spin unlock -----------------*/ /*-------------- spin unlock -----------------*/
return IRQ_HANDLED; return result;
} }
static int rcar_i2c_master_xfer(struct i2c_adapter *adap, static int rcar_i2c_master_xfer(struct i2c_adapter *adap,
......
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