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
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
