On Fri, 2016-06-10 at 11:43 -0700, Ben Greear wrote: > On 03/15/2016 01:20 PM, Johannes Berg wrote: > > On Tue, 2016-03-15 at 09:10 -0700, Ben Greear wrote: > > > > > > The logic I wrote is basically exactly this. It uses the > > > configured > > > rates to specify which of the hardware's rates are allowed and > > > disabled. > > > > > > > I understand that. I just take issue with the fact that we have to > > sprinkle "magic pixie dust" (in form of the override function > > calls) > > everywhere throughout the code. > > > > johannes > > > > So, I never could think up a way to make this more acceptable and > meet my own goals. > > Any chance I could get just the netlink enum in place, even if it is > called something like 'api_placeholder_x'? > No, I don't think I want to do that. But if you're not too worried about space consumption, you could pick a higher number: /* add attributes here, update the policy in nl80211.c */ NL80211_ATTR_TX_ADVERT_RATEMASK = 300, __NL80211_ATTR_AFTER_LAST, We currently use about 230 attributes, so that'll give you lots of breathing room. 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
