OSDN Git Service

qed: remove unused num_ooo_add_to_peninsula variable
authorTom Rix <trix@redhat.com>
Sun, 26 Mar 2023 00:17:33 +0000 (20:17 -0400)
committerJakub Kicinski <kuba@kernel.org>
Tue, 28 Mar 2023 02:43:45 +0000 (19:43 -0700)
clang with W=1 reports
drivers/net/ethernet/qlogic/qed/qed_ll2.c:649:6: error: variable
  'num_ooo_add_to_peninsula' set but not used [-Werror,-Wunused-but-set-variable]
        u32 num_ooo_add_to_peninsula = 0, cid;
            ^
This variable is not used so remove it.

Signed-off-by: Tom Rix <trix@redhat.com>
Reviewed-by: Simon Horman <simon.horman@corigine.com>
Link: https://lore.kernel.org/r/20230326001733.1343274-1-trix@redhat.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/qlogic/qed/qed_ll2.c

index e5116a8..717a0b3 100644 (file)
@@ -646,13 +646,13 @@ static int qed_ll2_lb_rxq_handler(struct qed_hwfn *p_hwfn,
        struct qed_ll2_rx_queue *p_rx = &p_ll2_conn->rx_queue;
        u16 packet_length = 0, parse_flags = 0, vlan = 0;
        struct qed_ll2_rx_packet *p_pkt = NULL;
-       u32 num_ooo_add_to_peninsula = 0, cid;
        union core_rx_cqe_union *cqe = NULL;
        u16 cq_new_idx = 0, cq_old_idx = 0;
        struct qed_ooo_buffer *p_buffer;
        struct ooo_opaque *ooo_opq;
        u8 placement_offset = 0;
        u8 cqe_type;
+       u32 cid;
 
        cq_new_idx = le16_to_cpu(*p_rx->p_fw_cons);
        cq_old_idx = qed_chain_get_cons_idx(&p_rx->rcq_chain);
@@ -762,7 +762,6 @@ static int qed_ll2_lb_rxq_handler(struct qed_hwfn *p_hwfn,
                                                   cid, ooo_opq->ooo_isle);
                                break;
                        case TCP_EVENT_ADD_PEN:
-                               num_ooo_add_to_peninsula++;
                                qed_ooo_put_ready_buffer(p_hwfn,
                                                         p_hwfn->p_ooo_info,
                                                         p_buffer, true);