[PATCH net v2 0/2] MAINTAINERS: WWAN subsystem changes

[PATCH net v2 0/2] MAINTAINERS: WWAN subsystem changes

[Bagas Sanjaya]

I have to make this little MAINTAINERS patch series when I was dealing
with a regression involving Intel WWAN IOSM driver [1]. The culprit
author was (AFAIK) MIA as well as its subsystem mailing list down.
So here is the series.

Changes since v1 [2]:
  * Alphabetically by surname sort CREDITS entry

[1]: https://lore.kernel.org/lkml/267abf02-4b60-4a2e-92cd-709e3da6f7d3@gmail.com/
[2]: https://lore.kernel.org/lkml/20231013014010.18338-1-bagasdotme@gmail.com/

Bagas Sanjaya (2):
  MAINTAINERS: Move M Chetan Kumar to CREDITS
  MAINTAINERS: Remove linuxwwan@intel.com mailing list

 CREDITS     | 5 +++++
 MAINTAINERS | 7 +------
 2 files changed, 6 insertions(+), 6 deletions(-)


base-commit: d2a0fc372aca561556e765d0a9ec365c7c12f0ad
-- 
An old man doll... just what I always wanted! - Clara

[PATCH net v2 1/2] MAINTAINERS: Move M Chetan Kumar to CREDITS

[Bagas Sanjaya]

Emails to him bounce with 550 error code:

```
5.1.0 - Unknown address error 550-'5.1.1 <m.chetan.kumar@linux.intel.com>: Recipient address rejected: User unknown in virtual mailbox table'
```

It looks like he had left Intel, so move him to CREDITS.

Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
---
 He is still listed by scripts/get_maintainer.pl, though (due to significant
 contribution):

 ```
 $ scripts/get_maintainer.pl drivers/net/wwan/iosm/
 ...
 M Chetan Kumar <m.chetan.kumar@linux.intel.com> (commit_signer:15/23=65%,authored:14/23=61%)
 ...
 ```

 If no one steps up maintaining IOSM, should INTEL WWAN IOSM DRIVER entry
 be removed then to get rid of him, like in [1]?

 [1]: https://lore.kernel.org/lkml/20210316081000.12596-1-krzysztof.kozlowski@canonical.com/

 CREDITS     | 5 +++++
 MAINTAINERS | 2 --
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/CREDITS b/CREDITS
index f33a33fd237170..0472398b6aa7e9 100644
--- a/CREDITS
+++ b/CREDITS
@@ -2191,6 +2191,11 @@ D: pirq addr, CS5535 alsa audio driver
 S: Gurgaon, India
 S: Kuala Lumpur, Malaysia
 
+N: M Chetan Kumar
+E: <m.chetan.kumar@intel.com>
+D: Intel WWAN IOSM driver
+D: Mediatek T7XX 5G WWAN reviewer
+
 N: Mohit Kumar
 D: ST Microelectronics SPEAr13xx PCI host bridge driver
 D: Synopsys DesignWare PCI host bridge driver
diff --git a/MAINTAINERS b/MAINTAINERS
index 7a7bd8bd80e9f2..08a7be6a5680ed 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -10879,7 +10879,6 @@ S:	Maintained
 F:	drivers/platform/x86/intel/wmi/thunderbolt.c
 
 INTEL WWAN IOSM DRIVER
-M:	M Chetan Kumar <m.chetan.kumar@intel.com>
 M:	Intel Corporation <linuxwwan@intel.com>
 L:	netdev@vger.kernel.org
 S:	Maintained
@@ -13504,7 +13503,6 @@ M:	Chandrashekar Devegowda <chandrashekar.devegowda@intel.com>
 M:	Intel Corporation <linuxwwan@intel.com>
 R:	Chiranjeevi Rapolu <chiranjeevi.rapolu@linux.intel.com>
 R:	Liu Haijun <haijun.liu@mediatek.com>
-R:	M Chetan Kumar <m.chetan.kumar@linux.intel.com>
 R:	Ricardo Martinez <ricardo.martinez@linux.intel.com>
 L:	netdev@vger.kernel.org
 S:	Supported
-- 
An old man doll... just what I always wanted! - Clara

Re: [PATCH net v2 1/2] MAINTAINERS: Move M Chetan Kumar to CREDITS

[Jakub Kicinski]

On Mon, 23 Oct 2023 10:29:04 +0700 Bagas Sanjaya wrote:
>  M Chetan Kumar <m.chetan.kumar@linux.intel.com> (commit_signer:15/23=65%,authored:14/23=61%)

14 patches authored and 15 signed off?
Let me be more clear this time - nak, please drop this patch.
-- 
pw-bot: cr

Re: [PATCH net v2 1/2] MAINTAINERS: Move M Chetan Kumar to CREDITS

[Bagas Sanjaya]

