Search Linux Wireless

Re: [RFC 1/2] mac80211: move ieee80211_set_channel function

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]


On Fri, 2012-05-11 at 08:36 +0200, Michal Kazior wrote:
> Johannes Berg wrote:
> > +static int ieee80211_set_channel(struct wiphy *wiphy,
> > +				 struct net_device *netdev,
> > +				 struct ieee80211_channel *chan,
> > +				 enum nl80211_channel_type channel_type)
> > +{
> > +	struct ieee80211_local *local = wiphy_priv(wiphy);
> > +	struct ieee80211_sub_if_data *sdata = NULL;
> > +
> > +	if (netdev)
> > +		sdata = IEEE80211_DEV_TO_SUB_IF(netdev);
> > +
> > +	switch (ieee80211_get_channel_mode(local, NULL)) {
> > +	case CHAN_MODE_HOPPING:
> > +		return -EBUSY;
> > +	case CHAN_MODE_FIXED:
> > +		if (local->oper_channel != chan)
> > +			return -EBUSY;
> 
> 
> > +		if (!sdata&&  local->_oper_channel_type == channel_type)
> > +			return 0;
> 
> Do we need to keep this part? If not we could remove it in your:
>    [RFC] mac80211: clean up ieee80211_set_channel

I think we do need to keep it.

> I think we could even remove the whole `ieee80211_get_channel_mode` 
> thing now since we enforce interface combinations, right? We just need 
> to adjust the interface combinations verification to refuse mixing ibss 
> with anything else for the moment. That shouldn't be bad, or is it?

Somebody is going to complain if we never allow IBSS+anything any more
because IBSS has a fixed-channel mode in mac80211 which can cooperate
better.

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


[Linux Kernel]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Photo]     [Yosemite]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]     [Free Dating]     [M2M Wireless]

Add to Google Powered by Linux