diff options
| author | Eddie James <eajames@linux.ibm.com> | 2020-11-10 22:47:36 +0100 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2020-11-11 13:34:29 +0100 |
| commit | ee4ad5d06509b3aea79b6a77bebd09ef891bed8d (patch) | |
| tree | 65de0b6cd50285e4b2c31d171a380f075bd87c77 /drivers/spi/spi-fsi.c | |
| parent | spi: fix client driver breakages when using GPIO descriptors (diff) | |
| download | linux-ee4ad5d06509b3aea79b6a77bebd09ef891bed8d.tar.xz linux-ee4ad5d06509b3aea79b6a77bebd09ef891bed8d.zip | |
spi: fsi: Fix transfer returning without finalizing message
In the case that the SPI mux isn't set, the transfer_one_message
function returns without finalizing the message. This means that
the transfer never completes, resulting in hung tasks and an
eventual kernel panic. Fix it by finalizing the transfer in this
case.
Fixes: 9211a441e606 ("spi: fsi: Check mux status before transfers")
Signed-off-by: Eddie James <eajames@linux.ibm.com>
Reviewed-by: Joel Stanley <joel@jms.id.au>
Link: https://lore.kernel.org/r/20201110214736.25718-1-eajames@linux.ibm.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/spi/spi-fsi.c')
| -rw-r--r-- | drivers/spi/spi-fsi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/spi/spi-fsi.c b/drivers/spi/spi-fsi.c index 8a440c7078ef..3920cd3286d8 100644 --- a/drivers/spi/spi-fsi.c +++ b/drivers/spi/spi-fsi.c @@ -477,7 +477,7 @@ static int fsi_spi_transfer_one_message(struct spi_controller *ctlr, rc = fsi_spi_check_mux(ctx->fsi, ctx->dev); if (rc) - return rc; + goto error; list_for_each_entry(transfer, &mesg->transfers, transfer_list) { struct fsi_spi_sequence seq; |