net: mvpp2: unify the txq size define use
authorAntoine Ténart <antoine.tenart@free-electrons.com>
Wed, 23 Aug 2017 07:46:55 +0000 (09:46 +0200)
committerDavid S. Miller <davem@davemloft.net>
Thu, 24 Aug 2017 03:42:09 +0000 (20:42 -0700)
The txq size is defined by MVPP2_AGGR_TXQ_SIZE, which is sometime not
used directly but through variables. As it is a fixed value use the
define everywhere in the driver.

Signed-off-by: Antoine Tenart <antoine.tenart@free-electrons.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/marvell/mvpp2.c

index 02c62cbbfe51604136f80ab571be7715d76e9744..eee878809c085c11a377f0e1ddea7a63d36303bb 100644 (file)
@@ -5284,15 +5284,14 @@ static unsigned int mvpp2_tx_done(struct mvpp2_port *port, u32 cause,
 
 /* Allocate and initialize descriptors for aggr TXQ */
 static int mvpp2_aggr_txq_init(struct platform_device *pdev,
-                              struct mvpp2_tx_queue *aggr_txq,
-                              int desc_num, int cpu,
+                              struct mvpp2_tx_queue *aggr_txq, int cpu,
                               struct mvpp2 *priv)
 {
        u32 txq_dma;
 
        /* Allocate memory for TX descriptors */
        aggr_txq->descs = dma_alloc_coherent(&pdev->dev,
-                               desc_num * MVPP2_DESC_ALIGNED_SIZE,
+                               MVPP2_AGGR_TXQ_SIZE * MVPP2_DESC_ALIGNED_SIZE,
                                &aggr_txq->descs_dma, GFP_KERNEL);
        if (!aggr_txq->descs)
                return -ENOMEM;
@@ -5313,7 +5312,8 @@ static int mvpp2_aggr_txq_init(struct platform_device *pdev,
                        MVPP22_AGGR_TXQ_DESC_ADDR_OFFS;
 
        mvpp2_write(priv, MVPP2_AGGR_TXQ_DESC_ADDR_REG(cpu), txq_dma);
-       mvpp2_write(priv, MVPP2_AGGR_TXQ_DESC_SIZE_REG(cpu), desc_num);
+       mvpp2_write(priv, MVPP2_AGGR_TXQ_DESC_SIZE_REG(cpu),
+                   MVPP2_AGGR_TXQ_SIZE);
 
        return 0;
 }
@@ -7445,8 +7445,7 @@ static int mvpp2_init(struct platform_device *pdev, struct mvpp2 *priv)
        for_each_present_cpu(i) {
                priv->aggr_txqs[i].id = i;
                priv->aggr_txqs[i].size = MVPP2_AGGR_TXQ_SIZE;
-               err = mvpp2_aggr_txq_init(pdev, &priv->aggr_txqs[i],
-                                         MVPP2_AGGR_TXQ_SIZE, i, priv);
+               err = mvpp2_aggr_txq_init(pdev, &priv->aggr_txqs[i], i, priv);
                if (err < 0)
                        return err;
        }