/drivers/net/usb/qmi_wwan.c: Added support for Cinterion's PLxx WWAN Interface
by adding QMI_FIXED_INTF with Cinterion's Vendor ID as well as Product ID and
WWAN Interface Number.
Signed-off-by: Hans-Christoph Schemmel <[email protected]>
Signed-off-by: Christian Schmiedl <[email protected]>
---
patch is against linux-3.9
--- linux-3.9/drivers/net/usb/qmi_wwan.c.orig 2013-05-06 08:10:11.170034451 +0200
+++ linux-3.9/drivers/net/usb/qmi_wwan.c 2013-05-06 08:58:00.800264179 +0200
@@ -569,6 +569,7 @@ static const struct usb_device_id produc
{QMI_FIXED_INTF(0x1bbb, 0x011e, 4)}, /* Telekom Speedstick LTE II (Alcatel One Touch L100V LTE) */
{QMI_FIXED_INTF(0x2357, 0x0201, 4)}, /* TP-LINK HSUPA Modem MA180 */
{QMI_FIXED_INTF(0x1bc7, 0x1200, 5)}, /* Telit LE920 */
+ {QMI_FIXED_INTF(0x1e2d, 0x12d1, 4)}, /* Cinterion PLxx */
/* 4. Gobi 1000 devices */
{QMI_GOBI1K_DEVICE(0x05c6, 0x9212)}, /* Acer Gobi Modem Device */
Schmiedl Christian <[email protected]> writes:
> /drivers/net/usb/qmi_wwan.c: Added support for Cinterion's PLxx WWAN Interface
> by adding QMI_FIXED_INTF with Cinterion's Vendor ID as well as Product ID and
> WWAN Interface Number.
Much better. But this patch need to go to the netdev list to get picked
up by the net patchworks and maintainer. It was posted there the last
time, so I don't understand why you dropped that now? The minimum To/Cc
list for usbnet drivers is:
[email protected], [email protected]
Anything else is a bonus, and adding linux-kernel is of course fine.
Hint: Use linux/scripts/get_maintainer.pl to get a list of addresses to
send a patch to.
> Signed-off-by: Hans-Christoph Schemmel <[email protected]>
> Signed-off-by: Christian Schmiedl <[email protected]>
This is of course fine, but the patch was sent by Hans-Christoph the
last time. If he is the original author, then you also want to preserve
that authorship in the git history (in addition to those SOBs). The
proper way to do that is to add a line like this as the first line of
the email body, followed by a single empty line:
From: Hans-Christoph Schemmel <[email protected]>
This will make "git am" use Hans-Christoph as the author, even if the
patch is posted by you.
And since you obviously have to resend this anyway: Please drop the file
name ("/drivers/net/usb/qmi_wwan.c: ") from the commit message. It is
part of the patch itself, and including it in the commit message just
adds noise.
Thanks,
Bjørn