| Message ID | 54F08D79.6060708@nokia.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 45b064d73d6a5cfcfa76012b91e2afc98e341664 |
| Headers | show |
On Fri, Feb 27, 2015 at 04:30:01PM +0100, Alexander Sverdlin wrote: > spi: pl022: Remove incorrect TxFIFO full reporting > > According to PL022 specification, TNF bit states for "Transmit FIFO Not full". > So the logic here is inverted. But "Receive Overrun Interrupt", which is handled > here, is only triggered on Rx errors. So instead of fixing the if statement, > remove the whole message. Applied, thanks.
diff --git a/drivers/spi/spi-pl022.c b/drivers/spi/spi-pl022.c index 8f67a46..f68cbf6 100644 --- a/drivers/spi/spi-pl022.c +++ b/drivers/spi/spi-pl022.c @@ -1254,9 +1254,6 @@ static irqreturn_t pl022_interrupt_handler(int irq, void *dev_id) if (readw(SSP_SR(pl022->virtbase)) & SSP_SR_MASK_RFF) dev_err(&pl022->adev->dev, "RXFIFO is full\n"); - if (readw(SSP_SR(pl022->virtbase)) & SSP_SR_MASK_TNF) - dev_err(&pl022->adev->dev, - "TXFIFO is full\n"); /* * Disable and clear interrupts, disable SSP,
spi: pl022: Remove incorrect TxFIFO full reporting According to PL022 specification, TNF bit states for "Transmit FIFO Not full". So the logic here is inverted. But "Receive Overrun Interrupt", which is handled here, is only triggered on Rx errors. So instead of fixing the if statement, remove the whole message. Signed-off-by: Alexander Sverdlin <alexander.sverdlin@nokia.com> --- drivers/spi/spi-pl022.c | 3 --- 1 files changed, 0 insertions(+), 3 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-spi" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html