This will later be used by others. Signed-off-by: Luis R. Rodriguez <lrodriguez@xxxxxxxxxxx> --- net/wireless/core.c | 2 +- net/wireless/core.h | 7 +++++++ 2 files changed, 8 insertions(+), 1 deletions(-) diff --git a/net/wireless/core.c b/net/wireless/core.c index 2b3e786..55be3b0 100644 --- a/net/wireless/core.c +++ b/net/wireless/core.c @@ -225,7 +225,7 @@ struct wiphy *wiphy_new(struct cfg80211_ops *ops, int sizeof_priv) drv->wiphy_idx = wiphy_counter++; - if (unlikely(drv->wiphy_idx < 0)) { + if (!wiphy_idx_valid(drv->wiphy_idx)) { wiphy_counter--; mutex_unlock(&cfg80211_drv_mutex); /* ugh, wrapped! */ diff --git a/net/wireless/core.h b/net/wireless/core.h index 36e2397..e24bc2a 100644 --- a/net/wireless/core.h +++ b/net/wireless/core.h @@ -62,6 +62,13 @@ struct cfg80211_registered_device *wiphy_to_dev(struct wiphy *wiphy) return container_of(wiphy, struct cfg80211_registered_device, wiphy); } +/* Note 0 is valid, hence phy0 */ +static inline +bool wiphy_idx_valid(int wiphy_idx) +{ + return (likely(wiphy_idx >= 0)); +} + extern struct mutex cfg80211_drv_mutex; extern struct list_head cfg80211_drv_list; -- 1.6.1.2.253.ga34a -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html