On 23/10/2023 23:38, Jakub Kicinski wrote:
> On Mon, 23 Oct 2023 10:29:04 +0700 Bagas Sanjaya wrote:
>>  M Chetan Kumar <m.chetan.kumar@linux.intel.com> (commit_signer:15/23=65%,authored:14/23=61%)
> 
> 14 patches authored and 15 signed off?
> Let me be more clear this time - nak, please drop this patch.

Or maybe as well drop INTEL WWAN IOSM DRIVER entry (and let
WWAN subsystem maintainers handle it)? I don't want
people get their inboxes spammed with bounces against
his addresses, though.

He's now in a state of limbo. He has significant contribution
(and gets listed by get_maintainer script with (AFAIK) no way
to filter him out), yet emails to him bounces. What will be
the resolution then?

-- 
An old man doll... just what I always wanted! - Clara

Re: [PATCH net v2 1/2] MAINTAINERS: Move M Chetan Kumar to CREDITS

[Jakub Kicinski]

On Tue, 24 Oct 2023 07:11:12 +0700 Bagas Sanjaya wrote:
> > 14 patches authored and 15 signed off?
> > Let me be more clear this time - nak, please drop this patch.  
> 
> Or maybe as well drop INTEL WWAN IOSM DRIVER entry (and let
> WWAN subsystem maintainers handle it)? I don't want
> people get their inboxes spammed with bounces against
> his addresses, though.

Right, sorry, still delete him from the MAINTAINERS.
Just don't add him to the CREDITS.

> He's now in a state of limbo. He has significant contribution
> (and gets listed by get_maintainer script with (AFAIK) no way
> to filter him out), yet emails to him bounces. What will be
> the resolution then?

