mt76: connac: move mt76_connac_mcu_rdd_cmd in mt76-connac module
authorLorenzo Bianconi <lorenzo@kernel.org>
Fri, 24 Dec 2021 10:58:11 +0000 (11:58 +0100)
committerFelix Fietkau <nbd@nbd.name>
Thu, 3 Feb 2022 12:57:58 +0000 (13:57 +0100)
Move mt76_connac_mcu_rdd_cmd routine in mt76-connac module and remove
duplicated code.

Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org>
Signed-off-by: Felix Fietkau <nbd@nbd.name>
drivers/net/wireless/mediatek/mt76/mt7615/mac.c
drivers/net/wireless/mediatek/mt76/mt7615/mcu.c
drivers/net/wireless/mediatek/mt76/mt7615/mt7615.h
drivers/net/wireless/mediatek/mt76/mt76_connac_mcu.c
drivers/net/wireless/mediatek/mt76/mt76_connac_mcu.h
drivers/net/wireless/mediatek/mt76/mt7915/debugfs.c
drivers/net/wireless/mediatek/mt76/mt7915/mac.c
drivers/net/wireless/mediatek/mt76/mt7915/mcu.c
drivers/net/wireless/mediatek/mt76/mt7915/mt7915.h

index ec25e5a95d442c3da6c08aca58795b1b0a767ca6..8f8a7bc0169b4b2db64d4d100c7087938cd190d3 100644 (file)
@@ -2160,21 +2160,24 @@ static void mt7615_dfs_stop_radar_detector(struct mt7615_phy *phy)
        struct mt7615_dev *dev = phy->dev;
 
        if (phy->rdd_state & BIT(0))
-               mt7615_mcu_rdd_cmd(dev, RDD_STOP, 0, MT_RX_SEL0, 0);
+               mt76_connac_mcu_rdd_cmd(&dev->mt76, RDD_STOP, 0,
+                                       MT_RX_SEL0, 0);
        if (phy->rdd_state & BIT(1))
-               mt7615_mcu_rdd_cmd(dev, RDD_STOP, 1, MT_RX_SEL0, 0);
+               mt76_connac_mcu_rdd_cmd(&dev->mt76, RDD_STOP, 1,
+                                       MT_RX_SEL0, 0);
 }
 
 static int mt7615_dfs_start_rdd(struct mt7615_dev *dev, int chain)
 {
        int err;
 
-       err = mt7615_mcu_rdd_cmd(dev, RDD_START, chain, MT_RX_SEL0, 0);
+       err = mt76_connac_mcu_rdd_cmd(&dev->mt76, RDD_START, chain,
+                                     MT_RX_SEL0, 0);
        if (err < 0)
                return err;
 
-       return mt7615_mcu_rdd_cmd(dev, RDD_DET_MODE, chain,
-                                 MT_RX_SEL0, 1);
+       return mt76_connac_mcu_rdd_cmd(&dev->mt76, RDD_DET_MODE, chain,
+                                      MT_RX_SEL0, 1);
 }
 
 static int mt7615_dfs_start_radar_detector(struct mt7615_phy *phy)
@@ -2185,7 +2188,8 @@ static int mt7615_dfs_start_radar_detector(struct mt7615_phy *phy)
        int err;
 
        /* start CAC */
-       err = mt7615_mcu_rdd_cmd(dev, RDD_CAC_START, ext_phy, MT_RX_SEL0, 0);
+       err = mt76_connac_mcu_rdd_cmd(&dev->mt76, RDD_CAC_START, ext_phy,
+                                     MT_RX_SEL0, 0);
        if (err < 0)
                return err;
 
@@ -2280,12 +2284,13 @@ int mt7615_dfs_init_radar_detector(struct mt7615_phy *phy)
                if (chandef->chan->dfs_state != NL80211_DFS_AVAILABLE)
                        return mt7615_dfs_start_radar_detector(phy);
 
-               return mt7615_mcu_rdd_cmd(dev, RDD_CAC_END, ext_phy,
-                                         MT_RX_SEL0, 0);
+               return mt76_connac_mcu_rdd_cmd(&dev->mt76, RDD_CAC_END, ext_phy,
+                                              MT_RX_SEL0, 0);
        }
 
 stop:
-       err = mt7615_mcu_rdd_cmd(dev, RDD_NORMAL_START, ext_phy, MT_RX_SEL0, 0);
+       err = mt76_connac_mcu_rdd_cmd(&dev->mt76, RDD_NORMAL_START, ext_phy,
+                                     MT_RX_SEL0, 0);
        if (err < 0)
                return err;
 
index 0a914494bb32fe378510a5d076b4af7ef023f2ef..f992e1285eaa3b2f813faa6f5afb0aca183d1908 100644 (file)
@@ -1891,27 +1891,6 @@ int mt7615_mcu_del_wtbl_all(struct mt7615_dev *dev)
                                 &req, sizeof(req), true);
 }
 
