* [PATCH] maintainers: assign meson.build for drivers with subtrees
@ 2023-09-01 12:03 David Marchand
2023-09-01 12:28 ` Thomas Monjalon
2023-09-01 12:41 ` [PATCH v2] " David Marchand
0 siblings, 2 replies; 5+ messages in thread
From: David Marchand @ 2023-09-01 12:03 UTC (permalink / raw)
To: dev; +Cc: thomas, ferruh.yigit, gakhil, jerinj, maxime.coquelin
When a new (for example, net) driver gets submitted, the CI scripts
can't determine which subtree (here, dpdk-next-net) this series is
associated to because the MAINTAINERS update for the new driver is
part of the series.
Put meson.build of the drivers classes under the responsibility of
subtree maintainers.
Signed-off-by: David Marchand <david.marchand@redhat.com>
---
MAINTAINERS | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index a926155f26..598edadc42 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -611,6 +611,7 @@ Networking Drivers
------------------
M: Ferruh Yigit <ferruh.yigit@amd.com>
T: git://dpdk.org/next/dpdk-next-net
+F: drivers/net/meson.build
F: doc/guides/nics/features/default.ini
Link bonding
@@ -1057,6 +1058,7 @@ F: doc/guides/nics/features/memif.ini
Crypto Drivers
--------------
T: git://dpdk.org/next/dpdk-next-crypto
+F: drivers/crypto/meson.build
F: doc/guides/cryptodevs/features/default.ini
AMD CCP Crypto
@@ -1193,6 +1195,7 @@ F: doc/guides/cryptodevs/features/virtio.ini
Compression Drivers
-------------------
T: git://dpdk.org/next/dpdk-next-crypto
+F: drivers/compress/meson.build
Cavium OCTEON TX zipvf
M: Ashish Gupta <ashish.gupta@marvell.com>
@@ -1290,6 +1293,7 @@ F: doc/guides/mldevs/cnxk.rst
vDPA Drivers
------------
T: git://dpdk.org/next/dpdk-next-virtio
+F: drivers/vdpa/meson.build
Intel ifc
M: Xiao Wang <xiao.w.wang@intel.com>
@@ -1315,6 +1319,7 @@ Eventdev Drivers
----------------
M: Jerin Jacob <jerinj@marvell.com>
T: git://dpdk.org/next/dpdk-next-eventdev
+F: drivers/event/meson.build
Cavium OCTEON TX ssovf
M: Jerin Jacob <jerinj@marvell.com>
@@ -1369,10 +1374,11 @@ F: doc/guides/eventdevs/opdl.rst
Baseband Drivers
----------------
+T: git://dpdk.org/next/dpdk-next-baseband
+F: drivers/baseband/meson.build
Intel baseband
M: Nicolas Chautru <nicolas.chautru@intel.com>
-T: git://dpdk.org/next/dpdk-next-baseband
F: drivers/baseband/turbo_sw/
F: doc/guides/bbdevs/turbo_sw.rst
F: doc/guides/bbdevs/features/turbo_sw.ini
@@ -1391,7 +1397,6 @@ F: doc/guides/bbdevs/features/vrb1.ini
Null baseband
M: Nicolas Chautru <nicolas.chautru@intel.com>
-T: git://dpdk.org/next/dpdk-next-baseband
F: drivers/baseband/null/
F: doc/guides/bbdevs/null.rst
F: doc/guides/bbdevs/features/null.ini
@@ -1399,7 +1404,6 @@ F: doc/guides/bbdevs/features/null.ini
NXP LA12xx
M: Gagandeep Singh <g.singh@nxp.com>
M: Hemant Agrawal <hemant.agrawal@nxp.com>
-T: git://dpdk.org/next/dpdk-next-baseband
F: drivers/baseband/la12xx/
F: doc/guides/bbdevs/la12xx.rst
F: doc/guides/bbdevs/features/la12xx.ini
--
2.41.0
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] maintainers: assign meson.build for drivers with subtrees
2023-09-01 12:03 [PATCH] maintainers: assign meson.build for drivers with subtrees David Marchand
@ 2023-09-01 12:28 ` Thomas Monjalon
2023-09-01 12:33 ` David Marchand
2023-09-01 12:41 ` [PATCH v2] " David Marchand
1 sibling, 1 reply; 5+ messages in thread
From: Thomas Monjalon @ 2023-09-01 12:28 UTC (permalink / raw)
To: David Marchand; +Cc: dev, ferruh.yigit, gakhil, jerinj, maxime.coquelin
01/09/2023 14:03, David Marchand:
> When a new (for example, net) driver gets submitted, the CI scripts
> can't determine which subtree (here, dpdk-next-net) this series is
> associated to because the MAINTAINERS update for the new driver is
> part of the series.
>
> Put meson.build of the drivers classes under the responsibility of
> subtree maintainers.
[...]
> @@ -1315,6 +1319,7 @@ Eventdev Drivers
> ----------------
> M: Jerin Jacob <jerinj@marvell.com>
> T: git://dpdk.org/next/dpdk-next-eventdev
> +F: drivers/event/meson.build
While at it, can we remove Jerin's name here?
He's already referenced for the whole git tree at the beginning of the file.
Other device classes don't add a name for drivers, it is redundant.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] maintainers: assign meson.build for drivers with subtrees
2023-09-01 12:28 ` Thomas Monjalon
@ 2023-09-01 12:33 ` David Marchand
0 siblings, 0 replies; 5+ messages in thread
From: David Marchand @ 2023-09-01 12:33 UTC (permalink / raw)
To: Thomas Monjalon; +Cc: dev, ferruh.yigit, gakhil, jerinj, maxime.coquelin
On Fri, Sep 1, 2023 at 2:28 PM Thomas Monjalon <thomas@monjalon.net> wrote:
>
> 01/09/2023 14:03, David Marchand:
> > When a new (for example, net) driver gets submitted, the CI scripts
> > can't determine which subtree (here, dpdk-next-net) this series is
> > associated to because the MAINTAINERS update for the new driver is
> > part of the series.
> >
> > Put meson.build of the drivers classes under the responsibility of
> > subtree maintainers.
> [...]
> > @@ -1315,6 +1319,7 @@ Eventdev Drivers
> > ----------------
> > M: Jerin Jacob <jerinj@marvell.com>
> > T: git://dpdk.org/next/dpdk-next-eventdev
> > +F: drivers/event/meson.build
>
> While at it, can we remove Jerin's name here?
> He's already referenced for the whole git tree at the beginning of the file.
> Other device classes don't add a name for drivers, it is redundant.
The same is true for Ferruh too.
v2 on the way.
--
David Marchand
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH v2] maintainers: assign meson.build for drivers with subtrees
2023-09-01 12:03 [PATCH] maintainers: assign meson.build for drivers with subtrees David Marchand
2023-09-01 12:28 ` Thomas Monjalon
@ 2023-09-01 12:41 ` David Marchand
2023-09-01 12:57 ` Thomas Monjalon
1 sibling, 1 reply; 5+ messages in thread
From: David Marchand @ 2023-09-01 12:41 UTC (permalink / raw)
To: dev; +Cc: thomas, ferruh.yigit, gakhil, jerinj, maxime.coquelin
When a new (for example, net) driver gets submitted, the CI scripts
can't determine which subtree (here, dpdk-next-net) this series is
associated to because the MAINTAINERS update for the new driver is
part of the series.
Put meson.build of the drivers classes under the responsibility of
subtree maintainers.
While at it, cleanup redundant statements:
- Ferruh or Jerin are maintainers of next-net / next-event but this
information is already present at the top of the MAINTAINERS file,
- the subtree for baseband drivers can be put at the start of the
baseband section,
Signed-off-by: David Marchand <david.marchand@redhat.com>
---
Changes since v1:
- added some minor cleanup,
---
MAINTAINERS | 12 +++++++-----
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index a926155f26..f3e3dda872 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -609,8 +609,8 @@ F: drivers/bus/vmbus/
Networking Drivers
------------------
-M: Ferruh Yigit <ferruh.yigit@amd.com>
T: git://dpdk.org/next/dpdk-next-net
+F: drivers/net/meson.build
F: doc/guides/nics/features/default.ini
Link bonding
@@ -1057,6 +1057,7 @@ F: doc/guides/nics/features/memif.ini
Crypto Drivers
--------------
T: git://dpdk.org/next/dpdk-next-crypto
+F: drivers/crypto/meson.build
F: doc/guides/cryptodevs/features/default.ini
AMD CCP Crypto
@@ -1193,6 +1194,7 @@ F: doc/guides/cryptodevs/features/virtio.ini
Compression Drivers
-------------------
T: git://dpdk.org/next/dpdk-next-crypto
+F: drivers/compress/meson.build
Cavium OCTEON TX zipvf
M: Ashish Gupta <ashish.gupta@marvell.com>
@@ -1290,6 +1292,7 @@ F: doc/guides/mldevs/cnxk.rst
vDPA Drivers
------------
T: git://dpdk.org/next/dpdk-next-virtio
+F: drivers/vdpa/meson.build
Intel ifc
M: Xiao Wang <xiao.w.wang@intel.com>
@@ -1313,8 +1316,8 @@ F: doc/guides/vdpadevs/features/sfc.ini
Eventdev Drivers
----------------
-M: Jerin Jacob <jerinj@marvell.com>
T: git://dpdk.org/next/dpdk-next-eventdev
+F: drivers/event/meson.build
Cavium OCTEON TX ssovf
M: Jerin Jacob <jerinj@marvell.com>
@@ -1369,10 +1372,11 @@ F: doc/guides/eventdevs/opdl.rst
Baseband Drivers
----------------
+T: git://dpdk.org/next/dpdk-next-baseband
+F: drivers/baseband/meson.build
Intel baseband
M: Nicolas Chautru <nicolas.chautru@intel.com>
-T: git://dpdk.org/next/dpdk-next-baseband
F: drivers/baseband/turbo_sw/
F: doc/guides/bbdevs/turbo_sw.rst
F: doc/guides/bbdevs/features/turbo_sw.ini
@@ -1391,7 +1395,6 @@ F: doc/guides/bbdevs/features/vrb1.ini
Null baseband
M: Nicolas Chautru <nicolas.chautru@intel.com>
-T: git://dpdk.org/next/dpdk-next-baseband
F: drivers/baseband/null/
F: doc/guides/bbdevs/null.rst
F: doc/guides/bbdevs/features/null.ini
@@ -1399,7 +1402,6 @@ F: doc/guides/bbdevs/features/null.ini
NXP LA12xx
M: Gagandeep Singh <g.singh@nxp.com>
M: Hemant Agrawal <hemant.agrawal@nxp.com>
-T: git://dpdk.org/next/dpdk-next-baseband
F: drivers/baseband/la12xx/
F: doc/guides/bbdevs/la12xx.rst
F: doc/guides/bbdevs/features/la12xx.ini
--
2.41.0
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] maintainers: assign meson.build for drivers with subtrees
2023-09-01 12:41 ` [PATCH v2] " David Marchand
@ 2023-09-01 12:57 ` Thomas Monjalon
0 siblings, 0 replies; 5+ messages in thread
From: Thomas Monjalon @ 2023-09-01 12:57 UTC (permalink / raw)
To: David Marchand; +Cc: dev, ferruh.yigit, gakhil, jerinj, maxime.coquelin
01/09/2023 14:41, David Marchand:
> When a new (for example, net) driver gets submitted, the CI scripts
> can't determine which subtree (here, dpdk-next-net) this series is
> associated to because the MAINTAINERS update for the new driver is
> part of the series.
>
> Put meson.build of the drivers classes under the responsibility of
> subtree maintainers.
>
> While at it, cleanup redundant statements:
> - Ferruh or Jerin are maintainers of next-net / next-event but this
> information is already present at the top of the MAINTAINERS file,
> - the subtree for baseband drivers can be put at the start of the
> baseband section,
>
> Signed-off-by: David Marchand <david.marchand@redhat.com>
Applied, thanks.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-09-01 12:57 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-09-01 12:03 [PATCH] maintainers: assign meson.build for drivers with subtrees David Marchand
2023-09-01 12:28 ` Thomas Monjalon
2023-09-01 12:33 ` David Marchand
2023-09-01 12:41 ` [PATCH v2] " David Marchand
2023-09-01 12:57 ` Thomas Monjalon
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).