Hi Andrew, On Sun, Mar 12, 2017 at 6:04 PM, Andrew Lunn <and...@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 <and...@lunn.ch> >> M: Vivien Didelot <vivien.dide...@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 <and...@lunn.ch> > M: Vivien Didelot <vivien.dide...@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 <and...@lunn.ch> > M: Vivien Didelot <vivien.dide...@savoirfairelinux.com> > M: Florian Fainelli <f.faine...@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 -- ge...@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