Re: [PATCH] net: usb: qmi_wwan: added support for Thales Cinterion PLSx3 modem family

Giacinto Cifelli [off-list ref] writes:

Hi Bjørn,
I have fixed and resent, but from your comment I might not have
selected the right line from maintaner.pl?
what I have is this:
$ ./scripts/get_maintainer.pl --file drivers/net/usb/qmi_wwan.c
"Bjørn Mork" [off-list ref] (maintainer:USB QMI WWAN NETWORK DRIVER)
"David S. Miller" [off-list ref] (maintainer:NETWORKING DRIVERS)
Jakub Kicinski [off-list ref] (maintainer:NETWORKING DRIVERS)
netdev@vger.kernel.org (open list:USB QMI WWAN NETWORK DRIVER)
<<<< this seems the right one
linux-usb@vger.kernel.org (open list:USB NETWORKING DRIVERS)
linux-kernel@vger.kernel.org (open list)

I have at the same time sent a patch for another enumeration of the
same product, for cdc_ether.  In that case, I have picked the
following line, which also looked the best fit:
  linux-usb@vger.kernel.org (open list:USB CDC ETHERNET DRIVER)

Did I misinterpret the results of the script?

Yes, but I'll be the first to admit that it isn't easy.  

netdev is definitely correct, and the most important one.

But in theory you are supposed to use all the listed addresses.  Except
that I don't think you need to CC David (and Jakub?) since they probably
read everything in netdev anyway.  And I believe many (most?) people
leave out the linux-kernel catch-all, since it doesn't provide any extra
coverage for networking. At least I do.

Then there's the two remaining addresses.  The linux-usb list is
traditionally CCed on patches touching USB drivers, since the USB
experts are there and not necessarily in netdev.  And I'd like a copy
because that's the only way I'll be able to catch these patches.  I
don't read any of the lists regularily.

This is my interpretation only.  I am sure there are other opinions. But
as usual, you cannot do anything wrong. The worst that can ever happen
is that you have to resend a patch or miss my review of it ;-)


Bjørn