On Mon, 2012-05-28 at 13:18 +0200, Michal Kazior wrote: > Having .set_monitor_channel work with non-monitor > interfaces running would make interface > combinations accounting ambiguous. > > Change-Id: I2d579b483fe3c2998c50dcf89600740188f79733 > Signed-off-by: Michal Kazior <michal.kazior@xxxxxxxxx> > --- > net/wireless/chan.c | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/net/wireless/chan.c b/net/wireless/chan.c > index 45b28ab..4d71b38 100644 > --- a/net/wireless/chan.c > +++ b/net/wireless/chan.c > @@ -85,6 +85,8 @@ int cfg80211_set_monitor_channel(struct cfg80211_registered_device *rdev, > > if (!rdev->ops->set_monitor_channel) > return -EOPNOTSUPP; > + if (!cfg80211_has_monitors_only(rdev)) > + return -EBUSY; Makes sense. 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