diff mbox series

[v7,04/17] net/r8169: implement core logic for Tx/Rx

Message ID 20241112095804.42091-5-howard_wang@realsil.com.cn (mailing list archive)
State Changes Requested, archived
Delegated to: Ferruh Yigit
Headers
Series modify code as suggested by the maintainer |

Checks

Context Check Description
ci/checkpatch warning coding style issues

Commit Message

Howard Wang Nov. 12, 2024, 9:57 a.m. UTC 
Add RX/TX function prototypes for further datapath development.

Signed-off-by: Howard Wang <howard_wang@realsil.com.cn>
---
 drivers/net/r8169/meson.build    |  1 +
 drivers/net/r8169/r8169_ethdev.c | 20 +++++++++++++-
 drivers/net/r8169/r8169_ethdev.h |  6 +++++
 drivers/net/r8169/r8169_rxtx.c   | 45 ++++++++++++++++++++++++++++++++
 4 files changed, 71 insertions(+), 1 deletion(-)
 create mode 100644 drivers/net/r8169/r8169_rxtx.c
diff mbox series

Patch

diff --git a/drivers/net/r8169/meson.build b/drivers/net/r8169/meson.build
index 3e5bd8ce52..3b6b6aa912 100644
--- a/drivers/net/r8169/meson.build
+++ b/drivers/net/r8169/meson.build
@@ -4,4 +4,5 @@ 
 sources = files(
         'r8169_ethdev.c',
         'r8169_hw.c',
+        'r8169_rxtx.c',
 )
diff --git a/drivers/net/r8169/r8169_ethdev.c b/drivers/net/r8169/r8169_ethdev.c
index 8208a54a00..3bfac5d011 100644
--- a/drivers/net/r8169/r8169_ethdev.c
+++ b/drivers/net/r8169/r8169_ethdev.c
@@ -17,6 +17,8 @@ 
 
 #include "r8169_ethdev.h"
 #include "r8169_compat.h"
+#include "r8169_logs.h"
+#include "r8169_hw.h"
 
 static int rtl_dev_configure(struct rte_eth_dev *dev);
 static int rtl_dev_start(struct rte_eth_dev *dev);
@@ -54,9 +56,23 @@  rtl_dev_configure(struct rte_eth_dev *dev __rte_unused)
  * It returns 0 on success.
  */
 static int
-rtl_dev_start(struct rte_eth_dev *dev  __rte_unused)
+rtl_dev_start(struct rte_eth_dev *dev)
 {
+	int err;
+
+	/* Initialize transmission unit */
+	rtl_tx_init(dev);
+
+	/* This can fail when allocating mbufs for descriptor rings */
+	err = rtl_rx_init(dev);
+	if (err) {
+		PMD_INIT_LOG(ERR, "Unable to initialize RX hardware");
+		goto error;
+	}
+
 	return 0;
+error:
+	return -EIO;
 }
 
 /*
@@ -88,6 +104,8 @@  static int
 rtl_dev_init(struct rte_eth_dev *dev)
 {
 	dev->dev_ops = &rtl_eth_dev_ops;
+	dev->tx_pkt_burst = &rtl_xmit_pkts;
+	dev->rx_pkt_burst = &rtl_recv_pkts;
 
 	/* For secondary processes, the primary process has done all the work */
 	if (rte_eal_process_type() != RTE_PROC_PRIMARY)
diff --git a/drivers/net/r8169/r8169_ethdev.h b/drivers/net/r8169/r8169_ethdev.h
index 6f7a01622e..3f179b0ebb 100644
--- a/drivers/net/r8169/r8169_ethdev.h
+++ b/drivers/net/r8169/r8169_ethdev.h
@@ -30,4 +30,10 @@  struct rtl_adapter {
 	struct rtl_sw_stats sw_stats;
 };
 
+int rtl_rx_init(struct rte_eth_dev *dev);
+int rtl_tx_init(struct rte_eth_dev *dev);
+
+uint16_t rtl_xmit_pkts(void *txq, struct rte_mbuf **tx_pkts, uint16_t nb_pkts);
+uint16_t rtl_recv_pkts(void *rxq, struct rte_mbuf **rx_pkts, uint16_t nb_pkts);
+
 #endif
diff --git a/drivers/net/r8169/r8169_rxtx.c b/drivers/net/r8169/r8169_rxtx.c
new file mode 100644
index 0000000000..23d36d2231
--- /dev/null
+++ b/drivers/net/r8169/r8169_rxtx.c
@@ -0,0 +1,45 @@ 
+/* SPDX-License-Identifier: BSD-3-Clause
+ * Copyright(c) 2024 Realtek Corporation. All rights reserved
+ */
+
+#include <stdio.h>
+#include <stdint.h>
+
+#include <rte_eal.h>
+
+#include <rte_common.h>
+#include <rte_pci.h>
+#include <bus_pci_driver.h>
+#include <ethdev_driver.h>
+#include <ethdev_pci.h>
+#include <dev_driver.h>
+
+#include "r8169_ethdev.h"
+
+/* ---------------------------------RX---------------------------------- */
+int
+rtl_rx_init(struct rte_eth_dev *dev __rte_unused)
+{
+	return 0;
+}
+
+uint16_t
+rtl_recv_pkts(void *rxq __rte_unused, struct rte_mbuf **rx_pkts __rte_unused,
+	      uint16_t nb_pkts __rte_unused)
+{
+	return 0;
+}
+
+/* ---------------------------------TX---------------------------------- */
+int
+rtl_tx_init(struct rte_eth_dev *dev __rte_unused)
+{
+	return 0;
+}
+
+uint16_t
+rtl_xmit_pkts(void *txq __rte_unused, struct rte_mbuf **tx_pkts __rte_unused,
+	      uint16_t nb_pkts __rte_unused)
+{
+	return 0;
+}