> - netif_carrier_on(dev);
> ifsta->flags |= IEEE80211_STA_PREV_BSSID_SET;
> memcpy(ifsta->prev_bssid, sdata->u.sta.bssid, ETH_ALEN);
> memcpy(wrqu.ap_addr.sa_data, sdata->u.sta.bssid, ETH_ALEN);
> ieee80211_sta_send_associnfo(dev, ifsta);
> } else {
> + netif_carrier_off(dev);
> ieee80211_sta_tear_down_BA_sessions(dev, ifsta->bssid);
> ifsta->flags &= ~IEEE80211_STA_ASSOCIATED;
> - netif_carrier_off(dev);
Maybe it should then be symmetric so that callers of the function have
to do it in both cases? Other than that, looks good.
johannes
Attachment:
signature.asc
Description: This is a digitally signed message part
