2020-02-04 15:07:05

by John Crispin

[permalink] [raw]
Subject: [PATCH] mac80211: add HE support to ieee80211_calculate_rx_timestamp

From: Miles Hu <[email protected]>

The RX timestamp was not calculated for HE yet. This patch adds the
missing case clause.

Signed-off-by: Miles Hu <[email protected]>
Tested-by: John Crispin <[email protected]>
---
net/mac80211/util.c | 7 +++++++
1 file changed, 7 insertions(+)

diff --git a/net/mac80211/util.c b/net/mac80211/util.c
index 780df3e9092e..876f2d92347b 100644
--- a/net/mac80211/util.c
+++ b/net/mac80211/util.c
@@ -3242,6 +3242,13 @@ u64 ieee80211_calculate_rx_timestamp(struct ieee80211_local *local,
if (status->enc_flags & RX_ENC_FLAG_SHORT_GI)
ri.flags |= RATE_INFO_FLAGS_SHORT_GI;
break;
+ case RX_ENC_HE:
+ ri.flags |= RATE_INFO_FLAGS_HE_MCS;
+ ri.mcs = status->rate_idx;
+ ri.nss = status->nss;
+ if (status->enc_flags & RX_ENC_FLAG_SHORT_GI)
+ ri.flags |= RATE_INFO_FLAGS_SHORT_GI;
+ break;
default:
WARN_ON(1);
/* fall through */
--
2.20.1


2020-02-10 13:46:27

by Kalle Valo

[permalink] [raw]
Subject: Re: [PATCH] mac80211: add HE support to ieee80211_calculate_rx_timestamp

John Crispin <[email protected]> writes:

> From: Miles Hu <[email protected]>
>
> The RX timestamp was not calculated for HE yet. This patch adds the
> missing case clause.
>
> Signed-off-by: Miles Hu <[email protected]>
> Tested-by: John Crispin <[email protected]>

John's s-o-b missing.

--
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches