diff mbox

[net,stable] net: huawei_cdc_ncm: add "subclass 3" devices

Message ID 1405596849-11800-1-git-send-email-bjorn@mork.no
State Accepted, archived
Delegated to: David Miller
Headers show

Commit Message

Bjørn Mork July 17, 2014, 11:34 a.m. UTC
Huawei's usage of the subclass and protocol fields is not 100%
clear to us, but there appears to be a very strict system.

A device with the "shared" device ID 12d1:1506 and this NCM
function was recently reported (showing only default altsetting):

    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        1
      bAlternateSetting       0
      bNumEndpoints           1
      bInterfaceClass       255 Vendor Specific Class
      bInterfaceSubClass      3
      bInterfaceProtocol     22
      iInterface              8 CDC Network Control Model (NCM)
      ** UNRECOGNIZED:  05 24 00 10 01
      ** UNRECOGNIZED:  06 24 1a 00 01 1f
      ** UNRECOGNIZED:  0c 24 1b 00 01 00 04 10 14 dc 05 20
      ** UNRECOGNIZED:  0d 24 0f 0a 0f 00 00 00 ea 05 03 00 01
      ** UNRECOGNIZED:  05 24 06 01 01
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x85  EP 5 IN
        bmAttributes            3
          Transfer Type            Interrupt
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0010  1x 16 bytes
        bInterval               9

Cc: Enrico Mioso <mrkiko.rs@gmail.com>
Signed-off-by: Bjørn Mork <bjorn@mork.no>
---
 drivers/net/usb/huawei_cdc_ncm.c | 3 +++
 1 file changed, 3 insertions(+)

Comments

David Miller July 17, 2014, 11:51 p.m. UTC | #1
From: Bjørn Mork <bjorn@mork.no>
Date: Thu, 17 Jul 2014 13:34:09 +0200

> Huawei's usage of the subclass and protocol fields is not 100%
> clear to us, but there appears to be a very strict system.
> 
> A device with the "shared" device ID 12d1:1506 and this NCM
> function was recently reported (showing only default altsetting):
 ...
> Cc: Enrico Mioso <mrkiko.rs@gmail.com>
> Signed-off-by: Bjørn Mork <bjorn@mork.no>

Applied and queued up for -stable.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Enrico Mioso July 18, 2014, 3:14 p.m. UTC | #2
Seems interesting
That's fine.

Acked-By: Enrico Mioso <mrkiko.rs@gmail.com>

On Thu, 17 Jul 2014, Bjørn Mork wrote:

==Date: Thu, 17 Jul 2014 13:34:09
==From: Bjørn Mork <bjorn@mork.no>
==To: netdev@vger.kernel.org
==Cc: linux-usb@vger.kernel.org, Bjørn Mork <bjorn@mork.no>,
==    Enrico Mioso <mrkiko.rs@gmail.com>
==Subject: [PATCH net,stable] net: huawei_cdc_ncm: add "subclass 3" devices
==
==Huawei's usage of the subclass and protocol fields is not 100%
==clear to us, but there appears to be a very strict system.
==
==A device with the "shared" device ID 12d1:1506 and this NCM
==function was recently reported (showing only default altsetting):
==
==    Interface Descriptor:
==      bLength                 9
==      bDescriptorType         4
==      bInterfaceNumber        1
==      bAlternateSetting       0
==      bNumEndpoints           1
==      bInterfaceClass       255 Vendor Specific Class
==      bInterfaceSubClass      3
==      bInterfaceProtocol     22
==      iInterface              8 CDC Network Control Model (NCM)
==      ** UNRECOGNIZED:  05 24 00 10 01
==      ** UNRECOGNIZED:  06 24 1a 00 01 1f
==      ** UNRECOGNIZED:  0c 24 1b 00 01 00 04 10 14 dc 05 20
==      ** UNRECOGNIZED:  0d 24 0f 0a 0f 00 00 00 ea 05 03 00 01
==      ** UNRECOGNIZED:  05 24 06 01 01
==      Endpoint Descriptor:
==        bLength                 7
==        bDescriptorType         5
==        bEndpointAddress     0x85  EP 5 IN
==        bmAttributes            3
==          Transfer Type            Interrupt
==          Synch Type               None
==          Usage Type               Data
==        wMaxPacketSize     0x0010  1x 16 bytes
==        bInterval               9
==
==Cc: Enrico Mioso <mrkiko.rs@gmail.com>
==Signed-off-by: Bjørn Mork <bjorn@mork.no>
==---
== drivers/net/usb/huawei_cdc_ncm.c | 3 +++
== 1 file changed, 3 insertions(+)
==
==diff --git a/drivers/net/usb/huawei_cdc_ncm.c b/drivers/net/usb/huawei_cdc_ncm.c
==index 5d95a13dbe2a..735f7dadb9a0 100644
==--- a/drivers/net/usb/huawei_cdc_ncm.c
==+++ b/drivers/net/usb/huawei_cdc_ncm.c
==@@ -194,6 +194,9 @@ static const struct usb_device_id huawei_cdc_ncm_devs[] = {
== 	{ USB_VENDOR_AND_INTERFACE_INFO(0x12d1, 0xff, 0x02, 0x76),
== 	  .driver_info = (unsigned long)&huawei_cdc_ncm_info,
== 	},
==+	{ USB_VENDOR_AND_INTERFACE_INFO(0x12d1, 0xff, 0x03, 0x16),
==+	  .driver_info = (unsigned long)&huawei_cdc_ncm_info,
==+	},
== 
== 	/* Terminating entry */
== 	{
==-- 
==2.0.1
==
==
diff mbox

Patch

diff --git a/drivers/net/usb/huawei_cdc_ncm.c b/drivers/net/usb/huawei_cdc_ncm.c
index 5d95a13dbe2a..735f7dadb9a0 100644
--- a/drivers/net/usb/huawei_cdc_ncm.c
+++ b/drivers/net/usb/huawei_cdc_ncm.c
@@ -194,6 +194,9 @@  static const struct usb_device_id huawei_cdc_ncm_devs[] = {
 	{ USB_VENDOR_AND_INTERFACE_INFO(0x12d1, 0xff, 0x02, 0x76),
 	  .driver_info = (unsigned long)&huawei_cdc_ncm_info,
 	},
+	{ USB_VENDOR_AND_INTERFACE_INFO(0x12d1, 0xff, 0x03, 0x16),
+	  .driver_info = (unsigned long)&huawei_cdc_ncm_info,
+	},
 
 	/* Terminating entry */
 	{