| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135 |
- From: Lorenzo Bianconi <[email protected]>
- Date: Fri, 20 May 2022 20:11:37 +0200
- Subject: [PATCH] net: ethernet: mtk_eth_soc: convert ring dma pointer to void
- Simplify the code converting {tx,rx} ring dma pointer to void
- Signed-off-by: Lorenzo Bianconi <[email protected]>
- Signed-off-by: David S. Miller <[email protected]>
- ---
- --- a/drivers/net/ethernet/mediatek/mtk_eth_soc.c
- +++ b/drivers/net/ethernet/mediatek/mtk_eth_soc.c
- @@ -946,18 +946,15 @@ static int mtk_init_fq_dma(struct mtk_et
- return 0;
- }
-
- -static inline void *mtk_qdma_phys_to_virt(struct mtk_tx_ring *ring, u32 desc)
- +static void *mtk_qdma_phys_to_virt(struct mtk_tx_ring *ring, u32 desc)
- {
- - void *ret = ring->dma;
- -
- - return ret + (desc - ring->phys);
- + return ring->dma + (desc - ring->phys);
- }
-
- static struct mtk_tx_buf *mtk_desc_to_tx_buf(struct mtk_tx_ring *ring,
- - struct mtk_tx_dma *txd,
- - u32 txd_size)
- + void *txd, u32 txd_size)
- {
- - int idx = ((void *)txd - (void *)ring->dma) / txd_size;
- + int idx = (txd - ring->dma) / txd_size;
-
- return &ring->buf[idx];
- }
- @@ -965,13 +962,12 @@ static struct mtk_tx_buf *mtk_desc_to_tx
- static struct mtk_tx_dma *qdma_to_pdma(struct mtk_tx_ring *ring,
- struct mtk_tx_dma *dma)
- {
- - return ring->dma_pdma - ring->dma + dma;
- + return ring->dma_pdma - (struct mtk_tx_dma *)ring->dma + dma;
- }
-
- -static int txd_to_idx(struct mtk_tx_ring *ring, struct mtk_tx_dma *dma,
- - u32 txd_size)
- +static int txd_to_idx(struct mtk_tx_ring *ring, void *dma, u32 txd_size)
- {
- - return ((void *)dma - (void *)ring->dma) / txd_size;
- + return (dma - ring->dma) / txd_size;
- }
-
- static void mtk_tx_unmap(struct mtk_eth *eth, struct mtk_tx_buf *tx_buf,
- @@ -1388,7 +1384,7 @@ static struct mtk_rx_ring *mtk_get_rx_ri
-
- ring = ð->rx_ring[i];
- idx = NEXT_DESP_IDX(ring->calc_idx, ring->dma_size);
- - rxd = (void *)ring->dma + idx * eth->soc->txrx.rxd_size;
- + rxd = ring->dma + idx * eth->soc->txrx.rxd_size;
- if (rxd->rxd2 & RX_DMA_DONE) {
- ring->calc_idx_update = true;
- return ring;
- @@ -1440,7 +1436,7 @@ static int mtk_poll_rx(struct napi_struc
- goto rx_done;
-
- idx = NEXT_DESP_IDX(ring->calc_idx, ring->dma_size);
- - rxd = (void *)ring->dma + idx * eth->soc->txrx.rxd_size;
- + rxd = ring->dma + idx * eth->soc->txrx.rxd_size;
- data = ring->data[idx];
-
- if (!mtk_rx_get_desc(eth, &trxd, rxd))
- @@ -1647,7 +1643,7 @@ static int mtk_poll_tx_pdma(struct mtk_e
-
- mtk_tx_unmap(eth, tx_buf, true);
-
- - desc = (void *)ring->dma + cpu * eth->soc->txrx.txd_size;
- + desc = ring->dma + cpu * eth->soc->txrx.txd_size;
- ring->last_free = desc;
- atomic_inc(&ring->free_count);
-
- @@ -1792,7 +1788,7 @@ static int mtk_tx_alloc(struct mtk_eth *
- int next = (i + 1) % MTK_DMA_SIZE;
- u32 next_ptr = ring->phys + next * sz;
-
- - txd = (void *)ring->dma + i * sz;
- + txd = ring->dma + i * sz;
- txd->txd2 = next_ptr;
- txd->txd3 = TX_DMA_LS0 | TX_DMA_OWNER_CPU;
- txd->txd4 = 0;
- @@ -1822,7 +1818,7 @@ static int mtk_tx_alloc(struct mtk_eth *
-
- ring->dma_size = MTK_DMA_SIZE;
- atomic_set(&ring->free_count, MTK_DMA_SIZE - 2);
- - ring->next_free = &ring->dma[0];
- + ring->next_free = ring->dma;
- ring->last_free = (void *)txd;
- ring->last_free_ptr = (u32)(ring->phys + ((MTK_DMA_SIZE - 1) * sz));
- ring->thresh = MAX_SKB_FRAGS;
- @@ -1937,7 +1933,7 @@ static int mtk_rx_alloc(struct mtk_eth *
- if (unlikely(dma_mapping_error(eth->dma_dev, dma_addr)))
- return -ENOMEM;
-
- - rxd = (void *)ring->dma + i * eth->soc->txrx.rxd_size;
- + rxd = ring->dma + i * eth->soc->txrx.rxd_size;
- rxd->rxd1 = (unsigned int)dma_addr;
-
- if (MTK_HAS_CAPS(eth->soc->caps, MTK_SOC_MT7628))
- @@ -1999,7 +1995,7 @@ static void mtk_rx_clean(struct mtk_eth
- if (!ring->data[i])
- continue;
-
- - rxd = (void *)ring->dma + i * eth->soc->txrx.rxd_size;
- + rxd = ring->dma + i * eth->soc->txrx.rxd_size;
- if (!rxd->rxd1)
- continue;
-
- --- a/drivers/net/ethernet/mediatek/mtk_eth_soc.h
- +++ b/drivers/net/ethernet/mediatek/mtk_eth_soc.h
- @@ -688,7 +688,7 @@ struct mtk_tx_buf {
- * are present
- */
- struct mtk_tx_ring {
- - struct mtk_tx_dma *dma;
- + void *dma;
- struct mtk_tx_buf *buf;
- dma_addr_t phys;
- struct mtk_tx_dma *next_free;
- @@ -718,7 +718,7 @@ enum mtk_rx_flags {
- * @calc_idx: The current head of ring
- */
- struct mtk_rx_ring {
- - struct mtk_rx_dma *dma;
- + void *dma;
- u8 **data;
- dma_addr_t phys;
- u16 frag_size;
|