-int mt7615_mcu_rdd_cmd(struct mt7615_dev *dev,
-                      enum mt7615_rdd_cmd cmd, u8 index,
-                      u8 rx_sel, u8 val)
-{
-       struct {
-               u8 ctrl;
-               u8 rdd_idx;
-               u8 rdd_rx_sel;
-               u8 val;
-               u8 rsv[4];
-       } req = {
-               .ctrl = cmd,
-               .rdd_idx = index,
-               .rdd_rx_sel = rx_sel,
-               .val = val,
-       };
-
-       return mt76_mcu_send_msg(&dev->mt76, MCU_EXT_CMD(SET_RDD_CTRL),
-                                &req, sizeof(req), true);
-}
-
 int mt7615_mcu_set_fcc5_lpn(struct mt7615_dev *dev, int val)
 {
        struct {
index 07b833654920e958e4e31830f1844cc25042f4c9..3b66aa749a215081322a2171412673f52b2c3754 100644 (file)
@@ -403,9 +403,6 @@ int mt7615_mcu_set_chan_info(struct mt7615_phy *phy, int cmd);
 int mt7615_mcu_set_wmm(struct mt7615_dev *dev, u8 queue,
                       const struct ieee80211_tx_queue_params *params);
 void mt7615_mcu_rx_event(struct mt7615_dev *dev, struct sk_buff *skb);
-int mt7615_mcu_rdd_cmd(struct mt7615_dev *dev,
-                      enum mt7615_rdd_cmd cmd, u8 index,
-                      u8 rx_sel, u8 val);
 int mt7615_mcu_rdd_send_pattern(struct mt7615_dev *dev);
 int mt7615_mcu_fw_log_2_host(struct mt7615_dev *dev, u8 ctrl);
 
index 2b647c19521bb445e8fcfb16b2b50dae908fb050..1b2340e4ce0cd0d89501cd09dd0a3a266159c326 100644 (file)
@@ -2764,5 +2764,26 @@ int mt76_connac_mcu_restart(struct mt76_dev *dev)
 }
 EXPORT_SYMBOL_GPL(mt76_connac_mcu_restart);
 
+int mt76_connac_mcu_rdd_cmd(struct mt76_dev *dev, int cmd, u8 index,
+                           u8 rx_sel, u8 val)
+{
+       struct {
+               u8 ctrl;
+               u8 rdd_idx;
+               u8 rdd_rx_sel;
+               u8 val;
+               u8 rsv[4];
+       } __packed req = {
+               .ctrl = cmd,
+               .rdd_idx = index,
+               .rdd_rx_sel = rx_sel,
+               .val = val,
+       };
+
+       return mt76_mcu_send_msg(dev, MCU_EXT_CMD(SET_RDD_CTRL), &req,
+                                sizeof(req), true);
+}
+EXPORT_SYMBOL_GPL(mt76_connac_mcu_rdd_cmd);
+
 MODULE_AUTHOR("Lorenzo Bianconi <lorenzo@kernel.org>");
 MODULE_LICENSE("Dual BSD/GPL");
index bce3134e36a13563a0394624e558b9122cb95a6b..9277a6a2c95e71492a41d495dbde88a1645ab9ab 100644 (file)
@@ -1649,4 +1649,6 @@ void mt76_connac_mcu_wtbl_smps_tlv(struct sk_buff *skb,
                                   void *sta_wtbl, void *wtbl_tlv);
 int mt76_connac_mcu_set_pm(struct mt76_dev *dev, int band, int enter);
 int mt76_connac_mcu_restart(struct mt76_dev *dev);
+int mt76_connac_mcu_rdd_cmd(struct mt76_dev *dev, int cmd, u8 index,
+                           u8 rx_sel, u8 val);
 #endif /* __MT76_CONNAC_MCU_H */
index 710f8cad624564689d592e2c287b3e9c3f837f20..c59ef08a5306e7249c4936385ad37545cdc240c2 100644 (file)
@@ -75,7 +75,7 @@ mt7915_radar_trigger(void *data, u64 val)
 {
        struct mt7915_dev *dev = data;
 
-       return mt7915_mcu_rdd_cmd(dev, RDD_RADAR_EMULATE, 1, 0, 0);
+       return mt76_connac_mcu_rdd_cmd(&dev->mt76, RDD_RADAR_EMULATE, 1, 0, 0);
 }
 
 DEFINE_DEBUGFS_ATTRIBUTE(fops_radar_trigger, NULL,
index d83e828b6e3e1e16a7abaf16deb91b6dcee6db40..e0200f84a2f9be92d9c053cac7bf69360e1f8da8 100644 (file)
@@ -2346,20 +2346,24 @@ static void mt7915_dfs_stop_radar_detector(struct mt7915_phy *phy)
        struct mt7915_dev *dev = phy->dev;
 
        if (phy->rdd_state & BIT(0))
-               mt7915_mcu_rdd_cmd(dev, RDD_STOP, 0, MT_RX_SEL0, 0);
+               mt76_connac_mcu_rdd_cmd(&dev->mt76, RDD_STOP, 0,
+                                       MT_RX_SEL0, 0);
        if (phy->rdd_state & BIT(1))
-               mt7915_mcu_rdd_cmd(dev, RDD_STOP, 1, MT_RX_SEL0, 0);
+               mt76_connac_mcu_rdd_cmd(&dev->mt76, RDD_STOP, 1,
+                                       MT_RX_SEL0, 0);
 }
 
 static int mt7915_dfs_start_rdd(struct mt7915_dev *dev, int chain)
 {
        int err;
 
-       err = mt7915_mcu_rdd_cmd(dev, RDD_START, chain, MT_RX_SEL0, 0);
+       err = mt76_connac_mcu_rdd_cmd(&dev->mt76, RDD_START, chain,
+                                     MT_RX_SEL0, 0);
        if (err < 0)
                return err;
 
-       return mt7915_mcu_rdd_cmd(dev, RDD_DET_MODE, chain, MT_RX_SEL0, 1);
+       return mt76_connac_mcu_rdd_cmd(&dev->mt76, RDD_DET_MODE, chain,
+                                      MT_RX_SEL0, 1);
 }
 
 static int mt7915_dfs_start_radar_detector(struct mt7915_phy *phy)
@@ -2370,7 +2374,8 @@ static int mt7915_dfs_start_radar_detector(struct mt7915_phy *phy)
        int err;
 
        /* start CAC */
-       err = mt7915_mcu_rdd_cmd(dev, RDD_CAC_START, ext_phy, MT_RX_SEL0, 0);
+       err = mt76_connac_mcu_rdd_cmd(&dev->mt76, RDD_CAC_START, ext_phy,
+                                     MT_RX_SEL0, 0);
        if (err < 0)
                return err;
 
@@ -2459,13 +2464,13 @@ int mt7915_dfs_init_radar_detector(struct mt7915_phy *phy)
                if (chandef->chan->dfs_state != NL80211_DFS_AVAILABLE)
                        return mt7915_dfs_start_radar_detector(phy);
 
-               return mt7915_mcu_rdd_cmd(dev, RDD_CAC_END, ext_phy,
-                                         MT_RX_SEL0, 0);
+               return mt76_connac_mcu_rdd_cmd(&dev->mt76, RDD_CAC_END,
+                                              ext_phy, MT_RX_SEL0, 0);
        }
 
 stop:
