Bug 17211

Summary: Support for Pirelli DP-L10 mobile: additional USB_DEVICE
Product: Drivers Reporter: A Lawrence (law_ence.dev)
Component: USBAssignee: Greg Kroah-Hartman (greg)
Status: RESOLVED INVALID    
Severity: normal CC: law_ence.dev
Priority: P1    
Hardware: All   
OS: Linux   
Kernel Version: 2.6.36-rc2 Subsystem:
Regression: No Bisected commit-id:
Attachments: patch for cp210x.c - additional USB_DEVICE
cp210x.c patch (with -p) for extra USB_DEVICE

Description A Lawrence 2010-08-28 19:55:20 UTC
The Pirelli DP-L10 mobile is sold under various brand names. One, already supported by cp210x, is the T-COM TC300. Here is the lsusb for that version:
-------------------------------------------------------------------
Bus 001 Device 002: ID 0489:e000 Foxconn / Hon Hai T-Com TC 300
Device Descriptor:
  bLength                18
  bDescriptorType         1
  bcdUSB               1.10
  bDeviceClass            0 (Defined at Interface level)
  bDeviceSubClass         0 
  bDeviceProtocol         0 
  bMaxPacketSize0        64
  idVendor           0x0489 Foxconn / Hon Hai
  idProduct          0xe000 T-Com TC 300
  bcdDevice            1.00
  iManufacturer           1 Silicon Labs
  iProduct                2 TC 300
  iSerial                 3 0001
    [snip]
---------------------------------------------------------------------------

However the native Pirelli DP-L10 is not supported:
------------------------------------------------------------------
Bus 001 Device 003: ID 0489:e003 Foxconn / Hon Hai Pirelli DP-L10
Device Descriptor:
  bLength                18
  bDescriptorType         1
  bcdUSB               1.10
  bDeviceClass            0 (Defined at Interface level)
  bDeviceSubClass         0 
  bDeviceProtocol         0 
  bMaxPacketSize0        64
  idVendor           0x0489 Foxconn / Hon Hai
  idProduct          0xe003 Pirelli DP-L10
  bcdDevice            1.00
  iManufacturer           1 Silicon Labs
  iProduct                2 DP-L10
  iSerial                 3 0001
    [snip]
-------------------------------------------------------------------------

All that is required is an extra USB_DEVICE entry:

{ USB_DEVICE(0x0489, 0xE003) }, /* Pirelli Broadband S.p.A, DP-L10 SIP/GSM
+Mobile */

The attached patch adds that entry. Tested under 2.6.36-rc2 from git.
Comment 1 A Lawrence 2010-08-28 19:59:10 UTC
Created attachment 28221 [details]
patch for cp210x.c - additional USB_DEVICE
Comment 2 Greg Kroah-Hartman 2010-08-29 04:21:16 UTC
> Created an attachment (id=28221)
>  --> (https://bugzilla.kernel.org/attachment.cgi?id=28221)
> patch for cp210x.c - additional USB_DEVICE

Please send this patch through email, to me and the
linux-usb@vger.kernel.org mailing list, in the format described in the
Documentation/SubmittingPatches file.
Comment 3 A Lawrence 2010-08-29 14:31:57 UTC
Created attachment 28311 [details]
cp210x.c patch (with -p) for extra USB_DEVICE

Previous patch did not include C-function context (arg -p to diff)
Comment 4 A Lawrence 2010-08-29 14:35:19 UTC
Sorry: forgot -p. Revised patch emailed as requested. Added here as well for the 
record.
Comment 5 Greg Kroah-Hartman 2012-02-22 21:52:03 UTC
All USB bugs should be sent to the linux-usb@vger.kernel.org mailing 
list, and not entered into bugzilla.  Please bring this issue up there,
if it is still a problem in the latest kernel release.
Comment 6 A Lawrence 2012-02-23 18:17:10 UTC
The patch was accepted (by you IIRC) so all subsequent kernels have
supported the Pirelli phones. Perhaps I should have closed this bug long
ago, but I overlooked that. Sorry.