blob: 0e8423234bd18ac4638e28042d0bdbceb9fad9a8 [file] [log] [blame]
From 37da0074c6570badb26e2fd527b55621bc571aca Mon Sep 17 00:00:00 2001
From: Bo Jiao <Bo.Jiao@mediatek.com>
Date: Tue, 6 Feb 2024 14:46:59 +0800
Subject: [PATCH 1048/1051] wifi: mt76: mt7915: add SER overlap handle
CR-ID: WCNCR00355921
Signed-off-by: Bo Jiao <Bo.Jiao@mediatek.com>
---
dma.c | 3 ++-
mcu.c | 7 +++++--
mt76.h | 9 +++++++++
mt7915/mac.c | 11 +++++++++++
mt7915/mcu.c | 7 +++++++
5 files changed, 34 insertions(+), 3 deletions(-)
diff --git a/dma.c b/dma.c
index 133a50dc..100d2aff 100644
--- a/dma.c
+++ b/dma.c
@@ -506,7 +506,8 @@ mt76_dma_tx_queue_skb_raw(struct mt76_dev *dev, struct mt76_queue *q,
dma_addr_t addr;
int ret = -ENOMEM;
- if (test_bit(MT76_MCU_RESET, &dev->phy.state)) {
+ if (test_bit(MT76_MCU_RESET, &dev->phy.state) ||
+ dev->recovery_state == MT76_RECOVERY_TRIGGERED) {
ret = -EAGAIN;
goto error;
}
diff --git a/mcu.c b/mcu.c
index de185cc9..1bc94e85 100644
--- a/mcu.c
+++ b/mcu.c
@@ -42,7 +42,9 @@ struct sk_buff *mt76_mcu_get_response(struct mt76_dev *dev,
timeout = expires - jiffies;
wait_event_timeout(dev->mcu.wait,
(!skb_queue_empty(&dev->mcu.res_q) ||
- test_bit(MT76_MCU_RESET, &dev->phy.state)),
+ test_bit(MT76_MCU_RESET, &dev->phy.state) ||
+ dev->recovery_state == MT76_RECOVERY_TRIGGERED ||
+ dev->recovery_state == MT76_RECOVERY_OVERLAP_TRIGGERED),
timeout);
return skb_dequeue(&dev->mcu.res_q);
}
@@ -92,7 +94,8 @@ int mt76_mcu_skb_send_and_get_msg(struct mt76_dev *dev, struct sk_buff *skb,
goto out;
if (retry && retry_cnt < MT76_MSG_MAX_RETRY_CNT) {
- if (test_bit(MT76_MCU_RESET, &dev->phy.state))
+ if (test_bit(MT76_MCU_RESET, &dev->phy.state) ||
+ dev->recovery_state == MT76_RECOVERY_TRIGGERED)
usleep_range(200000, 500000);
dev_err(dev->dev, "send message %08x timeout, try again(%d).\n",
cmd, (MT76_MSG_MAX_RETRY_CNT - retry_cnt));
diff --git a/mt76.h b/mt76.h
index b023f38d..fb50d88b 100644
--- a/mt76.h
+++ b/mt76.h
@@ -478,6 +478,14 @@ enum {
MT76_STATE_WED_RESET,
};
+enum {
+ MT76_RECOVERY_INIT,
+ MT76_RECOVERY_TRIGGERED,
+ MT76_RECOVERY_OVERLAP_TRIGGERED,
+ MT76_RECOVERY_PROCESS,
+ MT76_RECOVERY_DONE = MT76_RECOVERY_INIT,
+};
+
struct mt76_hw_cap {
bool has_2ghz;
bool has_5ghz;
@@ -944,6 +952,7 @@ struct mt76_dev {
wait_queue_head_t tx_wait;
/* spinclock used to protect wcid pktid linked list */
spinlock_t status_lock;
+ u16 recovery_state;
u32 wcid_mask[DIV_ROUND_UP(MT76_N_WCIDS, 32)];
u32 wcid_phy_mask[DIV_ROUND_UP(MT76_N_WCIDS, 32)];
diff --git a/mt7915/mac.c b/mt7915/mac.c
index dbc1095c..e9f50a38 100644
--- a/mt7915/mac.c
+++ b/mt7915/mac.c
@@ -1579,6 +1579,7 @@ void mt7915_mac_reset_work(struct work_struct *work)
if (!(READ_ONCE(dev->recovery.state) & MT_MCU_CMD_STOP_DMA))
return;
+ dev->mt76.recovery_state = MT76_RECOVERY_PROCESS;
dev_info(dev->mt76.dev,"\n%s L1 SER recovery start.",
wiphy_name(dev->mt76.hw->wiphy));
@@ -1645,6 +1646,7 @@ void mt7915_mac_reset_work(struct work_struct *work)
mt7915_update_beacons(dev);
+ dev->mt76.recovery_state = MT76_RECOVERY_DONE;
dev_info(dev->mt76.dev,"\n%s L1 SER recovery completed.",
wiphy_name(dev->mt76.hw->wiphy));
}
@@ -1783,6 +1785,15 @@ void mt7915_reset(struct mt7915_dev *dev)
return;
}
+ if ((READ_ONCE(dev->recovery.state) & MT_MCU_CMD_STOP_DMA)) {
+ if (dev->mt76.recovery_state != MT76_RECOVERY_DONE)
+ dev->mt76.recovery_state = MT76_RECOVERY_OVERLAP_TRIGGERED;
+ else
+ dev->mt76.recovery_state = MT76_RECOVERY_TRIGGERED;
+
+ wake_up(&dev->mt76.mcu.wait);
+ }
+
queue_work(dev->mt76.wq, &dev->reset_work);
wake_up(&dev->reset_wait);
}
diff --git a/mt7915/mcu.c b/mt7915/mcu.c
index c099c1d3..e6c2c3dc 100644
--- a/mt7915/mcu.c
+++ b/mt7915/mcu.c
@@ -198,6 +198,13 @@ mt7915_mcu_send_message(struct mt76_dev *mdev, struct sk_buff *skb,
enum mt76_mcuq_id qid;
int ret;
+ if (mdev->recovery_state == MT76_RECOVERY_OVERLAP_TRIGGERED) {
+ dev_info(dev->mt76.dev,"\n%s L1 SER recovery overlap, drop message %08x.",
+ wiphy_name(dev->mt76.hw->wiphy), cmd);
+ dev_kfree_skb(skb);
+ return -EPERM;
+ }
+
ret = mt76_connac2_mcu_fill_message(mdev, skb, cmd, wait_seq);
if (ret)
--
2.18.0