-       err = mt7915_mcu_rdd_cmd(dev, RDD_NORMAL_START, ext_phy,
-                                MT_RX_SEL0, 0);
+       err = mt76_connac_mcu_rdd_cmd(&dev->mt76, RDD_NORMAL_START, ext_phy,
+                                     MT_RX_SEL0, 0);
        if (err < 0)
                return err;
 
index eb9ce86d5303a661f85700e968c14f1fa27940c4..74cdfd3d13b9c182c35f5303cc5dfd6167ca0b09 100644 (file)
@@ -2528,27 +2528,6 @@ int mt7915_mcu_set_tx(struct mt7915_dev *dev, struct ieee80211_vif *vif)
        return mt7915_mcu_update_edca(dev, &req);
 }
 
-int mt7915_mcu_rdd_cmd(struct mt7915_dev *dev,
-                      enum mt7915_rdd_cmd cmd, u8 index,
-                      u8 rx_sel, u8 val)
-{
-       struct {
-               u8 ctrl;
-               u8 rdd_idx;
-               u8 rdd_rx_sel;
-               u8 val;
-               u8 rsv[4];
-       } __packed req = {
-               .ctrl = cmd,
-               .rdd_idx = index,
-               .rdd_rx_sel = rx_sel,
-               .val = val,
-       };
-
-       return mt76_mcu_send_msg(&dev->mt76, MCU_EXT_CMD(SET_RDD_CTRL), &req,
-                                sizeof(req), true);
-}
-
 int mt7915_mcu_set_fcc5_lpn(struct mt7915_dev *dev, int val)
 {
        struct {
index 3f32ae274600b78c713646fe607a2a33147a4582..5adde022d4e2912bd3b54903fd92d939c3d13b49 100644 (file)
@@ -457,8 +457,6 @@ int mt7915_mcu_get_temperature(struct mt7915_phy *phy);
 int mt7915_mcu_set_thermal_throttling(struct mt7915_phy *phy, u8 state);
 int mt7915_mcu_get_rx_rate(struct mt7915_phy *phy, struct ieee80211_vif *vif,
                           struct ieee80211_sta *sta, struct rate_info *rate);
-int mt7915_mcu_rdd_cmd(struct mt7915_dev *dev, enum mt7915_rdd_cmd cmd,
-                      u8 index, u8 rx_sel, u8 val);
 int mt7915_mcu_wa_cmd(struct mt7915_dev *dev, int cmd, u32 a1, u32 a2, u32 a3);
 int mt7915_mcu_fw_log_2_host(struct mt7915_dev *dev, u8 type, u8 ctrl);
 int mt7915_mcu_fw_dbg_ctrl(struct mt7915_dev *dev, u32 module, u8 level);