On Wed, 2016-04-06 at 09:38 +0000, Malinen, Jouni wrote: > > > +void cfg80211_nan_match(struct wireless_dev *wdev, > > + struct cfg80211_nan_match_params *match, > > gfp_t gfp); > Looks like this function is common for publish, subscribe and follow > up. If so, can we please have separate functions for publish, > subscribe and followup to match the spec primitives? Is it really worth exporting three functions just for the sake of that? Perhaps better if we provide inline wrappers though that set the value correctly? Would you extend this request also to the nl80211 command value? 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