Skip to contentSkip to conversation

Sierra Wireless - Add TRU-Install (c) Support

PATCH In review Kevin Lloyd 3 participants 3 messages Jul 11, 2007

Subsystems (3)
USB USB serial USB mass storage driver
lore
Share & tools
lore

Kevin Lloyd

author
#0 lore raw

PATCH 1/2 Sierra Wireless - Add TRU-Install (c) Support

From: Kevin Lloyd <[email protected]>

This patch adds compatibility with Sierra Wireless' new TRU-Install feature. Future devices that use this feature will not work unless this patch has been applied.

This patch was tested on the 2.6.21.1 kernel source patched with the following patches (found at http://www.kernel.org/pub/linux/kernel/people/gregkh/gregkh-2.6/patches/usb/):
usb-sierra-cleanup-urb-startup.patch
usb-sierra-fix-status-usage.patch
usb-sierra-status.patch

There were two previous submissions for similar support (6/6/2007 & 6/5/2007) they were however rejected so please disregard those.

Signed-off-by Kevin Lloyd <[email protected]>

---

drivers/usb/serial/sierra.c +0 -0
diff -uprN a/drivers/usb/serial/sierra.c b/drivers/usb/serial/sierra.c
@@ -1,7 +1,7 @@

/*
USB Driver for Sierra Wireless

-  Copyright (C) 2006  Kevin Lloyd <[email protected]>
+  Copyright (C) 2007  Kevin Lloyd <[email protected]>

IMPORTANT DISCLAIMER: This driver is not commercially supported by Sierra Wireless. Use at your own risk.
@@ -12,10 +12,9 @@

Portions based on the option driver by Matthias Urlichs <[email protected]> Whom based his on the Keyspan driver by Hugh Blemings <[email protected]>
-
*/

-#define DRIVER_VERSION "v.1.0.6"
+#define DRIVER_VERSION "v.1.2.4"
#define DRIVER_AUTHOR "Kevin Lloyd <[email protected]>" #define DRIVER_DESC "USB Driver for Sierra Wireless USB modems"

@@ -28,6 +27,70 @@
#include <linux/usb.h>
#include <linux/usb/serial.h>

+#define SWIMS_USB_REQUEST_SetMode 0x0B
+#define SWIMS_USB_REQUEST_TYPE_SetMode 0x40
+#define SWIMS_USB_INDEX_SetMode 0x0000
+#define SWIMS_SET_MODE_Modem 0x0001
+
+/* per port private data */
+#define N_IN_URB 4
+#define N_OUT_URB 4
+#define IN_BUFLEN 4096
+
+static int debug;
+
+enum devicetype {
+ DEVICE_3_PORT = 0,
+ DEVICE_1_PORT = 1,
+ DEVICE_INSTALLER = 2,
+};
+
+int sierra_set_power_state(struct usb_device *udev, __u16 swiState) +{
+ int result;
+ dev_dbg("SET POWER STATE");
+ result = usb_control_msg(udev, usb_sndctrlpipe(udev, 0),
+ 0x00, /* __u8 request */
+ 0x40, /* __u8 request type */
+ swiState, /* __u16 value */
+ 0, /* __u16 index */
+ NULL, /* void *data */
+ 0, /* __u16 size */
+ USB_CTRL_SET_TIMEOUT); /* int timeout */
+ return result;
+}
+
+int sierra_set_ms_mode(struct usb_device *udev, __u16 eSocMode) +{
+ int result;
+ dev_dbg("DEVICE MODE SWITCH");
+ result = usb_control_msg(udev, usb_sndctrlpipe(udev, 0),
+ SWIMS_USB_REQUEST_SetMode, /* __u8 request */
+ SWIMS_USB_REQUEST_TYPE_SetMode, /* __u8 request type */
+ eSocMode, /* __u16 value */
+ SWIMS_USB_INDEX_SetMode, /* __u16 index */
+ NULL, /* void *data */
+ 0, /* __u16 size */
+ USB_CTRL_SET_TIMEOUT); /* int timeout */
+ return result;
+}
+
+int sierra_probe(struct usb_interface *iface, const struct usb_device_id *id) +{
+ int result;
+ struct usb_device *udev;
+
+ udev = usb_get_dev(interface_to_usbdev(iface));
+
+ /* Check if in installer mode */
+ if (id->driver_info == DEVICE_INSTALLER){
+ dev_dbg("FOUND DEVICE(SW)\n");
+ result = sierra_set_ms_mode(udev, SWIMS_SET_MODE_Modem);
+ return 0;
+ }
+
+ return usb_serial_probe(iface, id);
+}

static struct usb_device_id id_table [] = {
	{ USB_DEVICE(0x1199, 0x0017) },	/* Sierra Wireless EM5625 */
@@ -43,8 +106,10 @@ static struct usb_device_id id_table [] 
	{ USB_DEVICE(0x1199, 0x6812) },	/* Sierra Wireless MC8775 */
	{ USB_DEVICE(0x1199, 0x6820) },	/* Sierra Wireless AirCard 875 */

- { USB_DEVICE(0x1199, 0x0112) }, /* Sierra Wireless AirCard 580 */
- { USB_DEVICE(0x0F3D, 0x0112) }, /* AirPrime/Sierra PC 5220 */
+ { USB_DEVICE(0x1199, 0x0112), .driver_info = DEVICE_1_PORT }, /* Sierra Wireless AirCard 580 */
+ { USB_DEVICE(0x0F3D, 0x0112), .driver_info = DEVICE_1_PORT }, /* Airprime/Sierra PC 5220 */
+
+ { USB_DEVICE(0x1199, 0x0FFF), .driver_info = DEVICE_INSTALLER},

	{ }
};

MODULE_DEVICE_TABLE(usb, id_table);
@@ -56,6 +121,7 @@ static struct usb_device_id id_table_1po };

