[RFC v3 17/26] bus/vdev: make driver-only headers private

[Xu, Rosen]

Hi,

> -----Original Message-----
> From: David Marchand <david.marchand at redhat.com>
> Sent: Thursday, July 28, 2022 23:27
> To: dev at dpdk.org
> Cc: Gagandeep Singh <g.singh at nxp.com>; Hemant Agrawal
> <hemant.agrawal at nxp.com>; Chautru, Nicolas <nicolas.chautru at intel.com>;
> Ray Kinsella <mdr at ashroe.eu>; Daly, Lee <lee.daly at intel.com>; Zhang, Roy
> Fan <roy.fan.zhang at intel.com>; Ashish Gupta <ashish.gupta at marvell.com>;
> Sunila Sahu <ssahu at marvell.com>; Ruifeng Wang <ruifeng.wang at arm.com>;
> Ajit Khaparde <ajit.khaparde at broadcom.com>; Raveendra Padasalagi
> <raveendra.padasalagi at broadcom.com>; Vikas Gupta
> <vikas.gupta at broadcom.com>; Chandubabu Namburu <chandu at amd.com>;
> De Lara Guarch, Pablo <pablo.de.lara.guarch at intel.com>; Michael Shamis
> <michaelsh at marvell.com>; Liron Himi <lironh at marvell.com>; Chengwen
> Feng <fengchengwen at huawei.com>; Laatz, Kevin <kevin.laatz at intel.com>;
> Richardson, Bruce <bruce.richardson at intel.com>; Sachin Saxena
> <sachin.saxena at oss.nxp.com>; Jerin Jacob <jerinj at marvell.com>; Liang Ma
> <liangma at liangbit.com>; Mccarthy, Peter <peter.mccarthy at intel.com>; Van
> Haaren, Harry <harry.van.haaren at intel.com>; John W. Linville
> <linville at tuxdriver.com>; Loftus, Ciara <ciara.loftus at intel.com>; Zhang, Qi Z
> <qi.z.zhang at intel.com>; Chas Williams <chas3 at att.com>; Min Hu (Connor)
> <humin29 at huawei.com>; Gaetan Rivet <grive at u256.net>; Xu, Rosen
> <rosen.xu at intel.com>; Jakub Grajciar <jgrajcia at cisco.com>; Zyta Szpak
> <zr at semihalf.com>; Tetsuya Mukawa <mtetsuyah at gmail.com>; Harman
> Kalra <hkalra at marvell.com>; Singh, Jasvinder <jasvinder.singh at intel.com>;
> Dumitrescu, Cristian <cristian.dumitrescu at intel.com>; Matan Azrad
> <matan at nvidia.com>; Maxime Coquelin <maxime.coquelin at redhat.com>;
> Xia, Chenbo <chenbo.xia at intel.com>; Jakub Palider <jpalider at marvell.com>;
> Tomasz Duszynski <tduszynski at marvell.com>; Zhang, Tianfei
> <tianfei.zhang at intel.com>; Thomas Monjalon <thomas at monjalon.net>;
> Ferruh Yigit <ferruh.yigit at xilinx.com>; Andrew Rybchenko
> <andrew.rybchenko at oktetlabs.ru>
> Subject: [RFC v3 17/26] bus/vdev: make driver-only headers private
> 
> The vdev bus interface is for drivers only.
> Mark as internal and move the header in the driver headers list.
> 
> While at it, cleanup the code:
> - fix indentation,
> - remove unneeded reference to bus specific singleton object,
> - remove unneeded list head structure type,
> - reorder the definitions and macro manipulating the bus singleton object,
> - remove inclusion of rte_bus.h and fix the code that relied on implicit
>   inclusion,
> 
> Signed-off-by: David Marchand <david.marchand at redhat.com>
> ---
> Changes since RFC v2:
> - squashed cleanup for this bus code from other patches,
> - updated release notes,
> 
> ---
>  app/test/test_vdev.c                          |   2 +-
>  doc/guides/rel_notes/release_22_11.rst        |   4 +-
>  drivers/baseband/la12xx/bbdev_la12xx.c        |   2 +-
>  drivers/baseband/null/bbdev_null.c            |   2 +-
>  .../baseband/turbo_sw/bbdev_turbo_software.c  |   2 +-
>  drivers/bus/vdev/bus_vdev_driver.h            | 151 ++++++++++++++++++
>  drivers/bus/vdev/meson.build                  |   1 +
>  drivers/bus/vdev/rte_bus_vdev.h               | 134 ----------------
>  drivers/bus/vdev/vdev.c                       |   8 +-
>  drivers/bus/vdev/vdev_params.c                |   1 -
>  drivers/bus/vdev/version.map                  |   9 +-
>  drivers/compress/isal/isal_compress_pmd.c     |   2 +-
>  drivers/compress/zlib/zlib_pmd.c              |   2 +-
>  drivers/crypto/armv8/rte_armv8_pmd.c          |   2 +-
>  drivers/crypto/bcmfs/bcmfs_device.h           |   2 +-
>  drivers/crypto/caam_jr/caam_jr.c              |   2 +-
>  drivers/crypto/ccp/rte_ccp_pmd.c              |   2 +-
>  drivers/crypto/ipsec_mb/ipsec_mb_private.c    |   2 +-
>  drivers/crypto/ipsec_mb/ipsec_mb_private.h    |   2 +-
>  drivers/crypto/ipsec_mb/pmd_kasumi.c          |   2 +-
>  drivers/crypto/mvsam/rte_mrvl_pmd.c           |   2 +-
>  drivers/crypto/null/null_crypto_pmd.c         |   2 +-
>  drivers/crypto/openssl/rte_openssl_pmd.c      |   2 +-
>  drivers/crypto/scheduler/scheduler_pmd.c      |   2 +-
>  drivers/dma/skeleton/skeleton_dmadev.c        |   2 +-
>  drivers/event/dpaa2/dpaa2_eventdev.c          |   2 +-
>  drivers/event/dpaa2/dpaa2_eventdev_selftest.c |   2 +-
>  drivers/event/octeontx/ssovf_evdev.c          |   2 +-
>  drivers/event/octeontx/ssovf_evdev_selftest.c |   2 +-
>  drivers/event/opdl/opdl_evdev.c               |   2 +-
>  drivers/event/opdl/opdl_evdev_init.c          |   2 +-
>  drivers/event/opdl/opdl_test.c                |   2 +-
>  drivers/event/skeleton/skeleton_eventdev.c    |   2 +-
>  drivers/event/sw/sw_evdev.c                   |   2 +-
>  drivers/event/sw/sw_evdev_selftest.c          |   2 +-
>  drivers/net/af_packet/rte_eth_af_packet.c     |   2 +-
>  drivers/net/af_xdp/rte_eth_af_xdp.c           |   2 +-
>  drivers/net/bonding/rte_eth_bond_api.c        |   2 +-
>  drivers/net/bonding/rte_eth_bond_pmd.c        |   2 +-
>  drivers/net/failsafe/failsafe.c               |   3 +-
>  drivers/net/ipn3ke/ipn3ke_ethdev.h            |   2 +-
>  drivers/net/kni/rte_eth_kni.c                 |   2 +-
>  drivers/net/memif/memif_socket.c              |   2 +-
>  drivers/net/memif/rte_eth_memif.c             |   2 +-
>  drivers/net/mvneta/mvneta_ethdev.c            |   2 +-
>  drivers/net/mvpp2/mrvl_ethdev.c               |   2 +-
>  drivers/net/null/rte_eth_null.c               |   2 +-
>  drivers/net/octeontx/octeontx_ethdev.c        |   2 +-
>  drivers/net/pcap/pcap_ethdev.c                |   2 +-
>  drivers/net/pfe/pfe_ethdev.c                  |   2 +-
>  drivers/net/ring/rte_eth_ring.c               |   2 +-
>  drivers/net/softnic/rte_eth_softnic.c         |   2 +-
>  drivers/net/tap/rte_eth_tap.c                 |   2 +-
>  drivers/net/vdev_netvsc/vdev_netvsc.c         |   2 +-
>  drivers/net/vhost/rte_eth_vhost.c             |   2 +-
>  drivers/net/virtio/virtio_user_ethdev.c       |   2 +-
>  drivers/raw/cnxk_gpio/cnxk_gpio.c             |   2 +-
>  drivers/raw/dpaa2_cmdif/dpaa2_cmdif.c         |   2 +-
>  drivers/raw/ifpga/ifpga_rawdev.c              |   2 +-
>  drivers/raw/skeleton/skeleton_rawdev.c        |   2 +-
>  drivers/raw/skeleton/skeleton_rawdev_test.c   |   2 +-
>  lib/ethdev/ethdev_vdev.h                      |   2 +-
>  lib/eventdev/eventdev_pmd_vdev.h              |   2 +-
>  63 files changed, 221 insertions(+), 200 deletions(-)  create mode 100644
> drivers/bus/vdev/bus_vdev_driver.h

Acked-by: Rosen Xu <rosen.xu at intel.com>