diff mbox

MAINTAINERS: arm64: Remove boot/dts/ directory from arm64 entry

Message ID 1530879736-7650-1-git-send-email-will.deacon@arm.com (mailing list archive)
State New, archived
Headers show

Commit Message

Will Deacon July 6, 2018, 12:22 p.m. UTC
The arm-soc tree does a good job handling .dts files, so exclude them
from the ARM64 entry in MAINTAINERS.

Cc: Catalin Marinas <catalin.marinas@arm.com>
Signed-off-by: Will Deacon <will.deacon@arm.com>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

Comments

Olof Johansson July 6, 2018, 5:29 p.m. UTC | #1
I have a batch going in today, I can add it there if you want, or:


On Fri, Jul 6, 2018 at 5:22 AM, Will Deacon <will.deacon@arm.com> wrote:
> The arm-soc tree does a good job handling .dts files, so exclude them
> from the ARM64 entry in MAINTAINERS.
>
> Cc: Catalin Marinas <catalin.marinas@arm.com>
> Signed-off-by: Will Deacon <will.deacon@arm.com>

Acked-by: Olof Johansson <olof@lixom.net>



-Olof
Will Deacon July 9, 2018, 11:56 a.m. UTC | #2
On Fri, Jul 06, 2018 at 10:29:00AM -0700, Olof Johansson wrote:
> I have a batch going in today, I can add it there if you want, or:

Sorry, I didn't see this on Friday.

> On Fri, Jul 6, 2018 at 5:22 AM, Will Deacon <will.deacon@arm.com> wrote:
> > The arm-soc tree does a good job handling .dts files, so exclude them
> > from the ARM64 entry in MAINTAINERS.
> >
> > Cc: Catalin Marinas <catalin.marinas@arm.com>
> > Signed-off-by: Will Deacon <will.deacon@arm.com>
> 
> Acked-by: Olof Johansson <olof@lixom.net>

Thanks. I'll queue this for 4.19, since it's not urgent.

Will
Geert Uytterhoeven July 10, 2018, 7:25 a.m. UTC | #3
Hi Will,

On Fri, Jul 6, 2018 at 2:22 PM Will Deacon <will.deacon@arm.com> wrote:
> The arm-soc tree does a good job handling .dts files, so exclude them
> from the ARM64 entry in MAINTAINERS.
>
> Cc: Catalin Marinas <catalin.marinas@arm.com>
> Signed-off-by: Will Deacon <will.deacon@arm.com>

Thanks for your patch!

> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -2264,6 +2264,7 @@ L:        linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
>  T:     git git://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git
>  S:     Maintained
>  F:     arch/arm64/
> +X:     arch/arm64/boot/dts/
>  F:     Documentation/arm64/

Please note that this does not cause checkpatch to point to the arm-soc
tree, but does drop the reference to linux-arm-kernel, which may not
be intentional?

--- scripts/get_maintainer.pl -f arch/arm64/boot/dts old 2018-07-10
09:21:51.595392363 +0200
+++ scripts/get_maintainer.pl -f arch/arm64/boot/dts new 2018-07-10
09:22:14.671368341 +0200
@@ -1,7 +1,4 @@
 Rob Herring <robh+dt@kernel.org> (maintainer:OPEN FIRMWARE AND
FLATTENED DEVICE TREE BINDINGS)
 Mark Rutland <mark.rutland@arm.com> (maintainer:OPEN FIRMWARE AND
FLATTENED DEVICE TREE BINDINGS)
-Catalin Marinas <catalin.marinas@arm.com> (maintainer:ARM64 PORT
(AARCH64 ARCHITECTURE))
-Will Deacon <will.deacon@arm.com> (maintainer:ARM64 PORT (AARCH64
ARCHITECTURE))
 devicetree@vger.kernel.org (open list:OPEN FIRMWARE AND FLATTENED
DEVICE TREE BINDINGS)
-linux-arm-kernel@lists.infradead.org (moderated list:ARM64 PORT
(AARCH64 ARCHITECTURE))
 linux-kernel@vger.kernel.org (open list)

Gr{oetje,eeting}s,

                        Geert
Olof Johansson July 10, 2018, 2:42 p.m. UTC | #4
Hi,

