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

[Geert Uytterhoeven]

Hi Michael,

On Wed, Mar 15, 2017 at 1:19 AM, Michael Ellerman <mpe@xxxxxxxxxxxxxx> wrote:
> Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> writes:
>> Submitters of device tree binding documentation may forget to CC
>> the subsystem maintainer if this is missing.
>>
>> Signed-off-by: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
>> Cc: Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx>
>> Cc: Paul Mackerras <paulus@xxxxxxxxx>
>> Cc: Michael Ellerman <mpe@xxxxxxxxxxxxxx>
>> Cc: linuxppc-dev@xxxxxxxxxxxxxxxx
>> ---
>> Please apply this patch directly if you want to be involved in device
>> tree binding documentation for your subsystem.
>>
>> v2:
>>   - Rebased on top of commit a42715830d552d7c ("MAINTAINERS: Remove
>>     powerpc's "opal" pattern match), which just added "powerpc/opal",
>>     while obviously the whole "powerpc" hierarchy is of interest.
>>
>> Impact on next-20170310:
>
> Actual diff ignoring ordering etc:
>
>  +Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx> (supporter:LINUX FOR POWERPC (32-BIT AND 64-BIT))
>  +Paul Mackerras <paulus@xxxxxxxxx> (supporter:LINUX FOR POWERPC (32-BIT AND 64-BIT))
>  +linuxppc-dev@xxxxxxxxxxxxxxxx (open list:LINUX FOR POWERPC (32-BIT AND 64-BIT))
>  -Scott Wood <oss@xxxxxxxxxxxx> (commit_signer:5/11=45%)
>  -Zhao Qiang <qiang.zhao@xxxxxxx> (commit_signer:4/11=36%,authored:4/11=36%)
>  -Christian Lamparter <chunkeey@xxxxxxxxxxxxxx> (commit_signer:1/11=9%)
>  -yangbo lu <yangbo.lu@xxxxxxx> (authored:1/11=9%)
>  -"Otto KekÃlÃinen" <otto@xxxxxxxxx> (authored:1/11=9%)
>  -Chris Packham <chris.packham@xxxxxxxxxxxxxxxxxxx> (authored:1/11=9%)
>  -York Sun <york.sun@xxxxxxx> (authored:1/11=9%)
>
> Which looks bad as all the NXP folks get dropped, but they should be
> reading linuxppc-dev. So I think I'll merge this, unless anyone
> disagrees.

They got dropped because in the absence of a maintainer entry, the
last committers are listed. If they need to be listed, I can send patches to add
more specific entries for 4xx and fsl DT bindings, like:

diff --git a/MAINTAINERS b/MAINTAINERS
index 0f1c2f96c99aa936..40b392a4f399adbe 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -5266,6 +5266,7 @@ M:        Scott Wood <oss@xxxxxxxxxxxx>
 L:     linuxppc-dev@xxxxxxxxxxxxxxxx
 L:     linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
 S:     Maintained
+F:     Documentation/devicetree/bindings/powerpc/fsl/
 F:     drivers/soc/fsl/
 F:     include/linux/fsl/

@@ -7540,6 +7541,7 @@ M:        Matt Porter <mporter@xxxxxxxxxxxxxxxxxxx>
 W:     http://www.penguinppc.org/
 L:     linuxppc-dev@xxxxxxxxxxxxxxxx
 S:     Maintained
+F:     Documentation/devicetree/bindings/powerpc/4xx/
 F:     arch/powerpc/platforms/40x/
 F:     arch/powerpc/platforms/44x/


OK?

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