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

[Michael Ellerman]

Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> writes:
> 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:

This is one is probably worth doing:

> 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/

I'll fold it in before applying?

But I wouldn't bother with 4xx, just falling back to linuxppc-dev is
fine for that.

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

cheers