Re: [PATCH 1/1] MAINTAINERS: Add maintainer for NXP S32G boards

[Arnd Bergmann]

On Sat, Feb 24, 2024, at 08:25, Chester Lin wrote:
> Hi all,
>
> Sorry for the late reply since I lost connection with upstream due to a
> health condition, which affected my eyesights for a while so I tried to use
> my eyes as less as possible. Please accept my apologies anyway.

No worries, and thanks for the clarifications.

> On Fri, Feb 23, 2024 at 10:42:58PM +0800, Shawn Guo wrote:
>> On Fri, Feb 23, 2024 at 01:29:10PM +0100, Arnd Bergmann wrote:
>
> Before leaving SUSE I reached NXP to see if anyone could take over it but I
> didn't get response unfortunately. Maybe it was too rush to find a right person
> at the moment but I still wish that someone can take over this role based on the
> following reasons:
>
> - Since I have returned my S32G boards to SUSE, currently I do not have 
> a platform
> to verify S32G patches unless I could get a new one. I wish I could 
> still help
> out but hardware & doc resources will be the biggest challenge to me.
>
> - My current employee may have competitive relationship with NXP in automotive
> field, which means I may not be fit in this role unless nobody cares.

In general, there no problem to just retire from a maintainer
position or mark it as 'Odd fixes' instead of 'Maintained' when
you are no longer planning to actively maintain it.

>> > 
>> > Added everyone there to Cc, having any s32 patches go through
>> > the imx tree would be the easiest way as far as I'm concerned.
>> > I've added the maintainers to Cc, let's see what they think.
>> 
>> It's unintentional that IMX entry covers s32 dts files, as they have a
>> dedicated entry.
>> 
>> ARM/NXP S32G ARCHITECTURE
>> M:      Chester Lin <chester62515@xxxxxxxxx>
>> R:      Andreas Färber <afaerber@xxxxxxx>
>> R:      Matthias Brugger <mbrugger@xxxxxxxx>
>> R:      NXP S32 Linux Team <s32@xxxxxxx>
>> L:      linux-arm-kernel@xxxxxxxxxxxxxxxxxxx (moderated for non-subscribers)
>> S:      Maintained
>> F:      arch/arm64/boot/dts/freescale/s32g*.dts*
>> 
>> However I'm fine with collecting and sending patches through IMX tree,
>> if S32G folks help review them.
>> 
>
> This looks good to me as well.

Ok, in this case I would suggest we change this section to
only have 'R:' entries and no 'M:' for the moment.

Between the four of you (Chester, Andreas, Matthias, Ghennadi),
I think we can choose to keep everyone or drop those that are
unlikely to actually review patches. Please let us know you
would like to be included as a reviewer or not.

For the pinctrl driver, I would add the files to the "freescale"
pinctrl entry in a similar way and end up with

diff --git a/MAINTAINERS b/MAINTAINERS
index efeaeb51f183..c1924c0053bc 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2555,13 +2555,14 @@ F:      drivers/*/*/*wpcm*
 F:     drivers/*/*wpcm*
 
 ARM/NXP S32G ARCHITECTURE
-M:     Chester Lin <chester62515@xxxxxxxxx>
+R:     Chester Lin <chester62515@xxxxxxxxx>
 R:     Andreas Färber <afaerber@xxxxxxx>
 R:     Matthias Brugger <mbrugger@xxxxxxxx>
 R:     NXP S32 Linux Team <s32@xxxxxxx>
 L:     linux-arm-kernel@xxxxxxxxxxxxxxxxxxx (moderated for non-subscribers)
 S:     Maintained
 F:     arch/arm64/boot/dts/freescale/s32g*.dts*
+F:     drivers/pinctrl/nxp/
 
 ARM/Orion SoC/Technologic Systems TS-78xx platform support
 M:     Alexander Clouter <alex@xxxxxxxxxxxxx>
@@ -17415,7 +17416,9 @@ R:      Pengutronix Kernel Team <kernel@xxxxxxxxxxxxxx>
 L:     linux-gpio@xxxxxxxxxxxxxxx
 S:     Maintained
 F:     Documentation/devicetree/bindings/pinctrl/fsl,*
+F:     Documentation/devicetree/bindings/pinctrl/nxp,s32*
 F:     drivers/pinctrl/freescale/
+F:     drivers/pinctrl/nxp/
 
 PIN CONTROLLER - INTEL
 M:     Mika Westerberg <mika.westerberg@xxxxxxxxxxxxxxx>
@@ -17469,14 +17472,6 @@ S:     Supported
 F:     drivers/gpio/gpio-sama5d2-piobu.c
 F:     drivers/pinctrl/pinctrl-at91*
 
-PIN CONTROLLER - NXP S32
-M:     Chester Lin <clin@xxxxxxxx>
-R:     NXP S32 Linux Team <s32@xxxxxxx>
-L:     linux-gpio@xxxxxxxxxxxxxxx
-S:     Maintained
-F:     Documentation/devicetree/bindings/pinctrl/nxp,s32*
-F:     drivers/pinctrl/nxp/
-
 PIN CONTROLLER - QUALCOMM
 M:     Bjorn Andersson <andersson@xxxxxxxxxx>
 L:     linux-arm-msm@xxxxxxxxxxxxxxx