hi Michal, On Mon, May 28, 2012 at 2:18 PM, Michal Kazior <michal.kazior@xxxxxxxxx> wrote: > Implements .set_monitor_enabled(wiphy, enabled). > > Notifies driver upon change of interface layout. > > If only monitor interfaces become present it is > called with 2nd argument being true. If > non-monitor interface appears then 2nd argument > is false. Driver is notified only upon change. > > This makes it more obvious about the fact that > cfg80211 supports single monitor channel. Once we > implement multi-channel we don't want to allow > setting monitor channel while other interface > types are running. Otherwise it would be ambiguous > once we start considering num_different_channels. > > Change-Id: Ibd82a70c256c2de584eb541ea2c36663a59f09d4 > Signed-off-by: Michal Kazior <michal.kazior@xxxxxxxxx> > --- [...] > @@ -879,6 +895,7 @@ static int cfg80211_netdev_notifier_call(struct notifier_block *nb, > break; > case NETDEV_DOWN: > dev_hold(dev); > + cfg80211_update_iface_num(rdev, wdev, -1); > queue_work(cfg80211_wq, &wdev->cleanup_work); > break; > case NETDEV_UP: > @@ -986,6 +1003,7 @@ static int cfg80211_netdev_notifier_call(struct notifier_block *nb, > ret = cfg80211_can_add_interface(rdev, wdev->iftype); > if (ret) > return notifier_from_errno(ret); > + cfg80211_update_iface_num(rdev, wdev, 1); > break; > } > what about interface type change? Eliad. -- 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
