On Wed, 2012-06-20 at 08:14 +0200, Michal Kazior 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. This makes sense :) > + * @set_monitor_enabled: Notify driver that there are only monitor > + * interfaces running. Maybe rename to "set_monitor_only()"? But I don't really care much. Ok totally makes sense, not sure why I thought yesterday I should take a closer look. Btw, I need to commend you on the quality of your code in general, I like all the locking assertions etc. :-) johannes -- 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
