<div dir="ltr">Acked-by:Ed Czeck <<a href="mailto:ed.czeck@atomicrules.com">ed.czeck@atomicrules.com</a>><br><br>On Fri, Sep 26, 2025 at 12:35 PM Stephen Hemminger <<a href="mailto:stephen@networkplumber.org">stephen@networkplumber.org</a>> wrote:<br>><br>> Rather than eth_ark_XXX use ark_XXX as prefix on all visible<br>> global variables. This makes device consistent with other PMD's.<br>><br>> Signed-off-by: Stephen Hemminger <<a href="mailto:stephen@networkplumber.org">stephen@networkplumber.org</a>><br>> ---<br>>  drivers/net/ark/ark_ethdev.c    | 154 ++++++++++++++++----------------<br>>  drivers/net/ark/ark_ethdev_rx.c |  50 +++++------<br>>  drivers/net/ark/ark_ethdev_rx.h |  18 ++--<br>>  drivers/net/ark/ark_ethdev_tx.c |  32 +++----<br>>  drivers/net/ark/ark_ethdev_tx.h |  14 +--<br>>  5 files changed, 134 insertions(+), 134 deletions(-)<br>><br>> diff --git a/drivers/net/ark/ark_ethdev.c b/drivers/net/ark/ark_ethdev.c<br>> index 8d3c0a1b02..04a15d006f 100644<br>> --- a/drivers/net/ark/ark_ethdev.c<br>> +++ b/drivers/net/ark/ark_ethdev.c<br>> @@ -22,32 +22,32 @@<br>>  #include "ark_pktchkr.h"<br>><br>>  /*  Internal prototypes */<br>> -static int eth_ark_check_args(struct ark_adapter *ark, const char *params);<br>> -static int eth_ark_dev_init(struct rte_eth_dev *dev);<br>> +static int ark_check_args(struct ark_adapter *ark, const char *params);<br>> +static int ark_dev_init(struct rte_eth_dev *dev);<br>>  static int ark_config_device(struct rte_eth_dev *dev);<br>> -static int eth_ark_dev_uninit(struct rte_eth_dev *eth_dev);<br>> -static int eth_ark_dev_configure(struct rte_eth_dev *dev);<br>> -static int eth_ark_dev_start(struct rte_eth_dev *dev);<br>> -static int eth_ark_dev_stop(struct rte_eth_dev *dev);<br>> -static int eth_ark_dev_close(struct rte_eth_dev *dev);<br>> -static int eth_ark_dev_info_get(struct rte_eth_dev *dev,<br>> +static int ark_dev_uninit(struct rte_eth_dev *eth_dev);<br>> +static int ark_dev_configure(struct rte_eth_dev *dev);<br>> +static int ark_dev_start(struct rte_eth_dev *dev);<br>> +static int ark_dev_stop(struct rte_eth_dev *dev);<br>> +static int ark_dev_close(struct rte_eth_dev *dev);<br>> +static int ark_dev_info_get(struct rte_eth_dev *dev,<br>>                                 struct rte_eth_dev_info *dev_info);<br>> -static int eth_ark_dev_link_update(struct rte_eth_dev *dev,<br>> +static int ark_dev_link_update(struct rte_eth_dev *dev,<br>>                                    int wait_to_complete);<br>> -static int eth_ark_dev_set_link_up(struct rte_eth_dev *dev);<br>> -static int eth_ark_dev_set_link_down(struct rte_eth_dev *dev);<br>> -static int eth_ark_dev_stats_get(struct rte_eth_dev *dev,<br>> +static int ark_dev_set_link_up(struct rte_eth_dev *dev);<br>> +static int ark_dev_set_link_down(struct rte_eth_dev *dev);<br>> +static int ark_dev_stats_get(struct rte_eth_dev *dev,<br>>                                   struct rte_eth_stats *stats);<br>> -static int eth_ark_dev_stats_reset(struct rte_eth_dev *dev);<br>> -static int eth_ark_set_default_mac_addr(struct rte_eth_dev *dev,<br>> +static int ark_dev_stats_reset(struct rte_eth_dev *dev);<br>> +static int ark_set_default_mac_addr(struct rte_eth_dev *dev,<br>>                                          struct rte_ether_addr *mac_addr);<br>> -static int eth_ark_macaddr_add(struct rte_eth_dev *dev,<br>> +static int ark_macaddr_add(struct rte_eth_dev *dev,<br>>                                struct rte_ether_addr *mac_addr,<br>>                                uint32_t index,<br>>                                uint32_t pool);<br>> -static void eth_ark_macaddr_remove(struct rte_eth_dev *dev,<br>> +static void ark_macaddr_remove(struct rte_eth_dev *dev,<br>>                                    uint32_t index);<br>> -static int  eth_ark_set_mtu(struct rte_eth_dev *dev, uint16_t size);<br>> +static int  ark_set_mtu(struct rte_eth_dev *dev, uint16_t size);<br>><br>>  /*<br>>   * The packet generator is a functional block used to generate packet<br>> @@ -145,7 +145,7 @@ ark_device_caps[] = {<br>>  };<br>><br>>  static int<br>> -eth_ark_pci_probe(struct rte_pci_driver *pci_drv __rte_unused,<br>> +ark_pci_probe(struct rte_pci_driver *pci_drv __rte_unused,<br>>                 struct rte_pci_device *pci_dev)<br>>  {<br>>         struct rte_eth_dev *eth_dev;<br>> @@ -156,7 +156,7 @@ eth_ark_pci_probe(struct rte_pci_driver *pci_drv __rte_unused,<br>>         if (eth_dev == NULL)<br>>                 return -ENOMEM;<br>><br>> -       ret = eth_ark_dev_init(eth_dev);<br>> +       ret = ark_dev_init(eth_dev);<br>>         if (ret)<br>>                 rte_eth_dev_release_port(eth_dev);<br>><br>> @@ -164,47 +164,47 @@ eth_ark_pci_probe(struct rte_pci_driver *pci_drv __rte_unused,<br>>  }<br>><br>>  static int<br>> -eth_ark_pci_remove(struct rte_pci_device *pci_dev)<br>> +ark_pci_remove(struct rte_pci_device *pci_dev)<br>>  {<br>> -       return rte_eth_dev_pci_generic_remove(pci_dev, eth_ark_dev_uninit);<br>> +       return rte_eth_dev_pci_generic_remove(pci_dev, ark_dev_uninit);<br>>  }<br>><br>>  static struct rte_pci_driver rte_ark_pmd = {<br>>         .id_table = pci_id_ark_map,<br>>         .drv_flags = RTE_PCI_DRV_NEED_MAPPING | RTE_PCI_DRV_INTR_LSC,<br>> -       .probe = eth_ark_pci_probe,<br>> -       .remove = eth_ark_pci_remove,<br>> +       .probe = ark_pci_probe,<br>> +       .remove = ark_pci_remove,<br>>  };<br>><br>>  static const struct eth_dev_ops ark_eth_dev_ops = {<br>> -       .dev_configure = eth_ark_dev_configure,<br>> -       .dev_start = eth_ark_dev_start,<br>> -       .dev_stop = eth_ark_dev_stop,<br>> -       .dev_close = eth_ark_dev_close,<br>> +       .dev_configure = ark_dev_configure,<br>> +       .dev_start = ark_dev_start,<br>> +       .dev_stop = ark_dev_stop,<br>> +       .dev_close = ark_dev_close,<br>><br>> -       .dev_infos_get = eth_ark_dev_info_get,<br>> +       .dev_infos_get = ark_dev_info_get,<br>><br>> -       .rx_queue_setup = eth_ark_dev_rx_queue_setup,<br>> -       .tx_queue_setup = eth_ark_tx_queue_setup,<br>> +       .rx_queue_setup = ark_dev_rx_queue_setup,<br>> +       .tx_queue_setup = ark_tx_queue_setup,<br>><br>> -       .link_update = eth_ark_dev_link_update,<br>> -       .dev_set_link_up = eth_ark_dev_set_link_up,<br>> -       .dev_set_link_down = eth_ark_dev_set_link_down,<br>> +       .link_update = ark_dev_link_update,<br>> +       .dev_set_link_up = ark_dev_set_link_up,<br>> +       .dev_set_link_down = ark_dev_set_link_down,<br>><br>> -       .rx_queue_start = eth_ark_rx_start_queue,<br>> -       .rx_queue_stop = eth_ark_rx_stop_queue,<br>> +       .rx_queue_start = ark_rx_start_queue,<br>> +       .rx_queue_stop = ark_rx_stop_queue,<br>><br>> -       .tx_queue_start = eth_ark_tx_queue_start,<br>> -       .tx_queue_stop = eth_ark_tx_queue_stop,<br>> +       .tx_queue_start = ark_tx_queue_start,<br>> +       .tx_queue_stop = ark_tx_queue_stop,<br>><br>> -       .stats_get = eth_ark_dev_stats_get,<br>> -       .stats_reset = eth_ark_dev_stats_reset,<br>> +       .stats_get = ark_dev_stats_get,<br>> +       .stats_reset = ark_dev_stats_reset,<br>><br>> -       .mac_addr_add = eth_ark_macaddr_add,<br>> -       .mac_addr_remove = eth_ark_macaddr_remove,<br>> -       .mac_addr_set = eth_ark_set_default_mac_addr,<br>> +       .mac_addr_add = ark_macaddr_add,<br>> +       .mac_addr_remove = ark_macaddr_remove,<br>> +       .mac_addr_set = ark_set_default_mac_addr,<br>><br>> -       .mtu_set = eth_ark_set_mtu,<br>> +       .mtu_set = ark_set_mtu,<br>>  };<br>><br>>  static int<br>> @@ -297,7 +297,7 @@ check_for_ext(struct ark_adapter *ark)<br>>  }<br>><br>>  static int<br>> -eth_ark_dev_init(struct rte_eth_dev *dev)<br>> +ark_dev_init(struct rte_eth_dev *dev)<br>>  {<br>>         struct ark_adapter *ark = dev->data->dev_private;<br>>         struct rte_pci_device *pci_dev;<br>> @@ -376,7 +376,7 @@ eth_ark_dev_init(struct rte_eth_dev *dev)<br>>                 return -1;<br>><br>>         dev->dev_ops = &ark_eth_dev_ops;<br>> -       dev->rx_queue_count = eth_ark_dev_rx_queue_count;<br>> +       dev->rx_queue_count = ark_dev_rx_queue_count;<br>><br>>         dev->data->mac_addrs = rte_zmalloc("ark", RTE_ETHER_ADDR_LEN, 0);<br>>         if (!dev->data->mac_addrs) {<br>> @@ -398,7 +398,7 @@ eth_ark_dev_init(struct rte_eth_dev *dev)<br>>         }<br>><br>>         if (pci_dev->device.devargs)<br>> -               ret = eth_ark_check_args(ark, pci_dev->device.devargs->args);<br>> +               ret = ark_check_args(ark, pci_dev->device.devargs->args);<br>>         else<br>>                 ARK_PMD_LOG(INFO, "No Device args found\n");<br>><br>> @@ -539,7 +539,7 @@ ark_config_device(struct rte_eth_dev *dev)<br>>  }<br>><br>>  static int<br>> -eth_ark_dev_uninit(struct rte_eth_dev *dev)<br>> +ark_dev_uninit(struct rte_eth_dev *dev)<br>>  {<br>>         struct ark_adapter *ark = dev->data->dev_private;<br>><br>> @@ -559,11 +559,11 @@ eth_ark_dev_uninit(struct rte_eth_dev *dev)<br>>  }<br>><br>>  static int<br>> -eth_ark_dev_configure(struct rte_eth_dev *dev)<br>> +ark_dev_configure(struct rte_eth_dev *dev)<br>>  {<br>>         struct ark_adapter *ark = dev->data->dev_private;<br>><br>> -       eth_ark_dev_set_link_up(dev);<br>> +       ark_dev_set_link_up(dev);<br>>         if (ark->user_ext.dev_configure)<br>>                 return ark->user_ext.dev_configure(dev,<br>>                            ark->user_data[dev->data->port_id]);<br>> @@ -571,23 +571,23 @@ eth_ark_dev_configure(struct rte_eth_dev *dev)<br>>  }<br>><br>>  static int<br>> -eth_ark_dev_start(struct rte_eth_dev *dev)<br>> +ark_dev_start(struct rte_eth_dev *dev)<br>>  {<br>>         struct ark_adapter *ark = dev->data->dev_private;<br>>         int i;<br>><br>>         /* RX Side */<br>>         for (i = 0; i < dev->data->nb_rx_queues; i++)<br>> -               eth_ark_rx_start_queue(dev, i);<br>> +               ark_rx_start_queue(dev, i);<br>><br>>         /* TX Side */<br>>         for (i = 0; i < dev->data->nb_tx_queues; i++)<br>> -               eth_ark_tx_queue_start(dev, i);<br>> +               ark_tx_queue_start(dev, i);<br>><br>>         ark->started = 1;<br>>         /* set xmit and receive function */<br>> -       dev->rx_pkt_burst = &eth_ark_recv_pkts;<br>> -       dev->tx_pkt_burst = &eth_ark_xmit_pkts;<br>> +       dev->rx_pkt_burst = &ark_recv_pkts;<br>> +       dev->tx_pkt_burst = &ark_xmit_pkts;<br>><br>>         if (!ark->isvf && ark->start_pg)<br>>                 ark_pktchkr_run(ark->pc);<br>> @@ -618,7 +618,7 @@ eth_ark_dev_start(struct rte_eth_dev *dev)<br>>  }<br>><br>>  static int<br>> -eth_ark_dev_stop(struct rte_eth_dev *dev)<br>> +ark_dev_stop(struct rte_eth_dev *dev)<br>>  {<br>>         uint16_t i;<br>>         int status;<br>> @@ -645,11 +645,11 @@ eth_ark_dev_stop(struct rte_eth_dev *dev)<br>><br>>         /* Stop RX Side */<br>>         for (i = 0; i < dev->data->nb_rx_queues; i++)<br>> -               eth_ark_rx_stop_queue(dev, i);<br>> +               ark_rx_stop_queue(dev, i);<br>><br>>         /* STOP TX Side */<br>>         for (i = 0; i < dev->data->nb_tx_queues; i++) {<br>> -               status = eth_ark_tx_queue_stop(dev, i);<br>> +               status = ark_tx_queue_stop(dev, i);<br>>                 if (status != 0) {<br>>                         uint16_t port = dev->data->port_id;<br>>                         ARK_PMD_LOG(ERR,<br>> @@ -662,7 +662,7 @@ eth_ark_dev_stop(struct rte_eth_dev *dev)<br>>         ark_udm_dump_stats(ark->udm.v, "Post stop");<br>><br>>         for (i = 0; i < dev->data->nb_rx_queues; i++)<br>> -               eth_ark_rx_dump_queue(dev, i, __func__);<br>> +               ark_rx_dump_queue(dev, i, __func__);<br>><br>>         /* Stop the packet checker if it is running */<br>>         if (!ark->isvf && ark->start_pg) {<br>> @@ -674,7 +674,7 @@ eth_ark_dev_stop(struct rte_eth_dev *dev)<br>>  }<br>><br>>  static int<br>> -eth_ark_dev_close(struct rte_eth_dev *dev)<br>> +ark_dev_close(struct rte_eth_dev *dev)<br>>  {<br>>         struct ark_adapter *ark = dev->data->dev_private;<br>>         uint16_t i;<br>> @@ -686,7 +686,7 @@ eth_ark_dev_close(struct rte_eth_dev *dev)<br>>                 ark->user_ext.dev_close(dev,<br>>                  ark->user_data[dev->data->port_id]);<br>><br>> -       eth_ark_dev_stop(dev);<br>> +       ark_dev_stop(dev);<br>><br>>         /*<br>>          * This should only be called once for the device during shutdown<br>> @@ -696,12 +696,12 @@ eth_ark_dev_close(struct rte_eth_dev *dev)<br>>                 ark_pktdir_setup(ark->pd, ARK_PKT_DIR_INIT_VAL);<br>><br>>         for (i = 0; i < dev->data->nb_tx_queues; i++) {<br>> -               eth_ark_tx_queue_release(dev->data->tx_queues[i]);<br>> +               ark_tx_queue_release(dev->data->tx_queues[i]);<br>>                 dev->data->tx_queues[i] = 0;<br>>         }<br>><br>>         for (i = 0; i < dev->data->nb_rx_queues; i++) {<br>> -               eth_ark_dev_rx_queue_release(dev->data->rx_queues[i]);<br>> +               ark_dev_rx_queue_release(dev->data->rx_queues[i]);<br>>                 dev->data->rx_queues[i] = 0;<br>>         }<br>><br>> @@ -709,7 +709,7 @@ eth_ark_dev_close(struct rte_eth_dev *dev)<br>>  }<br>><br>>  static int<br>> -eth_ark_dev_info_get(struct rte_eth_dev *dev,<br>> +ark_dev_info_get(struct rte_eth_dev *dev,<br>>                      struct rte_eth_dev_info *dev_info)<br>>  {<br>>         struct ark_adapter *ark = dev->data->dev_private;<br>> @@ -749,7 +749,7 @@ eth_ark_dev_info_get(struct rte_eth_dev *dev,<br>>  }<br>><br>>  static int<br>> -eth_ark_dev_link_update(struct rte_eth_dev *dev, int wait_to_complete)<br>> +ark_dev_link_update(struct rte_eth_dev *dev, int wait_to_complete)<br>>  {<br>>         ARK_PMD_LOG(DEBUG, "link status = %d\n",<br>>                         dev->data->dev_link.link_status);<br>> @@ -764,7 +764,7 @@ eth_ark_dev_link_update(struct rte_eth_dev *dev, int wait_to_complete)<br>>  }<br>><br>>  static int<br>> -eth_ark_dev_set_link_up(struct rte_eth_dev *dev)<br>> +ark_dev_set_link_up(struct rte_eth_dev *dev)<br>>  {<br>>         dev->data->dev_link.link_status = 1;<br>>         struct ark_adapter *ark = dev->data->dev_private;<br>> @@ -776,7 +776,7 @@ eth_ark_dev_set_link_up(struct rte_eth_dev *dev)<br>>  }<br>><br>>  static int<br>> -eth_ark_dev_set_link_down(struct rte_eth_dev *dev)<br>> +ark_dev_set_link_down(struct rte_eth_dev *dev)<br>>  {<br>>         dev->data->dev_link.link_status = 0;<br>>         struct ark_adapter *ark = dev->data->dev_private;<br>> @@ -788,7 +788,7 @@ eth_ark_dev_set_link_down(struct rte_eth_dev *dev)<br>>  }<br>><br>>  static int<br>> -eth_ark_dev_stats_get(struct rte_eth_dev *dev, struct rte_eth_stats *stats)<br>> +ark_dev_stats_get(struct rte_eth_dev *dev, struct rte_eth_stats *stats)<br>>  {<br>>         uint16_t i;<br>>         struct ark_adapter *ark = dev->data->dev_private;<br>> @@ -801,9 +801,9 @@ eth_ark_dev_stats_get(struct rte_eth_dev *dev, struct rte_eth_stats *stats)<br>>         stats->oerrors = 0;<br>><br>>         for (i = 0; i < dev->data->nb_tx_queues; i++)<br>> -               eth_tx_queue_stats_get(dev->data->tx_queues[i], stats);<br>> +               ark_tx_queue_stats_get(dev->data->tx_queues[i], stats);<br>>         for (i = 0; i < dev->data->nb_rx_queues; i++)<br>> -               eth_rx_queue_stats_get(dev->data->rx_queues[i], stats);<br>> +               ark_rx_queue_stats_get(dev->data->rx_queues[i], stats);<br>>         if (ark->user_ext.stats_get)<br>>                 return ark->user_ext.stats_get(dev, stats,<br>>                         ark->user_data[dev->data->port_id]);<br>> @@ -811,15 +811,15 @@ eth_ark_dev_stats_get(struct rte_eth_dev *dev, struct rte_eth_stats *stats)<br>>  }<br>><br>>  static int<br>> -eth_ark_dev_stats_reset(struct rte_eth_dev *dev)<br>> +ark_dev_stats_reset(struct rte_eth_dev *dev)<br>>  {<br>>         uint16_t i;<br>>         struct ark_adapter *ark = dev->data->dev_private;<br>><br>>         for (i = 0; i < dev->data->nb_tx_queues; i++)<br>> -               eth_tx_queue_stats_reset(dev->data->tx_queues[i]);<br>> +               ark_tx_queue_stats_reset(dev->data->tx_queues[i]);<br>>         for (i = 0; i < dev->data->nb_rx_queues; i++)<br>> -               eth_rx_queue_stats_reset(dev->data->rx_queues[i]);<br>> +               ark_rx_queue_stats_reset(dev->data->rx_queues[i]);<br>>         if (ark->user_ext.stats_reset)<br>>                 ark->user_ext.stats_reset(dev,<br>>                           ark->user_data[dev->data->port_id]);<br>> @@ -828,7 +828,7 @@ eth_ark_dev_stats_reset(struct rte_eth_dev *dev)<br>>  }<br>><br>>  static int<br>> -eth_ark_macaddr_add(struct rte_eth_dev *dev,<br>> +ark_macaddr_add(struct rte_eth_dev *dev,<br>>                     struct rte_ether_addr *mac_addr,<br>>                     uint32_t index,<br>>                     uint32_t pool)<br>> @@ -847,7 +847,7 @@ eth_ark_macaddr_add(struct rte_eth_dev *dev,<br>>  }<br>><br>>  static void<br>> -eth_ark_macaddr_remove(struct rte_eth_dev *dev, uint32_t index)<br>> +ark_macaddr_remove(struct rte_eth_dev *dev, uint32_t index)<br>>  {<br>>         struct ark_adapter *ark = dev->data->dev_private;<br>><br>> @@ -857,7 +857,7 @@ eth_ark_macaddr_remove(struct rte_eth_dev *dev, uint32_t index)<br>>  }<br>><br>>  static int<br>> -eth_ark_set_default_mac_addr(struct rte_eth_dev *dev,<br>> +ark_set_default_mac_addr(struct rte_eth_dev *dev,<br>>                              struct rte_ether_addr *mac_addr)<br>>  {<br>>         struct ark_adapter *ark = dev->data->dev_private;<br>> @@ -871,7 +871,7 @@ eth_ark_set_default_mac_addr(struct rte_eth_dev *dev,<br>>  }<br>><br>>  static int<br>> -eth_ark_set_mtu(struct rte_eth_dev *dev, uint16_t  size)<br>> +ark_set_mtu(struct rte_eth_dev *dev, uint16_t  size)<br>>  {<br>>         struct ark_adapter *ark = dev->data->dev_private;<br>><br>> @@ -936,7 +936,7 @@ process_file_args(const char *key, const char *value, void *extra_args)<br>>  }<br>><br>>  static int<br>> -eth_ark_check_args(struct ark_adapter *ark, const char *params)<br>> +ark_check_args(struct ark_adapter *ark, const char *params)<br>>  {<br>>         struct rte_kvargs *kvlist;<br>>         unsigned int k_idx;<br>> diff --git a/drivers/net/ark/ark_ethdev_rx.c b/drivers/net/ark/ark_ethdev_rx.c<br>> index 8b18d93eaa..f221137931 100644<br>> --- a/drivers/net/ark/ark_ethdev_rx.c<br>> +++ b/drivers/net/ark/ark_ethdev_rx.c<br>> @@ -22,11 +22,11 @@ struct ark_rx_meta;<br>><br>>  static void dump_mbuf_data(struct rte_mbuf *mbuf, uint16_t lo, uint16_t hi);<br>>  static void ark_ethdev_rx_dump(const char *name, struct ark_rx_queue *queue);<br>> -static uint32_t eth_ark_rx_jumbo(struct ark_rx_queue *queue,<br>> +static uint32_t ark_rx_jumbo(struct ark_rx_queue *queue,<br>>                                  struct ark_rx_meta *meta,<br>>                                  struct rte_mbuf *mbuf0,<br>>                                  uint32_t cons_index);<br>> -static inline int eth_ark_rx_seed_mbufs(struct ark_rx_queue *queue);<br>> +static inline int ark_rx_seed_mbufs(struct ark_rx_queue *queue);<br>><br>>  /* ************************************************************************* */<br>>  struct __rte_cache_aligned ark_rx_queue {<br>> @@ -67,7 +67,7 @@ struct __rte_cache_aligned ark_rx_queue {<br>><br>>  /* ************************************************************************* */<br>>  static int<br>> -eth_ark_rx_hw_setup(struct rte_eth_dev *dev,<br>> +ark_rx_hw_setup(struct rte_eth_dev *dev,<br>>                     struct ark_rx_queue *queue,<br>>                     uint16_t rx_queue_idx)<br>>  {<br>> @@ -100,18 +100,18 @@ eth_ark_rx_hw_setup(struct rte_eth_dev *dev,<br>>  }<br>><br>>  static inline void<br>> -eth_ark_rx_update_cons_index(struct ark_rx_queue *queue, uint32_t cons_index)<br>> +ark_rx_update_cons_index(struct ark_rx_queue *queue, uint32_t cons_index)<br>>  {<br>>         queue->cons_index = cons_index;<br>>         if ((cons_index + queue->queue_size - queue->seed_index) >= ARK_RX_MPU_CHUNK) {<br>> -               eth_ark_rx_seed_mbufs(queue);<br>> +               ark_rx_seed_mbufs(queue);<br>>                 ark_mpu_set_producer(queue->mpu, queue->seed_index);<br>>         }<br>>  }<br>><br>>  /* ************************************************************************* */<br>>  int<br>> -eth_ark_dev_rx_queue_setup(struct rte_eth_dev *dev,<br>> +ark_dev_rx_queue_setup(struct rte_eth_dev *dev,<br>>                            uint16_t queue_idx,<br>>                            uint16_t nb_desc,<br>>                            unsigned int socket_id,<br>> @@ -129,7 +129,7 @@ eth_ark_dev_rx_queue_setup(struct rte_eth_dev *dev,<br>><br>>         /* We may already be setup, free memory prior to re-allocation */<br>>         if (dev->data->rx_queues[queue_idx] != NULL) {<br>> -               eth_ark_dev_rx_queue_release(dev->data->rx_queues[queue_idx]);<br>> +               ark_dev_rx_queue_release(dev->data->rx_queues[queue_idx]);<br>>                 dev->data->rx_queues[queue_idx] = NULL;<br>>         }<br>><br>> @@ -215,7 +215,7 @@ eth_ark_dev_rx_queue_setup(struct rte_eth_dev *dev,<br>>         ark_udm_queue_stats_reset(queue->udm);<br>><br>>         /* populate mbuf reserve */<br>> -       status = eth_ark_rx_seed_mbufs(queue);<br>> +       status = ark_rx_seed_mbufs(queue);<br>><br>>         if (queue->seed_index != nb_desc) {<br>>                 ARK_PMD_LOG(ERR, "Failed to allocate %u mbufs for RX queue %d\n",<br>> @@ -224,7 +224,7 @@ eth_ark_dev_rx_queue_setup(struct rte_eth_dev *dev,<br>>         }<br>>         /* MPU Setup */<br>>         if (status == 0)<br>> -               status = eth_ark_rx_hw_setup(dev, queue, queue_idx);<br>> +               status = ark_rx_hw_setup(dev, queue, queue_idx);<br>><br>>         if (unlikely(status != 0)) {<br>>                 struct rte_mbuf **mbuf;<br>> @@ -248,7 +248,7 @@ eth_ark_dev_rx_queue_setup(struct rte_eth_dev *dev,<br>><br>>  /* ************************************************************************* */<br>>  uint16_t<br>> -eth_ark_recv_pkts(void *rx_queue,<br>> +ark_recv_pkts(void *rx_queue,<br>>                   struct rte_mbuf **rx_pkts,<br>>                   uint16_t nb_pkts)<br>>  {<br>> @@ -267,7 +267,7 @@ eth_ark_recv_pkts(void *rx_queue,<br>>         if (unlikely(nb_pkts == 0))<br>>                 return 0;<br>>         if (unlikely(queue->starvation))<br>> -               eth_ark_rx_seed_mbufs(queue);<br>> +               ark_rx_seed_mbufs(queue);<br>><br>>         prod_index = queue->prod_index;<br>>         cons_index = queue->cons_index;<br>> @@ -318,7 +318,7 @@ eth_ark_recv_pkts(void *rx_queue,<br>>                 }<br>><br>>                 if (unlikely(meta->pkt_len > queue->dataroom)) {<br>> -                       uint32_t tcons = eth_ark_rx_jumbo(queue, meta, mbuf, cons_index + 1);<br>> +                       uint32_t tcons = ark_rx_jumbo(queue, meta, mbuf, cons_index + 1);<br>>                         if ((int32_t)(prod_index - tcons) >= 0)<br>>                                 cons_index = tcons; /* nominal condition */<br>>                         else<br>> @@ -340,14 +340,14 @@ eth_ark_recv_pkts(void *rx_queue,<br>>                 rx_user_meta_hook(mbuf, meta->user_meta, queue->ext_user_data);<br>>         }<br>><br>> -       eth_ark_rx_update_cons_index(queue, cons_index);<br>> +       ark_rx_update_cons_index(queue, cons_index);<br>><br>>         return nb;<br>>  }<br>><br>>  /* ************************************************************************* */<br>>  static uint32_t<br>> -eth_ark_rx_jumbo(struct ark_rx_queue *queue,<br>> +ark_rx_jumbo(struct ark_rx_queue *queue,<br>>                  struct ark_rx_meta *meta,<br>>                  struct rte_mbuf *mbuf0,<br>>                  uint32_t cons_index)<br>> @@ -388,7 +388,7 @@ eth_ark_rx_jumbo(struct ark_rx_queue *queue,<br>><br>>  /* Drain the internal queue allowing hw to clear out. */<br>>  static void<br>> -eth_ark_rx_queue_drain(struct ark_rx_queue *queue)<br>> +ark_rx_queue_drain(struct ark_rx_queue *queue)<br>>  {<br>>         register uint32_t cons_index;<br>>         struct rte_mbuf *mbuf;<br>> @@ -400,12 +400,12 @@ eth_ark_rx_queue_drain(struct ark_rx_queue *queue)<br>>                 mbuf = queue->reserve_q[cons_index & queue->queue_mask];<br>>                 rte_pktmbuf_free(mbuf);<br>>                 cons_index++;<br>> -               eth_ark_rx_update_cons_index(queue, cons_index);<br>> +               ark_rx_update_cons_index(queue, cons_index);<br>>         }<br>>  }<br>><br>>  int<br>> -eth_ark_dev_rx_queue_count(void *rx_queue)<br>> +ark_dev_rx_queue_count(void *rx_queue)<br>>  {<br>>         struct ark_rx_queue *queue;<br>><br>> @@ -415,7 +415,7 @@ eth_ark_dev_rx_queue_count(void *rx_queue)<br>><br>>  /* ************************************************************************* */<br>>  int<br>> -eth_ark_rx_start_queue(struct rte_eth_dev *dev, uint16_t queue_id)<br>> +ark_rx_start_queue(struct rte_eth_dev *dev, uint16_t queue_id)<br>>  {<br>>         struct ark_rx_queue *queue;<br>><br>> @@ -438,7 +438,7 @@ eth_ark_rx_start_queue(struct rte_eth_dev *dev, uint16_t queue_id)<br>>  /* Queue can be restarted.   data remains<br>>   */<br>>  int<br>> -eth_ark_rx_stop_queue(struct rte_eth_dev *dev, uint16_t queue_id)<br>> +ark_rx_stop_queue(struct rte_eth_dev *dev, uint16_t queue_id)<br>>  {<br>>         struct ark_rx_queue *queue;<br>><br>> @@ -455,7 +455,7 @@ eth_ark_rx_stop_queue(struct rte_eth_dev *dev, uint16_t queue_id)<br>><br>>  /* ************************************************************************* */<br>>  static inline int<br>> -eth_ark_rx_seed_mbufs(struct ark_rx_queue *queue)<br>> +ark_rx_seed_mbufs(struct ark_rx_queue *queue)<br>>  {<br>>         uint32_t limit = RTE_ALIGN_FLOOR(queue->cons_index, ARK_RX_MPU_CHUNK) +<br>>                 queue->queue_size;<br>> @@ -539,7 +539,7 @@ eth_ark_rx_seed_mbufs(struct ark_rx_queue *queue)<br>>  }<br>><br>>  void<br>> -eth_ark_rx_dump_queue(struct rte_eth_dev *dev, uint16_t queue_id,<br>> +ark_rx_dump_queue(struct rte_eth_dev *dev, uint16_t queue_id,<br>>                       const char *msg)<br>>  {<br>>         struct ark_rx_queue *queue;<br>> @@ -552,7 +552,7 @@ eth_ark_rx_dump_queue(struct rte_eth_dev *dev, uint16_t queue_id,<br>>  /* ************************************************************************* */<br>>  /* Call on device closed no user API, queue is stopped */<br>>  void<br>> -eth_ark_dev_rx_queue_release(void *vqueue)<br>> +ark_dev_rx_queue_release(void *vqueue)<br>>  {<br>>         struct ark_rx_queue *queue;<br>><br>> @@ -565,7 +565,7 @@ eth_ark_dev_rx_queue_release(void *vqueue)<br>>         ark_mpu_stop(queue->mpu);<br>><br>>         /* Need to clear out mbufs here, dropping packets along the way */<br>> -       eth_ark_rx_queue_drain(queue);<br>> +       ark_rx_queue_drain(queue);<br>><br>>         rte_free(queue->reserve_q);<br>>         rte_free(queue->paddress_q);<br>> @@ -573,7 +573,7 @@ eth_ark_dev_rx_queue_release(void *vqueue)<br>>  }<br>><br>>  void<br>> -eth_rx_queue_stats_get(void *vqueue, struct rte_eth_stats *stats)<br>> +ark_rx_queue_stats_get(void *vqueue, struct rte_eth_stats *stats)<br>>  {<br>>         struct ark_rx_queue *queue;<br>>         struct ark_udm_t *udm;<br>> @@ -596,7 +596,7 @@ eth_rx_queue_stats_get(void *vqueue, struct rte_eth_stats *stats)<br>>  }<br>><br>>  void<br>> -eth_rx_queue_stats_reset(void *vqueue)<br>> +ark_rx_queue_stats_reset(void *vqueue)<br>>  {<br>>         struct ark_rx_queue *queue;<br>><br>> diff --git a/drivers/net/ark/ark_ethdev_rx.h b/drivers/net/ark/ark_ethdev_rx.h<br>> index 12d4f61637..7a5bab20a9 100644<br>> --- a/drivers/net/ark/ark_ethdev_rx.h<br>> +++ b/drivers/net/ark/ark_ethdev_rx.h<br>> @@ -11,21 +11,21 @@<br>>  #include <rte_mempool.h><br>>  #include <ethdev_driver.h><br>><br>> -int eth_ark_dev_rx_queue_setup(struct rte_eth_dev *dev,<br>> +int ark_dev_rx_queue_setup(struct rte_eth_dev *dev,<br>>                                uint16_t queue_idx,<br>>                                uint16_t nb_desc,<br>>                                unsigned int socket_id,<br>>                                const struct rte_eth_rxconf *rx_conf,<br>>                                struct rte_mempool *mp);<br>> -int eth_ark_dev_rx_queue_count(void *rx_queue);<br>> -int eth_ark_rx_stop_queue(struct rte_eth_dev *dev, uint16_t queue_id);<br>> -int eth_ark_rx_start_queue(struct rte_eth_dev *dev, uint16_t queue_id);<br>> -uint16_t eth_ark_recv_pkts(void *rx_queue, struct rte_mbuf **rx_pkts,<br>> +int ark_dev_rx_queue_count(void *rx_queue);<br>> +int ark_rx_stop_queue(struct rte_eth_dev *dev, uint16_t queue_id);<br>> +int ark_rx_start_queue(struct rte_eth_dev *dev, uint16_t queue_id);<br>> +uint16_t ark_recv_pkts(void *rx_queue, struct rte_mbuf **rx_pkts,<br>>                            uint16_t nb_pkts);<br>> -void eth_ark_dev_rx_queue_release(void *rx_queue);<br>> -void eth_rx_queue_stats_get(void *vqueue, struct rte_eth_stats *stats);<br>> -void eth_rx_queue_stats_reset(void *vqueue);<br>> -void eth_ark_rx_dump_queue(struct rte_eth_dev *dev, uint16_t queue_id,<br>> +void ark_dev_rx_queue_release(void *rx_queue);<br>> +void ark_rx_queue_stats_get(void *vqueue, struct rte_eth_stats *stats);<br>> +void ark_rx_queue_stats_reset(void *vqueue);<br>> +void ark_rx_dump_queue(struct rte_eth_dev *dev, uint16_t queue_id,<br>>                            const char *msg);<br>><br>>  #endif<br>> diff --git a/drivers/net/ark/ark_ethdev_tx.c b/drivers/net/ark/ark_ethdev_tx.c<br>> index ca6cd297a1..2c61cd5486 100644<br>> --- a/drivers/net/ark/ark_ethdev_tx.c<br>> +++ b/drivers/net/ark/ark_ethdev_tx.c<br>> @@ -53,15 +53,15 @@ struct __rte_cache_aligned ark_tx_queue {<br>>  };<br>><br>>  /* Forward declarations */<br>> -static int eth_ark_tx_jumbo(struct ark_tx_queue *queue,<br>> +static int ark_tx_jumbo(struct ark_tx_queue *queue,<br>>                             struct rte_mbuf *mbuf,<br>>                             uint32_t *user_meta, uint8_t meta_cnt);<br>> -static int eth_ark_tx_hw_queue_config(struct ark_tx_queue *queue);<br>> +static int ark_tx_hw_queue_config(struct ark_tx_queue *queue);<br>>  static void free_completed_tx(struct ark_tx_queue *queue);<br>><br>>  /* ************************************************************************* */<br>>  static inline void<br>> -eth_ark_tx_desc_fill(struct ark_tx_queue *queue,<br>> +ark_tx_desc_fill(struct ark_tx_queue *queue,<br>>                      struct rte_mbuf *mbuf,<br>>                      uint8_t  flags,<br>>                      uint32_t *user_meta,<br>> @@ -101,7 +101,7 @@ eth_ark_tx_desc_fill(struct ark_tx_queue *queue,<br>><br>>  /* ************************************************************************* */<br>>  uint16_t<br>> -eth_ark_xmit_pkts(void *vtxq, struct rte_mbuf **tx_pkts, uint16_t nb_pkts)<br>> +ark_xmit_pkts(void *vtxq, struct rte_mbuf **tx_pkts, uint16_t nb_pkts)<br>>  {<br>>         struct ark_tx_queue *queue;<br>>         struct rte_mbuf *mbuf;<br>> @@ -159,12 +159,12 @@ eth_ark_xmit_pkts(void *vtxq, struct rte_mbuf **tx_pkts, uint16_t nb_pkts)<br>>                         tx_user_meta_hook(mbuf, user_meta, &user_len,<br>>                                           queue->ext_user_data);<br>>                 if (unlikely(mbuf->nb_segs != 1)) {<br>> -                       stat = eth_ark_tx_jumbo(queue, mbuf,<br>> +                       stat = ark_tx_jumbo(queue, mbuf,<br>>                                                 user_meta, user_len);<br>>                         if (unlikely(stat != 0))<br>>                                 break;          /* Queue is full */<br>>                 } else {<br>> -                       eth_ark_tx_desc_fill(queue, mbuf,<br>> +                       ark_tx_desc_fill(queue, mbuf,<br>>                                              ARK_DDM_SOP | ARK_DDM_EOP,<br>>                                              user_meta, user_len);<br>>                 }<br>> @@ -195,7 +195,7 @@ eth_ark_xmit_pkts(void *vtxq, struct rte_mbuf **tx_pkts, uint16_t nb_pkts)<br>><br>>  /* ************************************************************************* */<br>>  static int<br>> -eth_ark_tx_jumbo(struct ark_tx_queue *queue, struct rte_mbuf *mbuf,<br>> +ark_tx_jumbo(struct ark_tx_queue *queue, struct rte_mbuf *mbuf,<br>>                  uint32_t *user_meta, uint8_t meta_cnt)<br>>  {<br>>         struct rte_mbuf *next;<br>> @@ -212,7 +212,7 @@ eth_ark_tx_jumbo(struct ark_tx_queue *queue, struct rte_mbuf *mbuf,<br>>                 next = mbuf->next;<br>>                 flags |= (next == NULL) ? ARK_DDM_EOP : 0;<br>><br>> -               eth_ark_tx_desc_fill(queue, mbuf, flags, user_meta, meta_cnt);<br>> +               ark_tx_desc_fill(queue, mbuf, flags, user_meta, meta_cnt);<br>><br>>                 flags &= ~ARK_DDM_SOP;  /* drop SOP flags */<br>>                 meta_cnt = 0;           /* Meta only on SOP */<br>> @@ -224,7 +224,7 @@ eth_ark_tx_jumbo(struct ark_tx_queue *queue, struct rte_mbuf *mbuf,<br>><br>>  /* ************************************************************************* */<br>>  int<br>> -eth_ark_tx_queue_setup(struct rte_eth_dev *dev,<br>> +ark_tx_queue_setup(struct rte_eth_dev *dev,<br>>                        uint16_t queue_idx,<br>>                        uint16_t nb_desc,<br>>                        unsigned int socket_id,<br>> @@ -291,7 +291,7 @@ eth_ark_tx_queue_setup(struct rte_eth_dev *dev,<br>>         queue->ddm = RTE_PTR_ADD(ark->ddm.v, qidx * ARK_DDM_QOFFSET);<br>>         queue->mpu = RTE_PTR_ADD(ark->mputx.v, qidx * ARK_MPU_QOFFSET);<br>><br>> -       status = eth_ark_tx_hw_queue_config(queue);<br>> +       status = ark_tx_hw_queue_config(queue);<br>><br>>         if (unlikely(status != 0)) {<br>>                 rte_free(queue->meta_q);<br>> @@ -305,7 +305,7 @@ eth_ark_tx_queue_setup(struct rte_eth_dev *dev,<br>><br>>  /* ************************************************************************* */<br>>  static int<br>> -eth_ark_tx_hw_queue_config(struct ark_tx_queue *queue)<br>> +ark_tx_hw_queue_config(struct ark_tx_queue *queue)<br>>  {<br>>         rte_iova_t queue_base, ring_base, cons_index_addr;<br>><br>> @@ -333,7 +333,7 @@ eth_ark_tx_hw_queue_config(struct ark_tx_queue *queue)<br>><br>>  /* ************************************************************************* */<br>>  void<br>> -eth_ark_tx_queue_release(void *vtx_queue)<br>> +ark_tx_queue_release(void *vtx_queue)<br>>  {<br>>         struct ark_tx_queue *queue;<br>><br>> @@ -352,7 +352,7 @@ eth_ark_tx_queue_release(void *vtx_queue)<br>><br>>  /* ************************************************************************* */<br>>  int<br>> -eth_ark_tx_queue_stop(struct rte_eth_dev *dev, uint16_t queue_id)<br>> +ark_tx_queue_stop(struct rte_eth_dev *dev, uint16_t queue_id)<br>>  {<br>>         struct ark_tx_queue *queue;<br>>         int cnt = 0;<br>> @@ -376,7 +376,7 @@ eth_ark_tx_queue_stop(struct rte_eth_dev *dev, uint16_t queue_id)<br>>  }<br>><br>>  int<br>> -eth_ark_tx_queue_start(struct rte_eth_dev *dev, uint16_t queue_id)<br>> +ark_tx_queue_start(struct rte_eth_dev *dev, uint16_t queue_id)<br>>  {<br>>         struct ark_tx_queue *queue;<br>><br>> @@ -415,7 +415,7 @@ free_completed_tx(struct ark_tx_queue *queue)<br>><br>>  /* ************************************************************************* */<br>>  void<br>> -eth_tx_queue_stats_get(void *vqueue, struct rte_eth_stats *stats)<br>> +ark_tx_queue_stats_get(void *vqueue, struct rte_eth_stats *stats)<br>>  {<br>>         struct ark_tx_queue *queue;<br>>         struct ark_ddm_t *ddm;<br>> @@ -435,7 +435,7 @@ eth_tx_queue_stats_get(void *vqueue, struct rte_eth_stats *stats)<br>>  }<br>><br>>  void<br>> -eth_tx_queue_stats_reset(void *vqueue)<br>> +ark_tx_queue_stats_reset(void *vqueue)<br>>  {<br>>         struct ark_tx_queue *queue;<br>>         struct ark_ddm_t *ddm;<br>> diff --git a/drivers/net/ark/ark_ethdev_tx.h b/drivers/net/ark/ark_ethdev_tx.h<br>> index 7134dbfeed..ee66ed19cc 100644<br>> --- a/drivers/net/ark/ark_ethdev_tx.h<br>> +++ b/drivers/net/ark/ark_ethdev_tx.h<br>> @@ -10,18 +10,18 @@<br>>  #include <ethdev_driver.h><br>><br>><br>> -uint16_t eth_ark_xmit_pkts(void *vtxq,<br>> +uint16_t ark_xmit_pkts(void *vtxq,<br>>                            struct rte_mbuf **tx_pkts,<br>>                            uint16_t nb_pkts);<br>> -int eth_ark_tx_queue_setup(struct rte_eth_dev *dev,<br>> +int ark_tx_queue_setup(struct rte_eth_dev *dev,<br>>                            uint16_t queue_idx,<br>>                            uint16_t nb_desc,<br>>                            unsigned int socket_id,<br>>                            const struct rte_eth_txconf *tx_conf);<br>> -void eth_ark_tx_queue_release(void *vtx_queue);<br>> -int eth_ark_tx_queue_stop(struct rte_eth_dev *dev, uint16_t queue_id);<br>> -int eth_ark_tx_queue_start(struct rte_eth_dev *dev, uint16_t queue_id);<br>> -void eth_tx_queue_stats_get(void *vqueue, struct rte_eth_stats *stats);<br>> -void eth_tx_queue_stats_reset(void *vqueue);<br>> +void ark_tx_queue_release(void *vtx_queue);<br>> +int ark_tx_queue_stop(struct rte_eth_dev *dev, uint16_t queue_id);<br>> +int ark_tx_queue_start(struct rte_eth_dev *dev, uint16_t queue_id);<br>> +void ark_tx_queue_stats_get(void *vqueue, struct rte_eth_stats *stats);<br>> +void ark_tx_queue_stats_reset(void *vqueue);<br>><br>>  #endif<br>> --<br>> 2.47.3<br>><br></div>