On Fri, Apr 03, 2015 at 09:46:26PM +0200, Wolfram Sang wrote:

quoted

+static void tegra_i2c_slave_write(struct tegra_i2c_dev *i2c_dev, u32 val)
+{
+	i2c_writel(i2c_dev, val, I2C_SL_RCVD);
+
+	/*
+	 * TODO: A correct fix needs to be found for this.
+	 *
+	 * We experience less incomplete messages with this delay than without
+	 * it, but we don't know why. Help is appreciated.
+	 */

Uh oh. So I assume this is another reason for staging?

I think we can live with this upstream, no need for staging.

quoted

+	if (!i2c_dev->slave || !i2c_dev->slave->slave_cb)
+		return -EINVAL;

The core checks for slave_cb being valid.

quoted

+		i2c_slave_event(i2c_dev->slave, I2C_SLAVE_STOP, &dummy);

You could use value here, too, or?

quoted

+	if (!tegra_i2c_slave_isr(irq, i2c_dev))
+		return IRQ_HANDLED;

Minor nit: I'd prefer == 0 here, since it reads "if not slave_isr return
handled".

quoted

+	if (slave->addr > 0x7F)
+		addr2 = (slave->addr >> 7) | I2C_SL_ADDR2_TEN_BIT_MODE;

Nope. There are 10 bit encodings of addresses 0x000-0x07f, too. So, you
need to check for the flag (slave->flags & I2C_CLIENT_TEN).

If you'd fix up these and resend, I'll happily include this patch for 4.3
already.

Thanks,

   Wolfram