From: Lorenzo Bianconi <lore...@kernel.org>

Move mt76x02_init_tx_queue and mt76x02_init_rx_queue in mt76
module in order to be reused adding support for mt7603 driver
and remove duplicated code.
Squash mt76x02_init_tx_queue and mt76x02_init_rx_queue in
mt76_dma_alloc_queue

Signed-off-by: Lorenzo Bianconi <lore...@kernel.org>
---
 drivers/net/wireless/mediatek/mt76/dma.c      |  9 ++++-
 drivers/net/wireless/mediatek/mt76/mt76.h     |  4 +-
 drivers/net/wireless/mediatek/mt76/mt76x02.h  | 32 ++++++++++++++++
 .../net/wireless/mediatek/mt76/mt76x02_mmio.c | 38 -------------------
 4 files changed, 43 insertions(+), 40 deletions(-)

diff --git a/drivers/net/wireless/mediatek/mt76/dma.c 
b/drivers/net/wireless/mediatek/mt76/dma.c
index 6eedc0ec7661..d2b7fa2c76d1 100644
--- a/drivers/net/wireless/mediatek/mt76/dma.c
+++ b/drivers/net/wireless/mediatek/mt76/dma.c
@@ -21,7 +21,9 @@
 #define DMA_DUMMY_TXWI ((void *) ~0)
 
 static int
-mt76_dma_alloc_queue(struct mt76_dev *dev, struct mt76_queue *q)
+mt76_dma_alloc_queue(struct mt76_dev *dev, struct mt76_queue *q,
+                    int idx, int n_desc, int bufsize,
+                    u32 ring_base)
 {
        int size;
        int i;
@@ -29,6 +31,11 @@ mt76_dma_alloc_queue(struct mt76_dev *dev, struct mt76_queue 
*q)
        spin_lock_init(&q->lock);
        INIT_LIST_HEAD(&q->swq);
 
+       q->regs = dev->mmio.regs + ring_base + idx * MT_RING_SIZE;
+       q->ndesc = n_desc;
+       q->buf_size = bufsize;
+       q->hw_idx = idx;
+
        size = q->ndesc * sizeof(struct mt76_desc);
        q->desc = dmam_alloc_coherent(dev->dev, size, &q->desc_dma, GFP_KERNEL);
        if (!q->desc)
diff --git a/drivers/net/wireless/mediatek/mt76/mt76.h 
b/drivers/net/wireless/mediatek/mt76/mt76.h
index 01f06b5766d7..e0524db171fc 100644
--- a/drivers/net/wireless/mediatek/mt76/mt76.h
+++ b/drivers/net/wireless/mediatek/mt76/mt76.h
@@ -147,7 +147,9 @@ struct mt76_mcu_ops {
 struct mt76_queue_ops {
        int (*init)(struct mt76_dev *dev);
 
-       int (*alloc)(struct mt76_dev *dev, struct mt76_queue *q);
+       int (*alloc)(struct mt76_dev *dev, struct mt76_queue *q,
+                    int idx, int n_desc, int bufsize,
+                    u32 ring_base);
 
        int (*add_buf)(struct mt76_dev *dev, struct mt76_queue *q,
                       struct mt76_queue_buf *buf, int nbufs, u32 info,
diff --git a/drivers/net/wireless/mediatek/mt76/mt76x02.h 
b/drivers/net/wireless/mediatek/mt76/mt76x02.h
index ecbdb4385885..ab9e58679833 100644
--- a/drivers/net/wireless/mediatek/mt76/mt76x02.h
+++ b/drivers/net/wireless/mediatek/mt76/mt76x02.h
@@ -203,6 +203,38 @@ static inline void mt76x02_irq_disable(struct mt76x02_dev 
*dev, u32 mask)
        mt76_set_irq_mask(&dev->mt76, MT_INT_MASK_CSR, mask, 0);
 }
 
+static inline int
+mt76x02_init_tx_queue(struct mt76x02_dev *dev, struct mt76_queue *q,
+                     int idx, int n_desc)
+{
+       int err;
+
+       err = mt76_queue_alloc(dev, q, idx, n_desc, 0,
+                              MT_TX_RING_BASE);
+       if (err < 0)
+               return err;
+
+       mt76x02_irq_enable(dev, MT_INT_TX_DONE(idx));
+
+       return 0;
+}
+
+static inline int
+mt76x02_init_rx_queue(struct mt76x02_dev *dev, struct mt76_queue *q,
+                     int idx, int n_desc, int bufsize)
+{
+       int err;
+
+       err = mt76_queue_alloc(dev, q, idx, n_desc, bufsize,
+                              MT_RX_RING_BASE);
+       if (err < 0)
+               return err;
+
+       mt76x02_irq_enable(dev, MT_INT_RX_DONE(idx));
+
+       return 0;
+}
+
 static inline bool
 mt76x02_wait_for_txrx_idle(struct mt76_dev *dev)
 {
diff --git a/drivers/net/wireless/mediatek/mt76/mt76x02_mmio.c 
b/drivers/net/wireless/mediatek/mt76/mt76x02_mmio.c
index c86af8c85fd4..a7ba9f5528aa 100644
--- a/drivers/net/wireless/mediatek/mt76/mt76x02_mmio.c
+++ b/drivers/net/wireless/mediatek/mt76/mt76x02_mmio.c
@@ -151,44 +151,6 @@ static void mt76x02_pre_tbtt_tasklet(unsigned long arg)
        spin_unlock_bh(&q->lock);
 }
 
-static int
-mt76x02_init_tx_queue(struct mt76x02_dev *dev, struct mt76_queue *q,
-                     int idx, int n_desc)
-{
-       int ret;
-
-       q->regs = dev->mt76.mmio.regs + MT_TX_RING_BASE + idx * MT_RING_SIZE;
-       q->ndesc = n_desc;
-       q->hw_idx = idx;
-
-       ret = mt76_queue_alloc(dev, q);
-       if (ret)
-               return ret;
-
-       mt76x02_irq_enable(dev, MT_INT_TX_DONE(idx));
-
-       return 0;
-}
-
-static int
-mt76x02_init_rx_queue(struct mt76x02_dev *dev, struct mt76_queue *q,
-                     int idx, int n_desc, int bufsize)
-{
-       int ret;
-
-       q->regs = dev->mt76.mmio.regs + MT_RX_RING_BASE + idx * MT_RING_SIZE;
-       q->ndesc = n_desc;
-       q->buf_size = bufsize;
-
-       ret = mt76_queue_alloc(dev, q);
-       if (ret)
-               return ret;
-
-       mt76x02_irq_enable(dev, MT_INT_RX_DONE(idx));
-
-       return 0;
-}
-
 static void mt76x02_process_tx_status_fifo(struct mt76x02_dev *dev)
 {
        struct mt76x02_tx_status stat;
-- 
2.20.1

Reply via email to