[PATCH] cdc_ether: add usb id for Ericsson F3507g

[PATCH] cdc_ether: add usb id for Ericsson F3507g

[Bjørn Mork]

The Ericsson F3507g wireless broadband module provides a CDC Ethernet
compliant interface, but identifies it as a "Mobile Direct Line" CDC
subclass, thereby preventing the CDC Ethernet class driver from picking
it up.  This patch adds the device id to cdc_ether.c as a workaround.

Ericsson has provided a "class" driver for this device:
http://kerneltrap.org/mailarchive/linux-net/2008/10/28/3832094
But closer inspection of that driver reveals that it adds little more
than duplication of code from cdc_ether.c.  See also
http://marc.info/?l=linux-usb&m=123334979706403&w=2


Signed-off-by: Bjørn Mork <bjorn@mork.no>

diff --git a/drivers/net/usb/cdc_ether.c b/drivers/net/usb/cdc_ether.c
index 0e061df..55e8ecc 100644
--- a/drivers/net/usb/cdc_ether.c
+++ b/drivers/net/usb/cdc_ether.c
@@ -559,6 +559,11 @@ static const struct usb_device_id	products [] = {
 	USB_INTERFACE_INFO(USB_CLASS_COMM, USB_CDC_SUBCLASS_ETHERNET,
 			USB_CDC_PROTO_NONE),
 	.driver_info = (unsigned long) &cdc_info,
+}, {
+	/* Ericsson F3507g */
+	USB_DEVICE_AND_INTERFACE_INFO(0x0bdb, 0x1900, USB_CLASS_COMM,
+			USB_CDC_SUBCLASS_MDLM, USB_CDC_PROTO_NONE),
+	.driver_info = (unsigned long) &cdc_info,
 },
 	{ },		// END
 };

Re: [PATCH] cdc_ether: add usb id for Ericsson F3507g

[David Miller]

From: Bjørn Mork <bjorn@mork.no>
Date: Wed, 25 Feb 2009 15:33:58 +0100

> The Ericsson F3507g wireless broadband module provides a CDC Ethernet
> compliant interface, but identifies it as a "Mobile Direct Line" CDC
> subclass, thereby preventing the CDC Ethernet class driver from picking
> it up.  This patch adds the device id to cdc_ether.c as a workaround.
> 
> Ericsson has provided a "class" driver for this device:
> http://kerneltrap.org/mailarchive/linux-net/2008/10/28/3832094
> But closer inspection of that driver reveals that it adds little more
> than duplication of code from cdc_ether.c.  See also
> http://marc.info/?l=linux-usb&m=123334979706403&w=2
> 
> Signed-off-by: Bjørn Mork <bjorn@mork.no>

Applied, thanks.