rndis_wlan: simplify is_associated()
authorzuoqilin <zuoqilin@yulong.com>
Fri, 23 Apr 2021 09:49:40 +0000 (17:49 +0800)
committerKalle Valo <kvalo@codeaurora.org>
Tue, 15 Jun 2021 13:34:19 +0000 (16:34 +0300)
It is not necessary to define the variable ret to receive
the return value of the get_bssid() method.

Signed-off-by: zuoqilin <zuoqilin@yulong.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/20210423094940.1593-1-zuoqilin1@163.com
drivers/net/wireless/rndis_wlan.c

index 9fe77556858ee362e5f1f37080921c9f1cef91ee..63ce2443f13647fb7e6a2ebfcd0f598f4ee5352a 100644 (file)
@@ -1036,14 +1036,11 @@ static bool is_associated(struct usbnet *usbdev)
 {
        struct rndis_wlan_private *priv = get_rndis_wlan_priv(usbdev);
        u8 bssid[ETH_ALEN];
-       int ret;
 
        if (!priv->radio_on)
                return false;
 
-       ret = get_bssid(usbdev, bssid);
-
-       return (ret == 0 && !is_zero_ether_addr(bssid));
+       return (get_bssid(usbdev, bssid) == 0 && !is_zero_ether_addr(bssid));
 }
 
 static int disassociate(struct usbnet *usbdev, bool reset_ssid)