diff options
| author |   David Jander <david@protonic.nl> | 2022-06-29 16:25:19 +0200 |
|---|---|---|
| committer |   Mark Brown <broonie@kernel.org> | 2022-06-30 13:40:36 +0100 |
| commit | 31d4c1bdf157421b26d51f61a4da95dd20d171e2 (patch) | |
| tree | 88948d3b66e2a444bfdb5dd75005d87084ffe459 /drivers/spi/spi.c | |
| parent | spi: spi.c: Fix comment style (diff) | |
| download | linux-dev-31d4c1bdf157421b26d51f61a4da95dd20d171e2.tar.xz linux-dev-31d4c1bdf157421b26d51f61a4da95dd20d171e2.zip | |
spi: spi.c: Remove redundant else block
Reported-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: David Jander <david@protonic.nl>
Link: https://lore.kernel.org/r/20220629142519.3985486-4-david@protonic.nl
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/spi/spi.c')
| -rw-r--r-- | drivers/spi/spi.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/drivers/spi/spi.c b/drivers/spi/spi.c index 6d69f5dc62bc..ebc47f63239f 100644 --- a/drivers/spi/spi.c +++ b/drivers/spi/spi.c @@ -1636,13 +1636,13 @@ static int __spi_pump_transfer_message(struct spi_controller *ctlr, dev_err(&ctlr->dev, "failed to transfer one message from queue\n"); return ret; - } else { - WRITE_ONCE(ctlr->cur_msg_need_completion, true); - smp_mb(); /* see spi_finalize_current_message()... */ - if (READ_ONCE(ctlr->cur_msg_incomplete)) - wait_for_completion(&ctlr->cur_msg_completion); } + WRITE_ONCE(ctlr->cur_msg_need_completion, true); + smp_mb(); /* See spi_finalize_current_message()... */ + if (READ_ONCE(ctlr->cur_msg_incomplete)) + wait_for_completion(&ctlr->cur_msg_completion); + return 0; } |