cfg80211: invert P2P-Device vs. netdev check ordering
authorJohannes Berg <johannes.berg@intel.com>
Fri, 19 Apr 2013 10:19:39 +0000 (12:19 +0200)
committerJohannes Berg <johannes.berg@intel.com>
Mon, 22 Apr 2013 13:31:55 +0000 (15:31 +0200)
In cfg80211_can_use_iftype_chan(), check for P2P Device
first, and then for netdevs. This doesn't really change
anything but makes the code a bit easier to read since
it may not be obvious for everyone at first that a P2P
device has no netdev.

Signed-off-by: Johannes Berg <johannes.berg@intel.com>
net/wireless/util.c

index 3d8a133..a7046a4 100644 (file)
@@ -1278,12 +1278,12 @@ int cfg80211_can_use_iftype_chan(struct cfg80211_registered_device *rdev,
        list_for_each_entry(wdev_iter, &rdev->wdev_list, list) {
                if (wdev_iter == wdev)
                        continue;
-               if (wdev_iter->netdev) {
-                       if (!netif_running(wdev_iter->netdev))
-                               continue;
-               } else if (wdev_iter->iftype == NL80211_IFTYPE_P2P_DEVICE) {
+               if (wdev_iter->iftype == NL80211_IFTYPE_P2P_DEVICE) {
                        if (!wdev_iter->p2p_started)
                                continue;
+               } else if (wdev_iter->netdev) {
+                       if (!netif_running(wdev_iter->netdev))
+                               continue;
                } else {
                        WARN_ON(1);
                }