scsi: scsi_transport_fc: Remove unused 'desc_cnt' variable
authorTom Rix <trix@redhat.com>
Sun, 26 Mar 2023 00:32:22 +0000 (20:32 -0400)
committerMartin K. Petersen <martin.petersen@oracle.com>
Mon, 3 Apr 2023 01:37:57 +0000 (21:37 -0400)
clang with W=1 reports:

drivers/scsi/scsi_transport_fc.c:908:6: error: variable
  'desc_cnt' set but not used [-Werror,-Wunused-but-set-variable]
        u32 desc_cnt = 0, bytes_remain;
            ^
This variable is not used so remove it.

Signed-off-by: Tom Rix <trix@redhat.com>
Link: https://lore.kernel.org/r/20230326003222.1343671-1-trix@redhat.com
Reviewed-by: Himanshu Madhani <himanshu.madhani@oracle.com>
Reviewed-by: Benjamin Block <bblock@linux.ibm.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/scsi_transport_fc.c

index f12e9467ebb430a1d9cfbcabccc708dded0cb128..64ff2629eaf98c196d67c0b2887cbac961ad30de 100644 (file)
@@ -905,7 +905,7 @@ fc_host_fpin_rcv(struct Scsi_Host *shost, u32 fpin_len, char *fpin_buf,
 {
        struct fc_els_fpin *fpin = (struct fc_els_fpin *)fpin_buf;
        struct fc_tlv_desc *tlv;
-       u32 desc_cnt = 0, bytes_remain;
+       u32 bytes_remain;
        u32 dtag;
        enum fc_host_event_code event_code =
                event_acknowledge ? FCH_EVT_LINK_FPIN_ACK : FCH_EVT_LINK_FPIN;
@@ -932,7 +932,6 @@ fc_host_fpin_rcv(struct Scsi_Host *shost, u32 fpin_len, char *fpin_buf,
                        fc_fpin_congn_stats_update(shost, tlv);
                }
 
-               desc_cnt++;
                bytes_remain -= FC_TLV_DESC_SZ_FROM_LENGTH(tlv);
                tlv = fc_tlv_next_desc(tlv);
        }