i2c-au1550: increase timeout waiting for master done
authorManuel Lauss <manuel.lauss@googlemail.com>
Sat, 22 Oct 2011 11:34:38 +0000 (13:34 +0200)
committerBen Dooks <ben-linux@fluff.org>
Sat, 29 Oct 2011 10:14:23 +0000 (11:14 +0100)
Double the timeout in the loop which busy-waits for the "master-done"
bit to be set.  This bit indicates whether an i2c transaction has
completed;  on the DB1300 and DB1550 boards this timeout is slightly
too short and causes transactions to the WM8731 codec to be falsely flagged
as failed.  The timeout itself is necessary since transactions to
non-existant slaves never set this bit in the first place (and cause
i2cdetect to hang).

With this change the WM8731 codec on the DB1300/DB1550 boards is correctly
detected and initialized.

Signed-off-by: Manuel Lauss <manuel.lauss@googlemail.com>
Signed-off-by: Ben Dooks <ben-linux@fluff.org>
drivers/i2c/busses/i2c-au1550.c

index 98ee11aa9e2c86aa56ebed5bf2017d782679276a..4f5150c417a205dba7d5fc1d7e9cb19d67924bf7 100644 (file)
@@ -101,7 +101,7 @@ static int wait_master_done(struct i2c_au1550_data *adap)
        int i;
 
        /* Wait for Master Done. */
-       for (i = 0; i < adap->xfer_timeout; i++) {
+       for (i = 0; i < 2 * adap->xfer_timeout; i++) {
                if ((RD(adap, PSC_SMBEVNT) & PSC_SMBEVNT_MD) != 0)
                        return 0;
                udelay(1);