ath9k: cleanup: remove unneeded null check
authorDan Carpenter <error27@gmail.com>
Sat, 22 May 2010 20:33:11 +0000 (22:33 +0200)
committerJohn W. Linville <linville@tuxdriver.com>
Wed, 2 Jun 2010 20:13:33 +0000 (16:13 -0400)
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 <error27@gmail.com>
Acked-by: Sujith <Sujith.Manoharan@atheros.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/ath/ath9k/wmi.c

index e23172c9caaf1e038a2ee0b91556c008062030cc..6260faa658a262ebd16017425deb2cdd28aa48ae 100644 (file)
@@ -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;