These functions are only two lines each and are used only once.

Signed-off-by: Pascal Mazon <pascal.ma...@6wind.com>
---
 drivers/net/tap/rte_eth_tap.c | 26 ++++----------------------
 1 file changed, 4 insertions(+), 22 deletions(-)

diff --git a/drivers/net/tap/rte_eth_tap.c b/drivers/net/tap/rte_eth_tap.c
index 0c89b63a2357..c9d107c25ac2 100644
--- a/drivers/net/tap/rte_eth_tap.c
+++ b/drivers/net/tap/rte_eth_tap.c
@@ -860,26 +860,6 @@ tap_setup_queue(struct rte_eth_dev *dev,
 }
 
 static int
-rx_setup_queue(struct rte_eth_dev *dev,
-               struct pmd_internals *internals,
-               uint16_t qid)
-{
-       dev->data->rx_queues[qid] = &internals->rxq[qid];
-
-       return tap_setup_queue(dev, internals, qid);
-}
-
-static int
-tx_setup_queue(struct rte_eth_dev *dev,
-               struct pmd_internals *internals,
-               uint16_t qid)
-{
-       dev->data->tx_queues[qid] = &internals->txq[qid];
-
-       return tap_setup_queue(dev, internals, qid);
-}
-
-static int
 tap_rx_queue_setup(struct rte_eth_dev *dev,
                   uint16_t rx_queue_id,
                   uint16_t nb_rx_desc,
@@ -917,7 +897,8 @@ tap_rx_queue_setup(struct rte_eth_dev *dev,
        }
        rxq->iovecs = iovecs;
 
-       fd = rx_setup_queue(dev, internals, rx_queue_id);
+       dev->data->rx_queues[rx_queue_id] = rxq;
+       fd = tap_setup_queue(dev, internals, rx_queue_id);
        if (fd == -1) {
                ret = fd;
                goto error;
@@ -968,7 +949,8 @@ tap_tx_queue_setup(struct rte_eth_dev *dev,
        if (tx_queue_id >= internals->nb_queues)
                return -1;
 
-       ret = tx_setup_queue(dev, internals, tx_queue_id);
+       dev->data->tx_queues[tx_queue_id] = &internals->txq[tx_queue_id];
+       ret = tap_setup_queue(dev, internals, tx_queue_id);
        if (ret == -1)
                return -1;
 
-- 
2.12.0.306.g4a9b9b3

Reply via email to