Add myself as co-maintainer for the S32G platform alongside Chester Lin, as I am
directly involved in developing and maintaining the downstream version of the
kernel and Linux BSP for this platform.
Ghennadi Procopciuc (1):
MAINTAINERS: Add maintainer for NXP S32G boards
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
--
2.43.2
From: Ghennadi Procopciuc <[email protected]>
Add myself as a maintainer of the NXP S32G DT files.
Signed-off-by: Ghennadi Procopciuc <[email protected]>
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 9ed4d3868539..09d7a0489952 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2543,6 +2543,7 @@ F: drivers/*/*wpcm*
ARM/NXP S32G ARCHITECTURE
M: Chester Lin <[email protected]>
+M: Ghennadi Procopciuc <[email protected]>
R: Andreas F??rber <[email protected]>
R: Matthias Brugger <[email protected]>
R: NXP S32 Linux Team <[email protected]>
--
2.43.2
On 21/02/2024 15:42, Krzysztof Kozlowski wrote:
> On 21/02/2024 13:01, Ghennadi Procopciuc wrote:
>> From: Ghennadi Procopciuc <[email protected]>
>>
>> Add myself as a maintainer of the NXP S32G DT files.
>
> No need for cover letters for single patches. OTOH, this commit msg is
> empty...
>
> Plus your patch looks corrupted (wrong encoding): F??rber
>
> BTW, did you contribute anything to the upstream Linux kernel? Do you
> know the process? Downstream does not really matter.
I found the answer:
From: Ghennadi Procopciuc <[email protected]>
Signed-off-by: Ghennadi Procopciuc <[email protected]>
Does not look like. Please get some upstream work experience first.
https://lore.kernel.org/all/?q=f%3Aghennadi.procopciuc%40oss.nxp.com
Best regards,
Krzysztof
On 21/02/2024 13:01, Ghennadi Procopciuc wrote:
> From: Ghennadi Procopciuc <[email protected]>
>
> Add myself as a maintainer of the NXP S32G DT files.
No need for cover letters for single patches. OTOH, this commit msg is
empty...
Plus your patch looks corrupted (wrong encoding): F??rber
BTW, did you contribute anything to the upstream Linux kernel? Do you
know the process? Downstream does not really matter.
Best regards,
Krzysztof
On 2/21/24 16:45, Krzysztof Kozlowski wrote:
> On 21/02/2024 15:42, Krzysztof Kozlowski wrote:
>> On 21/02/2024 13:01, Ghennadi Procopciuc wrote:
>>> From: Ghennadi Procopciuc <[email protected]>
>>>
>>> Add myself as a maintainer of the NXP S32G DT files.
>>
>> No need for cover letters for single patches. OTOH, this commit msg is
>> empty...
Thank you, I can correct that.
>> Plus your patch looks corrupted (wrong encoding): F??rber
Indeed, it is due to 'Content-Type: text/plain; charset="us-ascii"'.
I can fix this as part of v2.
>> BTW, did you contribute anything to the upstream Linux kernel? Do you
>> know the process? Downstream does not really matter.
>
> I found the answer:
>
> From: Ghennadi Procopciuc <[email protected]>
> Signed-off-by: Ghennadi Procopciuc <[email protected]>
>
> Does not look like. Please get some upstream work experience first.
>
> https://lore.kernel.org/all/?q=f%3Aghennadi.procopciuc%40oss.nxp.com
Although I am new to upstream communities and may make mistakes, I am
eager to learn and improve.
After leaving SuSe[0], the current maintainer of the NXP S32G DT files
became inactive[1]. As a result, this area is not currently being
maintained. This is the actual reason why I attempted to add myself as a
maintainer of that area. Although I acknowledge that I may not have
enough experience to become a maintainer, I am concerned that the NXP
S32G DT patch submission may be blocked for everyone due to the current
situation.
Should someone else from MAINTAINERS take over this area, or are there
other alternatives?
[0] https://lore.kernel.org/all/[email protected]/
[1]
https://lore.kernel.org/all/[email protected]/
>
> Best regards,
> Krzysztof
> Regards,
Ghennadi
On 21/02/2024 16:19, Ghennadi Procopciuc wrote:
> On 2/21/24 16:45, Krzysztof Kozlowski wrote:
>> On 21/02/2024 15:42, Krzysztof Kozlowski wrote:
>>> On 21/02/2024 13:01, Ghennadi Procopciuc wrote:
>>>> From: Ghennadi Procopciuc <[email protected]>
>>>>
>>>> Add myself as a maintainer of the NXP S32G DT files.
>>>
>>> No need for cover letters for single patches. OTOH, this commit msg is
>>> empty...
> Thank you, I can correct that.
>
>>> Plus your patch looks corrupted (wrong encoding): F??rber
>
> Indeed, it is due to 'Content-Type: text/plain; charset="us-ascii"'.
> I can fix this as part of v2.
>
>>> BTW, did you contribute anything to the upstream Linux kernel? Do you
>>> know the process? Downstream does not really matter.
>>
>> I found the answer:
>>
>> From: Ghennadi Procopciuc <[email protected]>
>> Signed-off-by: Ghennadi Procopciuc <[email protected]>
>>
>> Does not look like. Please get some upstream work experience first.
>>
>> https://lore.kernel.org/all/?q=f%3Aghennadi.procopciuc%40oss.nxp.com
>
> Although I am new to upstream communities and may make mistakes, I am
> eager to learn and improve.
>
> After leaving SuSe[0], the current maintainer of the NXP S32G DT files
> became inactive[1]. As a result, this area is not currently being
> maintained. This is the actual reason why I attempted to add myself as a
> maintainer of that area. Although I acknowledge that I may not have
> enough experience to become a maintainer, I am concerned that the NXP
> S32G DT patch submission may be blocked for everyone due to the current
> situation.
I would be just happy to see first actual contributions or any sort of
activity, like reviewing, before taking over something.
You do not need to become maintainer to provide reviews. By reviewing
patches you already reduce burden/work from the maintainers.
Best regards,
Krzysztof
On 2/21/24 17:43, Krzysztof Kozlowski wrote:
> On 21/02/2024 16:19, Ghennadi Procopciuc wrote:
>> On 2/21/24 16:45, Krzysztof Kozlowski wrote:
>>> On 21/02/2024 15:42, Krzysztof Kozlowski wrote:
>>>> On 21/02/2024 13:01, Ghennadi Procopciuc wrote:
>>>>> From: Ghennadi Procopciuc <[email protected]>
>>>>>
>>>>> Add myself as a maintainer of the NXP S32G DT files.
>>>>
>>>> No need for cover letters for single patches. OTOH, this commit msg is
>>>> empty...
>> Thank you, I can correct that.
>>
>>>> Plus your patch looks corrupted (wrong encoding): F??rber
>>
>> Indeed, it is due to 'Content-Type: text/plain; charset="us-ascii"'.
>> I can fix this as part of v2.
>>
>>>> BTW, did you contribute anything to the upstream Linux kernel? Do you
>>>> know the process? Downstream does not really matter.
>>>
>>> I found the answer:
>>>
>>> From: Ghennadi Procopciuc <[email protected]>
>>> Signed-off-by: Ghennadi Procopciuc <[email protected]>
>>>
>>> Does not look like. Please get some upstream work experience first.
>>>
>>> https://lore.kernel.org/all/?q=f%3Aghennadi.procopciuc%40oss.nxp.com
>>
>> Although I am new to upstream communities and may make mistakes, I am
>> eager to learn and improve.
>>
>> After leaving SuSe[0], the current maintainer of the NXP S32G DT files
>> became inactive[1]. As a result, this area is not currently being
>> maintained. This is the actual reason why I attempted to add myself as a
>> maintainer of that area. Although I acknowledge that I may not have
>> enough experience to become a maintainer, I am concerned that the NXP
>> S32G DT patch submission may be blocked for everyone due to the current
>> situation.
>
> I would be just happy to see first actual contributions or any sort of
> activity, like reviewing, before taking over something.
>
> You do not need to become maintainer to provide reviews. By reviewing
> patches you already reduce burden/work from the maintainers.
>
> Best regards,
> Krzysztof
>
I fully understand and agree with your perspective on this matter and
accept the fact that I will not be a maintainer as initially intended.
Furthermore, I am very willing to participate in any reviews related to
S32G SoCs.
Patches, including those I have created ([0], [1]), will likely be
submitted for this area. This is because each enabled driver for S32G
SoCs is expected to have at least one node in the device tree. These
patches will undergo review and receive feedback. However, the upstream
process will come to a halt at this point since there are no maintainers
to apply and integrate them.
I do not know how this situation should be handled, given my lack of
experience in upstreaming maintenance. The documentation for the Linux
kernel is insufficient in providing guidance [2] on how to handle
inactive maintainers and it is unclear who should take over their
responsibilities. This is likely not the first time this has happened in
the kernel's history.
Could you please guide me on how these patches should be integrated into
a maintainer's tree and by whom?
[0]
https://lore.kernel.org/all/[email protected]/
[1]
https://lore.kernel.org/all/[email protected]/
[2]
https://docs.kernel.org/maintainer/feature-and-driver-maintainers.html#non-compliance
Best regards,
Ghennadi
On 21/02/2024 18:00, Ghennadi Procopciuc wrote:
> On 2/21/24 17:43, Krzysztof Kozlowski wrote:
>> On 21/02/2024 16:19, Ghennadi Procopciuc wrote:
>>> On 2/21/24 16:45, Krzysztof Kozlowski wrote:
>>>> On 21/02/2024 15:42, Krzysztof Kozlowski wrote:
>>>>> On 21/02/2024 13:01, Ghennadi Procopciuc wrote:
>>>>>> From: Ghennadi Procopciuc <[email protected]>
>>>>>>
>>>>>> Add myself as a maintainer of the NXP S32G DT files.
>>>>>
>>>>> No need for cover letters for single patches. OTOH, this commit msg is
>>>>> empty...
>>> Thank you, I can correct that.
>>>
>>>>> Plus your patch looks corrupted (wrong encoding): F??rber
>>>
>>> Indeed, it is due to 'Content-Type: text/plain; charset="us-ascii"'.
>>> I can fix this as part of v2.
>>>
>>>>> BTW, did you contribute anything to the upstream Linux kernel? Do you
>>>>> know the process? Downstream does not really matter.
>>>>
>>>> I found the answer:
>>>>
>>>> From: Ghennadi Procopciuc <[email protected]>
>>>> Signed-off-by: Ghennadi Procopciuc <[email protected]>
>>>>
>>>> Does not look like. Please get some upstream work experience first.
>>>>
>>>> https://lore.kernel.org/all/?q=f%3Aghennadi.procopciuc%40oss.nxp.com
>>>
>>> Although I am new to upstream communities and may make mistakes, I am
>>> eager to learn and improve.
>>>
>>> After leaving SuSe[0], the current maintainer of the NXP S32G DT files
>>> became inactive[1]. As a result, this area is not currently being
>>> maintained. This is the actual reason why I attempted to add myself as a
>>> maintainer of that area. Although I acknowledge that I may not have
>>> enough experience to become a maintainer, I am concerned that the NXP
>>> S32G DT patch submission may be blocked for everyone due to the current
>>> situation.
>>
>> I would be just happy to see first actual contributions or any sort of
>> activity, like reviewing, before taking over something.
>>
>> You do not need to become maintainer to provide reviews. By reviewing
>> patches you already reduce burden/work from the maintainers.
>>
>> Best regards,
>> Krzysztof
>>
>
> I fully understand and agree with your perspective on this matter and
> accept the fact that I will not be a maintainer as initially intended.
> Furthermore, I am very willing to participate in any reviews related to
> S32G SoCs.
Just give it some time...
>
> Patches, including those I have created ([0], [1]), will likely be
> submitted for this area. This is because each enabled driver for S32G
> SoCs is expected to have at least one node in the device tree. These
> patches will undergo review and receive feedback. However, the upstream
> process will come to a halt at this point since there are no maintainers
> to apply and integrate them.
Indeed that's a problem.
>
> I do not know how this situation should be handled, given my lack of
> experience in upstreaming maintenance. The documentation for the Linux
> kernel is insufficient in providing guidance [2] on how to handle
> inactive maintainers and it is unclear who should take over their
> responsibilities. This is likely not the first time this has happened in
> the kernel's history.
>
> Could you please guide me on how these patches should be integrated into
> a maintainer's tree and by whom?
Chester left Suse, so maybe this also means less interest in maintaining
it? Stepping up in such case, so your proposal, is reasonable, so in
general I agree and thank you for trying to do something here.
Andreas or Matthias,
Maybe you could change your R: into M: and take s32g patches?
If not, then I will ack this patch making Ghennadi the maintainer.
Best regards,
Krzysztof
On 22/02/2024 12:13, Krzysztof Kozlowski wrote:
> On 21/02/2024 18:00, Ghennadi Procopciuc wrote:
>> On 2/21/24 17:43, Krzysztof Kozlowski wrote:
>>> On 21/02/2024 16:19, Ghennadi Procopciuc wrote:
>>>> On 2/21/24 16:45, Krzysztof Kozlowski wrote:
>>>>> On 21/02/2024 15:42, Krzysztof Kozlowski wrote:
>>>>>> On 21/02/2024 13:01, Ghennadi Procopciuc wrote:
>>>>>>> From: Ghennadi Procopciuc <[email protected]>
>>>>>>>
>>>>>>> Add myself as a maintainer of the NXP S32G DT files.
>>>>>>
>>>>>> No need for cover letters for single patches. OTOH, this commit msg is
>>>>>> empty...
>>>> Thank you, I can correct that.
>>>>
>>>>>> Plus your patch looks corrupted (wrong encoding): F??rber
>>>>
>>>> Indeed, it is due to 'Content-Type: text/plain; charset="us-ascii"'.
>>>> I can fix this as part of v2.
>>>>
>>>>>> BTW, did you contribute anything to the upstream Linux kernel? Do you
>>>>>> know the process? Downstream does not really matter.
>>>>>
>>>>> I found the answer:
>>>>>
>>>>> From: Ghennadi Procopciuc <[email protected]>
>>>>> Signed-off-by: Ghennadi Procopciuc <[email protected]>
>>>>>
>>>>> Does not look like. Please get some upstream work experience first.
>>>>>
>>>>> https://lore.kernel.org/all/?q=f%3Aghennadi.procopciuc%40oss.nxp.com
>>>>
>>>> Although I am new to upstream communities and may make mistakes, I am
>>>> eager to learn and improve.
>>>>
>>>> After leaving SuSe[0], the current maintainer of the NXP S32G DT files
>>>> became inactive[1]. As a result, this area is not currently being
>>>> maintained. This is the actual reason why I attempted to add myself as a
>>>> maintainer of that area. Although I acknowledge that I may not have
>>>> enough experience to become a maintainer, I am concerned that the NXP
>>>> S32G DT patch submission may be blocked for everyone due to the current
>>>> situation.
>>>
>>> I would be just happy to see first actual contributions or any sort of
>>> activity, like reviewing, before taking over something.
>>>
>>> You do not need to become maintainer to provide reviews. By reviewing
>>> patches you already reduce burden/work from the maintainers.
>>>
>>> Best regards,
>>> Krzysztof
>>>
>>
>> I fully understand and agree with your perspective on this matter and
>> accept the fact that I will not be a maintainer as initially intended.
>> Furthermore, I am very willing to participate in any reviews related to
>> S32G SoCs.
>
> Just give it some time...
>
>>
>> Patches, including those I have created ([0], [1]), will likely be
>> submitted for this area. This is because each enabled driver for S32G
>> SoCs is expected to have at least one node in the device tree. These
>> patches will undergo review and receive feedback. However, the upstream
>> process will come to a halt at this point since there are no maintainers
>> to apply and integrate them.
>
> Indeed that's a problem.
>
>>
>> I do not know how this situation should be handled, given my lack of
>> experience in upstreaming maintenance. The documentation for the Linux
>> kernel is insufficient in providing guidance [2] on how to handle
>> inactive maintainers and it is unclear who should take over their
>> responsibilities. This is likely not the first time this has happened in
>> the kernel's history.
>>
>> Could you please guide me on how these patches should be integrated into
>> a maintainer's tree and by whom?
>
> Chester left Suse, so maybe this also means less interest in maintaining
> it? Stepping up in such case, so your proposal, is reasonable, so in
> general I agree and thank you for trying to do something here.
>
> Andreas or Matthias,
> Maybe you could change your R: into M: and take s32g patches?
>
> If not, then I will ack this patch making Ghennadi the maintainer.
>
Normal process would be that Arnd would contact Chester to see if he is not able
to do the maintainer work any more. In that case maybe Arnd could take over.
I agree with you that some one should get maintainer for a SoC because he is
involved in the Linux kernel community and not because he is working on
downstream patches for the same silicon. Especially being paid by the company
that produces the silicon can quickly get you into dificult situation. Think for
example that NXP, which pays you, wants something to be added in the upstream
kernel, but the code is not in a shape to be part of Linux kernel. That can
generate conflict and for the upstream community it is important that the only
criteria to accept something upstream is the fact that it is in a good shape for
that, not any comercial roadmap by a company.
I'm not saying that Ghennadi won't be able to defend this, if it ever occurs.
Basically because I don't have a good track record of him due to missing
upstream collaboration.
I would prefer that Arnd (or Andreas) step up to take the maintainer role. I
already have one and it's difficult for me to find the time to do it in an
acceptable way.
Regards,
Matthias
On Fri, Feb 23, 2024, at 13:02, Matthias Brugger wrote:
> On 22/02/2024 12:13, Krzysztof Kozlowski wrote:
>> On 21/02/2024 18:00, Ghennadi Procopciuc wrote:
>>
>> Andreas or Matthias,
>> Maybe you could change your R: into M: and take s32g patches?
>>
>> If not, then I will ack this patch making Ghennadi the maintainer.
>>
>
> Normal process would be that Arnd would contact Chester to see if he is
> not able
> to do the maintainer work any more. In that case maybe Arnd could take
> over.
I was hoping to see a reply from Chester one way or another,
I see he has been on Cc for the entire thread but not
chimed in.
> I'm not saying that Ghennadi won't be able to defend this, if it ever occurs.
> Basically because I don't have a good track record of him due to missing
> upstream collaboration.
>
> I would prefer that Arnd (or Andreas) step up to take the maintainer role. I
> already have one and it's difficult for me to find the time to do it in an
> acceptable way.
It appears that we already cover the dts files in the IMX
entry, whether that is intentional or not:
ARM/FREESCALE IMX / MXC ARM ARCHITECTURE
M: Shawn Guo <[email protected]>
M: Sascha Hauer <[email protected]>
R: Pengutronix Kernel Team <[email protected]>
R: Fabio Estevam <[email protected]>
R: NXP Linux Team <[email protected]>
L: [email protected] (moderated for non-subscribers)
S: Maintained
T: git git://git.kernel.org/pub/scm/linux/kernel/git/shawnguo/linux.git
F: arch/arm/boot/dts/nxp/imx/
F: arch/arm/boot/dts/nxp/mxs/
F: arch/arm64/boot/dts/freescale/
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.
I also noticed that the pinctrl driver entry still has Chester's
old email address listed as the only maintainer, so we should
probably fix that as well:
PIN CONTROLLER - NXP S32
M: Chester Lin <[email protected]>
R: NXP S32 Linux Team <[email protected]>
L: [email protected]
S: Maintained
F: Documentation/devicetree/bindings/pinctrl/nxp,s32*
F: drivers/pinctrl/nxp/
Arnd
On Fri, Feb 23, 2024 at 01:29:10PM +0100, Arnd Bergmann wrote:
> On Fri, Feb 23, 2024, at 13:02, Matthias Brugger wrote:
> > On 22/02/2024 12:13, Krzysztof Kozlowski wrote:
> >> On 21/02/2024 18:00, Ghennadi Procopciuc wrote:
> >>
> >> Andreas or Matthias,
> >> Maybe you could change your R: into M: and take s32g patches?
> >>
> >> If not, then I will ack this patch making Ghennadi the maintainer.
> >>
> >
> > Normal process would be that Arnd would contact Chester to see if he is
> > not able
> > to do the maintainer work any more. In that case maybe Arnd could take
> > over.
>
> I was hoping to see a reply from Chester one way or another,
> I see he has been on Cc for the entire thread but not
> chimed in.
>
> > I'm not saying that Ghennadi won't be able to defend this, if it ever occurs.
> > Basically because I don't have a good track record of him due to missing
> > upstream collaboration.
> >
> > I would prefer that Arnd (or Andreas) step up to take the maintainer role. I
> > already have one and it's difficult for me to find the time to do it in an
> > acceptable way.
>
> It appears that we already cover the dts files in the IMX
> entry, whether that is intentional or not:
>
> ARM/FREESCALE IMX / MXC ARM ARCHITECTURE
> M: Shawn Guo <[email protected]>
> M: Sascha Hauer <[email protected]>
> R: Pengutronix Kernel Team <[email protected]>
> R: Fabio Estevam <[email protected]>
> R: NXP Linux Team <[email protected]>
> L: [email protected] (moderated for non-subscribers)
> S: Maintained
> T: git git://git.kernel.org/pub/scm/linux/kernel/git/shawnguo/linux.git
> F: arch/arm/boot/dts/nxp/imx/
> F: arch/arm/boot/dts/nxp/mxs/
> F: arch/arm64/boot/dts/freescale/
>
> 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 <[email protected]>
R: Andreas F?rber <[email protected]>
R: Matthias Brugger <[email protected]>
R: NXP S32 Linux Team <[email protected]>
L: [email protected] (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.
Shawn
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.
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:
> > On Fri, Feb 23, 2024, at 13:02, Matthias Brugger wrote:
> > > On 22/02/2024 12:13, Krzysztof Kozlowski wrote:
> > >> On 21/02/2024 18:00, Ghennadi Procopciuc wrote:
> > >>
> > >> Andreas or Matthias,
> > >> Maybe you could change your R: into M: and take s32g patches?
> > >>
> > >> If not, then I will ack this patch making Ghennadi the maintainer.
> > >>
> > >
> > > Normal process would be that Arnd would contact Chester to see if he is
> > > not able
> > > to do the maintainer work any more. In that case maybe Arnd could take
> > > over.
> >
> > I was hoping to see a reply from Chester one way or another,
> > I see he has been on Cc for the entire thread but not
> > chimed in.
> >
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.
> > > I'm not saying that Ghennadi won't be able to defend this, if it ever occurs.
> > > Basically because I don't have a good track record of him due to missing
> > > upstream collaboration.
> > >
> > > I would prefer that Arnd (or Andreas) step up to take the maintainer role. I
> > > already have one and it's difficult for me to find the time to do it in an
> > > acceptable way.
> >
> > It appears that we already cover the dts files in the IMX
> > entry, whether that is intentional or not:
> >
> > ARM/FREESCALE IMX / MXC ARM ARCHITECTURE
> > M: Shawn Guo <[email protected]>
> > M: Sascha Hauer <[email protected]>
> > R: Pengutronix Kernel Team <[email protected]>
> > R: Fabio Estevam <[email protected]>
> > R: NXP Linux Team <[email protected]>
> > L: [email protected] (moderated for non-subscribers)
> > S: Maintained
> > T: git git://git.kernel.org/pub/scm/linux/kernel/git/shawnguo/linux.git
> > F: arch/arm/boot/dts/nxp/imx/
> > F: arch/arm/boot/dts/nxp/mxs/
> > F: arch/arm64/boot/dts/freescale/
> >
> > 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 <[email protected]>
> R: Andreas F?rber <[email protected]>
> R: Matthias Brugger <[email protected]>
> R: NXP S32 Linux Team <[email protected]>
> L: [email protected] (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.
>
> Shawn
>
This looks good to me as well.
Regards,
Chester
On Sat, Feb 24, 2024 at 03:25:59PM +0800, Chester Lin wrote:
> Hi all,
>
> Sorry for the late reply since I lost connection with upstream due to a
> health condition, which affected my eyesight for a while so I tried to use
> my eyes as less as possible. Please accept my apologies anyway.
>
> 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:
> > > On Fri, Feb 23, 2024, at 13:02, Matthias Brugger wrote:
> > > > On 22/02/2024 12:13, Krzysztof Kozlowski wrote:
> > > >> On 21/02/2024 18:00, Ghennadi Procopciuc wrote:
> > > >>
> > > >> Andreas or Matthias,
> > > >> Maybe you could change your R: into M: and take s32g patches?
> > > >>
> > > >> If not, then I will ack this patch making Ghennadi the maintainer.
> > > >>
> > > >
> > > > Normal process would be that Arnd would contact Chester to see if he is
> > > > not able
> > > > to do the maintainer work any more. In that case maybe Arnd could take
> > > > over.
> > >
> > > I was hoping to see a reply from Chester one way or another,
> > > I see he has been on Cc for the entire thread but not
> > > chimed in.
> > >
>
> 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 rushed 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
"My current employer". Sorry for my stupid typo.
Chester
> field, which means I may not be fit in this role unless nobody cares.
>
> > > > I'm not saying that Ghennadi won't be able to defend this, if it ever occurs.
> > > > Basically because I don't have a good track record of him due to missing
> > > > upstream collaboration.
> > > >
> > > > I would prefer that Arnd (or Andreas) step up to take the maintainer role. I
> > > > already have one and it's difficult for me to find the time to do it in an
> > > > acceptable way.
> > >
> > > It appears that we already cover the dts files in the IMX
> > > entry, whether that is intentional or not:
> > >
> > > ARM/FREESCALE IMX / MXC ARM ARCHITECTURE
> > > M: Shawn Guo <[email protected]>
> > > M: Sascha Hauer <[email protected]>
> > > R: Pengutronix Kernel Team <[email protected]>
> > > R: Fabio Estevam <[email protected]>
> > > R: NXP Linux Team <[email protected]>
> > > L: [email protected] (moderated for non-subscribers)
> > > S: Maintained
> > > T: git git://git.kernel.org/pub/scm/linux/kernel/git/shawnguo/linux.git
> > > F: arch/arm/boot/dts/nxp/imx/
> > > F: arch/arm/boot/dts/nxp/mxs/
> > > F: arch/arm64/boot/dts/freescale/
> > >
> > > 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 <[email protected]>
> > R: Andreas F?rber <[email protected]>
> > R: Matthias Brugger <[email protected]>
> > R: NXP S32 Linux Team <[email protected]>
> > L: [email protected] (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.
> >
> > Shawn
> >
>
> This looks good to me as well.
>
> Regards,
> Chester
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 <[email protected]>
>> R: Andreas Färber <[email protected]>
>> R: Matthias Brugger <[email protected]>
>> R: NXP S32 Linux Team <[email protected]>
>> L: [email protected] (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 <[email protected]>
+R: Chester Lin <[email protected]>
R: Andreas Färber <[email protected]>
R: Matthias Brugger <[email protected]>
R: NXP S32 Linux Team <[email protected]>
L: [email protected] (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 <[email protected]>
@@ -17415,7 +17416,9 @@ R: Pengutronix Kernel Team <[email protected]>
L: [email protected]
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 <[email protected]>
@@ -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 <[email protected]>
-R: NXP S32 Linux Team <[email protected]>
-L: [email protected]
-S: Maintained
-F: Documentation/devicetree/bindings/pinctrl/nxp,s32*
-F: drivers/pinctrl/nxp/
-
PIN CONTROLLER - QUALCOMM
M: Bjorn Andersson <[email protected]>
L: [email protected]
On 2/24/24 13:42, Arnd Bergmann wrote:
> 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 <[email protected]>
>>> R: Andreas Färber <[email protected]>
>>> R: Matthias Brugger <[email protected]>
>>> R: NXP S32 Linux Team <[email protected]>
>>> L: [email protected] (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.
I agree, this looks good to me too.
>
> 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.
I would like to be included as reviewer.
Can we add Shawn as the maintainer since Chester agreed to pass on the
responsibility to him?
>
> 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 <[email protected]>
> +R: Chester Lin <[email protected]>
> R: Andreas Färber <[email protected]>
> R: Matthias Brugger <[email protected]>
> R: NXP S32 Linux Team <[email protected]>
> L: [email protected] (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 <[email protected]>
> @@ -17415,7 +17416,9 @@ R: Pengutronix Kernel Team <[email protected]>
> L: [email protected]
> S: Maintained
> F: Documentation/devicetree/bindings/pinctrl/fsl,*
> +F: Documentation/devicetree/bindings/pinctrl/nxp,s32*
> F: drivers/pinctrl/freescale/
> +F: drivers/pinctrl/nxp/
Could you please add 'L: NXP S32 Linux Team <[email protected]>', given that
this list is relevant for S32 patches ?
> PIN CONTROLLER - INTEL
> M: Mika Westerberg <[email protected]>
> @@ -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 <[email protected]>
> -R: NXP S32 Linux Team <[email protected]>
> -L: [email protected]
> -S: Maintained
> -F: Documentation/devicetree/bindings/pinctrl/nxp,s32*
> -F: drivers/pinctrl/nxp/
> -
> PIN CONTROLLER - QUALCOMM
> M: Bjorn Andersson <[email protected]>
> L: [email protected]
Best regards,
Ghennadi
On 24/02/2024 12:42, Arnd Bergmann wrote:
[...]
>>> ARM/NXP S32G ARCHITECTURE
>>> M: Chester Lin <[email protected]>
>>> R: Andreas Färber <[email protected]>
>>> R: Matthias Brugger <[email protected]>
>>> R: NXP S32 Linux Team <[email protected]>
>>> L: [email protected] (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.
>
Please include me as reviewer :)
Regards,
Matthias
On 26/02/2024 10:55, Matthias Brugger wrote:
>
>
> On 24/02/2024 12:42, Arnd Bergmann wrote:
> [...]
>>>> ARM/NXP S32G ARCHITECTURE
>>>> M: Chester Lin <[email protected]>
>>>> R: Andreas Färber <[email protected]>
>>>> R: Matthias Brugger <[email protected]>
>>>> R: NXP S32 Linux Team <[email protected]>
>>>> L: [email protected] (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 talked with Andreas today and he prefers to be dropped from the list.
Regards,
Matthias
>> 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.
>>
>
> Please include me as reviewer :)
>
> Regards,
> Matthias
>