Yes :( Not much we can do about that (even if we drop him from
maintainers all fixes will CC him based on the sign-off tags).

Re: [PATCH net v2 1/2] MAINTAINERS: Move M Chetan Kumar to CREDITS

[Bagas Sanjaya]

[-- Attachment #1: Type: text/plain, Size: 996 bytes --]

On Mon, Oct 23, 2023 at 06:52:21PM -0700, Jakub Kicinski wrote:
> On Tue, 24 Oct 2023 07:11:12 +0700 Bagas Sanjaya wrote:
> > > 14 patches authored and 15 signed off?
> > > Let me be more clear this time - nak, please drop this patch.  
> > 
> > Or maybe as well drop INTEL WWAN IOSM DRIVER entry (and let
> > WWAN subsystem maintainers handle it)? I don't want
> > people get their inboxes spammed with bounces against
> > his addresses, though.
> 
> Right, sorry, still delete him from the MAINTAINERS.
> Just don't add him to the CREDITS.
> 
> > He's now in a state of limbo. He has significant contribution
> > (and gets listed by get_maintainer script with (AFAIK) no way
> > to filter him out), yet emails to him bounces. What will be
> > the resolution then?
> 
> Yes :( Not much we can do about that (even if we drop him from
> maintainers all fixes will CC him based on the sign-off tags).

OK, thanks!

-- 
An old man doll... just what I always wanted! - Clara

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

Re: [PATCH net v2 1/2] MAINTAINERS: Move M Chetan Kumar to CREDITS

[Johannes Berg]

On Mon, 2023-10-23 at 18:52 -0700, Jakub Kicinski wrote:
> 
> > He's now in a state of limbo. He has significant contribution
> > (and gets listed by get_maintainer script with (AFAIK) no way
> > to filter him out), yet emails to him bounces. What will be
> > the resolution then?
> 
> Yes :( Not much we can do about that (even if we drop him from
> maintainers all fixes will CC him based on the sign-off tags).

I was hoping he'd chime in here - I did manage to find him at his
personal address, but I didn't want to just add his personal address to
the public list.

If he's OK with exposing his personal (or new work?) address I guess we
could add him to the .mailmap, to address this particular problem.

Anyway I've BCC'ed him now (Hi Chetan :) ), but he had also said he was
checking with his new work about this all.

johannes

Re: [PATCH net v2 1/2] MAINTAINERS: Move M Chetan Kumar to CREDITS

[Thorsten Leemhuis]

On 24.10.23 09:40, Johannes Berg wrote:
> On Mon, 2023-10-23 at 18:52 -0700, Jakub Kicinski wrote:
>>
>>> He's now in a state of limbo. He has significant contribution
>>> (and gets listed by get_maintainer script with (AFAIK) no way
>>> to filter him out), yet emails to him bounces. What will be
>>> the resolution then?
>>
>> Yes :( Not much we can do about that (even if we drop him from
>> maintainers all fixes will CC him based on the sign-off tags).
> 
> I was hoping he'd chime in here - I did manage to find him at his
> personal address, but I didn't want to just add his personal address to
> the public list.
> 
> If he's OK with exposing his personal (or new work?) address I guess we
> could add him to the .mailmap, to address this particular problem.
> 
> Anyway I've BCC'ed him now (Hi Chetan :) ), but he had also said he was
> checking with his new work about this all.

[reviving a old thread]

Johannes, if you have a minute: did anything ever come out of that? Or
did anyone else step in to take care of Intels's IOSM driver? Martian
(now CCed) struggles with a problem
(https://bugzilla.kernel.org/show_bug.cgi?id=218383 [no, not a
regression iirc]) for a while now and is kinda stuck. We discussed
off-list submitting a "mark as orphan" patch, but then I found this
discussion.

Ciao, Thorsten

Re: [PATCH net v2 1/2] MAINTAINERS: Move M Chetan Kumar to CREDITS

[Maciej W. Rozycki]

On Mon, 14 Oct 2024, Thorsten Leemhuis wrote:

> >>> He's now in a state of limbo. He has significant contribution
> >>> (and gets listed by get_maintainer script with (AFAIK) no way
> >>> to filter him out), yet emails to him bounces. What will be
> >>> the resolution then?
> >>
> >> Yes :( Not much we can do about that (even if we drop him from
> >> maintainers all fixes will CC him based on the sign-off tags).

 FWIW adding an entry to .get_maintainer.ignore is the way to sort it.

  Maciej

Re: [PATCH net v2 1/2] MAINTAINERS: Move M Chetan Kumar to CREDITS

[Johannes Berg]

On Mon, 2024-10-14 at 10:22 +0200, Thorsten Leemhuis wrote:
> > 
> > Anyway I've BCC'ed him now (Hi Chetan :) ), but he had also said he was
> > checking with his new work about this all.
> 
> [reviving a old thread]
> 
> Johannes, if you have a minute: did anything ever come out of that?

No, I guess I can try again, but everyone pretty much moved on from this
stuff... Would be surprised if Chetan were interested in any of this
now.

> Or did anyone else step in to take care of Intels's IOSM driver?

Not that I know of and I think the business was pretty much wound down?
No idea really though.

johannes

Re: [PATCH net v2 1/2] MAINTAINERS: Move M Chetan Kumar to CREDITS

[Thorsten Leemhuis]

On 22.10.24 16:27, Johannes Berg wrote:
> On Mon, 2024-10-14 at 10:22 +0200, Thorsten Leemhuis wrote:
>>>
>>> Anyway I've BCC'ed him now (Hi Chetan :) ), but he had also said he was
>>> checking with his new work about this all.
>>
>> [reviving a old thread]
>>
>> Johannes, if you have a minute: did anything ever come out of that?
> No, I guess I can try again, but everyone pretty much moved on from this
> stuff... Would be surprised if Chetan were interested in any of this
> now.
> 
>> Or did anyone else step in to take care of Intels's IOSM driver?
> Not that I know of and I think the business was pretty much wound down?
> No idea really though.

Thx for the reply. Then I guess it's time to mark this as orphaned in
MAINTAINERS, unless anyone objects.

Bagas, you about a year ago had a patch that did some of that. Want to
update and resubmit it? Please include a link to this thread in the
patch description.

Ciao, Thorsten

[PATCH net v2 2/2] MAINTAINERS: Remove linuxwwan@intel.com mailing list

[Bagas Sanjaya]

Messages submitted to the ML bounce (address not found error). In
fact, the ML was mistagged as person maintainer instead of mailing
list.

Remove the ML to keep Cc: lists a bit shorter and not to spam
everyone's inbox with postmaster notifications.

Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
---
 Intel people, if you still want to maintain int1092 (aka IOSM), please
 let me know.

 MAINTAINERS | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 08a7be6a5680ed..f164e573abdae8 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -10450,7 +10450,6 @@ F:	drivers/platform/x86/intel/atomisp2/led.c
 
 INTEL BIOS SAR INT1092 DRIVER
 M:	Shravan Sudhakar <s.shravan@intel.com>
-M:	Intel Corporation <linuxwwan@intel.com>
 L:	platform-driver-x86@vger.kernel.org
 S:	Maintained
 F:	drivers/platform/x86/intel/int1092/
@@ -10879,9 +10878,8 @@ S:	Maintained
 F:	drivers/platform/x86/intel/wmi/thunderbolt.c
 
 INTEL WWAN IOSM DRIVER
-M:	Intel Corporation <linuxwwan@intel.com>
 L:	netdev@vger.kernel.org
-S:	Maintained
+S:	Orphan
 F:	drivers/net/wwan/iosm/
 
 INTEL(R) TRACE HUB
@@ -13500,7 +13498,6 @@ F:	net/dsa/tag_mtk.c
 
 MEDIATEK T7XX 5G WWAN MODEM DRIVER
 M:	Chandrashekar Devegowda <chandrashekar.devegowda@intel.com>
-M:	Intel Corporation <linuxwwan@intel.com>
 R:	Chiranjeevi Rapolu <chiranjeevi.rapolu@linux.intel.com>
 R:	Liu Haijun <haijun.liu@mediatek.com>
 R:	Ricardo Martinez <ricardo.martinez@linux.intel.com>
-- 
An old man doll... just what I always wanted! - Clara