Thread (22 messages) flat view 22 messages, 4 authors, 2018-03-29

Re: [RFC 0/4] wireless: Per-sta NoAck and offload support

From: Johannes Berg <johannes@sipsolutions.net>
Date: 2018-03-28 08:03:35

On Wed, 2018-03-28 at 13:29 +0530, vthiagar@codeaurora.org wrote:
On 2018-03-28 13:07, Arend van Spriel wrote:
quoted
On 3/28/2018 8:09 AM, vthiagar@codeaurora.org wrote:
quoted
On 2018-03-27 22:18, Steve deRosier wrote:
quoted
Hi Vasanthakumar,

On Tue, Mar 27, 2018 at 1:42 AM, Vasanthakumar Thiagarajan
[off-list ref] wrote:
quoted
Adds infrastructure for driver to offload NoAck functionality, 
driver
like ath10k could make use of it. Also extends the current ndev wide
I'm not really much of a fan of adding a feature without some use of
the feature. Perhaps if drivers "like" ath10k could use it, maybe you
should add a patch(s) to the series where one of those drivers
actually uses the feature.  An API without an example of use is also
harder to evaluate effectively.
I agree driver patch using the new NoAck infrastructure would help 
with
understanding, ill post it once it is ready. But not sure the driver 
patch
can be part of the same series.
The API patches would go in mac80211-next tree and indeed the driver
patch would go through wireless-drivers-next tree. However, an option
would be to add the driver patch(es) as RFC in the series so Johannes
can ignore it and we still have an example to look at.
FWIW, you can just include it as a regular [PATCH] - Kalle and I have a
way of dealing with that - he just assigns it over to me initially, and
I assign back once the prerequisites have landed.

johannes
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help