Hi, There are a few options: 1) Keep as is, but not too scalable to keep on adding PMD APIs. Still there is no plan on adding more API, so no rush. 2) Keep API internal only (but in that case only work with static lib I believe) 3) Take out off DPDK, but that is used by users notably OEMs and for test purpose (bbdev-test) and hence proven valuable to have everything in one place. 4) Include under bbdev API, my concern is that this is not really meant to be used by something else than bbdev-test hence can be confusing to VRAN vendor by polluting the API.
For the sake of discussion I have captured what 4) may look like here: https://patches.dpdk.org/project/dpdk/patch/20221014014205.38074-2-nicolas.chau...@intel.com/ Note that the actual configuration is just a void* since that configuration structure is device specific (we cannot standardize this really, nor would there be any value to do this here). Note that this is definitely not blocking migration from one driver to another Thomas. That companion function is purely to help bbdev-test. Happy to discuss more, I don't believe we need a swift decision though. Thanks Nic > -----Original Message----- > From: Thomas Monjalon <tho...@monjalon.net> > Sent: Thursday, October 13, 2022 2:25 AM > To: Chautru, Nicolas <nicolas.chau...@intel.com>; Gagandeep Singh > <g.si...@nxp.com>; Hemant Agrawal <hemant.agra...@nxp.com> > Cc: Akhil Goyal <gak...@marvell.com>; david.march...@redhat.com; > maxime.coque...@redhat.com; Tom Rix <t...@redhat.com>; dev@dpdk.org > Subject: Re: baseband PMD APIs > > To bbdev maintainers, > > Do you think we can avoid having some PMD-specific configuration? > It looks blocking migration from a driver to another. > How do we manage that? > > > 13/10/2022 11:07, Akhil Goyal: > > There are 6 bbdev PMDs, out of which 3 have pmd APIs - all from Intel > > These are the PMD APIs for baseband drivers. > > rte_acc_configure -> this is being used by 2 drivers(acc100 and acc200) > merged in a single folder. > > rte_fpga_5gnr_fec_configure > > rte_fpga_lte_fec_configure > > > > The fpga ones have almost same syntax. > > >
