ath11k: debugfs: move some function declarations to correct header files
authorKalle Valo <kvalo@codeaurora.org>
Wed, 16 Sep 2020 13:55:23 +0000 (16:55 +0300)
committerKalle Valo <kvalo@codeaurora.org>
Tue, 22 Sep 2020 07:41:49 +0000 (10:41 +0300)
Some of the function declarations are for functions in debugfs_htt_stats.c and
debugfs_sta.c, move them to corresponding header files. As debugfs_sta.h didn't
exist create it.

Also in debugfs_htt_stats.h move dunction declarations to the end of the file.

No functional changes. Compile tested only.

Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/1600264523-12939-4-git-send-email-kvalo@codeaurora.org
drivers/net/wireless/ath/ath11k/debugfs.h
drivers/net/wireless/ath/ath11k/debugfs_htt_stats.c
drivers/net/wireless/ath/ath11k/debugfs_htt_stats.h
drivers/net/wireless/ath/ath11k/debugfs_sta.c
drivers/net/wireless/ath/ath11k/debugfs_sta.h [new file with mode: 0644]
drivers/net/wireless/ath/ath11k/dp_rx.c
drivers/net/wireless/ath/ath11k/dp_tx.c
drivers/net/wireless/ath/ath11k/mac.c

index 4b7867c..d1ed2e7 100644 (file)
@@ -110,12 +110,9 @@ int ath11k_debugfs_pdev_create(struct ath11k_base *ab);
 void ath11k_debugfs_pdev_destroy(struct ath11k_base *ab);
 int ath11k_debugfs_register(struct ath11k *ar);
 void ath11k_debugfs_unregister(struct ath11k *ar);
-void ath11k_dbg_htt_ext_stats_handler(struct ath11k_base *ab,
-                                     struct sk_buff *skb);
 void ath11k_debugfs_fw_stats_process(struct ath11k_base *ab, struct sk_buff *skb);
 
 void ath11k_debugfs_fw_stats_init(struct ath11k *ar);
-int ath11k_dbg_htt_stats_req(struct ath11k *ar);
 
 static inline bool ath11k_debugfs_is_pktlog_lite_mode_enabled(struct ath11k *ar)
 {
@@ -148,15 +145,6 @@ static inline int ath11k_debugfs_rx_filter(struct ath11k *ar)
        return ar->debug.rx_filter;
 }
 
-void ath11k_sta_add_debugfs(struct ieee80211_hw *hw, struct ieee80211_vif *vif,
-                           struct ieee80211_sta *sta, struct dentry *dir);
-void
-ath11k_accumulate_per_peer_tx_stats(struct ath11k_sta *arsta,
-                                   struct ath11k_per_peer_tx_stats *peer_stats,
-                                   u8 legacy_rate_idx);
-void ath11k_update_per_peer_stats_from_txcompl(struct ath11k *ar,
-                                              struct sk_buff *msdu,
-                                              struct hal_tx_status *ts);
 #else
 static inline int ath11k_debugfs_soc_create(struct ath11k_base *ab)
 {
@@ -209,7 +197,7 @@ static inline int ath11k_debugfs_is_extd_rx_stats_enabled(struct ath11k *ar)
        return 0;
 }
 
-static inline int ath11k_dbg_htt_stats_req(struct ath11k *ar)
+static inline int ath11k_debugfs_htt_stats_req(struct ath11k *ar)
 {
        return 0;
 }
@@ -234,20 +222,6 @@ static inline int ath11k_debugfs_rx_filter(struct ath11k *ar)
        return 0;
 }
 
-static inline void
-ath11k_accumulate_per_peer_tx_stats(struct ath11k_sta *arsta,
-                                   struct ath11k_per_peer_tx_stats *peer_stats,
-                                   u8 legacy_rate_idx)
-{
-}
-
-static inline void
-ath11k_update_per_peer_stats_from_txcompl(struct ath11k *ar,
-                                         struct sk_buff *msdu,
-                                         struct hal_tx_status *ts)
-{
-}
-
 #endif /* CONFIG_MAC80211_DEBUGFS*/
 
 #endif /* _ATH11K_DEBUGFS_H_ */
index 6f6e8ca..9191ffa 100644 (file)
@@ -4253,8 +4253,8 @@ static int ath11k_dbg_htt_ext_stats_parse(struct ath11k_base *ab,
        return 0;
 }
 
-void ath11k_dbg_htt_ext_stats_handler(struct ath11k_base *ab,
-                                     struct sk_buff *skb)
+void ath11k_debugfs_htt_ext_stats_handler(struct ath11k_base *ab,
+                                         struct sk_buff *skb)
 {
        struct ath11k_htt_extd_stats_msg *msg;
        struct debug_htt_stats_req *stats_req;
@@ -4402,7 +4402,7 @@ static int ath11k_prep_htt_stats_cfg_params(struct ath11k *ar, u8 type,
        return 0;
 }
 
-int ath11k_dbg_htt_stats_req(struct ath11k *ar)
+int ath11k_debugfs_htt_stats_req(struct ath11k *ar)
 {
        struct debug_htt_stats_req *stats_req = ar->debug.htt_stats.stats_req;
        u8 type = stats_req->type;
@@ -4476,7 +4476,7 @@ static int ath11k_open_htt_stats(struct inode *inode, struct file *file)
        ar->debug.htt_stats.stats_req = stats_req;
        stats_req->type = type;
 
-       ret = ath11k_dbg_htt_stats_req(ar);
+       ret = ath11k_debugfs_htt_stats_req(ar);
        if (ret < 0)
                goto out;
 
index d2d01bf..a36a4fe 100644 (file)
@@ -1660,8 +1660,6 @@ struct htt_pdev_obss_pd_stats_tlv {
        u32        num_obss_tx_ppdu_failure;
 };
 
-void ath11k_debugfs_htt_stats_init(struct ath11k *ar);
-
 struct htt_ring_backpressure_stats_tlv {
        u32 pdev_id;
        u32 current_head_idx;
@@ -1687,4 +1685,9 @@ struct htt_ring_backpressure_stats_tlv {
        u32 backpressure_hist[5];
 };
 
+void ath11k_debugfs_htt_stats_init(struct ath11k *ar);
+void ath11k_debugfs_htt_ext_stats_handler(struct ath11k_base *ab,
+                                         struct sk_buff *skb);
+int ath11k_debugfs_htt_stats_req(struct ath11k *ar);
+
 #endif
index 14f2e90..270c0ed 100644 (file)
@@ -5,16 +5,16 @@
 
 #include <linux/vmalloc.h>
 
+#include "debugfs_sta.h"
 #include "core.h"
 #include "peer.h"
 #include "debug.h"
 #include "dp_tx.h"
 #include "debugfs_htt_stats.h"
 
-void
-ath11k_accumulate_per_peer_tx_stats(struct ath11k_sta *arsta,
-                                   struct ath11k_per_peer_tx_stats *peer_stats,
-                                   u8 legacy_rate_idx)
+void ath11k_debugfs_sta_add_tx_stats(struct ath11k_sta *arsta,
+                                    struct ath11k_per_peer_tx_stats *peer_stats,
+                                    u8 legacy_rate_idx)
 {
        struct rate_info *txrate = &arsta->txrate;
        struct ath11k_htt_tx_stats *tx_stats;
@@ -125,9 +125,9 @@ ath11k_accumulate_per_peer_tx_stats(struct ath11k_sta *arsta,
        tx_stats->tx_duration += peer_stats->duration;
 }
 
-void ath11k_update_per_peer_stats_from_txcompl(struct ath11k *ar,
-                                              struct sk_buff *msdu,
-                                              struct hal_tx_status *ts)
+void ath11k_debugfs_sta_update_txcompl(struct ath11k *ar,
+                                      struct sk_buff *msdu,
+                                      struct hal_tx_status *ts)
 {
        struct ath11k_base *ab = ar->ab;
        struct ath11k_per_peer_tx_stats *peer_stats = &ar->cached_stats;
@@ -200,7 +200,8 @@ void ath11k_update_per_peer_stats_from_txcompl(struct ath11k *ar,
        arsta->txrate.nss = arsta->last_txrate.nss;
        arsta->txrate.bw = ath11k_mac_bw_to_mac80211_bw(bw);
 
-       ath11k_accumulate_per_peer_tx_stats(arsta, peer_stats, rate_idx);
+       ath11k_debugfs_sta_add_tx_stats(arsta, peer_stats, rate_idx);
+
 err_out:
        spin_unlock_bh(&ab->base_lock);
        rcu_read_unlock();
@@ -428,7 +429,7 @@ ath11k_dbg_sta_open_htt_peer_stats(struct inode *inode, struct file *file)
        ar->debug.htt_stats.stats_req = stats_req;
        stats_req->type = ATH11K_DBG_HTT_EXT_STATS_PEER_INFO;
        memcpy(stats_req->peer_addr, sta->addr, ETH_ALEN);
-       ret = ath11k_dbg_htt_stats_req(ar);
+       ret = ath11k_debugfs_htt_stats_req(ar);
        mutex_unlock(&ar->conf_mutex);
        if (ret < 0)
                goto out;
@@ -820,8 +821,8 @@ static const struct file_operations fops_htt_peer_stats_reset = {
        .llseek = default_llseek,
 };
 
-void ath11k_sta_add_debugfs(struct ieee80211_hw *hw, struct ieee80211_vif *vif,
-                           struct ieee80211_sta *sta, struct dentry *dir)
+void ath11k_debugfs_sta_op_add(struct ieee80211_hw *hw, struct ieee80211_vif *vif,
+                              struct ieee80211_sta *sta, struct dentry *dir)
 {
        struct ath11k *ar = hw->priv;
 
diff --git a/drivers/net/wireless/ath/ath11k/debugfs_sta.h b/drivers/net/wireless/ath/ath11k/debugfs_sta.h
new file mode 100644 (file)
index 0000000..18dc65d
--- /dev/null
@@ -0,0 +1,44 @@
+/* SPDX-License-Identifier: BSD-3-Clause-Clear */
+/*
+ * Copyright (c) 2018-2020 The Linux Foundation. All rights reserved.
+ */
+
+#ifndef _ATH11K_DEBUGFS_STA_H_
+#define _ATH11K_DEBUGFS_STA_H_
+
+#include <net/mac80211.h>
+
+#include "core.h"
+#include "hal_tx.h"
+
+#ifdef CONFIG_ATH11K_DEBUGFS
+
+void ath11k_debugfs_sta_op_add(struct ieee80211_hw *hw, struct ieee80211_vif *vif,
+                              struct ieee80211_sta *sta, struct dentry *dir);
+void ath11k_debugfs_sta_add_tx_stats(struct ath11k_sta *arsta,
+                                    struct ath11k_per_peer_tx_stats *peer_stats,
+                                    u8 legacy_rate_idx);
+void ath11k_debugfs_sta_update_txcompl(struct ath11k *ar,
+                                      struct sk_buff *msdu,
+                                      struct hal_tx_status *ts);
+
+#else /* CONFIG_ATH11K_DEBUGFS */
+
+#define ath11k_debugfs_sta_op_add NULL
+
+static inline void
+ath11k_debugfs_sta_add_tx_stats(struct ath11k_sta *arsta,
+                               struct ath11k_per_peer_tx_stats *peer_stats,
+                               u8 legacy_rate_idx)
+{
+}
+
+static inline void ath11k_debugfs_sta_update_txcompl(struct ath11k *ar,
+                                                    struct sk_buff *msdu,
+                                                    struct hal_tx_status *ts)
+{
+}
+
+#endif /* CONFIG_ATH11K_DEBUGFS */
+
+#endif /* _ATH11K_DEBUGFS_STA_H_ */
index 2c9251b..345eaa4 100644 (file)
@@ -9,6 +9,8 @@
 #include <crypto/hash.h>
 #include "core.h"
 #include "debug.h"
+#include "debugfs_htt_stats.h"
+#include "debugfs_sta.h"
 #include "hal_desc.h"
 #include "hw.h"
 #include "dp_rx.h"
@@ -1434,8 +1436,7 @@ ath11k_update_per_peer_tx_stats(struct ath11k *ar,
                        HTT_USR_CMPLTN_SHORT_RETRY(usr_stats->cmpltn_cmn.flags);
 
                if (ath11k_debugfs_is_extd_tx_stats_enabled(ar))
-                       ath11k_accumulate_per_peer_tx_stats(arsta,
-                                                           peer_stats, rate_idx);
+                       ath11k_debugfs_sta_add_tx_stats(arsta, peer_stats, rate_idx);
        }
 
        spin_unlock_bh(&ab->base_lock);
@@ -1658,7 +1659,7 @@ void ath11k_dp_htt_htc_t2h_msg_handler(struct ath11k_base *ab,
                ath11k_htt_pull_ppdu_stats(ab, skb);
                break;
        case HTT_T2H_MSG_TYPE_EXT_STATS_CONF:
-               ath11k_dbg_htt_ext_stats_handler(ab, skb);
+               ath11k_debugfs_htt_ext_stats_handler(ab, skb);
                break;
        case HTT_T2H_MSG_TYPE_PKTLOG:
                ath11k_htt_pktlog(ab, skb);
index 7975cc0..8625465 100644 (file)
@@ -6,6 +6,7 @@
 #include "core.h"
 #include "dp_tx.h"
 #include "debug.h"
+#include "debugfs_sta.h"
 #include "hw.h"
 #include "peer.h"
 
@@ -465,12 +466,12 @@ static void ath11k_dp_tx_complete_msdu(struct ath11k *ar,
                                   ar->cached_ppdu_id == ar->last_ppdu_id) {
                                ar->cached_ppdu_id = ar->last_ppdu_id;
                                ar->cached_stats.is_ampdu = true;
-                               ath11k_update_per_peer_stats_from_txcompl(ar, msdu, ts);
+                               ath11k_debugfs_sta_update_txcompl(ar, msdu, ts);
                                memset(&ar->cached_stats, 0,
                                       sizeof(struct ath11k_per_peer_tx_stats));
                        } else {
                                ar->cached_stats.is_ampdu = false;
-                               ath11k_update_per_peer_stats_from_txcompl(ar, msdu, ts);
+                               ath11k_debugfs_sta_update_txcompl(ar, msdu, ts);
                                memset(&ar->cached_stats, 0,
                                       sizeof(struct ath11k_per_peer_tx_stats));
                        }
index 6962116..86a1b98 100644 (file)
@@ -14,6 +14,7 @@
 #include "dp_rx.h"
 #include "testmode.h"
 #include "peer.h"
+#include "debugfs_sta.h"
 
 #define CHAN2G(_channel, _freq, _flags) { \
        .band                   = NL80211_BAND_2GHZ, \
@@ -5867,7 +5868,7 @@ static const struct ieee80211_ops ath11k_ops = {
        .sta_statistics                 = ath11k_mac_op_sta_statistics,
        CFG80211_TESTMODE_CMD(ath11k_tm_cmd)
 #ifdef CONFIG_ATH11K_DEBUGFS
-       .sta_add_debugfs                = ath11k_sta_add_debugfs,
+       .sta_add_debugfs                = ath11k_debugfs_sta_op_add,
 #endif
 };