wcn36xx: Remove unnecessary rcu_read_unlock in wcn36xx_bss_info_changed
authorJia-Ju Bai <baijiaju1990@163.com>
Fri, 27 Oct 2017 08:12:30 +0000 (11:12 +0300)
committerKalle Valo <kvalo@qca.qualcomm.com>
Fri, 27 Oct 2017 13:59:58 +0000 (16:59 +0300)
No rcu_read_lock is called, but rcu_read_unlock is still called.
Thus rcu_read_unlock should be removed.

Signed-off-by: Jia-Ju Bai <baijiaju1990@163.com>
Acked-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
drivers/net/wireless/ath/wcn36xx/main.c

index 35bd50bcbbd51e2615c7b63108fbff8d78ced466..b83f01d6e3ddd842e115a19266cb79d4dfcc6421 100644 (file)
@@ -812,7 +812,6 @@ static void wcn36xx_bss_info_changed(struct ieee80211_hw *hw,
                        if (!sta) {
                                wcn36xx_err("sta %pM is not found\n",
                                              bss_conf->bssid);
-                               rcu_read_unlock();
                                goto out;
                        }
                        sta_priv = wcn36xx_sta_to_priv(sta);