> -----Original Message----- > From: Jerin Jacob <jerinjac...@gmail.com> > Sent: Friday, March 26, 2021 6:01 AM > To: McDaniel, Timothy <timothy.mcdan...@intel.com> > Cc: dpdk-dev <dev@dpdk.org>; Jerin Jacob <jer...@marvell.com>; Van Haaren, > Harry <harry.van.haa...@intel.com>; Ray Kinsella <m...@ashroe.eu>; Neil > Horman <nhor...@tuxdriver.com>; Rao, Nikhil <nikhil....@intel.com>; > Carrillo, Erik G <erik.g.carri...@intel.com>; Gujjar, Abhinandan S > <abhinandan.guj...@intel.com>; Pavan Nikhilesh > <pbhagavat...@marvell.com>; Hemant Agrawal <hemant.agra...@nxp.com>; > mattias.ronnblom <mattias.ronnb...@ericsson.com>; Mccarthy, Peter > <peter.mccar...@intel.com> > Subject: Re: [dpdk-dev] [PATCH 01/25] event/dlb2: add dlb v2.5 probe > > On Thu, Mar 25, 2021 at 1:01 AM McDaniel, Timothy > <timothy.mcdan...@intel.com> wrote: > > > > > > > > > -----Original Message----- > > > From: Jerin Jacob <jerinjac...@gmail.com> > > > Sent: Sunday, March 21, 2021 4:48 AM > > > To: McDaniel, Timothy <timothy.mcdan...@intel.com> > > > Cc: dpdk-dev <dev@dpdk.org>; Jerin Jacob <jer...@marvell.com>; Van > Haaren, > > > Harry <harry.van.haa...@intel.com>; Ray Kinsella <m...@ashroe.eu>; Neil > > > Horman <nhor...@tuxdriver.com>; Rao, Nikhil <nikhil....@intel.com>; > > > Carrillo, Erik G <erik.g.carri...@intel.com>; Gujjar, Abhinandan S > > > <abhinandan.guj...@intel.com>; Pavan Nikhilesh > > > <pbhagavat...@marvell.com>; Hemant Agrawal > <hemant.agra...@nxp.com>; > > > mattias.ronnblom <mattias.ronnb...@ericsson.com>; Mccarthy, Peter > > > <peter.mccar...@intel.com> > > > Subject: Re: [dpdk-dev] [PATCH 01/25] event/dlb2: add dlb v2.5 probe > > > > > > On Wed, Mar 17, 2021 at 3:49 AM Timothy McDaniel > > > <timothy.mcdan...@intel.com> wrote: > > > > > > > > This commit adds dlb v2.5 probe support, and updates > > > > parameter parsing. > > > > > > > > The dlb v2.5 device differs from dlb v2, in that the > > > > number of resources (ports, queues, ...) is different, > > > > so macros have been added to take the device version > > > > into account. > > > > > > > > This commit also cleans up a few issues in the original > > > > dlb2 source: > > > > - eliminate duplicate constant definitions > > > > - removed unused constant definitions > > > > > > > > Signed-off-by: Timothy McDaniel <timothy.mcdan...@intel.com> > > > > --- > > > > > > > > > > > -#define EVDEV_DLB2_NAME_PMD dlb2_event > > > > +#define EVDEV_DLB2_NAME_PMD dlb_event > > > > > > Is this an intended change? why change the driver's name. > > > > Yes, This is an intentional change. We will be using the same driver name > going forward, regardless of the hardware version. > > Internally, we know which version of the hardware is present. > > Since the driver name is still driver/event/dlb2. Keep it as same > prefix scheme with other drivers. > > > > > > Thanks, > > Tim > >
Would it be acceptable to rename drivers/event/dlb2 to drivers/event/dlb? We may have additional dlb devices in the pipeline, such as v3, and we would really like to have them all use a common name.
