> 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. Regards, Sunil -----Original Message----- From: Johannes Berg [mailto:johannes@xxxxxxxxxxxxxxxx] Sent: Thursday, September 15, 2016 2:19 PM To: Kushwaha, Purushottam <pkushwah@xxxxxxxxxxxxxxxx> Cc: linux-wireless@xxxxxxxxxxxxxxx; Malinen, Jouni <jouni@xxxxxxxxxxxxxxxx>; Undekari, Sunil Dutt <usdutt@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 10:17 +0200, Johannes Berg wrote: > On Wed, 2016-09-14 at 17:38 +0530, Purushottam Kushwaha wrote: > > > > This allows an option to configure a single beacon tx rate (u8) for > > an AP. > > > Applied. > Actually. Moved to the pending branch - shouldn't we have some feature flags here to indicate that the driver actually supports this? johannes