projects
/
linux.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
805cb5a
)
wifi: cw1200: cleanup the code a bit
author
Bernard Zhao
<bernard@vivo.com>
Tue, 17 May 2022 01:41:36 +0000
(18:41 -0700)
committer
Kalle 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
patch
|
blob
|
history
diff --git
a/drivers/net/wireless/st/cw1200/bh.c
b/drivers/net/wireless/st/cw1200/bh.c
index 10e019cddcc6545384dec985d9042ba7793a9499..3b4ded2ac801cc57b2b3050198c931e6cf4f08ca 100644
(file)
--- a/
drivers/net/wireless/st/cw1200/bh.c
+++ b/
drivers/net/wireless/st/cw1200/bh.c
@@
-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;
}