spi: remove redundant assignment to variable ms
authorColin Ian King <colin.king@canonical.com>
Fri, 10 Apr 2020 12:23:15 +0000 (13:23 +0100)
committerMark Brown <broonie@kernel.org>
Tue, 14 Apr 2020 17:36:31 +0000 (18:36 +0100)
The variable ms is being initialized with a value that is never read
and it is being updated later with a new value.  The initialization is
redundant and can be removed.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Reviewed-by: Geert Uytterhoeven <geert@linux-m68k.org>
Addresses-Coverity: ("Unused value")
Link: https://lore.kernel.org/r/20200410122315.17523-1-colin.king@canonical.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/spi/spi.c

index c92c894..5446b07 100644 (file)
@@ -1075,7 +1075,7 @@ static int spi_transfer_wait(struct spi_controller *ctlr,
 {
        struct spi_statistics *statm = &ctlr->statistics;
        struct spi_statistics *stats = &msg->spi->statistics;
-       unsigned long long ms = 1;
+       unsigned long long ms;
 
        if (spi_controller_is_slave(ctlr)) {
                if (wait_for_completion_interruptible(&ctlr->xfer_completion)) {