Search Linux Wireless

Re: [PATCH] cfg80211: make action channel type optional

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

 



On Wed, 2010-05-19 at 20:48 +0200, Johannes Berg wrote:
> On Wed, 2010-05-19 at 14:19 -0400, Pavel Roskin wrote:
> > On Wed, 2010-05-19 at 12:17 +0200, Johannes Berg wrote:
> > 
> > > @@ -1168,6 +1168,7 @@ struct cfg80211_ops {
> > >  	int	(*action)(struct wiphy *wiphy, struct net_device *dev,
> > >  			  struct ieee80211_channel *chan,
> > >  			  enum nl80211_channel_type channel_type,
> > > +			  bool channel_type_valid,
> > >  			  const u8 *buf, size_t len, u64 *cookie);
> > 
> > Would not it be easier to introduce NL80211_CHAN_UNSPECIFIED in enum
> > nl80211_channel_type?
> 
> Yes, but it would change the userspace API so we can't do that.

I believe the new API would be a superset of the old API.  No userspace
caller would need to be changed.

Anyway, I just wanted to be sure that a simpler solution was considered.

-- 
Regards,
Pavel Roskin
--
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

[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux