[dpdk-dev] [PATCH v2 0/2] standardize rawdev names

Thomas Monjalon thomas at monjalon.net
Tue Jul 30 15:39:35 CEST 2019


The names of the rawdev drivers and the resultant libraries they built
were pretty inconsistent. In terms of directory names, some had "rawdev"
in the name, e.g. skeleton_rawdev, ifpga_rawdev, while others didn't.
Also, for the built .so/.a libraries, the names were not always
consistent between make and meson, as, while meson enforced a consistent
naming scheme, make did not, leading to differences - especially for
those drivers with "rawdev" in the directory name.

This patchset attempts to make things more standardized by removing
"rawdev" from all directory names - the drivers are in the "drivers/raw"
directory after all - and then standardizing the library names to
"librte_rawdev_<name>" for both make and meson.

Bruce Richardson (2):
  drivers/raw: remove rawdev from directory names
  drivers/raw: standardize naming

v2 (Thomas):
- update doc and MAINTAINERS file
- rebase for ntb and octeontx2

 MAINTAINERS                                        |  8 ++++----
 doc/guides/rawdevs/{ifpga_rawdev.rst => ifpga.rst} |  0
 doc/guides/rawdevs/index.rst                       |  4 ++--
 doc/guides/rawdevs/{ioat_rawdev.rst => ioat.rst}   |  0
 doc/guides/rel_notes/release_19_08.rst             |  3 +++
 drivers/raw/Makefile                               |  4 ++--
 drivers/raw/dpaa2_cmdif/Makefile                   |  4 ++--
 ...sion.map => rte_rawdev_dpaa2_cmdif_version.map} |  0
 drivers/raw/dpaa2_qdma/Makefile                    |  4 ++--
 ...rsion.map => rte_rawdev_dpaa2_qdma_version.map} |  0
 drivers/raw/{ifpga_rawdev => ifpga}/Makefile       |  6 +++---
 drivers/raw/{ifpga_rawdev => ifpga}/base/Makefile  |  0
 drivers/raw/{ifpga_rawdev => ifpga}/base/README    |  0
 .../raw/{ifpga_rawdev => ifpga}/base/ifpga_api.c   |  0
 .../raw/{ifpga_rawdev => ifpga}/base/ifpga_api.h   |  0
 .../{ifpga_rawdev => ifpga}/base/ifpga_compat.h    |  0
 .../{ifpga_rawdev => ifpga}/base/ifpga_defines.h   |  0
 .../{ifpga_rawdev => ifpga}/base/ifpga_enumerate.c |  0
 .../{ifpga_rawdev => ifpga}/base/ifpga_enumerate.h |  0
 .../base/ifpga_feature_dev.c                       |  0
 .../base/ifpga_feature_dev.h                       |  0
 .../raw/{ifpga_rawdev => ifpga}/base/ifpga_fme.c   |  0
 .../{ifpga_rawdev => ifpga}/base/ifpga_fme_dperf.c |  0
 .../{ifpga_rawdev => ifpga}/base/ifpga_fme_error.c |  0
 .../{ifpga_rawdev => ifpga}/base/ifpga_fme_iperf.c |  0
 .../{ifpga_rawdev => ifpga}/base/ifpga_fme_pr.c    |  0
 .../raw/{ifpga_rawdev => ifpga}/base/ifpga_hw.h    |  0
 .../raw/{ifpga_rawdev => ifpga}/base/ifpga_port.c  |  0
 .../base/ifpga_port_error.c                        |  0
 .../raw/{ifpga_rawdev => ifpga}/base/meson.build   |  0
 .../base/opae_at24_eeprom.c                        |  0
 .../base/opae_at24_eeprom.h                        |  0
 .../raw/{ifpga_rawdev => ifpga}/base/opae_debug.c  |  0
 .../raw/{ifpga_rawdev => ifpga}/base/opae_debug.h  |  0
 .../{ifpga_rawdev => ifpga}/base/opae_eth_group.c  |  0
 .../{ifpga_rawdev => ifpga}/base/opae_eth_group.h  |  0
 .../raw/{ifpga_rawdev => ifpga}/base/opae_hw_api.c |  0
 .../raw/{ifpga_rawdev => ifpga}/base/opae_hw_api.h |  0
 .../raw/{ifpga_rawdev => ifpga}/base/opae_i2c.c    |  0
 .../raw/{ifpga_rawdev => ifpga}/base/opae_i2c.h    |  0
 .../base/opae_ifpga_hw_api.c                       |  0
 .../base/opae_ifpga_hw_api.h                       |  0
 .../base/opae_intel_max10.c                        |  0
 .../base/opae_intel_max10.h                        |  0
 .../raw/{ifpga_rawdev => ifpga}/base/opae_osdep.h  |  0
 .../raw/{ifpga_rawdev => ifpga}/base/opae_spi.c    |  0
 .../raw/{ifpga_rawdev => ifpga}/base/opae_spi.h    |  0
 .../base/opae_spi_transaction.c                    |  0
 .../base/osdep_raw/osdep_generic.h                 |  0
 .../base/osdep_rte/osdep_generic.h                 |  0
 drivers/raw/{ifpga_rawdev => ifpga}/ifpga_rawdev.c |  0
 drivers/raw/{ifpga_rawdev => ifpga}/ifpga_rawdev.h |  0
 drivers/raw/{ifpga_rawdev => ifpga}/meson.build    |  0
 .../rte_rawdev_ifpga_version.map}                  |  0
 drivers/raw/ioat/Makefile                          |  4 ++--
 ...oat_version.map => rte_rawdev_ioat_version.map} |  0
 drivers/raw/meson.build                            |  6 +++---
 drivers/raw/ntb/Makefile                           |  4 ++--
 ..._ntb_version.map => rte_rawdev_ntb_version.map} |  0
 drivers/raw/octeontx2_dma/Makefile                 |  4 ++--
 ...on.map => rte_rawdev_octeontx2_dma_version.map} |  0
 drivers/raw/{skeleton_rawdev => skeleton}/Makefile |  4 ++--
 .../raw/{skeleton_rawdev => skeleton}/meson.build  |  0
 .../rte_rawdev_skeleton_version.map}               |  0
 .../skeleton_rawdev.c                              |  0
 .../skeleton_rawdev.h                              |  0
 .../skeleton_rawdev_test.c                         |  0
 mk/rte.app.mk                                      | 14 +++++++-------
 68 files changed, 36 insertions(+), 33 deletions(-)
 rename doc/guides/rawdevs/{ifpga_rawdev.rst => ifpga.rst} (100%)
 rename doc/guides/rawdevs/{ioat_rawdev.rst => ioat.rst} (100%)
 rename drivers/raw/dpaa2_cmdif/{rte_pmd_dpaa2_cmdif_version.map => rte_rawdev_dpaa2_cmdif_version.map} (100%)
 rename drivers/raw/dpaa2_qdma/{rte_pmd_dpaa2_qdma_version.map => rte_rawdev_dpaa2_qdma_version.map} (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/Makefile (82%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/Makefile (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/README (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/ifpga_api.c (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/ifpga_api.h (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/ifpga_compat.h (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/ifpga_defines.h (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/ifpga_enumerate.c (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/ifpga_enumerate.h (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/ifpga_feature_dev.c (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/ifpga_feature_dev.h (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/ifpga_fme.c (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/ifpga_fme_dperf.c (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/ifpga_fme_error.c (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/ifpga_fme_iperf.c (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/ifpga_fme_pr.c (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/ifpga_hw.h (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/ifpga_port.c (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/ifpga_port_error.c (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/meson.build (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/opae_at24_eeprom.c (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/opae_at24_eeprom.h (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/opae_debug.c (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/opae_debug.h (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/opae_eth_group.c (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/opae_eth_group.h (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/opae_hw_api.c (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/opae_hw_api.h (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/opae_i2c.c (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/opae_i2c.h (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/opae_ifpga_hw_api.c (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/opae_ifpga_hw_api.h (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/opae_intel_max10.c (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/opae_intel_max10.h (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/opae_osdep.h (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/opae_spi.c (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/opae_spi.h (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/opae_spi_transaction.c (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/osdep_raw/osdep_generic.h (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/base/osdep_rte/osdep_generic.h (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/ifpga_rawdev.c (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/ifpga_rawdev.h (100%)
 rename drivers/raw/{ifpga_rawdev => ifpga}/meson.build (100%)
 rename drivers/raw/{ifpga_rawdev/rte_pmd_ifpga_rawdev_version.map => ifpga/rte_rawdev_ifpga_version.map} (100%)
 rename drivers/raw/ioat/{rte_pmd_ioat_version.map => rte_rawdev_ioat_version.map} (100%)
 rename drivers/raw/ntb/{rte_pmd_ntb_version.map => rte_rawdev_ntb_version.map} (100%)
 rename drivers/raw/octeontx2_dma/{rte_pmd_octeontx2_dma_version.map => rte_rawdev_octeontx2_dma_version.map} (100%)
 rename drivers/raw/{skeleton_rawdev => skeleton}/Makefile (84%)
 rename drivers/raw/{skeleton_rawdev => skeleton}/meson.build (100%)
 rename drivers/raw/{skeleton_rawdev/rte_pmd_skeleton_rawdev_version.map => skeleton/rte_rawdev_skeleton_version.map} (100%)
 rename drivers/raw/{skeleton_rawdev => skeleton}/skeleton_rawdev.c (100%)
 rename drivers/raw/{skeleton_rawdev => skeleton}/skeleton_rawdev.h (100%)
 rename drivers/raw/{skeleton_rawdev => skeleton}/skeleton_rawdev_test.c (100%)

-- 
2.21.0



More information about the dev mailing list