scsi: qla2xxx: Remove redundant initialization of variable num_cnt
authorColin Ian King <colin.king@canonical.com>
Wed, 4 Aug 2021 13:13:44 +0000 (14:13 +0100)
committerMartin K. Petersen <martin.petersen@oracle.com>
Fri, 6 Aug 2021 03:21:08 +0000 (23:21 -0400)
The variable num_cnt 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/20210804131344.112635-1-colin.king@canonical.com
Reviewed-by: Himanshu Madhani <himanshu.madhani@oracle.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/qla2xxx/qla_edif.c

index fde4109..2db954a 100644 (file)
@@ -875,7 +875,7 @@ static int
 qla_edif_app_getfcinfo(scsi_qla_host_t *vha, struct bsg_job *bsg_job)
 {
        int32_t                 rval = 0;
-       int32_t                 num_cnt = 1;
+       int32_t                 num_cnt;
        struct fc_bsg_reply     *bsg_reply = bsg_job->reply;
        struct app_pinfo_req    app_req;
        struct app_pinfo_reply  *app_reply;