net/cnxk: unify the file name

Message ID 20220209165026.790880-1-jerinj@marvell.com (mailing list archive)
State Changes Requested, archived
Delegated to: Jerin Jacob
Headers
Series net/cnxk: unify the file name |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS
ci/github-robot: build success github build: passed
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-broadcom-Performance success Performance Testing PASS
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-aarch64-unit-testing success Testing PASS
ci/iol-x86_64-compile-testing success Testing PASS
ci/iol-abi-testing success Testing PASS
ci/iol-x86_64-unit-testing success Testing PASS

Commit Message

Jerin Jacob Kollanukkaran Feb. 9, 2022, 4:50 p.m. UTC
  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>
---
 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%)
  

Comments

Kiran Kumar Kokkilagadda Feb. 18, 2022, 6:18 a.m. UTC | #1
> -----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>

> ---
>  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
  
Jerin Jacob Feb. 18, 2022, 7:44 a.m. UTC | #2
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

    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
>
  
Jerin Jacob Feb. 22, 2022, 6:32 p.m. UTC | #3
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
> >
  

Patch

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',
 )