static struct usb_device_id id_table_3port [] = {
+ /* CDMA Devices */

	{ USB_DEVICE(0x1199, 0x0017) },	/* Sierra Wireless EM5625 */
	{ USB_DEVICE(0x1199, 0x0018) },	/* Sierra Wireless MC5720 */
	{ USB_DEVICE(0x1199, 0x0218) },	/* Sierra Wireless MC5720 */
@@ -63,6 +129,8 @@ static struct usb_device_id id_table_3po
	{ USB_DEVICE(0x1199, 0x0019) },	/* Sierra Wireless AirCard 595 */
	{ USB_DEVICE(0x1199, 0x0120) },	/* Sierra Wireless AirCard 595U */
	{ USB_DEVICE(0x1199, 0x0021) },	/* Sierra Wireless AirCard 597E */
+
+	/* GSM/UMTS Devices */
	{ USB_DEVICE(0x1199, 0x6802) },	/* Sierra Wireless MC8755 */
	{ USB_DEVICE(0x1199, 0x6804) },	/* Sierra Wireless MC8755 */
	{ USB_DEVICE(0x1199, 0x6803) },	/* Sierra Wireless MC8765 */

@@ -73,20 +141,13 @@ static struct usb_device_id id_table_3po

static struct usb_driver sierra_driver = {
.name = "sierra",
- .probe = usb_serial_probe,
+ .probe = sierra_probe,
.disconnect = usb_serial_disconnect,
.id_table = id_table,
.no_dynamic_id = 1,
};

-static int debug;
-
-/* per port private data */
-#define N_IN_URB 4
-#define N_OUT_URB 4
-#define IN_BUFLEN 4096
-
struct sierra_port_private {
spinlock_t lock; /* lock the structure */
int outstanding_urbs; /* number of out urbs in flight */
@@ -421,8 +482,7 @@ static int sierra_open(struct usb_serial

	int i;
	struct urb *urb;
	int result;
-	__u16 set_mode_dzero = 0x0000;
-
+	
	portdata = usb_get_serial_port_data(port);

	dbg("%s", __FUNCTION__);

@@ -457,12 +517,6 @@ static int sierra_open(struct usb_serial

	port->tty->low_latency = 1;

- /* set mode to D0 */
- result = usb_control_msg(serial->dev,
- usb_rcvctrlpipe(serial->dev, 0),
- 0x00, 0x40, set_mode_dzero, 0, NULL,
- 0, USB_CTRL_SET_TIMEOUT);
-

	sierra_send_setup(port);

	/* start up the interrupt endpoint if we have one */

@@ -510,6 +564,9 @@ static int sierra_startup(struct usb_ser

	dbg("%s", __FUNCTION__);

+ /*Set Device mode to D0 */
+ sierra_set_power_state(serial->dev, 0x0000);
+

	/* Now setup per port private data */
	for (i = 0; i < serial->num_ports; i++) {
		port = serial->port[i];
drivers/usb/storage/unusual_devs.h +0 -0
diff -uprN a/drivers/usb/storage/unusual_devs.h b/drivers/usb/storage/unusual_devs.h
@@ -1384,6 +1384,17 @@ UNUSUAL_DEV(  0x1019, 0x0c55, 0x0000, 0x

		US_SC_DEVICE, US_PR_DEVICE, usb_stor_ucr61s2b_init,
		0 ),

+/* Reported by Kevin Lloyd <[email protected]>
+ * Entry is needed for the initializer function override, + * which instructs the device to load as a modem
+ * device.
+ */
+UNUSUAL_DEV( 0x1199, 0x0fff, 0x0000, 0x9999,
+ "Sierra Wireless",
+ "USB MMC Storage",
+ US_SC_DEVICE, US_PR_DEVICE, NULL, + US_FL_IGNORE_DEVICE),
+
/* Reported by Jaco Kroon <[email protected]>
* The usb-storage module found on the Digitech GNX4 (and supposedly other
* devices) misbehaves and causes a bunch of invalid I/O errors.

#1 lore raw

Am Donnerstag, 12. Juli 2007 schrieb Kevin Lloyd:

Quoted 3 lines

From: Kevin Lloyd <[email protected]>

This patch adds compatibility with Sierra Wireless' new TRU-Install feature. Future devices that use this feature will not work unless this patch has been applied.

Is this some type of CD-ROM simulation to provide drivers?

Review

Quoted 12 lines
(quoted diff) +12 -0
+int sierra_probe(struct usb_interface *iface, const struct usb_device_id *id)
+{
+	int result;	
+	struct usb_device *udev;
+
+	udev = usb_get_dev(interface_to_usbdev(iface));
+
+	/* Check if in installer mode */
+	if (id->driver_info == DEVICE_INSTALLER){
+		dev_dbg("FOUND DEVICE(SW)\n");
+		result = sierra_set_ms_mode(udev, SWIMS_SET_MODE_Modem);
+		return 0;

This is not a good idea in the long run. If you don't return an error here, disconnect() will be called for your driver and will have to deal with a semiinitialized device.

Regards
Oliver

#2 lore raw

Unfortunately I can not myself comment on the feature at this point, however the code is pretty obvious :).

I'll go ahead and change it so that it returns the result of sierra_set_ms_mode.

Also, I realized that the second patch (02/02) fixes a bug in the first patch (01/02), particularly the dev_dbg messages are implemented incorrectly in the first patch. I'll move that fix up to the first patch.

I'll wait another day for other comments and then resubmit with those changes.

- Kevin

-----Original Message-----
From: Oliver Neukum [mailto:[email protected]]
Sent: Wednesday, July 11, 2007 3:37 PM
To: [email protected]
Cc: Kevin Lloyd; [email protected]; Linux Development Group; [email protected]; [email protected]
Subject: Re: [linux-usb-devel] [PATCH 01/02] Sierra Wireless - Add TRU-Install (c) Support

Am Donnerstag, 12. Juli 2007 schrieb Kevin Lloyd:

Quoted 3 lines

From: Kevin Lloyd <[email protected]>

This patch adds compatibility with Sierra Wireless' new TRU-Install

feature. Future devices that use this feature will not work unless this patch has been applied.

Is this some type of CD-ROM simulation to provide drivers?

Review

Quoted 12 lines
(quoted diff) +12 -0
+int sierra_probe(struct usb_interface *iface, const struct 
+usb_device_id *id) {
+	int result;	
+	struct usb_device *udev;
+
+	udev = usb_get_dev(interface_to_usbdev(iface));
+
+	/* Check if in installer mode */
+	if (id->driver_info == DEVICE_INSTALLER){
+		dev_dbg("FOUND DEVICE(SW)\n");
+		result = sierra_set_ms_mode(udev, SWIMS_SET_MODE_Modem);
+		return 0;

This is not a good idea in the long run. If you don't return an error here,
disconnect() will be called for your driver and will have to deal with a semiinitialized device.

Regards
Oliver