<div dir="ltr">I have been using the driver with this modification for the past 6 months and it has been stable in an industrial environment.I had made a few other changes that also improve reliability (using ppc in_8 and out_8 and eieio barriers to ensure in-order execution. This lets you remove the unneeded double read of the status register. I also added a more robust recovery function to handle force of bus master-ship, and clearing the arb lost interrupt that is generated. currently this can cause the isr to trigger and cause superfluous interrupts. I have not posted this patch because of the extensive changes,<div><br></div><div>I will ack this patch.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 30, 2015 at 2:24 AM, Amit Tomar <span dir="ltr"><<a href="mailto:Amit.Tomar@freescale.com" target="_blank">Amit.Tomar@freescale.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ISR should not return IRQ_HANDLED for not handling anything.<br>
This patch fixes the return value of ISR for the same case.<br>
<br>
<br>
Signed-off-by: Amit Singh Tomar <<a href="mailto:amit.tomar@freescale.com">amit.tomar@freescale.com</a>><br>
---<br>
drivers/i2c/busses/i2c-mpc.c |    3 ++-<br>
1 file changed, 2 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/drivers/i2c/busses/i2c-mpc.c b/drivers/i2c/busses/i2c-mpc.c index 0edf630..7a3136f 100644<br>
--- a/drivers/i2c/busses/i2c-mpc.c<br>
+++ b/drivers/i2c/busses/i2c-mpc.c<br>
@@ -95,8 +95,9 @@ static irqreturn_t mpc_i2c_isr(int irq, void *dev_id)<br>
        i2c->interrupt = readb(i2c->base + MPC_I2C_SR);<br>
        writeb(0, i2c->base + MPC_I2C_SR);<br>
        wake_up(&i2c->queue);<br>
+       return IRQ_HANDLED;<br>
    }<br>
-  return IRQ_HANDLED;<br>
+  return IRQ_NONE;<br>
}<br>
<br>
/* Sometimes 9th clock pulse isn't generated, and slave doesn't release<br>
--<br>
1.7.9.5<br>
<br>
--<br>
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in<br>
the body of a message to <a href="mailto:majordomo@vger.kernel.org">majordomo@vger.kernel.org</a><br>
More majordomo info at  <a href="http://vger.kernel.org/majordomo-info.html" target="_blank">http://vger.kernel.org/majordomo-info.html</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div style="margin-left:40px">- Danielle Costantino<br></div></div>
</div>