RE: [PATCHv3 RESEND 01/11] cfg80211: add start / stop NAN commands

2016-04-06 Thread Otcheretianski, Andrei
> -Original Message- > From: Malinen, Jouni [mailto:jo...@qca.qualcomm.com] > Sent: Wednesday, April 6, 2016 12:34 PM > To: Grumbach, Emmanuel > Cc: johan...@sipsolutions.net; linux-wireless@vger.kernel.org; > Otcheretianski, Andrei > Subject: Re: [PATCHv3 RESEND

RE: [PATCHv3 RESEND 01/11] cfg80211: add start / stop NAN commands

2016-04-06 Thread Grumbach, Emmanuel
> > On Tue, Mar 29, 2016 at 12:34:59PM +0300, Emmanuel Grumbach wrote: > > This allows user space to start/stop NAN interface. > > A NAN interface is like P2P device in a few aspects: it doesn't have a > > netdev associated to it. > > Add the new interface type and prevent operations that can't be

Re: [PATCHv3 RESEND 01/11] cfg80211: add start / stop NAN commands

2016-04-06 Thread Malinen, Jouni
On Tue, Mar 29, 2016 at 12:34:59PM +0300, Emmanuel Grumbach wrote: > This allows user space to start/stop NAN interface. > A NAN interface is like P2P device in a few aspects: it > doesn't have a netdev associated to it. > Add the new interface type and prevent operations that > can't be executed o

RE: [PATCHv3 RESEND 01/11] cfg80211: add start / stop NAN commands

2016-04-06 Thread Grumbach, Emmanuel
> On Tue, Mar 29, 2016 at 12:34:59PM +0300, Emmanuel Grumbach wrote: > > Define several attributes that may be configured by user space when > > starting NAN functionality (master preference and dual band operation) > > > diff --git a/include/uapi/linux/nl80211.h > > b/include/uapi/linux/nl80211

Re: [PATCHv3 RESEND 01/11] cfg80211: add start / stop NAN commands

2016-04-06 Thread Johannes Berg
> Why NL80211_ATTR_NAN_MASTER_PREF is mandatory? The spec suggests to > assume master preference value of 128 if not provided. Please see > quoted text from NAN 1.0 specification: > "A NAN Device which out-of-the-box will have a Master Preference > greater than or equal to 128 with the intention o

Re: [PATCHv3 RESEND 01/11] cfg80211: add start / stop NAN commands

2016-04-06 Thread Malinen, Jouni
On Tue, Mar 29, 2016 at 12:34:59PM +0300, Emmanuel Grumbach wrote: > Define several attributes that may be configured by user space > when starting NAN functionality (master preference and dual > band operation) > diff --git a/include/uapi/linux/nl80211.h b/include/uapi/linux/nl80211.h > @@ -826,6

Re: [PATCHv3 RESEND 01/11] cfg80211: add start / stop NAN commands

2016-04-06 Thread Johannes Berg
>  /** > + * struct cfg80211_nan_conf - nan configuration > + * > + * This struct defines nan configuration parameters I think you should consistently capitalize "NAN" in comments. > + * @start_nan: Start the NAN interface. > + * @stop_nan: Stop the NAN interface. johannes -- To unsubscribe from

[PATCHv3 RESEND 01/11] cfg80211: add start / stop NAN commands

2016-03-29 Thread Emmanuel Grumbach
This allows user space to start/stop NAN interface. A NAN interface is like P2P device in a few aspects: it doesn't have a netdev associated to it. Add the new interface type and prevent operations that can't be executed on NAN interface like scan. Define several attributes that may be configured b