On Tue, 2012-09-04 at 14:49 +0530, Vasanthakumar Thiagarajan wrote: > >> + if (dev->ieee80211_ptr->iftype != NL80211_IFTYPE_AP&& > >> + dev->ieee80211_ptr->iftype != NL80211_IFTYPE_P2P_GO) > >> + return -EOPNOTSUPP; > > > > This also needs to check that the interface is operating already in AP > > mode. > > Sorry, not sure I get this. Is it not that configuring acl list > when dev->ieee80211_ptr->iftype == NL80211_IFTYPE_AP is already > taken care. Well you say that on start_ap the list must be cleared. Therefore, updating the list only makes sense after start_ap, not before, so you should refuse it being set before start_ap. 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