[PATCH] MAINTAINERS: Add ARM64 defconfig to its section

[PATCH] MAINTAINERS: Add ARM64 defconfig to its section

[Fabio Estevam]

From: Fabio Estevam <festevam@denx.de>

Add arch/arm64/configs/defconfig to the ARM64 MAINTAINERS section so that
changes to this file are automatically sent to the appropriate maintainers
and mailing lists.

Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 20e07e61a148..82bd864b1972 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1937,6 +1937,7 @@ T:	git git://git.kernel.org/pub/scm/linux/kernel/git/soc/soc.git
 F:	Documentation/process/maintainer-soc*.rst
 F:	arch/arm/boot/dts/Makefile
 F:	arch/arm64/boot/dts/Makefile
+F:	arch/arm64/configs/defconfig
 
 ARM ARCHITECTED TIMER DRIVER
 M:	Mark Rutland <mark.rutland@arm.com>
-- 
2.34.1

Re: [PATCH] MAINTAINERS: Add ARM64 defconfig to its section

[Krzysztof Kozlowski]

On 16/05/2025 20:30, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
> 
> Add arch/arm64/configs/defconfig to the ARM64 MAINTAINERS section so that
> changes to this file are automatically sent to the appropriate maintainers
> and mailing lists.
I don't think that's correct. As always stated, the defconfig goes via
soc vendor tree. It was always like that, so if you intend to change it,
provide some rationale and make it explicit (and considering you did not
Cc other soc maintainers how can we even know that policy changed?).
Putting it here will unnecessarily include soc on every random arm64
defconfig change.

Best regards,
Krzysztof

Re: [PATCH] MAINTAINERS: Add ARM64 defconfig to its section

[Krzysztof Kozlowski]

On 19/05/2025 19:22, Krzysztof Kozlowski wrote:
> On 16/05/2025 20:30, Fabio Estevam wrote:
>> From: Fabio Estevam <festevam@denx.de>
>>
>> Add arch/arm64/configs/defconfig to the ARM64 MAINTAINERS section so that
>> changes to this file are automatically sent to the appropriate maintainers
>> and mailing lists.
> I don't think that's correct. As always stated, the defconfig goes via
> soc vendor tree. It was always like that, so if you intend to change it,
> provide some rationale and make it explicit (and considering you did not
> Cc other soc maintainers how can we even know that policy changed?).
> Putting it here will unnecessarily include soc on every random arm64
> defconfig change.


P.S. We have it explicitly documented in maintainer-soc profile, so with
this change probably you need to update that document (with its own
rationale).

Best regards,
Krzysztof

Re: [PATCH] MAINTAINERS: Add ARM64 defconfig to its section

[Fabio Estevam]

On Mon, May 19, 2025 at 2:37 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:

> P.S. We have it explicitly documented in maintainer-soc profile, so with
> this change probably you need to update that document (with its own
> rationale).

The reason I submitted this change was that Arnd asked me to resend a
patch that touched arm64 defconfig with soc@lists.linux.dev on Cc:

https://lore.kernel.org/linux-arm-kernel/ece2df8d-acaa-4554-8c82-24ed45f3a8a4@app.fastmail.com/

Re: [PATCH] MAINTAINERS: Add ARM64 defconfig to its section

[Krzysztof Kozlowski]

On 19/05/2025 20:06, Fabio Estevam wrote:
> On Mon, May 19, 2025 at 2:37 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:
> 
>> P.S. We have it explicitly documented in maintainer-soc profile, so with
>> this change probably you need to update that document (with its own
>> rationale).
> 
> The reason I submitted this change was that Arnd asked me to resend a
> patch that touched arm64 defconfig with soc@lists.linux.dev on Cc:
> 
> https://lore.kernel.org/linux-arm-kernel/ece2df8d-acaa-4554-8c82-24ed45f3a8a4@app.fastmail.com/


Yeah, I understand, I saw it, but that was an exception. One of the very
few cases of defconfig not belonging to vendor SoC maintainer... and
even not originating from such subsystem (Ard's patch).

Best regards,
Krzysztof