diff mbox series

[v1] MAINTAINERS: Add ARM64 DEFCONFIG entry

Message ID 3d1862f6-f83d-dd8f-061f-8feff9588133@free.fr (mailing list archive)
State New, archived
Headers show
Series [v1] MAINTAINERS: Add ARM64 DEFCONFIG entry | expand

Commit Message

Marc Gonzalez April 18, 2019, 12:50 p.m. UTC
As it stands, get_maintainer.pl scans the changelog to determine the
recipients for arm64 defconfig patch reviews:

$ scripts/get_maintainer.pl arch/arm64/configs/defconfig
Catalin Marinas <catalin.marinas@arm.com> (maintainer:ARM64 PORT (AARCH64 ARCHITECTURE))
Will Deacon <will.deacon@arm.com> (maintainer:ARM64 PORT (AARCH64 ARCHITECTURE))
Heiko Stuebner <heiko@sntech.de> (commit_signer:10/73=14%)
Simon Horman <horms+renesas@verge.net.au> (commit_signer:9/73=12%)
Maxime Ripard <maxime.ripard@bootlin.com> (commit_signer:8/73=11%)
Olof Johansson <olof@lixom.net> (commit_signer:8/73=11%,added_lines:39/248=16%,removed_lines:63/135=47%)
Andy Gross <andy.gross@linaro.org> (commit_signer:8/73=11%)
Jagan Teki <jagan@amarulasolutions.com> (authored:7/73=10%)
Bjorn Andersson <bjorn.andersson@linaro.org> (authored:7/73=10%,added_lines:32/248=13%)
Enric Balletbo i Serra <enric.balletbo@collabora.com> (authored:5/73=7%,added_lines:29/248=12%)
Stefan Wahren <stefan.wahren@i2se.com> (authored:4/73=5%)
Ezequiel Garcia <ezequiel@collabora.com> (authored:4/73=5%)
Marc Gonzalez <marc.w.gonzalez@free.fr> (added_lines:53/248=21%,removed_lines:63/135=47%)
linux-arm-kernel@lists.infradead.org (moderated list:ARM64 PORT (AARCH64 ARCHITECTURE))
linux-kernel@vger.kernel.org (open list)

Exclude arm64 defconfig from AArch64 purview, and provide an explicit
maintainers list, in order to limit the noise from trivial patches:

$ scripts/get_maintainer.pl arch/arm64/configs/defconfig
Arnd Bergmann <arnd@arndb.de> (maintainer:ARM64 DEFCONFIG)
Olof Johansson <olof@lixom.net> (maintainer:ARM64 DEFCONFIG)
linux-arm-kernel@lists.infradead.org (moderated list:ARM64 DEFCONFIG)
linux-kernel@vger.kernel.org (open list)

Signed-off-by: Marc Gonzalez <marc.w.gonzalez@free.fr>
---
Or maybe just
M:	ARM SoC Justice League <arm@kernel.org>
---
 MAINTAINERS | 8 ++++++++
 1 file changed, 8 insertions(+)

Comments

Mark Rutland April 18, 2019, 1:01 p.m. UTC | #1
On Thu, Apr 18, 2019 at 02:50:04PM +0200, Marc Gonzalez wrote:
> As it stands, get_maintainer.pl scans the changelog to determine the
> recipients for arm64 defconfig patch reviews:
> 
> $ scripts/get_maintainer.pl arch/arm64/configs/defconfig
> Catalin Marinas <catalin.marinas@arm.com> (maintainer:ARM64 PORT (AARCH64 ARCHITECTURE))
> Will Deacon <will.deacon@arm.com> (maintainer:ARM64 PORT (AARCH64 ARCHITECTURE))
> Heiko Stuebner <heiko@sntech.de> (commit_signer:10/73=14%)
> Simon Horman <horms+renesas@verge.net.au> (commit_signer:9/73=12%)
> Maxime Ripard <maxime.ripard@bootlin.com> (commit_signer:8/73=11%)
> Olof Johansson <olof@lixom.net> (commit_signer:8/73=11%,added_lines:39/248=16%,removed_lines:63/135=47%)
> Andy Gross <andy.gross@linaro.org> (commit_signer:8/73=11%)
> Jagan Teki <jagan@amarulasolutions.com> (authored:7/73=10%)
> Bjorn Andersson <bjorn.andersson@linaro.org> (authored:7/73=10%,added_lines:32/248=13%)
> Enric Balletbo i Serra <enric.balletbo@collabora.com> (authored:5/73=7%,added_lines:29/248=12%)
> Stefan Wahren <stefan.wahren@i2se.com> (authored:4/73=5%)
> Ezequiel Garcia <ezequiel@collabora.com> (authored:4/73=5%)
> Marc Gonzalez <marc.w.gonzalez@free.fr> (added_lines:53/248=21%,removed_lines:63/135=47%)
> linux-arm-kernel@lists.infradead.org (moderated list:ARM64 PORT (AARCH64 ARCHITECTURE))
> linux-kernel@vger.kernel.org (open list)
> 
> Exclude arm64 defconfig from AArch64 purview, and provide an explicit
> maintainers list, in order to limit the noise from trivial patches:

