ath11k: Remove some duplicate code
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Fri, 25 Jun 2021 19:38:00 +0000 (22:38 +0300)
committerKalle Valo <kvalo@codeaurora.org>
Fri, 2 Jul 2021 11:40:24 +0000 (14:40 +0300)
'ATH11K_HE_MCS_MAX' is 11, so these 2 blocks of code are exactly the same.
Remove the one that uses a hard-coded constant.

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/a65952db7f4eb8aaaa654b77dcd4930482f5c49b.1624483438.git.christophe.jaillet@wanadoo.fr
drivers/net/wireless/ath/ath11k/dp_rx.c

index 603d2f9..9a22481 100644 (file)
@@ -1406,11 +1406,6 @@ ath11k_update_per_peer_tx_stats(struct ath11k *ar,
         * Firmware rate's control to be skipped for this?
         */
 
-       if (flags == WMI_RATE_PREAMBLE_HE && mcs > 11) {
-               ath11k_warn(ab, "Invalid HE mcs %d peer stats",  mcs);
-               return;
-       }
-
        if (flags == WMI_RATE_PREAMBLE_HE && mcs > ATH11K_HE_MCS_MAX) {
                ath11k_warn(ab, "Invalid HE mcs %d peer stats",  mcs);
                return;