scsi: elx: efct: Remove redundant initialization of variable lun
authorColin Ian King <colin.king@canonical.com>
Wed, 16 Jun 2021 17:16:21 +0000 (18:16 +0100)
committerMartin K. Petersen <martin.petersen@oracle.com>
Sat, 19 Jun 2021 03:01:03 +0000 (23:01 -0400)
The variable "lun" 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/20210616171621.16176-1-colin.king@canonical.com
Reviewed-by: James Smart <jsmart2021@gmail.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/elx/efct/efct_unsol.c

index a9384c9..e6addab 100644 (file)
@@ -342,7 +342,7 @@ efct_dispatch_fcp_cmd(struct efct_node *node, struct efc_hw_sequence *seq)
        struct fc_frame_header *fchdr = seq->header->dma.virt;
        struct fcp_cmnd *cmnd = NULL;
        struct efct_io *io = NULL;
-       u32 lun = U32_MAX;
+       u32 lun;
 
        if (!seq->payload) {
                efc_log_err(efct, "Sequence payload is NULL.\n");