[PATCH] softmac: correctly use netif_carrier_{on,off}
authorJohannes Berg <johannes@sipsolutions.net>
Fri, 6 Jan 2006 17:11:23 +0000 (18:11 +0100)
committerJohn W. Linville <linville@tuxdriver.com>
Thu, 23 Mar 2006 03:16:52 +0000 (22:16 -0500)
TODO: add callbacks for ifup/ifdown (see mailing list)

Signed-off-by: John W. Linville <linville@tuxdriver.com>
net/ieee80211/softmac/ieee80211softmac_assoc.c
net/ieee80211/softmac/ieee80211softmac_auth.c
net/ieee80211/softmac/ieee80211softmac_module.c

index 98487448f2d3a17c46933590d9cf156f8c85bb04..085022fd23c0c3084ce45f6acf1ede873ae8dfb5 100644 (file)
@@ -268,7 +268,7 @@ ieee80211softmac_associated(struct ieee80211softmac_device *mac,
        if (mac->set_bssid_filter)
                mac->set_bssid_filter(mac->dev, net->bssid);
        memcpy(mac->ieee->bssid, net->bssid, ETH_ALEN);
-       mac->dev->flags |= IFF_RUNNING;
+       netif_carrier_on(mac->dev);
        
        mac->association_id = le16_to_cpup(&resp->aid);
 }
@@ -346,7 +346,7 @@ ieee80211softmac_handle_disassoc(struct net_device * dev,
        struct ieee80211softmac_device *mac = ieee80211_priv(dev);
        unsigned long flags;
        dprintk(KERN_INFO PFX "got disassoc frame\n");
-       
+       netif_carrier_off(dev);
        spin_lock_irqsave(&mac->lock, flags);
        mac->associnfo.bssvalid = 0;
        mac->associated = 0;
index 5a773528110fbc204841dc8fbb9677eed5c1e479..6eab2be9187065379dcb7654f85f02e1899c69be 100644 (file)
@@ -278,6 +278,8 @@ ieee80211softmac_deauth_from_net(struct ieee80211softmac_device *mac,
                kfree(net->challenge);
        kfree(net);
        
+       /* can't transmit data right now... */
+       netif_carrier_off(mac->dev);
        /* let's try to re-associate */
        schedule_work(&mac->associnfo.work);
        spin_unlock_irqrestore(&mac->lock, flags);
index 79ef959a2c11852d45858ee486d9620e6a1e864d..ea4a19e1122c305d0607a3d0cacb374c0ef5d5ad 100644 (file)
@@ -40,21 +40,10 @@ struct net_device *alloc_ieee80211softmac(int sizeof_priv)
        softmac->txrates.default_rate = IEEE80211_CCK_RATE_1MB;
        softmac->txrates.default_fallback = IEEE80211_CCK_RATE_1MB;
 
-       /* should we also assign softmac->mgmt_xmit here so
-        * that it is always valid? If so, we probably want
-        * to define a new function for that which just
-        * wraps ieee80211_tx_frame
-        */
+       /* to start with, we can't send anything ... */
+       netif_carrier_off(dev);
        
-       /* until associated, we're not ready */
-       dev->flags &= ~IFF_RUNNING;
-
        return dev;
-
-err_free_ieee80211:
-       free_ieee80211(dev);
-
-       return NULL;
 }
 
 /* Clears the pending work queue items, stops all scans, etc. */