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

Reply via email to