Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1600964 > unrolled thread

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

Started byMichael Ellerman <mpe@ellerman.id.au>
First post2017-03-15 01:30 +0100
Last post2017-03-15 10:40 +0100
Articles 4 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH v2 16/23] MAINTAINERS: Add file patterns for powerpc device tree bindings Michael Ellerman <mpe@ellerman.id.au> - 2017-03-15 01:30 +0100
    Re: [PATCH v2 16/23] MAINTAINERS: Add file patterns for powerpc  device tree bindings Geert Uytterhoeven <geert@linux-m68k.org> - 2017-03-15 09:10 +0100
      Re: [PATCH v2 16/23] MAINTAINERS: Add file patterns for powerpc device tree bindings Michael Ellerman <mpe@ellerman.id.au> - 2017-03-15 10:40 +0100
        Re: [PATCH v2 16/23] MAINTAINERS: Add file patterns for powerpc  device tree bindings Geert Uytterhoeven <geert@linux-m68k.org> - 2017-03-15 10:40 +0100

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

FromMichael Ellerman <mpe@ellerman.id.au>
Date2017-03-15 01:30 +0100
SubjectRe: [PATCH v2 16/23] MAINTAINERS: Add file patterns for powerpc device tree bindings
Message-ID<tl4Zc-83n-5@gated-at.bofh.it>
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.

cheers

[toc] | [next] | [standalone]


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

FromGeert Uytterhoeven <geert@linux-m68k.org>
Date2017-03-15 09:10 +0100
SubjectRe: [PATCH v2 16/23] MAINTAINERS: Add file patterns for powerpc device tree bindings
Message-ID<tlcal-4OJ-1@gated-at.bofh.it>
In reply to#1600964
Hi Michael,

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:

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/

@@ -7540,6 +7541,7 @@ M:        Matt Porter <mporter@kernel.crashing.org>
 W:     http://www.penguinppc.org/
 L:     linuxppc-dev@lists.ozlabs.org
 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@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

[toc] | [prev] | [next] | [standalone]


#1601175

FromMichael Ellerman <mpe@ellerman.id.au>
Date2017-03-15 10:40 +0100
Message-ID<tldzs-5Hw-9@gated-at.bofh.it>
In reply to#1601127
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?

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@kernel.crashing.org>
>  W:     http://www.penguinppc.org/
>  L:     linuxppc-dev@lists.ozlabs.org
>  S:     Maintained
> +F:     Documentation/devicetree/bindings/powerpc/4xx/
>  F:     arch/powerpc/platforms/40x/
>  F:     arch/powerpc/platforms/44x/

cheers

[toc] | [prev] | [next] | [standalone]


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

FromGeert Uytterhoeven <geert@linux-m68k.org>
Date2017-03-15 10:40 +0100
SubjectRe: [PATCH v2 16/23] MAINTAINERS: Add file patterns for powerpc device tree bindings
Message-ID<tldzs-5Hw-13@gated-at.bofh.it>
In reply to#1601175
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

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web