Thanks Stephen. Agree. Will remove dependency on PCI for now. And num_queues should have been declared as uint16_t.
> -----Original Message----- > From: Stephen Hemminger [mailto:step...@networkplumber.org] > Sent: Friday 1 September 2017 21:04 > To: Mokhtar, Amr <amr.mokh...@intel.com> > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] [RFC] Wireless Base Band Device (bbdev) > > > > +/* Forward declaration */ > > +struct rte_pci_device; > > + > > +/** Device information structure used by an application to discover a > > +devices > > + * capabilities and current configuration */ struct rte_bbdev_info { > > + int socket_id; /**< NUMA socket that device is on */ > > + const char *dev_name; /**< Unique device name */ > > + const struct rte_pci_device *pci_dev; /**< PCI information */ > > + unsigned int num_queues; /**< Number of queues currently configured > */ > > + struct rte_bbdev_conf conf; /**< Current device configuration */ > > + bool started; /**< Set if device is currently started */ > > + struct rte_bbdev_driver_info drv; /**< Info from device driver */ > > +}; > > Please don't build in dependency on PCI from the beginning. > Number of queues can be uint16_t ?
