Re: [PATCH 1/4] rtw88: pci: Change type of rtw_hw_queue_mapping() and ac_to_hwq to enum

From: Martin Blumenstingl
Date: Mon Jan 17 2022 - 12:48:46 EST


Hi Kalle,

On Mon, Jan 17, 2022 at 1:11 PM Kalle Valo <kvalo@xxxxxxxxxx> wrote:
[...]
> > -static u8 ac_to_hwq[] = {
> > +static enum rtw_tx_queue_type ac_to_hwq[] = {
> > [IEEE80211_AC_VO] = RTW_TX_QUEUE_VO,
> > [IEEE80211_AC_VI] = RTW_TX_QUEUE_VI,
> > [IEEE80211_AC_BE] = RTW_TX_QUEUE_BE,
>
> Shouldn't ac_to_hwq be static const?
Good point, thanks for this suggestion!
I will include that in v2 of this series (which I will send in a few
days so others can share their thoughts about these patches as well).


Best regards,
Martin