spi: davinci: do not treat Tx interrupt being set as error
authorSekhar Nori <nsekhar@ti.com>
Fri, 20 Aug 2010 10:57:08 +0000 (16:27 +0530)
committerSekhar Nori <nsekhar@ti.com>
Thu, 18 Nov 2010 13:08:29 +0000 (18:38 +0530)
In davinci_spi_check_error(), Tx interrupt being set is
treated as error. This function is only meant to flag
bus error conditions and Tx interrupt being set at that
point is not a bus error but rather a driver bug.

Stop checking for Tx interrupt and flagging that as an
IO error.

Tested-By: Michael Williamson <michael.williamson@criticallink.com>
Tested-By: Brian Niebuhr <bniebuhr@efjohnson.com>
Signed-off-by: Sekhar Nori <nsekhar@ti.com>
drivers/spi/davinci_spi.c

index ad814f2..a60a65c 100644 (file)
@@ -590,10 +590,6 @@ static int davinci_spi_check_error(struct davinci_spi *davinci_spi,
                        dev_dbg(sdev, "SPI Data Overrun error\n");
                        return -EIO;
                }
-               if (int_status & SPIFLG_TX_INTR_MASK) {
-                       dev_dbg(sdev, "SPI TX intr bit set\n");
-                       return -EIO;
-               }
                if (int_status & SPIFLG_BUF_INIT_ACTIVE_MASK) {
                        dev_dbg(sdev, "SPI Buffer Init Active\n");
                        return -EBUSY;