[PATCH net] MAINTAINERS: merge entries for felix and ocelot drivers

[PATCH net] MAINTAINERS: merge entries for felix and ocelot drivers

[Vladimir Oltean]

From: Vladimir Oltean <vladimir.oltean@nxp.com>

The ocelot switchdev driver also provides a set of library functions for
the felix DSA driver, which in practice means that most of the patches
will be of interest to both groups of driver maintainers.

So, as also suggested in the discussion here, let's merge the 2 entries
into a single larger one:
https://www.spinics.net/lists/netdev/msg657412.html

Note that the entry has been renamed into "OCELOT SWITCH" since neither
Vitesse nor Microsemi exist any longer as company names, instead they
are now named Microchip (which again might be subject to change in the
future), so use the device family name instead.

Suggested-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
---
 MAINTAINERS | 28 ++++++++++++----------------
 1 file changed, 12 insertions(+), 16 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index f08f290df174..621474172fdf 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -11339,14 +11339,6 @@ L:	dmaengine@vger.kernel.org
 S:	Supported
 F:	drivers/dma/at_xdmac.c
 
-MICROSEMI ETHERNET SWITCH DRIVER
-M:	Alexandre Belloni <alexandre.belloni@bootlin.com>
-M:	Microchip Linux Driver Support <UNGLinuxDriver@microchip.com>
-L:	netdev@vger.kernel.org
-S:	Supported
-F:	drivers/net/ethernet/mscc/
-F:	include/soc/mscc/ocelot*
-
 MICROSEMI MIPS SOCS
 M:	Alexandre Belloni <alexandre.belloni@bootlin.com>
 M:	Microchip Linux Driver Support <UNGLinuxDriver@microchip.com>
@@ -12305,6 +12297,18 @@ M:	Peter Zijlstra <peterz@infradead.org>
 S:	Supported
 F:	tools/objtool/
 
