scsi: qedf: Remove redundant initialization of variable rc
authorColin Ian King <colin.king@canonical.com>
Wed, 27 May 2020 11:52:42 +0000 (12:52 +0100)
committerMartin K. Petersen <martin.petersen@oracle.com>
Wed, 3 Jun 2020 01:44:14 +0000 (21:44 -0400)
The variable rc 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.

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

index bb82f08..ad6a56c 100644 (file)
@@ -20,7 +20,7 @@ void qedf_fcoe_send_vlan_req(struct qedf_ctx *qedf)
 #define MY_FIP_ALL_FCF_MACS        ((__u8[6]) { 1, 0x10, 0x18, 1, 0, 2 })
        static u8 my_fcoe_all_fcfs[ETH_ALEN] = MY_FIP_ALL_FCF_MACS;
        unsigned long flags = 0;
-       int rc = -1;
+       int rc;
 
        skb = dev_alloc_skb(sizeof(struct fip_vlan));
        if (!skb) {