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

[Geert Uytterhoeven <geert@linux-m68k.org>]

Hi Michael,

On Wed, Mar 15, 2017 at 10:30 AM, Michael Ellerman <mpe@ellerman.id.au> wrote:
> Geert Uytterhoeven <geert@linux-m68k.org> writes:
>> On Wed, Mar 15, 2017 at 1:19 AM, Michael Ellerman <mpe@ellerman.id.au> wrote:
>>> Geert Uytterhoeven <geert@linux-m68k.org> writes:
>>>> Submitters of device tree binding documentation may forget to CC
>>>> the subsystem maintainer if this is missing.
>>>>
>>>> Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
>>>> Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
>>>> Cc: Paul Mackerras <paulus@samba.org>
>>>> Cc: Michael Ellerman <mpe@ellerman.id.au>
>>>> Cc: linuxppc-dev@lists.ozlabs.org
>>>> ---
>>>> 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@kernel.crashing.org> (supporter:LINUX FOR POWERPC (32-BIT AND 64-BIT))
>>>  +Paul Mackerras <paulus@samba.org> (supporter:LINUX FOR POWERPC (32-BIT AND 64-BIT))
>>>  +linuxppc-dev@lists.ozlabs.org (open list:LINUX FOR POWERPC (32-BIT AND 64-BIT))
>>>  -Scott Wood <oss@buserror.net> (commit_signer:5/11=45%)
>>>  -Zhao Qiang <qiang.zhao@nxp.com> (commit_signer:4/11=36%,authored:4/11=36%)
>>>  -Christian Lamparter <chunkeey@googlemail.com> (commit_signer:1/11=9%)
>>>  -yangbo lu <yangbo.lu@nxp.com> (authored:1/11=9%)
>>>  -"Otto Kekäläinen" <otto@seravo.fi> (authored:1/11=9%)
>>>  -Chris Packham <chris.packham@alliedtelesis.co.nz> (authored:1/11=9%)
>>>  -York Sun <york.sun@nxp.com> (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@buserror.net>
>>  L:     linuxppc-dev@lists.ozlabs.org
>>  L:     linux-arm-kernel@lists.infradead.org
>>  S:     Maintained
>> +F:     Documentation/devicetree/bindings/powerpc/fsl/
>>  F:     drivers/soc/fsl/
>>  F:     include/linux/fsl/
>
> I'll fold it in before applying?

Fine (less work) for me! Thanks!

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

OK.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@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