[PATCH][OPTION]: Add id for Onda MSA501HS HSDPA modem

[PATCH][OPTION]: Add id for Onda MSA501HS HSDPA modem

[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 <acme@redhat.com>

Best Regards,

- Arnaldo

diff --git a/drivers/usb/serial/option.c b/drivers/usb/serial/option.c
index e7e016e..47be78b 100644
--- a/drivers/usb/serial/option.c
+++ b/drivers/usb/serial/option.c
@@ -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) },

Re: [PATCH][OPTION]: Add id for Onda MSA501HS HSDPA modem

[Greg KH]

On Fri, May 16, 2008 at 03:41:40PM -0300, Arnaldo Carvalho de Melo wrote:
> 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 <acme@redhat.com>

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