wifi: ath11k: Relocate the func ath11k_mac_bitrate_mask_num_ht_rates() and change...
authorMaharaja Kennadyrajan <quic_mkenna@quicinc.com>
Tue, 9 May 2023 17:07:23 +0000 (20:07 +0300)
committerKalle Valo <quic_kvalo@quicinc.com>
Wed, 17 May 2023 08:18:42 +0000 (11:18 +0300)
Relocate the function ath11k_mac_bitrate_mask_num_ht_rates() definition
to call this function from other functions which helps to avoid the
compilation error (function not defined).

ht_mcs[] is 1 byte array and it is enough to use hweight8() instead
of hweight16(). Hence, fixed the same.

Tested on: Compile tested only.

Signed-off-by: Maharaja Kennadyrajan <quic_mkenna@quicinc.com>
Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
Link: https://lore.kernel.org/r/20230504092033.3542456-2-quic_mkenna@quicinc.com
drivers/net/wireless/ath/ath11k/mac.c

index a164b57dcf30d6fa29d5f6a1564cf3c4a775a63b..33620a7a444b75445f6560e9fd1aacb4f9e27387 100644 (file)
@@ -1,7 +1,7 @@
 // SPDX-License-Identifier: BSD-3-Clause-Clear
 /*
  * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved.
- * Copyright (c) 2021-2022 Qualcomm Innovation Center, Inc. All rights reserved.
+ * Copyright (c) 2021-2023 Qualcomm Innovation Center, Inc. All rights reserved.
  */
 
 #include <net/mac80211.h>
@@ -4337,6 +4337,20 @@ exit:
        return ret;
 }
 
+static int
+ath11k_mac_bitrate_mask_num_ht_rates(struct ath11k *ar,
+                                    enum nl80211_band band,
+                                    const struct cfg80211_bitrate_mask *mask)
+{
+       int num_rates = 0;
+       int i;
+
+       for (i = 0; i < ARRAY_SIZE(mask->control[band].ht_mcs); i++)
+               num_rates += hweight8(mask->control[band].ht_mcs[i]);
+
+       return num_rates;
+}
+
 static int
 ath11k_mac_bitrate_mask_num_vht_rates(struct ath11k *ar,
                                      enum nl80211_band band,
@@ -7791,20 +7805,6 @@ static void ath11k_mac_op_flush(struct ieee80211_hw *hw, struct ieee80211_vif *v
        ath11k_mac_flush_tx_complete(ar);
 }
 
-static int
-ath11k_mac_bitrate_mask_num_ht_rates(struct ath11k *ar,
-                                    enum nl80211_band band,
-                                    const struct cfg80211_bitrate_mask *mask)
-{
-       int num_rates = 0;
-       int i;
-
-       for (i = 0; i < ARRAY_SIZE(mask->control[band].ht_mcs); i++)
-               num_rates += hweight16(mask->control[band].ht_mcs[i]);
-
-       return num_rates;
-}
-
 static bool
 ath11k_mac_has_single_legacy_rate(struct ath11k *ar,
                                  enum nl80211_band band,