On Thu, Jul 9, 2015 at 11:01 PM, Wolfram Sang <[email protected]> wrote: >> static int xiic_bus_busy(struct xiic_i2c *i2c) >> @@ -602,16 +601,21 @@ static void xiic_start_send(struct xiic_i2c *i2c) >> static irqreturn_t xiic_isr(int irq, void *dev_id) >> { >> struct xiic_i2c *i2c = dev_id; >> - >> - spin_lock(&i2c->lock); >> + u32 pend, isr, ier; >> + irqreturn_t ret = IRQ_HANDLED; >> + /* Do not processes a devices interrupts if the device has no >> + * interrupts pending >> + */ > > Shouldn't you init 'ret' to IRQ_NONE then? >
Indeed I missed it. -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
