2016-08-11 14:34:08 +00:00
|
|
|
From 8e830bd17e945e74964a5b61353d74e34c0791cd Mon Sep 17 00:00:00 2001
|
|
|
|
From: Andy Gross <andy.gross@linaro.org>
|
|
|
|
Date: Fri, 29 Jan 2016 22:06:50 -0600
|
|
|
|
Subject: [PATCH] spi: qup: Fix transaction done signaling
|
|
|
|
|
|
|
|
Wait to signal done until we get all of the interrupts we are expecting
|
|
|
|
to get for a transaction. If we don't wait for the input done flag, we
|
|
|
|
can be inbetween transactions when the done flag comes in and this can
|
|
|
|
mess up the next transaction.
|
|
|
|
|
|
|
|
Change-Id: I08d78376e71590663158d6434a3fb7c0623264c9
|
|
|
|
CC: Grant Grundler <grundler@chromium.org>
|
|
|
|
CC: Sarthak Kukreti <skukreti@codeaurora.org>
|
|
|
|
Signed-off-by: Andy Gross <andy.gross@linaro.org>
|
|
|
|
---
|
|
|
|
drivers/spi/spi-qup.c | 3 ++-
|
|
|
|
1 file changed, 2 insertions(+), 1 deletion(-)
|
|
|
|
|
|
|
|
--- a/drivers/spi/spi-qup.c
|
|
|
|
+++ b/drivers/spi/spi-qup.c
|
2016-08-22 17:05:45 +00:00
|
|
|
@@ -447,7 +447,8 @@ static irqreturn_t spi_qup_qup_irq(int i
|
2016-08-11 14:34:08 +00:00
|
|
|
controller->xfer = xfer;
|
|
|
|
spin_unlock_irqrestore(&controller->lock, flags);
|
2016-08-22 17:05:45 +00:00
|
|
|
|
2016-08-11 14:34:08 +00:00
|
|
|
- if (controller->rx_bytes == xfer->len || error)
|
|
|
|
+ if ((controller->rx_bytes == xfer->len &&
|
|
|
|
+ (opflags & QUP_OP_MAX_INPUT_DONE_FLAG)) || error)
|
|
|
|
complete(&controller->done);
|
2016-08-22 17:05:45 +00:00
|
|
|
|
2016-08-11 14:34:08 +00:00
|
|
|
return IRQ_HANDLED;
|