ipw2100: Fix order of device registration
authorBen Hutchings <ben@decadent.org.uk>
Sun, 22 Jan 2012 03:11:12 +0000 (03:11 +0000)
committerJohn W. Linville <linville@tuxdriver.com>
Wed, 9 May 2012 01:53:47 +0000 (21:53 -0400)
Currently cfg80211 fails to create a "phy80211" symlink in sysfs from
the net device to the wiphy device.  The latter needs to be registered
first.

Compile-tested only.

Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/ipw2x00/ipw2100.c

index c72136c..9cfae0c 100644 (file)
@@ -1963,10 +1963,8 @@ static int ipw2100_wdev_init(struct net_device *dev)
        wdev->wiphy->n_cipher_suites = ARRAY_SIZE(ipw_cipher_suites);
 
        set_wiphy_dev(wdev->wiphy, &priv->pci_dev->dev);
-       if (wiphy_register(wdev->wiphy)) {
-               ipw2100_down(priv);
+       if (wiphy_register(wdev->wiphy))
                return -EIO;
-       }
        return 0;
 }
 
@@ -6331,6 +6329,11 @@ static int ipw2100_pci_init_one(struct pci_dev *pci_dev,
        printk(KERN_INFO DRV_NAME
               ": Detected Intel PRO/Wireless 2100 Network Connection\n");
 
+       err = ipw2100_wdev_init(dev);
+       if (err)
+               goto fail;
+       registered = 1;
+
        /* Bring up the interface.  Pre 0.46, after we registered the
         * network device we would call ipw2100_up.  This introduced a race
         * condition with newer hotplug configurations (network was coming
@@ -6347,11 +6350,7 @@ static int ipw2100_pci_init_one(struct pci_dev *pci_dev,
                       "Error calling register_netdev.\n");
                goto fail;
        }
-       registered = 1;
-
-       err = ipw2100_wdev_init(dev);
-       if (err)
-               goto fail;
+       registered = 2;
 
        mutex_lock(&priv->action_mutex);
 
@@ -6390,13 +6389,16 @@ out:
 
       fail_unlock:
        mutex_unlock(&priv->action_mutex);
-       wiphy_unregister(priv->ieee->wdev.wiphy);
-       kfree(priv->ieee->bg_band.channels);
       fail:
        if (dev) {
-               if (registered)
+               if (registered >= 2)
                        unregister_netdev(dev);
 
+               if (registered) {
+                       wiphy_unregister(priv->ieee->wdev.wiphy);
+                       kfree(priv->ieee->bg_band.channels);
+               }
+
                ipw2100_hw_stop_adapter(priv);
 
                ipw2100_disable_interrupts(priv);