2022-04-21 09:17:35

by Felix Fietkau

[permalink] [raw]
Subject: Re: [PATCH] mt76: mt7915: disable RX_HDR_TRANS_SHORT


On 20.04.22 09:11, Bo Jiao wrote:
> From: Bo Jiao <[email protected]>
>
> This patch disables RX_TRANS_SHORT to make MDP to do header translation
> when payload less than 8 bytes, hence the (QoS) null data can be encapsulated
> to 802.3 format. However, WDS requires (QoS) null data in 802.11 format
> to created vlan AP interfaces.
>
> Signed-off-by: Ryder Lee <[email protected]>
> Signed-off-by: lian.chen <[email protected]>
> Signed-off-by: Sujuan Chen <[email protected]>
> Signed-off-by: Bo Jiao <[email protected]>
> ---
> drivers/net/wireless/mediatek/mt76/agg-rx.c | 3 +++
> drivers/net/wireless/mediatek/mt76/mt7915/init.c | 3 +++
> drivers/net/wireless/mediatek/mt76/mt7915/mac.c | 4 ----
> drivers/net/wireless/mediatek/mt76/mt7915/regs.h | 3 +++
> 4 files changed, 9 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/net/wireless/mediatek/mt76/agg-rx.c b/drivers/net/wireless/mediatek/mt76/agg-rx.c
> index 7262222..efd1a85 100644
> --- a/drivers/net/wireless/mediatek/mt76/agg-rx.c
> +++ b/drivers/net/wireless/mediatek/mt76/agg-rx.c
> @@ -167,6 +167,9 @@ void mt76_rx_aggr_reorder(struct sk_buff *skb, struct sk_buff_head *frames)
> return;
> }
>
> + if (!status->aggr)
> + return;
> +
> /* not part of a BA session */
> ackp = status->qos_ctl & IEEE80211_QOS_CTL_ACK_POLICY_MASK;
> if (ackp != IEEE80211_QOS_CTL_ACK_POLICY_BLOCKACK &&
This is an unrelated change, for which I've just sent a replacement
patch. The rest is applied.

Thanks,

- Felix