Signed-off-by: Alok Chauhan <alokc@xxxxxxxxxx> -----Original Message----- From: Stephen Warren Sent: Saturday, February 04, 2012 5:40 AM To: Ben Dooks; Wolfram Sang Cc: linux-i2c@xxxxxxxxxxxxxxx; linux-tegra@xxxxxxxxxxxxxxx; Alok Chauhan; Stephen Warren Subject: [PATCH 2/2] i2c: tegra: Remove unnecessary write to INT_STATUS From: Alok Chauhan <alokc@xxxxxxxxxx> The write is not necessary and may cause the I2C controller to misbehave. With this fix, I2C on Tegra30 works (at least, running i2cdump repeatedly on the WM8903 on Cardhu's I2C5/DVC bus). Originally by Alok Chauhan <alokc@xxxxxxxxxx>, but S-o-b missing in our internal repo. [swarren: Reworded commit description] Signed-off-by: Stephen Warren <swarren@xxxxxxxxxx> --- drivers/i2c/busses/i2c-tegra.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/drivers/i2c/busses/i2c-tegra.c b/drivers/i2c/busses/i2c-tegra.c index a546ede..edac27b 100644 --- a/drivers/i2c/busses/i2c-tegra.c +++ b/drivers/i2c/busses/i2c-tegra.c @@ -456,7 +456,6 @@ static int tegra_i2c_xfer_msg(struct tegra_i2c_dev *i2c_dev, int ret; tegra_i2c_flush_fifos(i2c_dev); - i2c_writel(i2c_dev, 0xFF, I2C_INT_STATUS); if (msg->len == 0) return -EINVAL; -- 1.7.0.4 -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html