14/10/2022 03:46, Chautru, Nicolas: > 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.
If it is only for testing, it is not an issue. I just want to avoid having different API paths for major API functions. > 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. > > > > > > > >
