From: Dan Carpenter Date: Sat, 22 May 2010 20:33:11 +0000 (+0200) Subject: ath9k: cleanup: remove unneeded null check X-Git-Tag: firefly_0821_release~9833^2~1306^2~107^2~413 X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=43163f0f8a11822607c36249c6f55f0613e0d73d;p=firefly-linux-kernel-4.4.55.git ath9k: cleanup: remove unneeded null check We dereference "wmi" on the line before and also when we initialize "ah". This check has always been after a dereference since the first commit a couple months ago. Looking through the code, it looks like "wmi" can't actually be null here so I just removed the check. Signed-off-by: Dan Carpenter Acked-by: Sujith Signed-off-by: John W. Linville --- diff --git a/drivers/net/wireless/ath/ath9k/wmi.c b/drivers/net/wireless/ath/ath9k/wmi.c index e23172c9caaf..6260faa658a2 100644 --- a/drivers/net/wireless/ath/ath9k/wmi.c +++ b/drivers/net/wireless/ath/ath9k/wmi.c @@ -279,9 +279,6 @@ int ath9k_wmi_cmd(struct wmi *wmi, enum wmi_cmd_id cmd_id, if (wmi->drv_priv->op_flags & OP_UNPLUGGED) return 0; - if (!wmi) - return -EINVAL; - skb = alloc_skb(headroom + cmd_len, GFP_ATOMIC); if (!skb) return -ENOMEM;