On Tue, Jul 10, 2018 at 12:25 AM, Geert Uytterhoeven
<geert@linux-m68k.org> wrote:
> Hi Will,
>
> On Fri, Jul 6, 2018 at 2:22 PM Will Deacon <will.deacon@arm.com> wrote:
>> The arm-soc tree does a good job handling .dts files, so exclude them
>> from the ARM64 entry in MAINTAINERS.
>>
>> Cc: Catalin Marinas <catalin.marinas@arm.com>
>> Signed-off-by: Will Deacon <will.deacon@arm.com>
>
> Thanks for your patch!
>
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -2264,6 +2264,7 @@ L:        linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
>>  T:     git git://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git
>>  S:     Maintained
>>  F:     arch/arm64/
>> +X:     arch/arm64/boot/dts/
>>  F:     Documentation/arm64/
>
> Please note that this does not cause checkpatch to point to the arm-soc
> tree, but does drop the reference to linux-arm-kernel, which may not
> be intentional?

It is intentional. If people start sending patches to arm@kernel.org
we'll drown. The model relies on platform maintainers picking up the
contents and sending us pull requests.


-Olof
Geert Uytterhoeven July 10, 2018, 2:51 p.m. UTC | #5
Hi Olof,

On Tue, Jul 10, 2018 at 4:43 PM Olof Johansson <olof@lixom.net> wrote:
> On Tue, Jul 10, 2018 at 12:25 AM, Geert Uytterhoeven
> <geert@linux-m68k.org> wrote:
> > On Fri, Jul 6, 2018 at 2:22 PM Will Deacon <will.deacon@arm.com> wrote:
> >> The arm-soc tree does a good job handling .dts files, so exclude them
> >> from the ARM64 entry in MAINTAINERS.
> >>
> >> Cc: Catalin Marinas <catalin.marinas@arm.com>
> >> Signed-off-by: Will Deacon <will.deacon@arm.com>
> >
> > Thanks for your patch!
> >
> >> --- a/MAINTAINERS
> >> +++ b/MAINTAINERS
> >> @@ -2264,6 +2264,7 @@ L:        linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
> >>  T:     git git://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git
> >>  S:     Maintained
> >>  F:     arch/arm64/
> >> +X:     arch/arm64/boot/dts/
> >>  F:     Documentation/arm64/
> >
> > Please note that this does not cause checkpatch to point to the arm-soc
> > tree, but does drop the reference to linux-arm-kernel, which may not
> > be intentional?
>
> It is intentional. If people start sending patches to arm@kernel.org

They won't, as that address is nowhere to be found in MAINTAINERS ;-)

I really meant linux-arm-kernel@lists.infradead.org

> we'll drown. The model relies on platform maintainers picking up the
> contents and sending us pull requests.

OK. So a few platform maintainers still have to add their dts directory to
their MAINTAINERS entry.

Gr{oetje,eeting}s,

                        Geert
Will Deacon July 11, 2018, 9:43 a.m. UTC | #6
Hi Geert,

On Tue, Jul 10, 2018 at 09:25:58AM +0200, Geert Uytterhoeven wrote:
> On Fri, Jul 6, 2018 at 2:22 PM Will Deacon <will.deacon@arm.com> wrote:
> > The arm-soc tree does a good job handling .dts files, so exclude them
> > from the ARM64 entry in MAINTAINERS.
> >
> > Cc: Catalin Marinas <catalin.marinas@arm.com>
> > Signed-off-by: Will Deacon <will.deacon@arm.com>
> 
> Thanks for your patch!
> 
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -2264,6 +2264,7 @@ L:        linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
> >  T:     git git://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git
> >  S:     Maintained
> >  F:     arch/arm64/
> > +X:     arch/arm64/boot/dts/
> >  F:     Documentation/arm64/
> 
> Please note that this does not cause checkpatch to point to the arm-soc
> tree, but does drop the reference to linux-arm-kernel, which may not
> be intentional?

I think this is ok, since all of the individual sub-directories do pick
up the mailing list. Otherwise, somebody needs to claim responsibility
for the top-level directory, but realistically if you're adding a new
subdirectory there then you're going to be working with the right folks
anyway.

Will
diff mbox

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index 07d1576fc766..61301440e66f 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2264,6 +2264,7 @@  L:	linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
 T:	git git://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git
 S:	Maintained
 F:	arch/arm64/
+X:	arch/arm64/boot/dts/
 F:	Documentation/arm64/
 
 AS3645A LED FLASH CONTROLLER DRIVER