[dpdk-dev] [PATCH 01/25] event/dlb2: add dlb v2.5 probe

[Jerin Jacob]

On Thu, Mar 25, 2021 at 1:01 AM McDaniel, Timothy
<timothy.mcdaniel at intel.com> wrote:
>
>
>
> > -----Original Message-----
> > From: Jerin Jacob <jerinjacobk at gmail.com>
> > Sent: Sunday, March 21, 2021 4:48 AM
> > To: McDaniel, Timothy <timothy.mcdaniel at intel.com>
> > Cc: dpdk-dev <dev at dpdk.org>; Jerin Jacob <jerinj at marvell.com>; Van Haaren,
> > Harry <harry.van.haaren at intel.com>; Ray Kinsella <mdr at ashroe.eu>; Neil
> > Horman <nhorman at tuxdriver.com>; Rao, Nikhil <nikhil.rao at intel.com>;
> > Carrillo, Erik G <Erik.G.Carrillo at intel.com>; Gujjar, Abhinandan S
> > <abhinandan.gujjar at intel.com>; Pavan Nikhilesh
> > <pbhagavatula at marvell.com>; Hemant Agrawal <hemant.agrawal at nxp.com>;
> > mattias.ronnblom <mattias.ronnblom at ericsson.com>; Mccarthy, Peter
> > <Peter.Mccarthy at 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.mcdaniel at 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.mcdaniel at 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
>