Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Toke =?utf-8?Q?H=C3=B8iland-J=C3=B8rgensen?= <toke@toke.dk>
To:     Alexander Wetzel <alexander@wetzel-home.de>,
        linux-wireless@vger.kernel.org
Cc:     johannes@sipsolutions.net,
        Alexander Wetzel <alexander@wetzel-home.de>
Subject: Re: [PATCH v2 1/3] wifi: mac80211: add internal handler for
 wake_tx_queue
In-Reply-To: <20221009163040.25637-1-alexander@wetzel-home.de>
References: <20221009163040.25637-1-alexander@wetzel-home.de>
Date:   Mon, 10 Oct 2022 16:44:36 +0200
Message-ID: <87y1tnrb57.fsf@toke.dk>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Alexander Wetzel <alexander@wetzel-home.de> writes:

> Start to align the TX handling to only use internal TX queues (iTXQs):
>
> Provide a handler for drivers not having a custom wake_tx_queue
> callback and update the documentation.
>
> Signed-off-by: Alexander Wetzel <alexander@wetzel-home.de>
> ---
>
> This is the v2 for "wifi: mac80211: Use internal TX queues for all driver=
s"
> https://patchwork.kernel.org/project/linux-wireless/patch/20220926161303.=
13035-1-alexander@wetzel-home.de/
>
> Changes compared to V1:
>  - broken out the new wake_tx_queue handler as exported function
>  - update drivers to provide wake_tx_queue instead handling it internally
>  - Minor docu updates
>  - Some sentences of the edited docu were referring to wrong netif
>    functions. Fixed that with the edit, too.
>  - added a comment explaining why we call ieee80211_next_txq()
>  - removed now pointless checks for sta.txq

I think this version looks quite reasonable. For the series:

Acked-by: Toke H=C3=B8iland-J=C3=B8rgensen <toke@toke.dk>