[PATCH v1] cfg80211/nl80211: Add support for NL80211_STA_INFO_RX_DURATION

Mohammed Shafi Shajakhan mohammed at codeaurora.org
Tue Mar 8 22:50:15 PST 2016


On Wed, Mar 09, 2016 at 12:09:15PM +0530, Mohammed Shafi Shajakhan wrote:
> Hi Johannes,
> 
> On Tue, Mar 08, 2016 at 05:52:23PM +0100, Johannes Berg wrote:
> > On Mon, 2016-03-07 at 16:33 +0530, Mohammed Shafi Shajakhan wrote:
> > > From: Mohammed Shafi Shajakhan <mohammed at qti.qualcomm.com>
> > > 
> > > Add support for new netlink attribute 'NL80211_STA_INFO_RX_DURATION'
> > > This flag will be set when drivers can fill rx_duration (vendor
> > > specific total air time(usecs) for data/management frames from the
> > > connected client) via 'drv_sta_statistics' callback
> > > 
> > > Also make sta_info flags 'filled' as 64 bit to accommodate for new
> > > per station stats. Extend 'PUT_SINFO' for supporting rx_duration
> > > field and any new per sta information in future
> > > 
> > > Signed-off-by: Mohammed Shafi Shajakhan <mohammed at qti.qualcomm.com>
> > > ---
> > > [v1] 
> > > 1. Fixed typo for accommodate
> > > 2. changed the doc for rx_duration as 'vendor specific
> > > implementation'
> > 
> > No, I want it defined for all cases, not left undefined.
> 
> [shafi] Let me know if the below documentation is fine with you please
> 
> "@rx_duration: approximate total air time(usecs) for all data/management
>     frames from the connected client "

[shafi] correction (made it all frames (not only data/management frames)

 "@rx_duration: approximate total air time(usecs) for all frames
		from the connected client "

> 
> thanks for reviewing this and Jouni for his initial suggestions.
> 
> regards,
> shafi
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html



More information about the ath10k mailing list