> I'm even willing to add it myself - should we just have three feature flags? I guess , Yes. Regards, Sunil -----Original Message----- From: Johannes Berg [mailto:johannes@xxxxxxxxxxxxxxxx] Sent: Thursday, September 15, 2016 2:34 PM To: Undekari, Sunil Dutt <usdutt@xxxxxxxxxxxxxxxx>; Kushwaha, Purushottam <pkushwah@xxxxxxxxxxxxxxxx> Cc: linux-wireless@xxxxxxxxxxxxxxx; Malinen, Jouni <jouni@xxxxxxxxxxxxxxxx>; Hullur Subramanyam, Amarnath <amarnath@xxxxxxxxxxxxxxxx> Subject: Re: [PATCH v5] cfg80211: Add support to configure a beacon data rate On Thu, 2016-09-15 at 09:00 +0000, Undekari, Sunil Dutt wrote: > > > > shouldn't we have some feature flags here to indicate that the > > driver actually supports this? > Yes. Can you please accept this with a new commit. > I'm even willing to add it myself - should we just have three feature flags? AP_BEACON_RATE_LEGACY AP_BEACON_RATE_HT AP_BEACON_RATE_VHT ? johannes
