* [dpdk-dev] [PATCH] net/cnxk: unify the file name
@ 2022-02-09 16:50 jerinj
2022-02-18 6:18 ` Kiran Kumar Kokkilagadda
2022-02-22 18:32 ` [dpdk-dev] [PATCH v2] net/cnxk: unify file names jerinj
0 siblings, 2 replies; 7+ messages in thread
From: jerinj @ 2022-02-09 16:50 UTC (permalink / raw)
To: dev, Nithin Dabilpuram, Kiran Kumar K, Sunil Kumar Kori, Satha Rao
Cc: ferruh.yigit, psatheesh, Jerin Jacob
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%)
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
^ permalink raw reply [flat|nested] 7+ messages in thread
* RE: [dpdk-dev] [PATCH] net/cnxk: unify the file name
2022-02-09 16:50 [dpdk-dev] [PATCH] net/cnxk: unify the file name jerinj
@ 2022-02-18 6:18 ` Kiran Kumar Kokkilagadda
2022-02-18 7:44 ` Jerin Jacob
2022-02-22 18:32 ` [dpdk-dev] [PATCH v2] net/cnxk: unify file names jerinj
1 sibling, 1 reply; 7+ messages in thread
From: Kiran Kumar Kokkilagadda @ 2022-02-18 6:18 UTC (permalink / raw)
To: Jerin Jacob Kollanukkaran, dev, Nithin Kumar Dabilpuram,
Sunil Kumar Kori, Satha Koteswara Rao Kottidi
Cc: ferruh.yigit, Satheesh Paul, Jerin Jacob Kollanukkaran
> -----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
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [dpdk-dev] [PATCH] net/cnxk: unify the file name
2022-02-18 6:18 ` Kiran Kumar Kokkilagadda
@ 2022-02-18 7:44 ` Jerin Jacob
2022-02-22 18:32 ` Jerin Jacob
0 siblings, 1 reply; 7+ messages in thread
From: Jerin Jacob @ 2022-02-18 7:44 UTC (permalink / raw)
To: Kiran Kumar Kokkilagadda, Ferruh Yigit
Cc: Jerin Jacob Kollanukkaran, dev, Nithin Kumar Dabilpuram,
Sunil Kumar Kori, Satha Koteswara Rao Kottidi, Satheesh Paul
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
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* [dpdk-dev] [PATCH v2] net/cnxk: unify file names
2022-02-09 16:50 [dpdk-dev] [PATCH] net/cnxk: unify the file name jerinj
2022-02-18 6:18 ` Kiran Kumar Kokkilagadda
@ 2022-02-22 18:32 ` jerinj
2022-02-23 9:08 ` Jerin Jacob
1 sibling, 1 reply; 7+ messages in thread
From: jerinj @ 2022-02-22 18:32 UTC (permalink / raw)
To: dev, Nithin Dabilpuram, Kiran Kumar K, Sunil Kumar Kori, Satha Rao
Cc: ferruh.yigit, psatheesh, Jerin Jacob
From: Jerin Jacob <jerinj@marvell.com>
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 K <kirankumark@marvell.com>
---
v2..v1:
-Missed renaming cn10k_rte* and cn9k_rte* and fixed it
drivers/net/cnxk/cn10k_ethdev.c | 2 +-
drivers/net/cnxk/{cn10k_rte_flow.c => cn10k_flow.c} | 4 ++--
drivers/net/cnxk/{cn10k_rte_flow.h => cn10k_flow.h} | 0
drivers/net/cnxk/cn9k_ethdev.c | 2 +-
drivers/net/cnxk/{cn9k_rte_flow.c => cn9k_flow.c} | 4 ++--
drivers/net/cnxk/{cn9k_rte_flow.h => cn9k_flow.h} | 0
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 | 6 +++---
9 files changed, 10 insertions(+), 10 deletions(-)
rename drivers/net/cnxk/{cn10k_rte_flow.c => cn10k_flow.c} (99%)
rename drivers/net/cnxk/{cn10k_rte_flow.h => cn10k_flow.h} (100%)
rename drivers/net/cnxk/{cn9k_rte_flow.c => cn9k_flow.c} (97%)
rename drivers/net/cnxk/{cn9k_rte_flow.h => cn9k_flow.h} (100%)
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_ethdev.c b/drivers/net/cnxk/cn10k_ethdev.c
index 9bb08e1824..f3003dd3df 100644
--- a/drivers/net/cnxk/cn10k_ethdev.c
+++ b/drivers/net/cnxk/cn10k_ethdev.c
@@ -2,7 +2,7 @@
* Copyright(C) 2021 Marvell.
*/
#include "cn10k_ethdev.h"
-#include "cn10k_rte_flow.h"
+#include "cn10k_flow.h"
#include "cn10k_rx.h"
#include "cn10k_tx.h"
diff --git a/drivers/net/cnxk/cn10k_rte_flow.c b/drivers/net/cnxk/cn10k_flow.c
similarity index 99%
rename from drivers/net/cnxk/cn10k_rte_flow.c
rename to drivers/net/cnxk/cn10k_flow.c
index 529fb0e4b7..63281d41a4 100644
--- a/drivers/net/cnxk/cn10k_rte_flow.c
+++ b/drivers/net/cnxk/cn10k_flow.c
@@ -1,8 +1,8 @@
/* SPDX-License-Identifier: BSD-3-Clause
* Copyright(C) 2020 Marvell.
*/
-#include <cnxk_rte_flow.h>
-#include "cn10k_rte_flow.h"
+#include <cnxk_flow.h>
+#include "cn10k_flow.h"
#include "cn10k_ethdev.h"
#include "cn10k_rx.h"
diff --git a/drivers/net/cnxk/cn10k_rte_flow.h b/drivers/net/cnxk/cn10k_flow.h
similarity index 100%
rename from drivers/net/cnxk/cn10k_rte_flow.h
rename to drivers/net/cnxk/cn10k_flow.h
diff --git a/drivers/net/cnxk/cn9k_ethdev.c b/drivers/net/cnxk/cn9k_ethdev.c
index 01e3850561..a890829065 100644
--- a/drivers/net/cnxk/cn9k_ethdev.c
+++ b/drivers/net/cnxk/cn9k_ethdev.c
@@ -2,7 +2,7 @@
* Copyright(C) 2021 Marvell.
*/
#include "cn9k_ethdev.h"
-#include "cn9k_rte_flow.h"
+#include "cn9k_flow.h"
#include "cn9k_rx.h"
#include "cn9k_tx.h"
diff --git a/drivers/net/cnxk/cn9k_rte_flow.c b/drivers/net/cnxk/cn9k_flow.c
similarity index 97%
rename from drivers/net/cnxk/cn9k_rte_flow.c
rename to drivers/net/cnxk/cn9k_flow.c
index b94d29e547..d67580bf27 100644
--- a/drivers/net/cnxk/cn9k_rte_flow.c
+++ b/drivers/net/cnxk/cn9k_flow.c
@@ -1,9 +1,9 @@
/* 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_flow.h"
#include "cn9k_rx.h"
struct rte_flow *
diff --git a/drivers/net/cnxk/cn9k_rte_flow.h b/drivers/net/cnxk/cn9k_flow.h
similarity index 100%
rename from drivers/net/cnxk/cn9k_rte_flow.h
rename to drivers/net/cnxk/cn9k_flow.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 83af1f4d0c..016a79b2a8 100644
--- a/drivers/net/cnxk/meson.build
+++ b/drivers/net/cnxk/meson.build
@@ -19,7 +19,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',
)
@@ -28,7 +28,7 @@ sources = files(
sources += files(
'cn9k_ethdev.c',
'cn9k_ethdev_sec.c',
- 'cn9k_rte_flow.c',
+ 'cn9k_flow.c',
'cn9k_rx_select.c',
'cn9k_tx_select.c',
)
@@ -107,7 +107,7 @@ sources += files(
sources += files(
'cn10k_ethdev.c',
'cn10k_ethdev_sec.c',
- 'cn10k_rte_flow.c',
+ 'cn10k_flow.c',
'cn10k_rx_select.c',
'cn10k_tx_select.c',
)
--
2.35.1
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [dpdk-dev] [PATCH] net/cnxk: unify the file name
2022-02-18 7:44 ` Jerin Jacob
@ 2022-02-22 18:32 ` Jerin Jacob
0 siblings, 0 replies; 7+ messages in thread
From: Jerin Jacob @ 2022-02-22 18:32 UTC (permalink / raw)
To: Kiran Kumar Kokkilagadda, Ferruh Yigit
Cc: Jerin Jacob Kollanukkaran, dev, Nithin Kumar Dabilpuram,
Sunil Kumar Kori, Satha Koteswara Rao Kottidi, Satheesh Paul
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
> >
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [dpdk-dev] [PATCH v2] net/cnxk: unify file names
2022-02-22 18:32 ` [dpdk-dev] [PATCH v2] net/cnxk: unify file names jerinj
@ 2022-02-23 9:08 ` Jerin Jacob
2022-02-23 12:18 ` Ferruh Yigit
0 siblings, 1 reply; 7+ messages in thread
From: Jerin Jacob @ 2022-02-23 9:08 UTC (permalink / raw)
To: Jerin Jacob
Cc: dpdk-dev, Nithin Dabilpuram, Kiran Kumar K, Sunil Kumar Kori,
Satha Rao, Ferruh Yigit, Satheesh Paul
On Wed, Feb 23, 2022 at 12:01 AM <jerinj@marvell.com> wrote:
>
> From: Jerin Jacob <jerinj@marvell.com>
>
> 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 K <kirankumark@marvell.com>
Applied to dpdk-next-net-mrvl/for-next-net. Thanks
> ---
>
> v2..v1:
>
> -Missed renaming cn10k_rte* and cn9k_rte* and fixed it
>
>
> drivers/net/cnxk/cn10k_ethdev.c | 2 +-
> drivers/net/cnxk/{cn10k_rte_flow.c => cn10k_flow.c} | 4 ++--
> drivers/net/cnxk/{cn10k_rte_flow.h => cn10k_flow.h} | 0
> drivers/net/cnxk/cn9k_ethdev.c | 2 +-
> drivers/net/cnxk/{cn9k_rte_flow.c => cn9k_flow.c} | 4 ++--
> drivers/net/cnxk/{cn9k_rte_flow.h => cn9k_flow.h} | 0
> 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 | 6 +++---
> 9 files changed, 10 insertions(+), 10 deletions(-)
> rename drivers/net/cnxk/{cn10k_rte_flow.c => cn10k_flow.c} (99%)
> rename drivers/net/cnxk/{cn10k_rte_flow.h => cn10k_flow.h} (100%)
> rename drivers/net/cnxk/{cn9k_rte_flow.c => cn9k_flow.c} (97%)
> rename drivers/net/cnxk/{cn9k_rte_flow.h => cn9k_flow.h} (100%)
> 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_ethdev.c b/drivers/net/cnxk/cn10k_ethdev.c
> index 9bb08e1824..f3003dd3df 100644
> --- a/drivers/net/cnxk/cn10k_ethdev.c
> +++ b/drivers/net/cnxk/cn10k_ethdev.c
> @@ -2,7 +2,7 @@
> * Copyright(C) 2021 Marvell.
> */
> #include "cn10k_ethdev.h"
> -#include "cn10k_rte_flow.h"
> +#include "cn10k_flow.h"
> #include "cn10k_rx.h"
> #include "cn10k_tx.h"
>
> diff --git a/drivers/net/cnxk/cn10k_rte_flow.c b/drivers/net/cnxk/cn10k_flow.c
> similarity index 99%
> rename from drivers/net/cnxk/cn10k_rte_flow.c
> rename to drivers/net/cnxk/cn10k_flow.c
> index 529fb0e4b7..63281d41a4 100644
> --- a/drivers/net/cnxk/cn10k_rte_flow.c
> +++ b/drivers/net/cnxk/cn10k_flow.c
> @@ -1,8 +1,8 @@
> /* SPDX-License-Identifier: BSD-3-Clause
> * Copyright(C) 2020 Marvell.
> */
> -#include <cnxk_rte_flow.h>
> -#include "cn10k_rte_flow.h"
> +#include <cnxk_flow.h>
> +#include "cn10k_flow.h"
> #include "cn10k_ethdev.h"
> #include "cn10k_rx.h"
>
> diff --git a/drivers/net/cnxk/cn10k_rte_flow.h b/drivers/net/cnxk/cn10k_flow.h
> similarity index 100%
> rename from drivers/net/cnxk/cn10k_rte_flow.h
> rename to drivers/net/cnxk/cn10k_flow.h
> diff --git a/drivers/net/cnxk/cn9k_ethdev.c b/drivers/net/cnxk/cn9k_ethdev.c
> index 01e3850561..a890829065 100644
> --- a/drivers/net/cnxk/cn9k_ethdev.c
> +++ b/drivers/net/cnxk/cn9k_ethdev.c
> @@ -2,7 +2,7 @@
> * Copyright(C) 2021 Marvell.
> */
> #include "cn9k_ethdev.h"
> -#include "cn9k_rte_flow.h"
> +#include "cn9k_flow.h"
> #include "cn9k_rx.h"
> #include "cn9k_tx.h"
>
> diff --git a/drivers/net/cnxk/cn9k_rte_flow.c b/drivers/net/cnxk/cn9k_flow.c
> similarity index 97%
> rename from drivers/net/cnxk/cn9k_rte_flow.c
> rename to drivers/net/cnxk/cn9k_flow.c
> index b94d29e547..d67580bf27 100644
> --- a/drivers/net/cnxk/cn9k_rte_flow.c
> +++ b/drivers/net/cnxk/cn9k_flow.c
> @@ -1,9 +1,9 @@
> /* 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_flow.h"
> #include "cn9k_rx.h"
>
> struct rte_flow *
> diff --git a/drivers/net/cnxk/cn9k_rte_flow.h b/drivers/net/cnxk/cn9k_flow.h
> similarity index 100%
> rename from drivers/net/cnxk/cn9k_rte_flow.h
> rename to drivers/net/cnxk/cn9k_flow.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 83af1f4d0c..016a79b2a8 100644
> --- a/drivers/net/cnxk/meson.build
> +++ b/drivers/net/cnxk/meson.build
> @@ -19,7 +19,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',
> )
> @@ -28,7 +28,7 @@ sources = files(
> sources += files(
> 'cn9k_ethdev.c',
> 'cn9k_ethdev_sec.c',
> - 'cn9k_rte_flow.c',
> + 'cn9k_flow.c',
> 'cn9k_rx_select.c',
> 'cn9k_tx_select.c',
> )
> @@ -107,7 +107,7 @@ sources += files(
> sources += files(
> 'cn10k_ethdev.c',
> 'cn10k_ethdev_sec.c',
> - 'cn10k_rte_flow.c',
> + 'cn10k_flow.c',
> 'cn10k_rx_select.c',
> 'cn10k_tx_select.c',
> )
> --
> 2.35.1
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [dpdk-dev] [PATCH v2] net/cnxk: unify file names
2022-02-23 9:08 ` Jerin Jacob
@ 2022-02-23 12:18 ` Ferruh Yigit
0 siblings, 0 replies; 7+ messages in thread
From: Ferruh Yigit @ 2022-02-23 12:18 UTC (permalink / raw)
To: Jerin Jacob, Jerin Jacob
Cc: dpdk-dev, Nithin Dabilpuram, Kiran Kumar K, Sunil Kumar Kori,
Satha Rao, Satheesh Paul
On 2/23/2022 9:08 AM, Jerin Jacob wrote:
> On Wed, Feb 23, 2022 at 12:01 AM <jerinj@marvell.com> wrote:
>>
>> From: Jerin Jacob <jerinj@marvell.com>
>>
>> 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 K <kirankumark@marvell.com>
>
> Applied to dpdk-next-net-mrvl/for-next-net. Thanks
>
>
v2 updated in next-net, can you please rebase on latest next-net?
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-02-23 12:18 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-09 16:50 [dpdk-dev] [PATCH] net/cnxk: unify the file name jerinj
2022-02-18 6:18 ` Kiran Kumar Kokkilagadda
2022-02-18 7:44 ` Jerin Jacob
2022-02-22 18:32 ` Jerin Jacob
2022-02-22 18:32 ` [dpdk-dev] [PATCH v2] net/cnxk: unify file names jerinj
2022-02-23 9:08 ` Jerin Jacob
2022-02-23 12:18 ` Ferruh Yigit
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).