Signed-off-by: Alok Chauhan <[email protected]>
-----Original Message----- From: Stephen Warren Sent: Saturday, February 04, 2012 5:40 AM To: Ben Dooks; Wolfram Sang Cc: [email protected]; [email protected]; Alok Chauhan; Stephen Warren Subject: [PATCH 2/2] i2c: tegra: Remove unnecessary write to INT_STATUS From: Alok Chauhan <[email protected]> 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 <[email protected]>, but S-o-b missing in our internal repo. [swarren: Reworded commit description] Signed-off-by: Stephen Warren <[email protected]> --- 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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
