Hi Andrew,

On Sun, Mar 12, 2017 at 6:04 PM, Andrew Lunn <[email protected]> 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 <[email protected]>
>>  M:   Vivien Didelot <[email protected]>
>>  L:   [email protected]
>>  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 <[email protected]>
> M:      Vivien Didelot <[email protected]>
> L:      [email protected]
> 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 <[email protected]>
> M:      Vivien Didelot <[email protected]>
> M:      Florian Fainelli <[email protected]>

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 -- [email protected]

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

Reply via email to