Re: [dpdk-dev] [PATCH] net/cnxk: unify the file name

[Jerin Jacob <jerinjacobk@gmail.com>]

On Fri, Feb 18, 2022 at 1:14 PM Jerin Jacob <jerinjacobk@gmail.com> wrote:
>
> On Fri, Feb 18, 2022 at 11:48 AM Kiran Kumar Kokkilagadda
> <kirankumark@marvell.com> wrote:
> >
> >
> >
> > > -----Original Message-----
> > > From: jerinj@marvell.com <jerinj@marvell.com>
> > > Sent: Wednesday, February 9, 2022 10:20 PM
> > > To: dev@dpdk.org; Nithin Kumar Dabilpuram <ndabilpuram@marvell.com>;
> > > Kiran Kumar Kokkilagadda <kirankumark@marvell.com>; Sunil Kumar Kori
> > > <skori@marvell.com>; Satha Koteswara Rao Kottidi
> > > <skoteshwar@marvell.com>
> > > Cc: ferruh.yigit@intel.com; Satheesh Paul <psatheesh@marvell.com>; Jerin
> > > Jacob Kollanukkaran <jerinj@marvell.com>
> > > Subject: [dpdk-dev] [PATCH] net/cnxk: unify the file name
> > >
> > > From: Jerin Jacob <jerinj@marvell.com>
> > >
> > > expect cn*_rte_flow* files, none of the other file has rte in the file name,
> > > remove the rte to unify the file name across the directory.
> > >
> > > Signed-off-by: Jerin Jacob <jerinj@marvell.com>
> >
> > Acked-by: Kiran Kumar Kokkilagadda <kirankumark@marvell.com>
>
>
>
> Updated the commit message as follows and Applied to
> dpdk-next-net-mrvl/for-next-net. Thanks

Self reply: There another instance of rte found. Fixed it in v2
http://patches.dpdk.org/project/dpdk/patch/20220222183204.2378596-1-jerinj@marvell.com/

>
>     net/cnxk: unify the file names
>
>     Except for the cn*_rte_flow* files, none of the other
>     file has rte in the file name, remove the
>     rte to unify the file name across the directory.
>
>     Signed-off-by: Jerin Jacob <jerinj@marvell.com>
>     Acked-by: Kiran Kumar Kokkilagadda <kirankumark@marvell.com>
> >
> > > ---
> > >  drivers/net/cnxk/cn10k_rte_flow.c                 | 2 +-
> > >  drivers/net/cnxk/cn9k_rte_flow.c                  | 2 +-
> > >  drivers/net/cnxk/{cnxk_rte_flow.c => cnxk_flow.c} | 2 +-
> > > drivers/net/cnxk/{cnxk_rte_flow.h => cnxk_flow.h} | 0
> > >  drivers/net/cnxk/meson.build                      | 2 +-
> > >  5 files changed, 4 insertions(+), 4 deletions(-)  rename
> > > drivers/net/cnxk/{cnxk_rte_flow.c => cnxk_flow.c} (99%)  rename
> > > drivers/net/cnxk/{cnxk_rte_flow.h => cnxk_flow.h} (100%)
> > >
> > > diff --git a/drivers/net/cnxk/cn10k_rte_flow.c
> > > b/drivers/net/cnxk/cn10k_rte_flow.c
> > > index 529fb0e4b7..aed187c849 100644
> > > --- a/drivers/net/cnxk/cn10k_rte_flow.c
> > > +++ b/drivers/net/cnxk/cn10k_rte_flow.c
> > > @@ -1,7 +1,7 @@
> > >  /* SPDX-License-Identifier: BSD-3-Clause
> > >   * Copyright(C) 2020 Marvell.
> > >   */
> > > -#include <cnxk_rte_flow.h>
> > > +#include <cnxk_flow.h>
> > >  #include "cn10k_rte_flow.h"
> > >  #include "cn10k_ethdev.h"
> > >  #include "cn10k_rx.h"
> > > diff --git a/drivers/net/cnxk/cn9k_rte_flow.c
> > > b/drivers/net/cnxk/cn9k_rte_flow.c
> > > index b94d29e547..6460672afa 100644
> > > --- a/drivers/net/cnxk/cn9k_rte_flow.c
> > > +++ b/drivers/net/cnxk/cn9k_rte_flow.c
> > > @@ -1,7 +1,7 @@
> > >  /* SPDX-License-Identifier: BSD-3-Clause
> > >   * Copyright(C) 2020 Marvell.
> > >   */
> > > -#include <cnxk_rte_flow.h>
> > > +#include <cnxk_flow.h>
> > >  #include "cn9k_ethdev.h"
> > >  #include "cn9k_rte_flow.h"
> > >  #include "cn9k_rx.h"
> > > diff --git a/drivers/net/cnxk/cnxk_rte_flow.c b/drivers/net/cnxk/cnxk_flow.c
> > > similarity index 99% rename from drivers/net/cnxk/cnxk_rte_flow.c rename to
> > > drivers/net/cnxk/cnxk_flow.c index b08d7c34fa..8763ca63d6 100644
> > > --- a/drivers/net/cnxk/cnxk_rte_flow.c
> > > +++ b/drivers/net/cnxk/cnxk_flow.c
> > > @@ -1,7 +1,7 @@
> > >  /* SPDX-License-Identifier: BSD-3-Clause
> > >   * Copyright(C) 2021 Marvell.
> > >   */
> > > -#include <cnxk_rte_flow.h>
> > > +#include <cnxk_flow.h>
> > >
> > >  const struct cnxk_rte_flow_term_info term[] = {
> > >       [RTE_FLOW_ITEM_TYPE_ETH] = {ROC_NPC_ITEM_TYPE_ETH, diff --git
> > > a/drivers/net/cnxk/cnxk_rte_flow.h b/drivers/net/cnxk/cnxk_flow.h similarity
> > > index 100% rename from drivers/net/cnxk/cnxk_rte_flow.h rename to
> > > drivers/net/cnxk/cnxk_flow.h diff --git a/drivers/net/cnxk/meson.build
> > > b/drivers/net/cnxk/meson.build index cd8c13bd1c..af5e09dc65 100644
> > > --- a/drivers/net/cnxk/meson.build
> > > +++ b/drivers/net/cnxk/meson.build
> > > @@ -18,7 +18,7 @@ sources = files(
> > >          'cnxk_link.c',
> > >          'cnxk_lookup.c',
> > >          'cnxk_ptp.c',
> > > -        'cnxk_rte_flow.c',
> > > +        'cnxk_flow.c',
> > >          'cnxk_stats.c',
> > >          'cnxk_tm.c',
> > >  )
> > > --
> > > 2.35.1
> >