+OCELOT ETHERNET SWITCH DRIVER
+M:	Microchip Linux Driver Support <UNGLinuxDriver@microchip.com>
+M:	Vladimir Oltean <vladimir.oltean@nxp.com>
+M:	Claudiu Manoil <claudiu.manoil@nxp.com>
+M:	Alexandre Belloni <alexandre.belloni@bootlin.com>
+L:	netdev@vger.kernel.org
+S:	Supported
+F:	include/soc/mscc/ocelot*
+F:	drivers/net/ethernet/mscc/
+F:	drivers/net/dsa/ocelot/*
+F:	net/dsa/tag_ocelot.c
+
 OCXL (Open Coherent Accelerator Processor Interface OpenCAPI) DRIVER
 M:	Frederic Barrat <fbarrat@linux.ibm.com>
 M:	Andrew Donnellan <ajd@linux.ibm.com>
@@ -18188,14 +18192,6 @@ S:	Maintained
 F:	drivers/input/serio/userio.c
 F:	include/uapi/linux/userio.h
 
-VITESSE FELIX ETHERNET SWITCH DRIVER
-M:	Vladimir Oltean <vladimir.oltean@nxp.com>
-M:	Claudiu Manoil <claudiu.manoil@nxp.com>
-L:	netdev@vger.kernel.org
-S:	Maintained
-F:	drivers/net/dsa/ocelot/*
-F:	net/dsa/tag_ocelot.c
-
 VIVID VIRTUAL VIDEO DRIVER
 M:	Hans Verkuil <hverkuil@xs4all.nl>
 L:	linux-media@vger.kernel.org
-- 
2.25.1

Re: [PATCH net] MAINTAINERS: merge entries for felix and ocelot drivers

[Horatiu Vultur]

The 06/14/2020 01:07, Vladimir Oltean wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> From: Vladimir Oltean <vladimir.oltean@nxp.com>
> 
> The ocelot switchdev driver also provides a set of library functions for
> the felix DSA driver, which in practice means that most of the patches
> will be of interest to both groups of driver maintainers.
> 
> So, as also suggested in the discussion here, let's merge the 2 entries
> into a single larger one:
> https://www.spinics.net/lists/netdev/msg657412.html
> 
> Note that the entry has been renamed into "OCELOT SWITCH" since neither
> Vitesse nor Microsemi exist any longer as company names, instead they
> are now named Microchip (which again might be subject to change in the
> future), so use the device family name instead.
> 
> Suggested-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
> Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
> ---
>  MAINTAINERS | 28 ++++++++++++----------------
>  1 file changed, 12 insertions(+), 16 deletions(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index f08f290df174..621474172fdf 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -11339,14 +11339,6 @@ L:     dmaengine@vger.kernel.org
>  S:     Supported
>  F:     drivers/dma/at_xdmac.c
> 
> -MICROSEMI ETHERNET SWITCH DRIVER
> -M:     Alexandre Belloni <alexandre.belloni@bootlin.com>
> -M:     Microchip Linux Driver Support <UNGLinuxDriver@microchip.com>
> -L:     netdev@vger.kernel.org
> -S:     Supported
> -F:     drivers/net/ethernet/mscc/
> -F:     include/soc/mscc/ocelot*
> -
>  MICROSEMI MIPS SOCS
>  M:     Alexandre Belloni <alexandre.belloni@bootlin.com>
>  M:     Microchip Linux Driver Support <UNGLinuxDriver@microchip.com>
> @@ -12305,6 +12297,18 @@ M:     Peter Zijlstra <peterz@infradead.org>
>  S:     Supported
>  F:     tools/objtool/
> 
> +OCELOT ETHERNET SWITCH DRIVER
> +M:     Microchip Linux Driver Support <UNGLinuxDriver@microchip.com>
> +M:     Vladimir Oltean <vladimir.oltean@nxp.com>
> +M:     Claudiu Manoil <claudiu.manoil@nxp.com>
> +M:     Alexandre Belloni <alexandre.belloni@bootlin.com>
> +L:     netdev@vger.kernel.org
> +S:     Supported
> +F:     include/soc/mscc/ocelot*
> +F:     drivers/net/ethernet/mscc/
> +F:     drivers/net/dsa/ocelot/*
> +F:     net/dsa/tag_ocelot.c
> +
>  OCXL (Open Coherent Accelerator Processor Interface OpenCAPI) DRIVER
>  M:     Frederic Barrat <fbarrat@linux.ibm.com>
>  M:     Andrew Donnellan <ajd@linux.ibm.com>
> @@ -18188,14 +18192,6 @@ S:     Maintained
>  F:     drivers/input/serio/userio.c
>  F:     include/uapi/linux/userio.h
> 
> -VITESSE FELIX ETHERNET SWITCH DRIVER
> -M:     Vladimir Oltean <vladimir.oltean@nxp.com>
> -M:     Claudiu Manoil <claudiu.manoil@nxp.com>
> -L:     netdev@vger.kernel.org
> -S:     Maintained
> -F:     drivers/net/dsa/ocelot/*
> -F:     net/dsa/tag_ocelot.c
> -
>  VIVID VIRTUAL VIDEO DRIVER
>  M:     Hans Verkuil <hverkuil@xs4all.nl>
>  L:     linux-media@vger.kernel.org
> --
> 2.25.1
> 

Acked-by: Horatiu Vultur<horatiu.vultur@microchip.com>

-- 
/Horatiu

Re: [PATCH net] MAINTAINERS: merge entries for felix and ocelot drivers

[Jakub Kicinski]

On Sun, 14 Jun 2020 01:07:53 +0300 Vladimir Oltean wrote:
> From: Vladimir Oltean <vladimir.oltean@nxp.com>
> 
> The ocelot switchdev driver also provides a set of library functions for
> the felix DSA driver, which in practice means that most of the patches
> will be of interest to both groups of driver maintainers.
> 
> So, as also suggested in the discussion here, let's merge the 2 entries
> into a single larger one:
> https://www.spinics.net/lists/netdev/msg657412.html
> 
> Note that the entry has been renamed into "OCELOT SWITCH" since neither
> Vitesse nor Microsemi exist any longer as company names, instead they
> are now named Microchip (which again might be subject to change in the
> future), so use the device family name instead.
> 
> Suggested-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
> Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>

Looks like checkpatch is unhappy about the order of files?

WARNING: Misordered MAINTAINERS entry - list file patterns in alphabetic order
#58: FILE: MAINTAINERS:12308:
+F:	include/soc/mscc/ocelot*
+F:	drivers/net/ethernet/mscc/

WARNING: Misordered MAINTAINERS entry - list file patterns in alphabetic order
#59: FILE: MAINTAINERS:12309:
+F:	drivers/net/ethernet/mscc/
+F:	drivers/net/dsa/ocelot/*

total: 0 errors, 2 warnings, 0 checks, 46 lines checked

Re: [PATCH net] MAINTAINERS: merge entries for felix and ocelot drivers

[Vladimir Oltean]

On Mon, 15 Jun 2020 at 19:37, Jakub Kicinski <kuba@kernel.org> wrote:
>
> On Sun, 14 Jun 2020 01:07:53 +0300 Vladimir Oltean wrote:
> > From: Vladimir Oltean <vladimir.oltean@nxp.com>
> >
> > The ocelot switchdev driver also provides a set of library functions for
> > the felix DSA driver, which in practice means that most of the patches
> > will be of interest to both groups of driver maintainers.
> >
> > So, as also suggested in the discussion here, let's merge the 2 entries
> > into a single larger one:
> > https://www.spinics.net/lists/netdev/msg657412.html
> >
> > Note that the entry has been renamed into "OCELOT SWITCH" since neither
> > Vitesse nor Microsemi exist any longer as company names, instead they
> > are now named Microchip (which again might be subject to change in the
> > future), so use the device family name instead.
> >
> > Suggested-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
> > Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
>
> Looks like checkpatch is unhappy about the order of files?
>
> WARNING: Misordered MAINTAINERS entry - list file patterns in alphabetic order
> #58: FILE: MAINTAINERS:12308:
> +F:     include/soc/mscc/ocelot*
> +F:     drivers/net/ethernet/mscc/
>
> WARNING: Misordered MAINTAINERS entry - list file patterns in alphabetic order
> #59: FILE: MAINTAINERS:12309:
> +F:     drivers/net/ethernet/mscc/
> +F:     drivers/net/dsa/ocelot/*
>
> total: 0 errors, 2 warnings, 0 checks, 46 lines checked

Thanks for checking, Jakub. I sent a v2 now.

-Vladimir