| Message ID | 20210514220644.1.Iad576de95836b74aba80a5fc28d7131940eca190@changeid (mailing list archive) |
|---|---|
| State | Awaiting Upstream |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | ath10k: remove unused variable in ath10k_htt_rx_h_frag_pn_check | expand |
| Context | Check | Description |
|---|---|---|
| netdev/tree_selection | success | Not a local patch |
Abhishek Kumar <kuabhs@chromium.org> wrote: > The local variable more_frags in ath10k_htt_rx_h_frag_pn_check is not > used. This patch is to remove that. > > Fixes: a1166b2653db ("ath10k: add CCMP PN replay protection for fragmented > frames for PCIe") > Signed-off-by: Abhishek Kumar <kuabhs@chromium.org> Fixed already by another identical commit: https://git.kernel.org/linus/e0a6120f6816 Patch set to Superseded.
diff --git a/drivers/net/wireless/ath/ath10k/htt_rx.c b/drivers/net/wireless/ath/ath10k/htt_rx.c index 7ffb5d5b2a70..adbaeb67eedf 100644 --- a/drivers/net/wireless/ath/ath10k/htt_rx.c +++ b/drivers/net/wireless/ath/ath10k/htt_rx.c @@ -1787,7 +1787,6 @@ static bool ath10k_htt_rx_h_frag_pn_check(struct ath10k *ar, struct ath10k_peer *peer; union htt_rx_pn_t *last_pn, new_pn = {0}; struct ieee80211_hdr *hdr; - bool more_frags; u8 tid, frag_number; u32 seq; @@ -1805,7 +1804,6 @@ static bool ath10k_htt_rx_h_frag_pn_check(struct ath10k *ar, last_pn = &peer->frag_tids_last_pn[tid]; new_pn.pn48 = ath10k_htt_rx_h_get_pn(ar, skb, offset, enctype); - more_frags = ieee80211_has_morefrags(hdr->frame_control); frag_number = le16_to_cpu(hdr->seq_ctrl) & IEEE80211_SCTL_FRAG; seq = (__le16_to_cpu(hdr->seq_ctrl) & IEEE80211_SCTL_SEQ) >> 4;
The local variable more_frags in ath10k_htt_rx_h_frag_pn_check is not used. This patch is to remove that. Fixes: a1166b2653db ("ath10k: add CCMP PN replay protection for fragmented frames for PCIe") Signed-off-by: Abhishek Kumar <kuabhs@chromium.org> --- drivers/net/wireless/ath/ath10k/htt_rx.c | 2 -- 1 file changed, 2 deletions(-)