Re: [PATCH v2 06/23] MAINTAINERS: Add file patterns for dsa device tree bindings

From: Geert Uytterhoeven <geert@linux-m68k.org>
To: Andrew Lunn <andrew@lunn.ch>
Cc: "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	Rob Herring <robh+dt@kernel.org>,
	Mark Rutland <mark.rutland@arm.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Vivien Didelot <vivien.didelot@savoirfairelinux.com>,
	"netdev@vger.kernel.org" <netdev@vger.kernel.org>
Subject: Re: [PATCH v2 06/23] MAINTAINERS: Add file patterns for dsa device tree bindings
Date: Thu, 16 Mar 2017 13:58:54 +0100	[thread overview]
Message-ID: <CAMuHMdWVTzYjmRCduVWxB4ZWZJSYXZQcaQR6goyUvrn_zD0fRA@mail.gmail.com> (raw)
In-Reply-To: <20170312170407.GJ15842@lunn.ch>

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