spi: spi-fsl-dspi: Clear completion counter before initiating transfer

In target mode, extra interrupts can be received between the end of a
transfer and halting the module if the host continues sending more data.
If the interrupt from this occurs after the reinit_completion() then the
completion counter is left at a non-zero value. The next unrelated
transfer initiated by userspace will then complete immediately without
waiting for the interrupt or writing to the RX buffer.

Fix it by resetting the counter before the transfer so that lingering
values are cleared. This is done after clearing the FIFOs and the
status register but before the transfer is initiated, so no interrupts
should be received at this point resulting in other race conditions.

Fixes: 4f5ee75ea171 ("spi: spi-fsl-dspi: Replace interruptible wait queue with a simple completion")
Signed-off-by: James Clark <james.clark@linaro.org>
Reviewed-by: Frank Li <Frank.Li@nxp.com>
Link: https://patch.msgid.link/20250627-james-nxp-spi-dma-v4-1-178dba20c120@linaro.org
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/drivers/spi/spi-fsl-dspi.c b/drivers/spi/spi-fsl-dspi.c
index 863781ba6c1601f5d32cbf5a2d904374d33268b2..0dcd49114095051777256bc59180984d959e3575 100644 (file)
--- a/drivers/spi/spi-fsl-dspi.c
+++ b/drivers/spi/spi-fsl-dspi.c
@@ -983,11 +983,20 @@ static int dspi_transfer_one_message(struct spi_controller *ctlr,
                if (dspi->devtype_data->trans_mode == DSPI_DMA_MODE) {
                        status = dspi_dma_xfer(dspi);
                } else {
+                       /*
+                        * Reinitialize the completion before transferring data
+                        * to avoid the case where it might remain in the done
+                        * state due to a spurious interrupt from a previous
+                        * transfer. This could falsely signal that the current
+                        * transfer has completed.
+                        */
+                       if (dspi->irq)
+                               reinit_completion(&dspi->xfer_done);
+
                        dspi_fifo_write(dspi);
 
                        if (dspi->irq) {
                                wait_for_completion(&dspi->xfer_done);
-                               reinit_completion(&dspi->xfer_done);
                        } else {
                                do {
                                        status = dspi_poll(dspi);