]> git.itanic.dy.fi Git - linux-stable/commitdiff
scsi: pm8001: Remove redundant initialization of variable 'rv'
authorColin Ian King <colin.king@canonical.com>
Wed, 4 Aug 2021 14:33:19 +0000 (15:33 +0100)
committerMartin K. Petersen <martin.petersen@oracle.com>
Tue, 10 Aug 2021 03:47:28 +0000 (23:47 -0400)
The variable 'rv' is being initialized with a value that is never read, it
is being updated later on. The assignment is redundant and can be removed.

Link: https://lore.kernel.org/r/20210804143319.115340-1-colin.king@canonical.com
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Addresses-Coverity: ("Unused value")

drivers/scsi/pm8001/pm8001_hwi.c

index 17c0f26e683a9c95178ee5e665f7e7b3d44af0e2..63690508313b76d43df04fd8cff6c44539d226a5 100644 (file)
@@ -1323,7 +1323,7 @@ int pm8001_mpi_build_cmd(struct pm8001_hba_info *pm8001_ha,
        void *pMessage;
        unsigned long flags;
        int q_index = circularQ - pm8001_ha->inbnd_q_tbl;
-       int rv = -1;
+       int rv;
 
        WARN_ON(q_index >= PM8001_MAX_INB_NUM);
        spin_lock_irqsave(&circularQ->iq_lock, flags);