06/06/2023 12:02, Nipun Gupta: > +/* Forward declarations */ > +struct rte_cdx_device; > +struct rte_cdx_driver; > +struct rte_cdx_bus;
You should be very careful about what you export. > + > +#define CDX_BUS_DEVICES_PATH "/sys/bus/cdx/devices" This define is not needed in the .h file. > + > +#define CDX_MAX_RESOURCE 4 This size looks required for rte_cdx_device, but as an exported constant, it should be prefixed with RTE_ > + > +/** Any CDX device identifier (vendor, device). */ > +#define RTE_CDX_ANY_ID (0xffff)