I'm not sure this makes sense.

Catalin and Will do send defconfig patches through the arm64 tree (even
if the majority of patches are soc updates), and are in charge of the
architecture portions of defconfig.

Thanks,
Mark.

> $ scripts/get_maintainer.pl arch/arm64/configs/defconfig
> Arnd Bergmann <arnd@arndb.de> (maintainer:ARM64 DEFCONFIG)
> Olof Johansson <olof@lixom.net> (maintainer:ARM64 DEFCONFIG)
> linux-arm-kernel@lists.infradead.org (moderated list:ARM64 DEFCONFIG)
> linux-kernel@vger.kernel.org (open list)
> 
> Signed-off-by: Marc Gonzalez <marc.w.gonzalez@free.fr>
> ---
> Or maybe just
> M:	ARM SoC Justice League <arm@kernel.org>
> ---
>  MAINTAINERS | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 3671fdea5010..d8dad615cb94 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -2486,8 +2486,16 @@ T:	git git://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git
>  S:	Maintained
>  F:	arch/arm64/
>  X:	arch/arm64/boot/dts/
> +X:	arch/arm64/configs/defconfig
>  F:	Documentation/arm64/
>  
> +ARM64 DEFCONFIG
> +M:	Arnd Bergmann <arnd@arndb.de>
> +M:	Olof Johansson <olof@lixom.net>
> +L:	linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
> +S:	Maintained
> +F:	arch/arm64/configs/defconfig
> +
>  AS3645A LED FLASH CONTROLLER DRIVER
>  M:	Sakari Ailus <sakari.ailus@iki.fi>
>  L:	linux-leds@vger.kernel.org
> -- 
> 2.17.1
Catalin Marinas April 18, 2019, 1:10 p.m. UTC | #2
On Thu, Apr 18, 2019 at 02:50:04PM +0200, Marc Gonzalez wrote:
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 3671fdea5010..d8dad615cb94 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -2486,8 +2486,16 @@ T:	git git://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git
>  S:	Maintained
>  F:	arch/arm64/
>  X:	arch/arm64/boot/dts/
> +X:	arch/arm64/configs/defconfig
>  F:	Documentation/arm64/
>  
> +ARM64 DEFCONFIG
> +M:	Arnd Bergmann <arnd@arndb.de>
> +M:	Olof Johansson <olof@lixom.net>
> +L:	linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
> +S:	Maintained
> +F:	arch/arm64/configs/defconfig

NAK on this patch, firstly because we don't have any arm-soc entry in
MAINTAINERS (similarly arch/arm64/Kconfig.platforms is maintained by the
arm-soc folk without any MAINTAINERS entry). In addition, I'd like to
keep an eye on what's happening with this file, it's not only about SoC.

If you are bothered about get_maintainer.pl, just add this:

diff --git a/MAINTAINERS b/MAINTAINERS
index 3671fdea5010..df0818652a03 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2485,6 +2485,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/
+F:	arch/arm64/configs/
 X:	arch/arm64/boot/dts/
 F:	Documentation/arm64/
diff mbox series

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index 3671fdea5010..d8dad615cb94 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2486,8 +2486,16 @@  T:	git git://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git
 S:	Maintained
 F:	arch/arm64/
 X:	arch/arm64/boot/dts/
+X:	arch/arm64/configs/defconfig
 F:	Documentation/arm64/
 
+ARM64 DEFCONFIG
+M:	Arnd Bergmann <arnd@arndb.de>
+M:	Olof Johansson <olof@lixom.net>
+L:	linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
+S:	Maintained
+F:	arch/arm64/configs/defconfig
+
 AS3645A LED FLASH CONTROLLER DRIVER
 M:	Sakari Ailus <sakari.ailus@iki.fi>
 L:	linux-leds@vger.kernel.org