[v2,06/23] MAINTAINERS: Add file patterns for dsa device tree bindings
diff mbox series

Message ID 1489324627-19126-7-git-send-email-geert@linux-m68k.org
State New, archived
Headers show
Series
  • MAINTAINERS: Add file patterns for subsystem device tree bindings
Related show

Commit Message

Geert Uytterhoeven March 12, 2017, 1:16 p.m. UTC
Submitters of device tree binding documentation may forget to CC
the subsystem maintainer if this is missing.

Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Andrew Lunn <andrew@lunn.ch>
Cc: Vivien Didelot <vivien.didelot@savoirfairelinux.com>
Cc: netdev@vger.kernel.org
---
Please apply this patch directly if you want to be involved in device
tree binding documentation for your subsystem.

v2:
  - Rebased on top of commit 0d3cd4b6b49865e8 ("net: dsa: mv88e6xxx:
    move driver in its own folder").

Impact on next-20170310:

+Andrew Lunn <andrew@lunn.ch> (maintainer:MARVELL 88E6XXX ETHERNET SWITCH FABRIC DRIVER)
+Vivien Didelot <vivien.didelot@savoirfairelinux.com> (maintainer:MARVELL 88E6XXX ETHERNET SWITCH FABRIC DRIVER)
 Rob Herring <robh+dt@kernel.org> (maintainer:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS)
 Mark Rutland <mark.rutland@arm.com> (maintainer:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS)
-"David S. Miller" <davem@davemloft.net> (commit_signer:11/12=92%)
-Andrew Lunn <andrew@lunn.ch> (commit_signer:8/12=67%,authored:7/12=58%)
-Florian Fainelli <f.fainelli@gmail.com> (commit_signer:4/12=33%,authored:2/12=17%)
-Vivien Didelot <vivien.didelot@savoirfairelinux.com> (commit_signer:2/12=17%,authored:1/12=8%)
-John Crispin <john@phrozen.org> (commit_signer:1/12=8%,authored:1/12=8%)
-"Otto Kekäläinen" <otto@seravo.fi> (authored:1/12=8%)
-netdev@vger.kernel.org (open list:NETWORKING DRIVERS)
+netdev@vger.kernel.org (open list:MARVELL 88E6XXX ETHERNET SWITCH FABRIC DRIVER)
 devicetree@vger.kernel.org (open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS)
 linux-kernel@vger.kernel.org (open list)
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

Comments

Andrew Lunn March 12, 2017, 5:04 p.m. UTC | #1
On Sun, Mar 12, 2017 at 02:16:50PM +0100, Geert Uytterhoeven wrote:
> Submitters of device tree binding documentation may forget to CC
> the subsystem maintainer if this is missing.

> diff --git a/MAINTAINERS b/MAINTAINERS
> index ce461fefec6c9463..c0822d4fa5c3ade5 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -7812,6 +7812,7 @@ M:	Andrew Lunn <andrew@lunn.ch>
>  M:	Vivien Didelot <vivien.didelot@savoirfairelinux.com>
>  L:	netdev@vger.kernel.org
>  S:	Maintained
> +F:	Documentation/devicetree/bindings/net/dsa/
>  F:	drivers/net/dsa/mv88e6xxx/
>  F:	Documentation/devicetree/bindings/net/dsa/marvell.txt

Hi Geert

If i'm reading this patch correctly, you are putting it under

MARVELL 88E6XXX ETHERNET SWITCH FABRIC DRIVER
M:      Andrew Lunn <andrew@lunn.ch>
M:      Vivien Didelot <vivien.didelot@savoirfairelinux.com>
L:      netdev@vger.kernel.org
S:      Maintained
F:      drivers/net/dsa/mv88e6xxx/
F:      Documentation/devicetree/bindings/net/dsa/marvell.txt

I would says the correct place for
Documentation/devicetree/bindings/net/dsa/ is

NETWORKING [DSA]
M:      Andrew Lunn <andrew@lunn.ch>
M:      Vivien Didelot <vivien.didelot@savoirfairelinux.com>
M:      Florian Fainelli <f.fainelli@gmail.com>
S:      Maintained
F:      net/dsa/
F:      include/net/dsa.h
F:      drivers/net/dsa/

	Andrew
Geert Uytterhoeven March 16, 2017, 12:58 p.m. UTC | #2
Hi Andrew,

On Sun, Mar 12, 2017 at 6:04 PM, Andrew Lunn <andrew@lunn.ch> wrote:
> On Sun, Mar 12, 2017 at 02:16:50PM +0100, Geert Uytterhoeven wrote:
>> Submitters of device tree binding documentation may forget to CC
>> the subsystem maintainer if this is missing.
>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index ce461fefec6c9463..c0822d4fa5c3ade5 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -7812,6 +7812,7 @@ M:      Andrew Lunn <andrew@lunn.ch>
>>  M:   Vivien Didelot <vivien.didelot@savoirfairelinux.com>
>>  L:   netdev@vger.kernel.org
>>  S:   Maintained
>> +F:   Documentation/devicetree/bindings/net/dsa/
>>  F:   drivers/net/dsa/mv88e6xxx/
>>  F:   Documentation/devicetree/bindings/net/dsa/marvell.txt
>
> If i'm reading this patch correctly, you are putting it under
>
> MARVELL 88E6XXX ETHERNET SWITCH FABRIC DRIVER
> M:      Andrew Lunn <andrew@lunn.ch>
> M:      Vivien Didelot <vivien.didelot@savoirfairelinux.com>
> L:      netdev@vger.kernel.org
> S:      Maintained
> F:      drivers/net/dsa/mv88e6xxx/
> F:      Documentation/devicetree/bindings/net/dsa/marvell.txt
>
> I would says the correct place for
> Documentation/devicetree/bindings/net/dsa/ is
>
> NETWORKING [DSA]
> M:      Andrew Lunn <andrew@lunn.ch>
> M:      Vivien Didelot <vivien.didelot@savoirfairelinux.com>
> M:      Florian Fainelli <f.fainelli@gmail.com>

Thanks, that would indeed be better. Seems that entry didn't exist yet in v4.6,
when I did the original version of the patch.

Will send an update...

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

Patch
diff mbox series

diff --git a/MAINTAINERS b/MAINTAINERS
index ce461fefec6c9463..c0822d4fa5c3ade5 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -7812,6 +7812,7 @@  M:	Andrew Lunn <andrew@lunn.ch>
 M:	Vivien Didelot <vivien.didelot@savoirfairelinux.com>
 L:	netdev@vger.kernel.org
 S:	Maintained
+F:	Documentation/devicetree/bindings/net/dsa/
 F:	drivers/net/dsa/mv88e6xxx/
 F:	Documentation/devicetree/bindings/net/dsa/marvell.txt