Skip to contentSkip to conversation

: Add id for Onda MSA501HS HSDPA modem

PATCH In review Arnaldo Carvalho de Melo 2 participants 2 messages May 16, 2008

Subsystems (2)
USB USB serial
lore
Share & tools
lore

Arnaldo Carvalho de Melo

author
#0 lore raw

PATCH 1/1 : Add id for Onda MSA501HS HSDPA modem

Hi Matthias,

Please consider adding the following USB product id to the
option GSM modem driver.

Using the generic usbserial kernel driver I can only get to
~490 Kbit/s, while when using your excellent driver I'm being able to saturate the 1 Mbit/s bandwidth I subscribed from TIM WEB here in Brazil.

Signed-off-by Arnaldo Carvalho de Melo <[email protected]>

Best Regards,

- Arnaldo

drivers/usb/serial/option.c +2 -0
index e7e016e..47be78b 100644
@@ -183,6 +183,7 @@ static int  option_send_setup(struct usb_serial_port *port);
 #define AXESSTEL_PRODUCT_MV110H			0x1000
 
 #define ONDA_VENDOR_ID				0x19d2
+#define ONDA_PRODUCT_MSA501HS			0x0001
 #define ONDA_PRODUCT_ET502HS			0x0002
 
 #define BANDRICH_VENDOR_ID			0x1A8D

@@ -297,6 +298,7 @@ static struct usb_device_id option_ids[] = {
 	{ USB_DEVICE(ANYDATA_VENDOR_ID, ANYDATA_PRODUCT_ADU_E100A) },
 	{ USB_DEVICE(ANYDATA_VENDOR_ID, ANYDATA_PRODUCT_ADU_500A) },
 	{ USB_DEVICE(AXESSTEL_VENDOR_ID, AXESSTEL_PRODUCT_MV110H) },
+	{ USB_DEVICE(ONDA_VENDOR_ID, ONDA_PRODUCT_MSA501HS) },
 	{ USB_DEVICE(ONDA_VENDOR_ID, ONDA_PRODUCT_ET502HS) },
 	{ USB_DEVICE(BANDRICH_VENDOR_ID, BANDRICH_PRODUCT_C100_1) },
 	{ USB_DEVICE(BANDRICH_VENDOR_ID, BANDRICH_PRODUCT_C100_2) },
#1 lore raw
Quoted 10 lines · Arnaldo Carvalho de Melo

Hi Matthias,

Please consider adding the following USB product id to the
option GSM modem driver.

Using the generic usbserial kernel driver I can only get to
~490 Kbit/s, while when using your excellent driver I'm being able to saturate the 1 Mbit/s bandwidth I subscribed from TIM WEB here in Brazil.

Signed-off-by Arnaldo Carvalho de Melo <[email protected]>

Hm, I already added this device to the driver a few days ago, but it was under a different section and name. I'll go fix it up to use these defines as that makes more sense, sorry for missing that previously.

thanks,

greg k-h