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

[Geert Uytterhoeven]

Hi Andrew,

On Sun, Mar 12, 2017 at 6:04 PM, Andrew Lunn <andrew@xxxxxxx> 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@xxxxxxx>
>>  M:   Vivien Didelot <vivien.didelot@xxxxxxxxxxxxxxxxxxxx>
>>  L:   netdev@xxxxxxxxxxxxxxx
>>  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@xxxxxxx>
> M:      Vivien Didelot <vivien.didelot@xxxxxxxxxxxxxxxxxxxx>
> L:      netdev@xxxxxxxxxxxxxxx
> 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@xxxxxxx>
> M:      Vivien Didelot <vivien.didelot@xxxxxxxxxxxxxxxxxxxx>
> M:      Florian Fainelli <f.fainelli@xxxxxxxxx>

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@xxxxxxxxxxxxxx

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