On 10/14/2015 03:15 PM, Martin Kepplinger wrote:
[...]
> + if (irq1 > 0)
> + client->irq = irq1;
You must not overwrite client->irq, that field is manged by the I2C core and
is supposed to be read only for device drivers.
> + dev_info(&client->dev, "using interrupt line INT1\n");
This should probably be a dev_dbg()
> + } else {
> + client->irq = irq2;
> + dev_info(&client->dev, "using interrupt line INT2\n");
Same here
> + }
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
