wifi: cw1200: cleanup the code a bit
authorBernard Zhao <bernard@vivo.com>
Tue, 17 May 2022 01:41:36 +0000 (18:41 -0700)
committerKalle Valo <kvalo@kernel.org>
Mon, 30 May 2022 08:29:15 +0000 (11:29 +0300)
Delete if NULL check before dev_kfree_skb call.
This change is to cleanup the code a bit.

Signed-off-by: Bernard Zhao <bernard@vivo.com>
Signed-off-by: Kalle Valo <kvalo@kernel.org>
Link: https://lore.kernel.org/r/20220517014136.410450-1-bernard@vivo.com
drivers/net/wireless/st/cw1200/bh.c

index 10e019c..3b4ded2 100644 (file)
@@ -327,18 +327,12 @@ static int cw1200_bh_rx_helper(struct cw1200_common *priv,
        if (WARN_ON(wsm_handle_rx(priv, wsm_id, wsm, &skb_rx)))
                goto err;
 
-       if (skb_rx) {
-               dev_kfree_skb(skb_rx);
-               skb_rx = NULL;
-       }
+       dev_kfree_skb(skb_rx);
 
        return 0;
 
 err:
-       if (skb_rx) {
-               dev_kfree_skb(skb_rx);
-               skb_rx = NULL;
-       }
+       dev_kfree_skb(skb_rx);
        return -1;
 }