DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 634FE20840
Message-ID: <218bac09a559cd0d1c6cdc5d9c4dbd4b33ffca1c.camel@sipsolutions.net>
Subject: Re: [PATCH 0/6] wireless: Per-sta NoAck and offload support
From:   Johannes Berg <johannes@sipsolutions.net>
To:     Tamizh chelvam <tamizhr@codeaurora.org>, ath10k@lists.infradead.org
Cc:     linux-wireless@vger.kernel.org
Date:   Fri, 09 Nov 2018 12:28:01 +0100
In-Reply-To: <1539715271-14325-1-git-send-email-tamizhr@codeaurora.org> (sfid-20181016_204254_690922_6A9D8E64)
References: <1539715271-14325-1-git-send-email-tamizhr@codeaurora.org>
         (sfid-20181016_204254_690922_6A9D8E64)
Content-Type: text/plain; charset="UTF-8"
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-wireless-owner@vger.kernel.org
Precedence: bulk

Hi,

> Adds infrastructure for driver to offload NoAck functionality,
> driver like ath10k could make use of it. Also extends the
> current ndev wide NoAck policy to per-station, with sta level
> NoAck policy configuration userspace could selectively turn off/on
> Noack based on various connection parameters of the station.

So ... we have this, but we also have

[PATCH 0/4] cfg80211/mac80211: Add support for TID specific configuration

which you were involved in.

Unless you can cite a good reason not to, I think it would make sense to
combine the two and make "noack" just another attribute in the TID
configuration.

johannes