[PATCH 03/24] wifi: mt76: Serialize MMIO reads with outer cache maintenance
Navid Ghahremani
ghahramani.navid at gmail.com
Sat Oct 10 05:59:41 PDT 2026
Apply the existing workaround to current mainline mt76. This depends on
the preceding ARM API and needs review together with that API.
Assisted-by: LLM
The implementation is refreshed from the existing locally signed mt76
patch; this new draft still requires review of the final dependency/API.
Signed-off-by: Navid Ghahremani <ghahramani.navid at gmail.com>
---
drivers/net/wireless/mediatek/mt76/dma.h | 2 +-
drivers/net/wireless/mediatek/mt76/mmio.c | 6 ++---
drivers/net/wireless/mediatek/mt76/mt76.h | 23 +++++++++++++++++++
.../net/wireless/mediatek/mt76/mt76x02_mmio.c | 2 +-
.../net/wireless/mediatek/mt76/mt7915/pci.c | 2 +-
5 files changed, 29 insertions(+), 6 deletions(-)
diff --git a/drivers/net/wireless/mediatek/mt76/dma.h b/drivers/net/wireless/mediatek/mt76/dma.h
index 7694764feb..721c54b92f 100644
--- a/drivers/net/wireless/mediatek/mt76/dma.h
+++ b/drivers/net/wireless/mediatek/mt76/dma.h
@@ -107,7 +107,7 @@ mt76_dma_handle_write(struct mt76_queue *q, u32 offset, u32 val)
u32 _offset = offsetof(struct mt76_queue_regs, _field); \
u32 _val; \
if (!mt76_dma_handle_read(_q, _offset, &_val)) \
- _val = readl(&(_q)->regs->_field); \
+ _val = mt76_readl(&(_q)->regs->_field); \
_val; \
})
diff --git a/drivers/net/wireless/mediatek/mt76/mmio.c b/drivers/net/wireless/mediatek/mt76/mmio.c
index 73d47608bf..95c6fe27f3 100644
--- a/drivers/net/wireless/mediatek/mt76/mmio.c
+++ b/drivers/net/wireless/mediatek/mt76/mmio.c
@@ -11,7 +11,7 @@ static u32 mt76_mmio_rr(struct mt76_dev *dev, u32 offset)
{
u32 val;
- val = readl(dev->mmio.regs + offset);
+ val = mt76_readl(dev->mmio.regs + offset);
trace_reg_rr(dev, offset, val);
return val;
@@ -53,13 +53,13 @@ static void mt76_mmio_read_copy(struct mt76_dev *dev, u32 offset,
int i;
for (i = 0; i + 4 <= len; i += 4)
- put_unaligned_le32(readl(dev->mmio.regs + offset + i),
+ put_unaligned_le32(mt76_readl(dev->mmio.regs + offset + i),
data + i);
if (i < len) {
u8 tmp[4];
- put_unaligned_le32(readl(dev->mmio.regs + offset + i), tmp);
+ put_unaligned_le32(mt76_readl(dev->mmio.regs + offset + i), tmp);
memcpy(data + i, tmp, len - i);
}
}
diff --git a/drivers/net/wireless/mediatek/mt76/mt76.h b/drivers/net/wireless/mediatek/mt76/mt76.h
index 62b41c8bb7..54b63ba614 100644
--- a/drivers/net/wireless/mediatek/mt76/mt76.h
+++ b/drivers/net/wireless/mediatek/mt76/mt76.h
@@ -20,6 +20,29 @@
#include "util.h"
#include "testmode.h"
+#ifdef CONFIG_CACHE_L2X0_IO_LOCK
+#include <asm/outercache.h>
+#endif
+
+/*
+ * Some SoCs deadlock when an outer cache maintenance operation runs while a
+ * PCIe read is outstanding (see CONFIG_CACHE_L2X0_IO_LOCK). On those, take
+ * the platform lock around device register reads.
+ */
+static inline u32 mt76_readl(const void __iomem *addr)
+{
+#ifdef CONFIG_CACHE_L2X0_IO_LOCK
+ unsigned long flags = outer_io_lock();
+ u32 val = readl(addr);
+
+ outer_io_unlock(flags);
+
+ return val;
+#else
+ return readl(addr);
+#endif
+}
+
#define MT_MCU_RING_SIZE 32
#define MT_RX_BUF_SIZE 2048
#define MT_SKB_HEAD_LEN 256
diff --git a/drivers/net/wireless/mediatek/mt76/mt76x02_mmio.c b/drivers/net/wireless/mediatek/mt76/mt76x02_mmio.c
index dc7c03d231..0fdf2e3a2a 100644
--- a/drivers/net/wireless/mediatek/mt76/mt76x02_mmio.c
+++ b/drivers/net/wireless/mediatek/mt76/mt76x02_mmio.c
@@ -357,7 +357,7 @@ static bool mt76x02_tx_hang(struct mt76x02_dev *dev)
q = dev->mphy.q_tx[i];
prev_dma_idx = dev->mt76.tx_dma_idx[i];
- dma_idx = readl(&q->regs->dma_idx);
+ dma_idx = mt76_readl(&q->regs->dma_idx);
dev->mt76.tx_dma_idx[i] = dma_idx;
if (!q->queued || prev_dma_idx != dma_idx) {
diff --git a/drivers/net/wireless/mediatek/mt76/mt7915/pci.c b/drivers/net/wireless/mediatek/mt76/mt7915/pci.c
index 8007e62004..4c57e66a21 100644
--- a/drivers/net/wireless/mediatek/mt76/mt7915/pci.c
+++ b/drivers/net/wireless/mediatek/mt76/mt7915/pci.c
@@ -36,7 +36,7 @@ static struct mt7915_hif *mt7915_pci_get_hif2(u32 idx)
spin_lock_bh(&hif_lock);
list_for_each_entry(hif, &hif_list, list) {
- val = readl(hif->regs + MT_PCIE_RECOG_ID);
+ val = mt76_readl(hif->regs + MT_PCIE_RECOG_ID);
val &= MT_PCIE_RECOG_ID_MASK;
if (val != idx)
continue;
--
2.55.0
More information about the linux-arm-kernel
mailing list