> 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:johan...@sipsolutions.net] Sent: Thursday, September 15, 2016 2:34 PM To: Undekari, Sunil Dutt <usd...@qti.qualcomm.com>; Kushwaha, Purushottam <pkush...@qti.qualcomm.com> Cc: linux-wireless@vger.kernel.org; Malinen, Jouni <jo...@qca.qualcomm.com>; Hullur Subramanyam, Amarnath <amarn...@qca.qualcomm.com> 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