Re: [PATCH] net/usb/hso: Add support for Option GTM671WFS

From: Dan Williams
Date: Tue Aug 05 2014 - 11:57:34 EST

Next message: Dan Williams: "Re: [PATCH] net/usb/hso: Add support for Option GTM671WFS"
Previous message: Jean-Michel Hautbois: "Re: patch "firmware loader: allow disabling of udev as firmware loader" added to driver-core tree"
In reply to: Ricardo Ribalda Delgado: "Re: [PATCH] net/usb/hso: Add support for Option GTM671WFS"
Next in thread: Dan Williams: "Re: [PATCH] net/usb/hso: Add support for Option GTM671WFS"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

On Tue, 2014-08-05 at 08:59 +0200, Ricardo Ribalda Delgado wrote:
> Hello Dan.
>
> I have also been able to start/stop the gps via the gps control port
> (ttyHS2) and get nmea data port through the gps port. (ttyHS1)
>
> Please tell me if you need more tests

No, this looks good enough. Because (a) 'hsotype' is valid for the
ports and (b) we get the hso0 net device and (c) you can get an IP
address on that interface, then this device should definitely be driven
by 'hso' and not option. I'll ack the hso patch.

Dan

> Regards!
>
> root@qt5022:~# cat /sys/class/tty/*/hsotype
> Diagnostic
> GPS
> GPS Control
> Application
> Control
> Modem
> root@qt5022:~# hso_connect up
> Using /etc/conninfo.ini as connection file
> Using /dev/ttyHS3 application port.
> Initializing...
> Trying internet ...
> Connecting...
> trying
> Connected
> Setting IP address to 10.198.XXX.115
> Adding route
> Done.
> root@qt5022:~# ifconfig hso0
> hso0 Link encap:UNSPEC HWaddr
> 00-00-00-00-00-00-00-00-00-00-00-00-00-00-00-00
> inet addr:10.198.XXX.115 P-t-P:10.198.XXX.115 Mask:255.255.255.255
> UP POINTOPOINT RUNNING NOARP MULTICAST MTU:1486 Metric:1
> RX packets:0 errors:0 dropped:0 overruns:0 frame:0
> TX packets:0 errors:0 dropped:0 overruns:0 carrier:0
> collisions:0 txqueuelen:10
> RX bytes:0 (0.0 B) TX bytes:0 (0.0 B)
>
> root@qt5022:~# ping 8.8.8.8
> PING 8.8.8.8 (8.8.8.8) 56(84) bytes of data.
> 64 bytes from 8.8.8.8: icmp_seq=1 ttl=46 time=1163 ms
> 64 bytes from 8.8.8.8: icmp_seq=2 ttl=46 time=304 ms
> 64 bytes from 8.8.8.8: icmp_seq=3 ttl=46 time=72.5 ms
> 64 bytes from 8.8.8.8: icmp_seq=4 ttl=46 time=60.7 ms
> ^C
> --- 8.8.8.8 ping statistics ---
> 4 packets transmitted, 4 received, 0% packet loss, time 3003ms
> rtt min/avg/max/mdev = 60.790/400.296/1163.670/451.296 ms, pipe 2
> root@qt5022:~# cat /dev/ttyHS1
> $GPGSV,4,1,16,24,,,,27,,,,12,,,,32,,,*7A
> $GPGSV,4,2,16,31,,,,30,,,,29,,,,28,,,*78
> $GPGSV,4,3,16,26,,,,25,,,,23,,,,22,,,*7B
> $GPGSV,4,4,16,21,,,,20,,,,19,,,,18,,,*7E
> $GPGGA,,,,,,0,,,,,,,,*66
> $PQXFI,,,,,,,,,,*56
> $GPVTG,,T,,M,,N,,K,N*2C
> $GPRMC,,V,,,,,,,,,,N*53
> $GPGSA,A,1,,,,,,,,,,,,,,,*1E
> $GPGSV,4,1,16,04,,,,24,,,,27,,,,12,,,*7F
> $GPGSV,4,2,16,32,,,,31,,,,30,,,,29,,,*73
> $GPGSV,4,3,16,28,,,,26,,,,25,,,,23,,,*71
> $GPGSV,4,4,16,22,,,,21,,,,20,,,,19,,,*77
> $GPGGA,,,,,,0,,,,,,,,*66
> $PQXFI,,,,,,,,,,*56
> $GPVTG,,T,,M,,N,,K,N*2C
> $GPRMC,,V,,,,,,,,,,N*53
> $GPGSA,A,1,,,,,,,,,,,,,,,*1E
>
> On Mon, Aug 4, 2014 at 8:30 PM, Dan Williams <dcbw@xxxxxxxxxx> wrote:
> > On Mon, 2014-08-04 at 11:20 +0200, Ricardo Ribalda Delgado wrote:
> >> Suggested-by: Dan Williams <dcbw@xxxxxxxxxx>
> >
> > Before we apply this patch though, can you grab for the following for
> > me?
> >
> > cat /sys/class/tty/*/hsotype
> >
> > and lets see if the firmware actually responds. Also, do you get an
> > 'hso0' network interface as reported by ifconfig or /sbin/ip? If you
> > do, then lets do some additional verification to ensure it should be
> > driven by 'hso' instead of option.
> >
> > Dan
> >
> >> On Mon, Aug 4, 2014 at 11:11 AM, Ricardo Ribalda Delgado
> >> <ricardo.ribalda@xxxxxxxxx> wrote:
> >> > After this patch:
> >> >
> >> > [ 32.985530] hso: drivers/net/usb/hso.c: Option Wireless
> >> > [ 33.000452] hso 2-1.4:1.7: Not our interface
> >> > [ 33.001849] usbcore: registered new interface driver hso
> >> >
> >> > root@qt5022:~# ls /dev/ttyHS*
> >> > /dev/ttyHS0 /dev/ttyHS1 /dev/ttyHS2 /dev/ttyHS3 /dev/ttyHS4
> >> > /dev/ttyHS5
> >> >
> >> > root@qt5022:~# lsusb -d 0af0: -vvv
> >> >
> >> > Bus 002 Device 003: ID 0af0:9200 Option
> >> > Device Descriptor:
> >> > bLength 18
> >> > bDescriptorType 1
> >> > bcdUSB 2.00
> >> > bDeviceClass 255 Vendor Specific Class
> >> > bDeviceSubClass 255 Vendor Specific Subclass
> >> > bDeviceProtocol 255 Vendor Specific Protocol
> >> > bMaxPacketSize0 64
> >> > idVendor 0x0af0 Option
> >> > idProduct 0x9200
> >> > bcdDevice 0.00
> >> > iManufacturer 3 Option N.V.
> >> > iProduct 2 Globetrotter HSUPA Modem
> >> > iSerial 0
> >> > bNumConfigurations 1
> >> > Configuration Descriptor:
> >> > bLength 9
> >> > bDescriptorType 2
> >> > wTotalLength 200
> >> > bNumInterfaces 8
> >> > bConfigurationValue 1
> >> > iConfiguration 1 Option Configuration
> >> > bmAttributes 0xe0
> >> > Self Powered
> >> > Remote Wakeup
> >> > MaxPower 100mA
> >> > Interface Descriptor:
> >> > bLength 9
> >> > bDescriptorType 4
> >> > bInterfaceNumber 0
> >> > bAlternateSetting 0
> >> > bNumEndpoints 2
> >> > bInterfaceClass 255 Vendor Specific Class
> >> > bInterfaceSubClass 255 Vendor Specific Subclass
> >> > bInterfaceProtocol 255 Vendor Specific Protocol
> >> > iInterface 0
> >> > Endpoint Descriptor:
> >> > bLength 7
> >> > bDescriptorType 5
> >> > bEndpointAddress 0x81 EP 1 IN
> >> > bmAttributes 2
> >> > Transfer Type Bulk
> >> > Synch Type None
> >> > Usage Type Data
> >> > wMaxPacketSize 0x0200 1x 512 bytes
> >> > bInterval 32
> >> > Endpoint Descriptor:
> >> > bLength 7
> >> > bDescriptorType 5
> >> > bEndpointAddress 0x01 EP 1 OUT
> >> > bmAttributes 2
> >> > Transfer Type Bulk
> >> > Synch Type None
> >> > Usage Type Data
> >> > wMaxPacketSize 0x0200 1x 512 bytes
> >> > bInterval 32
> >> > Interface Descriptor:
> >> > bLength 9
> >> > bDescriptorType 4
> >> > bInterfaceNumber 1
> >> > bAlternateSetting 0
> >> > bNumEndpoints 2
> >> > bInterfaceClass 255 Vendor Specific Class
> >> > bInterfaceSubClass 255 Vendor Specific Subclass
> >> > bInterfaceProtocol 255 Vendor Specific Protocol
> >> > iInterface 0
> >> > Endpoint Descriptor:
> >> > bLength 7
> >> > bDescriptorType 5
> >> > bEndpointAddress 0x82 EP 2 IN
> >> > bmAttributes 2
> >> > Transfer Type Bulk
> >> > Synch Type None
> >> > Usage Type Data
> >> > wMaxPacketSize 0x0200 1x 512 bytes
> >> > bInterval 32
> >> > Endpoint Descriptor:
> >> > bLength 7
> >> > bDescriptorType 5
> >> > bEndpointAddress 0x02 EP 2 OUT
> >> > bmAttributes 2
> >> > Transfer Type Bulk
> >> > Synch Type None
> >> > Usage Type Data
> >> > wMaxPacketSize 0x0200 1x 512 bytes
> >> > bInterval 32
> >> > Interface Descriptor:
> >> > bLength 9
> >> > bDescriptorType 4
> >> > bInterfaceNumber 2
> >> > bAlternateSetting 0
> >> > bNumEndpoints 2
> >> > bInterfaceClass 255 Vendor Specific Class
> >> > bInterfaceSubClass 255 Vendor Specific Subclass
> >> > bInterfaceProtocol 255 Vendor Specific Protocol
> >> > iInterface 0
> >> > Endpoint Descriptor:
> >> > bLength 7
> >> > bDescriptorType 5
> >> > bEndpointAddress 0x83 EP 3 IN
> >> > bmAttributes 2
> >> > Transfer Type Bulk
> >> > Synch Type None
> >> > Usage Type Data
> >> > wMaxPacketSize 0x0200 1x 512 bytes
> >> > bInterval 32
> >> > Endpoint Descriptor:
> >> > bLength 7
> >> > bDescriptorType 5
> >> > bEndpointAddress 0x03 EP 3 OUT
> >> > bmAttributes 2
> >> > Transfer Type Bulk
> >> > Synch Type None
> >> > Usage Type Data
> >> > wMaxPacketSize 0x0200 1x 512 bytes
> >> > bInterval 32
> >> > Interface Descriptor:
> >> > bLength 9
> >> > bDescriptorType 4
> >> > bInterfaceNumber 3
> >> > bAlternateSetting 0
> >> > bNumEndpoints 2
> >> > bInterfaceClass 255 Vendor Specific Class
> >> > bInterfaceSubClass 255 Vendor Specific Subclass
> >> > bInterfaceProtocol 255 Vendor Specific Protocol
> >> > iInterface 0
> >> > Endpoint Descriptor:
> >> > bLength 7
> >> > bDescriptorType 5
> >> > bEndpointAddress 0x84 EP 4 IN
> >> > bmAttributes 2
> >> > Transfer Type Bulk
> >> > Synch Type None
> >> > Usage Type Data
> >> > wMaxPacketSize 0x0200 1x 512 bytes
> >> > bInterval 32
> >> > Endpoint Descriptor:
> >> > bLength 7
> >> > bDescriptorType 5
> >> > bEndpointAddress 0x04 EP 4 OUT
> >> > bmAttributes 2
> >> > Transfer Type Bulk
> >> > Synch Type None
> >> > Usage Type Data
> >> > wMaxPacketSize 0x0200 1x 512 bytes
> >> > bInterval 32
> >> > Interface Descriptor:
> >> > bLength 9
> >> > bDescriptorType 4
> >> > bInterfaceNumber 4
> >> > bAlternateSetting 0
> >> > bNumEndpoints 2
> >> > bInterfaceClass 255 Vendor Specific Class
> >> > bInterfaceSubClass 255 Vendor Specific Subclass
> >> > bInterfaceProtocol 255 Vendor Specific Protocol
> >> > iInterface 0
> >> > Endpoint Descriptor:
> >> > bLength 7
> >> > bDescriptorType 5
> >> > bEndpointAddress 0x85 EP 5 IN
> >> > bmAttributes 2
> >> > Transfer Type Bulk
> >> > Synch Type None
> >> > Usage Type Data
> >> > wMaxPacketSize 0x0200 1x 512 bytes
> >> > bInterval 32
> >> > Endpoint Descriptor:
> >> > bLength 7
> >> > bDescriptorType 5
> >> > bEndpointAddress 0x05 EP 5 OUT
> >> > bmAttributes 2
> >> > Transfer Type Bulk
> >> > Synch Type None
> >> > Usage Type Data
> >> > wMaxPacketSize 0x0200 1x 512 bytes
> >> > bInterval 32
> >> > Interface Descriptor:
> >> > bLength 9
> >> > bDescriptorType 4
> >> > bInterfaceNumber 5
> >> > bAlternateSetting 0
> >> > bNumEndpoints 2
> >> > bInterfaceClass 255 Vendor Specific Class
> >> > bInterfaceSubClass 255 Vendor Specific Subclass
> >> > bInterfaceProtocol 255 Vendor Specific Protocol
> >> > iInterface 0
> >> > Endpoint Descriptor:
> >> > bLength 7
> >> > bDescriptorType 5
> >> > bEndpointAddress 0x06 EP 6 OUT
> >> > bmAttributes 2
> >> > Transfer Type Bulk
> >> > Synch Type None
> >> > Usage Type Data
> >> > wMaxPacketSize 0x0200 1x 512 bytes
> >> > bInterval 32
> >> > Endpoint Descriptor:
> >> > bLength 7
> >> > bDescriptorType 5
> >> > bEndpointAddress 0x86 EP 6 IN
> >> > bmAttributes 2
> >> > Transfer Type Bulk
> >> > Synch Type None
> >> > Usage Type Data
> >> > wMaxPacketSize 0x0200 1x 512 bytes
> >> > bInterval 32
> >> > Interface Descriptor:
> >> > bLength 9
> >> > bDescriptorType 4
> >> > bInterfaceNumber 6
> >> > bAlternateSetting 0
> >> > bNumEndpoints 3
> >> > bInterfaceClass 255 Vendor Specific Class
> >> > bInterfaceSubClass 255 Vendor Specific Subclass
> >> > bInterfaceProtocol 255 Vendor Specific Protocol
> >> > iInterface 0
> >> > Endpoint Descriptor:
> >> > bLength 7
> >> > bDescriptorType 5
> >> > bEndpointAddress 0x87 EP 7 IN
> >> > bmAttributes 3
> >> > Transfer Type Interrupt
> >> > Synch Type None
> >> > Usage Type Data
> >> > wMaxPacketSize 0x0040 1x 64 bytes
> >> > bInterval 5
> >> > Endpoint Descriptor:
> >> > bLength 7
> >> > bDescriptorType 5
> >> > bEndpointAddress 0x88 EP 8 IN
> >> > bmAttributes 2
> >> > Transfer Type Bulk
> >> > Synch Type None
> >> > Usage Type Data
> >> > wMaxPacketSize 0x0200 1x 512 bytes
> >> > bInterval 32
> >> > Endpoint Descriptor:
> >> > bLength 7
> >> > bDescriptorType 5
> >> > bEndpointAddress 0x07 EP 7 OUT
> >> > bmAttributes 2
> >> > Transfer Type Bulk
> >> > Synch Type None
> >> > Usage Type Data
> >> > wMaxPacketSize 0x0200 1x 512 bytes
> >> > bInterval 32
> >> > Interface Descriptor:
> >> > bLength 9
> >> > bDescriptorType 4
> >> > bInterfaceNumber 7
> >> > bAlternateSetting 0
> >> > bNumEndpoints 2
> >> > bInterfaceClass 8 Mass Storage
> >> > bInterfaceSubClass 6 SCSI
> >> > bInterfaceProtocol 80 Bulk-Only
> >> > iInterface 0
> >> > Endpoint Descriptor:
> >> > bLength 7
> >> > bDescriptorType 5
> >> > bEndpointAddress 0x08 EP 8 OUT
> >> > bmAttributes 2
> >> > Transfer Type Bulk
> >> > Synch Type None
> >> > Usage Type Data
> >> > wMaxPacketSize 0x0200 1x 512 bytes
> >> > bInterval 1
> >> > Endpoint Descriptor:
> >> > bLength 7
> >> > bDescriptorType 5
> >> > bEndpointAddress 0x89 EP 9 IN
> >> > bmAttributes 2
> >> > Transfer Type Bulk
> >> > Synch Type None
> >> > Usage Type Data
> >> > wMaxPacketSize 0x0200 1x 512 bytes
> >> > bInterval 1
> >> > Device Qualifier (for other device speed):
> >> > bLength 10
> >> > bDescriptorType 6
> >> > bcdUSB 2.00
> >> > bDeviceClass 255 Vendor Specific Class
> >> > bDeviceSubClass 255 Vendor Specific Subclass
> >> > bDeviceProtocol 255 Vendor Specific Protocol
> >> > bMaxPacketSize0 64
> >> > bNumConfigurations 1
> >> > Device Status: 0x0001
> >> > Self Powered
> >> >
> >> > Signed-off-by: Ricardo Ribalda Delgado <ricardo.ribalda@xxxxxxxxx>
> >> > ---
> >> > drivers/net/usb/hso.c | 1 +
> >> > 1 file changed, 1 insertion(+)
> >> >
> >> > diff --git a/drivers/net/usb/hso.c b/drivers/net/usb/hso.c
> >> > index a3a0586..eb12261 100644
> >> > --- a/drivers/net/usb/hso.c
> >> > +++ b/drivers/net/usb/hso.c
> >> > @@ -469,6 +469,7 @@ static const struct usb_device_id hso_ids[] = {
> >> > {USB_DEVICE(0x0af0, 0x8800)},
> >> > {USB_DEVICE(0x0af0, 0x8900)},
> >> > {USB_DEVICE(0x0af0, 0x9000)},
> >> > + {USB_DEVICE(0x0af0, 0x9200)}, /* Option GTM671WFS */
> >> > {USB_DEVICE(0x0af0, 0xd035)},
> >> > {USB_DEVICE(0x0af0, 0xd055)},
> >> > {USB_DEVICE(0x0af0, 0xd155)},
> >> > --
> >> > 2.0.1
> >> >
> >>
> >>
> >>
> >
> >
>
>
>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

Next message: Dan Williams: "Re: [PATCH] net/usb/hso: Add support for Option GTM671WFS"
Previous message: Jean-Michel Hautbois: "Re: patch "firmware loader: allow disabling of udev as firmware loader" added to driver-core tree"
In reply to: Ricardo Ribalda Delgado: "Re: [PATCH] net/usb/hso: Add support for Option GTM671WFS"
Next in thread: Dan Williams: "Re: [PATCH] net/usb/hso: Add support for Option GTM671WFS"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]