OSDN Git Service

wifi: iwlegacy: remove redundant variable len
authorColin Ian King <colin.i.king@gmail.com>
Fri, 4 Nov 2022 13:50:36 +0000 (13:50 +0000)
committerKalle Valo <kvalo@kernel.org>
Tue, 8 Nov 2022 07:40:18 +0000 (09:40 +0200)
Variable len is being assigned and modified but it is never
used. The variable is redundant and can be removed.

Cleans up clang scan build warning:
warning: variable 'len' set but not used [-Wunused-but-set-variable]

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Acked-by: Stanislaw Gruszka <stf_xl@wp.pl>
Signed-off-by: Kalle Valo <kvalo@kernel.org>
Link: https://lore.kernel.org/r/20221104135036.225628-1-colin.i.king@gmail.com
drivers/net/wireless/intel/iwlegacy/3945-mac.c

index c450330..d7e99d5 100644 (file)
@@ -1202,8 +1202,6 @@ il3945_rx_handle(struct il_priv *il)
                D_RX("r = %d, i = %d\n", r, i);
 
        while (i != r) {
-               int len;
-
                rxb = rxq->queue[i];
 
                /* If an RXB doesn't have a Rx queue slot associated with it,
@@ -1217,10 +1215,6 @@ il3945_rx_handle(struct il_priv *il)
                               PAGE_SIZE << il->hw_params.rx_page_order,
                               DMA_FROM_DEVICE);
                pkt = rxb_addr(rxb);
-
-               len = le32_to_cpu(pkt->len_n_flags) & IL_RX_FRAME_SIZE_MSK;
-               len += sizeof(u32);     /* account for status word */
-
                reclaim = il_need_reclaim(il, pkt);
 
                /* Based on type of command response or notification,