* Re: [PATCH] usb id addition for Terratec Cinergy T Stick Dual rev. 2
[not found] <5064A3AD.70009@free.fr>
@ 2012-09-27 19:41 ` Antti Palosaari
2012-09-28 16:34 ` Damien Bally
0 siblings, 1 reply; 13+ messages in thread
From: Antti Palosaari @ 2012-09-27 19:41 UTC (permalink / raw)
To: Damien Bally; +Cc: linux-media, tvboxspy
On 09/27/2012 10:06 PM, Damien Bally wrote:
> This patch adds support for new Terratec Cinergy T Stick Dual rev. 2.
>
> Signed-off-by: Damien Bally <biribi@free.fr>
I will NACK that initially because that USB ID already used by AF9015
driver. You have to explain / study what happens when AF9015 driver
claims that device same time.
Antti
--
http://palosaari.fi/
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] usb id addition for Terratec Cinergy T Stick Dual rev. 2
2012-09-27 19:41 ` [PATCH] usb id addition for Terratec Cinergy T Stick Dual rev. 2 Antti Palosaari
@ 2012-09-28 16:34 ` Damien Bally
2012-09-28 17:55 ` Antti Palosaari
0 siblings, 1 reply; 13+ messages in thread
From: Damien Bally @ 2012-09-28 16:34 UTC (permalink / raw)
To: linux-media; +Cc: tvboxspy, Antti Palosaari
> I will NACK that initially because that USB ID already used by AF9015
> driver. You have to explain / study what happens when AF9015 driver
> claims that device same time.
>
Hi Antti
With the Cinergy stick alone, dvb_usb_af9015 is predictably loaded, but
doesn't prevent dvb_usb_it913x from working nicely.
If an afatech 9015 stick is connected, such as an AverTV Volar Black HD
(A850), it will be recognized and doesn't affect the other device.
*But* it runs into trouble if the two devices were connected at bootup,
or if the Cinergy stick is inserted after the other one :
-----------------------------------------------------------------------
[ 1.264018] usb 2-1: new high speed USB device using ehci_hcd and
address 2
[ 1.382487] usb 2-1: New USB device found, idVendor=0ccd, idProduct=0099
[ 1.382490] usb 2-1: New USB device strings: Mfr=1, Product=2,
SerialNumber=0
[ 1.382492] usb 2-1: Product: DVB-T TV Stick
[ 1.382494] usb 2-1: Manufacturer: ITE Technologies, Inc.
[ 1.385073] input: ITE Technologies, Inc. DVB-T TV Stick as
/devices/pci0000:00/0000:00:1d.7/usb2/2-1/2-1:1.1/input/input1
[ 1.385147] generic-usb 0003:0CCD:0099.0001: input,hidraw0: USB HID
v1.01 Keyboard [ITE Technologies, Inc. DVB-T TV Stick] on
usb-0000:00:1d.7-1 input1
[ 5.045527] usbcore: registered new interface driver dvb_usb_it913x
[ 5.147276] it913x: Chip Version=01 Chip Type=9135
[ 5.147524] it913x: Firmware Version 33684956
[ 5.148649] it913x: Remote HID mode NOT SUPPORTED
[ 5.149024] it913x: Dual mode=3 Tuner Type=0
[ 5.149028] usb 2-1: dvb_usbv2: found a 'ITE 9135(9006) Generic' in
warm state
[ 5.149077] usb 2-1: dvb_usbv2: will pass the complete MPEG2
transport stream to the software demuxer
[ 5.149307] DVB: registering new adapter (ITE 9135(9006) Generic)
[ 5.174907] usb 1-4: dvb_usbv2: downloading firmware from file
'dvb-usb-af9015.fw'
[ 5.241934] usb 1-4: dvb_usbv2: found a 'AverMedia AVerTV Volar Black
HD (A850)' in warm state
[ 5.614827] usb 1-4: dvb_usbv2: will pass the complete MPEG2
transport stream to the software demuxer
[ 5.614866] DVB: registering new adapter (AverMedia AVerTV Volar
Black HD (A850))
[ 5.710026] af9013: firmware version 4.95.0.0
[ 5.712151] DVB: registering adapter 1 frontend 0 (Afatech AF9013)...
[ 5.813139] MXL5005S: Attached at address 0xc6
[ 5.818896] usb 1-4: dvb_usbv2: 'AverMedia AVerTV Volar Black HD
(A850)' successfully initialized and connected
[ 7.266161] usb 2-1: dvb_usbv2: 2nd usb_bulk_msg() failed=-110
[ 7.266247] it913x-fe: ADF table value :00
[ 9.267200] usb 2-1: dvb_usbv2: 2nd usb_bulk_msg() failed=-110
[ 11.267153] usb 2-1: dvb_usbv2: 2nd usb_bulk_msg() failed=-110
[ 13.267250] usb 2-1: dvb_usbv2: 2nd usb_bulk_msg() failed=-110
[ 15.267089] usb 2-1: dvb_usbv2: 2nd usb_bulk_msg() failed=-110
[ 17.267162] usb 2-1: dvb_usbv2: 2nd usb_bulk_msg() failed=-110
[ 19.267139] usb 2-1: dvb_usbv2: 2nd usb_bulk_msg() failed=-110
[ 19.267218] it913x-fe: Crystal Frequency :12000000 Adc Frequency
:20250000 ADC X2: 01
[ 19.267296] usb 2-1: dvb_usbv2: 'ITE 9135(9006) Generic' error while
loading driver (-19)
[ 19.267472] usb 2-1: dvb_usbv2: 'ITE 9135(9006) Generic' successfully
deinitialized and disconnected
-----------------------------------------------------------------------
I'm unfortunately not able to rewrite the driver, but I'm willing to
provide any information about the device to help its correct
identification. Here is what lsusb yields :
-----------------------------------------------------------------------
Bus 002 Device 003: ID 0ccd:0099 TerraTec Electronic GmbH
Device Descriptor:
bLength 18
bDescriptorType 1
bcdUSB 2.00
bDeviceClass 0 (Defined at Interface level)
bDeviceSubClass 0
bDeviceProtocol 0
bMaxPacketSize0 64
idVendor 0x0ccd TerraTec Electronic GmbH
idProduct 0x0099
bcdDevice 2.00
iManufacturer 1 ITE Technologies, Inc.
iProduct 2 DVB-T TV Stick
iSerial 0
bNumConfigurations 1
Configuration Descriptor:
bLength 9
bDescriptorType 2
wTotalLength 71
bNumInterfaces 2
bConfigurationValue 1
iConfiguration 0
bmAttributes 0x80
(Bus Powered)
MaxPower 500mA
Interface Descriptor:
bLength 9
bDescriptorType 4
bInterfaceNumber 0
bAlternateSetting 0
bNumEndpoints 4
bInterfaceClass 255 Vendor Specific Class
bInterfaceSubClass 0
bInterfaceProtocol 0
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 0
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 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 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 0
Interface Descriptor:
bLength 9
bDescriptorType 4
bInterfaceNumber 1
bAlternateSetting 0
bNumEndpoints 1
bInterfaceClass 3 Human Interface Device
bInterfaceSubClass 0 No Subclass
bInterfaceProtocol 1 Keyboard
iInterface 0
HID Device Descriptor:
bLength 9
bDescriptorType 33
bcdHID 1.01
bCountryCode 0 Not supported
bNumDescriptors 1
bDescriptorType 34 Report
wDescriptorLength 65
Report Descriptors:
** UNAVAILABLE **
Endpoint Descriptor:
bLength 7
bDescriptorType 5
bEndpointAddress 0x83 EP 3 IN
bmAttributes 3
Transfer Type Interrupt
Synch Type None
Usage Type Data
wMaxPacketSize 0x0040 1x 64 bytes
bInterval 10
Device Qualifier (for other device speed):
bLength 10
bDescriptorType 6
bcdUSB 2.00
bDeviceClass 0 (Defined at Interface level)
bDeviceSubClass 0
bDeviceProtocol 0
bMaxPacketSize0 64
bNumConfigurations 1
Device Status: 0x0000
(Bus Powered)
Hope that helps...
Damien
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] usb id addition for Terratec Cinergy T Stick Dual rev. 2
2012-09-28 16:34 ` Damien Bally
@ 2012-09-28 17:55 ` Antti Palosaari
2012-09-28 19:00 ` Damien Bally
2012-09-28 19:30 ` Malcolm Priestley
0 siblings, 2 replies; 13+ messages in thread
From: Antti Palosaari @ 2012-09-28 17:55 UTC (permalink / raw)
To: Damien Bally; +Cc: linux-media, tvboxspy
On 09/28/2012 07:34 PM, Damien Bally wrote:
> > I will NACK that initially because that USB ID already used by AF9015
>> driver. You have to explain / study what happens when AF9015 driver
>> claims that device same time.
>>
>
> Hi Antti
>
> With the Cinergy stick alone, dvb_usb_af9015 is predictably loaded, but
> doesn't prevent dvb_usb_it913x from working nicely.
>
> If an afatech 9015 stick is connected, such as an AverTV Volar Black HD
> (A850), it will be recognized and doesn't affect the other device.
>
> *But* it runs into trouble if the two devices were connected at bootup,
> or if the Cinergy stick is inserted after the other one :
I am not sure what you do here but let it be clear.
There is same ID used by af9015 and it913x. Both drivers are loaded when
that ID appears. What I understand *both* drivers, af9015 and it913x
should detect if device is correct or not. If device is af9015 then
it913x should reject it with -ENODEV most likely without a I/O. If
device is it913x then af9015 should reject the device similarly. And you
must find out how to do that. It is not acceptable both drivers starts
doing I/O for same device same time.
regards
Antti
> -----------------------------------------------------------------------
> [ 1.264018] usb 2-1: new high speed USB device using ehci_hcd and
> address 2
> [ 1.382487] usb 2-1: New USB device found, idVendor=0ccd, idProduct=0099
> [ 1.382490] usb 2-1: New USB device strings: Mfr=1, Product=2,
> SerialNumber=0
> [ 1.382492] usb 2-1: Product: DVB-T TV Stick
> [ 1.382494] usb 2-1: Manufacturer: ITE Technologies, Inc.
> [ 1.385073] input: ITE Technologies, Inc. DVB-T TV Stick as
> /devices/pci0000:00/0000:00:1d.7/usb2/2-1/2-1:1.1/input/input1
> [ 1.385147] generic-usb 0003:0CCD:0099.0001: input,hidraw0: USB HID
> v1.01 Keyboard [ITE Technologies, Inc. DVB-T TV Stick] on
> usb-0000:00:1d.7-1 input1
> [ 5.045527] usbcore: registered new interface driver dvb_usb_it913x
> [ 5.147276] it913x: Chip Version=01 Chip Type=9135
> [ 5.147524] it913x: Firmware Version 33684956
> [ 5.148649] it913x: Remote HID mode NOT SUPPORTED
> [ 5.149024] it913x: Dual mode=3 Tuner Type=0
> [ 5.149028] usb 2-1: dvb_usbv2: found a 'ITE 9135(9006) Generic' in
> warm state
> [ 5.149077] usb 2-1: dvb_usbv2: will pass the complete MPEG2
> transport stream to the software demuxer
> [ 5.149307] DVB: registering new adapter (ITE 9135(9006) Generic)
> [ 5.174907] usb 1-4: dvb_usbv2: downloading firmware from file
> 'dvb-usb-af9015.fw'
> [ 5.241934] usb 1-4: dvb_usbv2: found a 'AverMedia AVerTV Volar Black
> HD (A850)' in warm state
> [ 5.614827] usb 1-4: dvb_usbv2: will pass the complete MPEG2
> transport stream to the software demuxer
> [ 5.614866] DVB: registering new adapter (AverMedia AVerTV Volar
> Black HD (A850))
> [ 5.710026] af9013: firmware version 4.95.0.0
> [ 5.712151] DVB: registering adapter 1 frontend 0 (Afatech AF9013)...
> [ 5.813139] MXL5005S: Attached at address 0xc6
> [ 5.818896] usb 1-4: dvb_usbv2: 'AverMedia AVerTV Volar Black HD
> (A850)' successfully initialized and connected
> [ 7.266161] usb 2-1: dvb_usbv2: 2nd usb_bulk_msg() failed=-110
> [ 7.266247] it913x-fe: ADF table value :00
> [ 9.267200] usb 2-1: dvb_usbv2: 2nd usb_bulk_msg() failed=-110
> [ 11.267153] usb 2-1: dvb_usbv2: 2nd usb_bulk_msg() failed=-110
> [ 13.267250] usb 2-1: dvb_usbv2: 2nd usb_bulk_msg() failed=-110
> [ 15.267089] usb 2-1: dvb_usbv2: 2nd usb_bulk_msg() failed=-110
> [ 17.267162] usb 2-1: dvb_usbv2: 2nd usb_bulk_msg() failed=-110
> [ 19.267139] usb 2-1: dvb_usbv2: 2nd usb_bulk_msg() failed=-110
> [ 19.267218] it913x-fe: Crystal Frequency :12000000 Adc Frequency
> :20250000 ADC X2: 01
> [ 19.267296] usb 2-1: dvb_usbv2: 'ITE 9135(9006) Generic' error while
> loading driver (-19)
> [ 19.267472] usb 2-1: dvb_usbv2: 'ITE 9135(9006) Generic' successfully
> deinitialized and disconnected
> -----------------------------------------------------------------------
>
> I'm unfortunately not able to rewrite the driver, but I'm willing to
> provide any information about the device to help its correct
> identification. Here is what lsusb yields :
> -----------------------------------------------------------------------
> Bus 002 Device 003: ID 0ccd:0099 TerraTec Electronic GmbH
> Device Descriptor:
> bLength 18
> bDescriptorType 1
> bcdUSB 2.00
> bDeviceClass 0 (Defined at Interface level)
> bDeviceSubClass 0
> bDeviceProtocol 0
> bMaxPacketSize0 64
> idVendor 0x0ccd TerraTec Electronic GmbH
> idProduct 0x0099
> bcdDevice 2.00
> iManufacturer 1 ITE Technologies, Inc.
> iProduct 2 DVB-T TV Stick
> iSerial 0
> bNumConfigurations 1
> Configuration Descriptor:
> bLength 9
> bDescriptorType 2
> wTotalLength 71
> bNumInterfaces 2
> bConfigurationValue 1
> iConfiguration 0
> bmAttributes 0x80
> (Bus Powered)
> MaxPower 500mA
> Interface Descriptor:
> bLength 9
> bDescriptorType 4
> bInterfaceNumber 0
> bAlternateSetting 0
> bNumEndpoints 4
> bInterfaceClass 255 Vendor Specific Class
> bInterfaceSubClass 0
> bInterfaceProtocol 0
> 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 0
> 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 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 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 0
> Interface Descriptor:
> bLength 9
> bDescriptorType 4
> bInterfaceNumber 1
> bAlternateSetting 0
> bNumEndpoints 1
> bInterfaceClass 3 Human Interface Device
> bInterfaceSubClass 0 No Subclass
> bInterfaceProtocol 1 Keyboard
> iInterface 0
> HID Device Descriptor:
> bLength 9
> bDescriptorType 33
> bcdHID 1.01
> bCountryCode 0 Not supported
> bNumDescriptors 1
> bDescriptorType 34 Report
> wDescriptorLength 65
> Report Descriptors:
> ** UNAVAILABLE **
> Endpoint Descriptor:
> bLength 7
> bDescriptorType 5
> bEndpointAddress 0x83 EP 3 IN
> bmAttributes 3
> Transfer Type Interrupt
> Synch Type None
> Usage Type Data
> wMaxPacketSize 0x0040 1x 64 bytes
> bInterval 10
> Device Qualifier (for other device speed):
> bLength 10
> bDescriptorType 6
> bcdUSB 2.00
> bDeviceClass 0 (Defined at Interface level)
> bDeviceSubClass 0
> bDeviceProtocol 0
> bMaxPacketSize0 64
> bNumConfigurations 1
> Device Status: 0x0000
> (Bus Powered)
>
> Hope that helps...
>
> Damien
--
http://palosaari.fi/
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] usb id addition for Terratec Cinergy T Stick Dual rev. 2
2012-09-28 17:55 ` Antti Palosaari
@ 2012-09-28 19:00 ` Damien Bally
2012-09-28 19:30 ` Malcolm Priestley
1 sibling, 0 replies; 13+ messages in thread
From: Damien Bally @ 2012-09-28 19:00 UTC (permalink / raw)
To: Antti Palosaari; +Cc: linux-media, tvboxspy
Le 28/09/2012 19:55, Antti Palosaari a écrit :
> I am not sure what you do here but let it be clear.
> There is same ID used by af9015 and it913x. Both drivers are loaded when
> that ID appears. What I understand *both* drivers, af9015 and it913x
> should detect if device is correct or not. If device is af9015 then
> it913x should reject it with -ENODEV most likely without a I/O. If
> device is it913x then af9015 should reject the device similarly. And you
> must find out how to do that. It is not acceptable both drivers starts
> doing I/O for same device same time.
>
I'd gladly implement this, but I'm not a developper, for the moment ;-)
>>
>> I'm unfortunately not able to rewrite the driver, but I'm willing to
>> provide any information about the device to help its correct
>> identification.
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] usb id addition for Terratec Cinergy T Stick Dual rev. 2
2012-09-28 17:55 ` Antti Palosaari
2012-09-28 19:00 ` Damien Bally
@ 2012-09-28 19:30 ` Malcolm Priestley
2012-09-29 17:33 ` Mauro Carvalho Chehab
1 sibling, 1 reply; 13+ messages in thread
From: Malcolm Priestley @ 2012-09-28 19:30 UTC (permalink / raw)
To: Antti Palosaari; +Cc: Damien Bally, linux-media, MauroCarvalhoChehab
On Fri, 2012-09-28 at 20:55 +0300, Antti Palosaari wrote:
> On 09/28/2012 07:34 PM, Damien Bally wrote:
> > > I will NACK that initially because that USB ID already used by AF9015
> >> driver. You have to explain / study what happens when AF9015 driver
> >> claims that device same time.
> >>
> >
> > Hi Antti
> >
> > With the Cinergy stick alone, dvb_usb_af9015 is predictably loaded, but
> > doesn't prevent dvb_usb_it913x from working nicely.
> >
> > If an afatech 9015 stick is connected, such as an AverTV Volar Black HD
> > (A850), it will be recognized and doesn't affect the other device.
> >
> > *But* it runs into trouble if the two devices were connected at bootup,
> > or if the Cinergy stick is inserted after the other one :
>
> I am not sure what you do here but let it be clear.
> There is same ID used by af9015 and it913x. Both drivers are loaded when
> that ID appears. What I understand *both* drivers, af9015 and it913x
> should detect if device is correct or not. If device is af9015 then
> it913x should reject it with -ENODEV most likely without a I/O. If
> device is it913x then af9015 should reject the device similarly. And you
> must find out how to do that. It is not acceptable both drivers starts
> doing I/O for same device same time.
>
Hi All
Which module is loaded first depends on the order in
/lib/modules/$(uname -r)/modules.usbmap
Its is likely that af9015 will be first, so the it913x will need to be
loaded first by added it to /etc/modules.
I recall a similar problem exists with the DiB3000M-B driver with its
faulty IDs.
A solution may be for Cinergy to have its own module with extern access
to both dvb_usb_device_properties structures and dvb_usbv2_probe them
twice.
Regards
Malcolm
> regards
> Antti
>
> > -----------------------------------------------------------------------
> > [ 1.264018] usb 2-1: new high speed USB device using ehci_hcd and
> > address 2
> > [ 1.382487] usb 2-1: New USB device found, idVendor=0ccd, idProduct=0099
> > [ 1.382490] usb 2-1: New USB device strings: Mfr=1, Product=2,
> > SerialNumber=0
> > [ 1.382492] usb 2-1: Product: DVB-T TV Stick
> > [ 1.382494] usb 2-1: Manufacturer: ITE Technologies, Inc.
> > [ 1.385073] input: ITE Technologies, Inc. DVB-T TV Stick as
> > /devices/pci0000:00/0000:00:1d.7/usb2/2-1/2-1:1.1/input/input1
> > [ 1.385147] generic-usb 0003:0CCD:0099.0001: input,hidraw0: USB HID
> > v1.01 Keyboard [ITE Technologies, Inc. DVB-T TV Stick] on
> > usb-0000:00:1d.7-1 input1
> > [ 5.045527] usbcore: registered new interface driver dvb_usb_it913x
> > [ 5.147276] it913x: Chip Version=01 Chip Type=9135
> > [ 5.147524] it913x: Firmware Version 33684956
> > [ 5.148649] it913x: Remote HID mode NOT SUPPORTED
> > [ 5.149024] it913x: Dual mode=3 Tuner Type=0
> > [ 5.149028] usb 2-1: dvb_usbv2: found a 'ITE 9135(9006) Generic' in
> > warm state
> > [ 5.149077] usb 2-1: dvb_usbv2: will pass the complete MPEG2
> > transport stream to the software demuxer
> > [ 5.149307] DVB: registering new adapter (ITE 9135(9006) Generic)
> > [ 5.174907] usb 1-4: dvb_usbv2: downloading firmware from file
> > 'dvb-usb-af9015.fw'
> > [ 5.241934] usb 1-4: dvb_usbv2: found a 'AverMedia AVerTV Volar Black
> > HD (A850)' in warm state
> > [ 5.614827] usb 1-4: dvb_usbv2: will pass the complete MPEG2
> > transport stream to the software demuxer
> > [ 5.614866] DVB: registering new adapter (AverMedia AVerTV Volar
> > Black HD (A850))
> > [ 5.710026] af9013: firmware version 4.95.0.0
> > [ 5.712151] DVB: registering adapter 1 frontend 0 (Afatech AF9013)...
> > [ 5.813139] MXL5005S: Attached at address 0xc6
> > [ 5.818896] usb 1-4: dvb_usbv2: 'AverMedia AVerTV Volar Black HD
> > (A850)' successfully initialized and connected
> > [ 7.266161] usb 2-1: dvb_usbv2: 2nd usb_bulk_msg() failed=-110
> > [ 7.266247] it913x-fe: ADF table value :00
> > [ 9.267200] usb 2-1: dvb_usbv2: 2nd usb_bulk_msg() failed=-110
> > [ 11.267153] usb 2-1: dvb_usbv2: 2nd usb_bulk_msg() failed=-110
> > [ 13.267250] usb 2-1: dvb_usbv2: 2nd usb_bulk_msg() failed=-110
> > [ 15.267089] usb 2-1: dvb_usbv2: 2nd usb_bulk_msg() failed=-110
> > [ 17.267162] usb 2-1: dvb_usbv2: 2nd usb_bulk_msg() failed=-110
> > [ 19.267139] usb 2-1: dvb_usbv2: 2nd usb_bulk_msg() failed=-110
> > [ 19.267218] it913x-fe: Crystal Frequency :12000000 Adc Frequency
> > :20250000 ADC X2: 01
> > [ 19.267296] usb 2-1: dvb_usbv2: 'ITE 9135(9006) Generic' error while
> > loading driver (-19)
> > [ 19.267472] usb 2-1: dvb_usbv2: 'ITE 9135(9006) Generic' successfully
> > deinitialized and disconnected
> > -----------------------------------------------------------------------
> >
> > I'm unfortunately not able to rewrite the driver, but I'm willing to
> > provide any information about the device to help its correct
> > identification. Here is what lsusb yields :
> > -----------------------------------------------------------------------
> > Bus 002 Device 003: ID 0ccd:0099 TerraTec Electronic GmbH
> > Device Descriptor:
> > bLength 18
> > bDescriptorType 1
> > bcdUSB 2.00
> > bDeviceClass 0 (Defined at Interface level)
> > bDeviceSubClass 0
> > bDeviceProtocol 0
> > bMaxPacketSize0 64
> > idVendor 0x0ccd TerraTec Electronic GmbH
> > idProduct 0x0099
> > bcdDevice 2.00
> > iManufacturer 1 ITE Technologies, Inc.
> > iProduct 2 DVB-T TV Stick
> > iSerial 0
> > bNumConfigurations 1
> > Configuration Descriptor:
> > bLength 9
> > bDescriptorType 2
> > wTotalLength 71
> > bNumInterfaces 2
> > bConfigurationValue 1
> > iConfiguration 0
> > bmAttributes 0x80
> > (Bus Powered)
> > MaxPower 500mA
> > Interface Descriptor:
> > bLength 9
> > bDescriptorType 4
> > bInterfaceNumber 0
> > bAlternateSetting 0
> > bNumEndpoints 4
> > bInterfaceClass 255 Vendor Specific Class
> > bInterfaceSubClass 0
> > bInterfaceProtocol 0
> > 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 0
> > 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 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 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 0
> > Interface Descriptor:
> > bLength 9
> > bDescriptorType 4
> > bInterfaceNumber 1
> > bAlternateSetting 0
> > bNumEndpoints 1
> > bInterfaceClass 3 Human Interface Device
> > bInterfaceSubClass 0 No Subclass
> > bInterfaceProtocol 1 Keyboard
> > iInterface 0
> > HID Device Descriptor:
> > bLength 9
> > bDescriptorType 33
> > bcdHID 1.01
> > bCountryCode 0 Not supported
> > bNumDescriptors 1
> > bDescriptorType 34 Report
> > wDescriptorLength 65
> > Report Descriptors:
> > ** UNAVAILABLE **
> > Endpoint Descriptor:
> > bLength 7
> > bDescriptorType 5
> > bEndpointAddress 0x83 EP 3 IN
> > bmAttributes 3
> > Transfer Type Interrupt
> > Synch Type None
> > Usage Type Data
> > wMaxPacketSize 0x0040 1x 64 bytes
> > bInterval 10
> > Device Qualifier (for other device speed):
> > bLength 10
> > bDescriptorType 6
> > bcdUSB 2.00
> > bDeviceClass 0 (Defined at Interface level)
> > bDeviceSubClass 0
> > bDeviceProtocol 0
> > bMaxPacketSize0 64
> > bNumConfigurations 1
> > Device Status: 0x0000
> > (Bus Powered)
> >
> > Hope that helps...
> >
> > Damien
>
>
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] usb id addition for Terratec Cinergy T Stick Dual rev. 2
2012-09-28 19:30 ` Malcolm Priestley
@ 2012-09-29 17:33 ` Mauro Carvalho Chehab
2012-09-30 17:36 ` Damien Bally
0 siblings, 1 reply; 13+ messages in thread
From: Mauro Carvalho Chehab @ 2012-09-29 17:33 UTC (permalink / raw)
To: Malcolm Priestley; +Cc: Antti Palosaari, Damien Bally, linux-media
Em Fri, 28 Sep 2012 20:30:17 +0100
Malcolm Priestley <tvboxspy@gmail.com> escreveu:
> On Fri, 2012-09-28 at 20:55 +0300, Antti Palosaari wrote:
> > On 09/28/2012 07:34 PM, Damien Bally wrote:
> > > > I will NACK that initially because that USB ID already used by AF9015
> > >> driver. You have to explain / study what happens when AF9015 driver
> > >> claims that device same time.
> > >>
> > >
> > > Hi Antti
> > >
> > > With the Cinergy stick alone, dvb_usb_af9015 is predictably loaded, but
> > > doesn't prevent dvb_usb_it913x from working nicely.
> > >
> > > If an afatech 9015 stick is connected, such as an AverTV Volar Black HD
> > > (A850), it will be recognized and doesn't affect the other device.
> > >
> > > *But* it runs into trouble if the two devices were connected at bootup,
> > > or if the Cinergy stick is inserted after the other one :
> >
> > I am not sure what you do here but let it be clear.
> > There is same ID used by af9015 and it913x. Both drivers are loaded when
> > that ID appears. What I understand *both* drivers, af9015 and it913x
> > should detect if device is correct or not. If device is af9015 then
> > it913x should reject it with -ENODEV most likely without a I/O. If
> > device is it913x then af9015 should reject the device similarly. And you
> > must find out how to do that. It is not acceptable both drivers starts
> > doing I/O for same device same time.
> >
> Hi All
>
> Which module is loaded first depends on the order in
>
> /lib/modules/$(uname -r)/modules.usbmap
>
> Its is likely that af9015 will be first, so the it913x will need to be
> loaded first by added it to /etc/modules.
Well, builtin modules are loaded before, so if it931x is built with 'Y'
and af9015 with 'M', then it913x will be the first one.
As a general rule, don't trust on one being loaded before the other: the
best thing to do is to detect different USB ID and bcdDevice at the USB
Table. It seems that the it931x variant has bcdDevice equal to 2.00,
from Damien's email:
idVendor 0x0ccd TerraTec Electronic GmbH
idProduct 0x0099
bcdDevice 2.00
iManufacturer 1 ITE Technologies, Inc.
iProduct 2 DVB-T TV Stick
iSerial 0
If the af9015 variant uses another bcdDevice, the fix should be simple.
Btw, cx231xx and dib0700 does that for some Pixelviel devices. This is
how cx231xx declares his device (bcd 40.00 to 40.01):
{USB_DEVICE_VER(USB_VID_PIXELVIEW, USB_PID_PIXELVIEW_SBTVD, 0x4000, 0x4001),
.driver_info = CX231XX_BOARD_PV_PLAYTV_USB_HYBRID},
And dib0700 declares the same USB ID with bcdDevice range from 0.00 to 3f.00:
{ USB_DEVICE_VER(USB_VID_PIXELVIEW, USB_PID_PIXELVIEW_SBTVD, 0x000, 0x3f00) },
If the bcdDevice is identical, then the driver should find a way to check it during
.probe time, and return -ENODEV if the device doesn't match. The other USB driver
will handle it properly.
Regards,
Mauro
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] usb id addition for Terratec Cinergy T Stick Dual rev. 2
2012-09-29 17:33 ` Mauro Carvalho Chehab
@ 2012-09-30 17:36 ` Damien Bally
2012-10-01 11:15 ` Mauro Carvalho Chehab
0 siblings, 1 reply; 13+ messages in thread
From: Damien Bally @ 2012-09-30 17:36 UTC (permalink / raw)
To: Mauro Carvalho Chehab; +Cc: Malcolm Priestley, Antti Palosaari, linux-media
[-- Attachment #1: Type: text/plain, Size: 681 bytes --]
Le 29/09/2012 19:33, Mauro Carvalho Chehab a écrit :
It seems that the it931x variant has bcdDevice equal to 2.00,
> from Damien's email:
>
> idVendor 0x0ccd TerraTec Electronic GmbH
> idProduct 0x0099
> bcdDevice 2.00
> iManufacturer 1 ITE Technologies, Inc.
> iProduct 2 DVB-T TV Stick
> iSerial 0
>
> If the af9015 variant uses another bcdDevice, the fix should be simple.
Alas, according to
http://www.linuxtv.org/wiki/index.php/TerraTec_Cinergy_T_USB_Dual_RC the
af9015 variant appears to have the same bcdDevice. I join both lsusb
outputs for comparison.
Regards
Damien
[-- Attachment #2: af9015 --]
[-- Type: text/plain, Size: 4125 bytes --]
Bus 003 Device 003: ID 0ccd:0099 TerraTec Electronic GmbH
Device Descriptor:
bLength 18
bDescriptorType 1
bcdUSB 2.00
bDeviceClass 0 (Defined at Interface level)
bDeviceSubClass 0
bDeviceProtocol 0
bMaxPacketSize0 64
idVendor 0x0ccd TerraTec Electronic GmbH
idProduct 0x0099
bcdDevice 2.00
iManufacturer 1 Afatech
iProduct 2 DVB-T 2
iSerial 3 xxxxxxxxxxxxxx
bNumConfigurations 1
Configuration Descriptor:
bLength 9
bDescriptorType 2
wTotalLength 71
bNumInterfaces 2
bConfigurationValue 1
iConfiguration 0
bmAttributes 0x80
(Bus Powered)
MaxPower 500mA
Interface Descriptor:
bLength 9
bDescriptorType 4
bInterfaceNumber 0
bAlternateSetting 0
bNumEndpoints 4
bInterfaceClass 255 Vendor Specific Class
bInterfaceSubClass 0
bInterfaceProtocol 0
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 0
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 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 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 0
Interface Descriptor:
bLength 9
bDescriptorType 4
bInterfaceNumber 1
bAlternateSetting 0
bNumEndpoints 1
bInterfaceClass 3 Human Interface Device
bInterfaceSubClass 0 No Subclass
bInterfaceProtocol 1 Keyboard
iInterface 0
HID Device Descriptor:
bLength 9
bDescriptorType 33
bcdHID 1.01
bCountryCode 0 Not supported
bNumDescriptors 1
bDescriptorType 34 Report
wDescriptorLength 65
Report Descriptors:
** UNAVAILABLE **
Endpoint Descriptor:
bLength 7
bDescriptorType 5
bEndpointAddress 0x83 EP 3 IN
bmAttributes 3
Transfer Type Interrupt
Synch Type None
Usage Type Data
wMaxPacketSize 0x0040 1x 64 bytes
bInterval 10
Device Qualifier (for other device speed):
bLength 10
bDescriptorType 6
bcdUSB 2.00
bDeviceClass 0 (Defined at Interface level)
bDeviceSubClass 0
bDeviceProtocol 0
bMaxPacketSize0 64
bNumConfigurations 1
Device Status: 0x0000
(Bus Powered)
[-- Attachment #3: it913x --]
[-- Type: text/plain, Size: 4121 bytes --]
Bus 002 Device 003: ID 0ccd:0099 TerraTec Electronic GmbH
Device Descriptor:
bLength 18
bDescriptorType 1
bcdUSB 2.00
bDeviceClass 0 (Defined at Interface level)
bDeviceSubClass 0
bDeviceProtocol 0
bMaxPacketSize0 64
idVendor 0x0ccd TerraTec Electronic GmbH
idProduct 0x0099
bcdDevice 2.00
iManufacturer 1 ITE Technologies, Inc.
iProduct 2 DVB-T TV Stick
iSerial 0
bNumConfigurations 1
Configuration Descriptor:
bLength 9
bDescriptorType 2
wTotalLength 71
bNumInterfaces 2
bConfigurationValue 1
iConfiguration 0
bmAttributes 0x80
(Bus Powered)
MaxPower 500mA
Interface Descriptor:
bLength 9
bDescriptorType 4
bInterfaceNumber 0
bAlternateSetting 0
bNumEndpoints 4
bInterfaceClass 255 Vendor Specific Class
bInterfaceSubClass 0
bInterfaceProtocol 0
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 0
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 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 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 0
Interface Descriptor:
bLength 9
bDescriptorType 4
bInterfaceNumber 1
bAlternateSetting 0
bNumEndpoints 1
bInterfaceClass 3 Human Interface Device
bInterfaceSubClass 0 No Subclass
bInterfaceProtocol 1 Keyboard
iInterface 0
HID Device Descriptor:
bLength 9
bDescriptorType 33
bcdHID 1.01
bCountryCode 0 Not supported
bNumDescriptors 1
bDescriptorType 34 Report
wDescriptorLength 65
Report Descriptors:
** UNAVAILABLE **
Endpoint Descriptor:
bLength 7
bDescriptorType 5
bEndpointAddress 0x83 EP 3 IN
bmAttributes 3
Transfer Type Interrupt
Synch Type None
Usage Type Data
wMaxPacketSize 0x0040 1x 64 bytes
bInterval 10
Device Qualifier (for other device speed):
bLength 10
bDescriptorType 6
bcdUSB 2.00
bDeviceClass 0 (Defined at Interface level)
bDeviceSubClass 0
bDeviceProtocol 0
bMaxPacketSize0 64
bNumConfigurations 1
Device Status: 0x0000
(Bus Powered)
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] usb id addition for Terratec Cinergy T Stick Dual rev. 2
2012-09-30 17:36 ` Damien Bally
@ 2012-10-01 11:15 ` Mauro Carvalho Chehab
2012-10-01 11:21 ` Antti Palosaari
0 siblings, 1 reply; 13+ messages in thread
From: Mauro Carvalho Chehab @ 2012-10-01 11:15 UTC (permalink / raw)
To: Damien Bally; +Cc: Malcolm Priestley, Antti Palosaari, linux-media
Em Sun, 30 Sep 2012 19:36:50 +0200
Damien Bally <biribi@free.fr> escreveu:
>
>
> Le 29/09/2012 19:33, Mauro Carvalho Chehab a écrit :
> It seems that the it931x variant has bcdDevice equal to 2.00,
> > from Damien's email:
> >
> > idVendor 0x0ccd TerraTec Electronic GmbH
> > idProduct 0x0099
> > bcdDevice 2.00
> > iManufacturer 1 ITE Technologies, Inc.
> > iProduct 2 DVB-T TV Stick
> > iSerial 0
> >
> > If the af9015 variant uses another bcdDevice, the fix should be simple.
>
> Alas, according to
> http://www.linuxtv.org/wiki/index.php/TerraTec_Cinergy_T_USB_Dual_RC the
> af9015 variant appears to have the same bcdDevice. I join both lsusb
> outputs for comparison.
Well, then the alternative is to let both drivers to handle this USB ID,
and add a code there on each of them that will check if the device is the
right one, perhaps by looking at iProduct string. If the driver doesn't
recognize it, it should return -ENODEV at .probe() time. The USB core will
call the second driver.
Regards,
Mauro
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] usb id addition for Terratec Cinergy T Stick Dual rev. 2
2012-10-01 11:15 ` Mauro Carvalho Chehab
@ 2012-10-01 11:21 ` Antti Palosaari
2012-10-06 15:40 ` Mauro Carvalho Chehab
0 siblings, 1 reply; 13+ messages in thread
From: Antti Palosaari @ 2012-10-01 11:21 UTC (permalink / raw)
To: Mauro Carvalho Chehab; +Cc: Damien Bally, Malcolm Priestley, linux-media
On 10/01/2012 02:15 PM, Mauro Carvalho Chehab wrote:
> Em Sun, 30 Sep 2012 19:36:50 +0200
> Damien Bally <biribi@free.fr> escreveu:
>
>>
>>
>> Le 29/09/2012 19:33, Mauro Carvalho Chehab a écrit :
>> It seems that the it931x variant has bcdDevice equal to 2.00,
>>> from Damien's email:
>>>
>>> idVendor 0x0ccd TerraTec Electronic GmbH
>>> idProduct 0x0099
>>> bcdDevice 2.00
>>> iManufacturer 1 ITE Technologies, Inc.
>>> iProduct 2 DVB-T TV Stick
>>> iSerial 0
>>>
>>> If the af9015 variant uses another bcdDevice, the fix should be simple.
>>
>> Alas, according to
>> http://www.linuxtv.org/wiki/index.php/TerraTec_Cinergy_T_USB_Dual_RC the
>> af9015 variant appears to have the same bcdDevice. I join both lsusb
>> outputs for comparison.
>
> Well, then the alternative is to let both drivers to handle this USB ID,
> and add a code there on each of them that will check if the device is the
> right one, perhaps by looking at iProduct string. If the driver doesn't
> recognize it, it should return -ENODEV at .probe() time. The USB core will
> call the second driver.
It is the easiest solution, but there should be very careful. Those
strings could change from device to device. I used earlier af9015 eeprom
hash (those string as coming from the eeprom) to map TerraTec dual
remote controller and git bug report quite soon as it didn't worked.
After I looked the reason I found out they was changed some not
meaningful value.
t. Antti
--
http://palosaari.fi/
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] usb id addition for Terratec Cinergy T Stick Dual rev. 2
2012-10-01 11:21 ` Antti Palosaari
@ 2012-10-06 15:40 ` Mauro Carvalho Chehab
2013-01-07 19:37 ` Antti Palosaari
0 siblings, 1 reply; 13+ messages in thread
From: Mauro Carvalho Chehab @ 2012-10-06 15:40 UTC (permalink / raw)
To: Antti Palosaari; +Cc: Damien Bally, Malcolm Priestley, linux-media
Em Mon, 01 Oct 2012 14:21:34 +0300
Antti Palosaari <crope@iki.fi> escreveu:
> On 10/01/2012 02:15 PM, Mauro Carvalho Chehab wrote:
> > Em Sun, 30 Sep 2012 19:36:50 +0200
> > Damien Bally <biribi@free.fr> escreveu:
> >
> >>
> >>
> >> Le 29/09/2012 19:33, Mauro Carvalho Chehab a écrit :
> >> It seems that the it931x variant has bcdDevice equal to 2.00,
> >>> from Damien's email:
> >>>
> >>> idVendor 0x0ccd TerraTec Electronic GmbH
> >>> idProduct 0x0099
> >>> bcdDevice 2.00
> >>> iManufacturer 1 ITE Technologies, Inc.
> >>> iProduct 2 DVB-T TV Stick
> >>> iSerial 0
> >>>
> >>> If the af9015 variant uses another bcdDevice, the fix should be simple.
> >>
> >> Alas, according to
> >> http://www.linuxtv.org/wiki/index.php/TerraTec_Cinergy_T_USB_Dual_RC the
> >> af9015 variant appears to have the same bcdDevice. I join both lsusb
> >> outputs for comparison.
> >
> > Well, then the alternative is to let both drivers to handle this USB ID,
> > and add a code there on each of them that will check if the device is the
> > right one, perhaps by looking at iProduct string. If the driver doesn't
> > recognize it, it should return -ENODEV at .probe() time. The USB core will
> > call the second driver.
>
> It is the easiest solution, but there should be very careful. Those
> strings could change from device to device. I used earlier af9015 eeprom
> hash (those string as coming from the eeprom) to map TerraTec dual
> remote controller and git bug report quite soon as it didn't worked.
> After I looked the reason I found out they was changed some not
> meaningful value.
Yeah, those strings can change, especially when vendors don't care enough
to use a different USB ID/bcdDevice for different models. Yet, seems to
be the cleaner approach, among the alternatives.
Regards,
Mauro
>
> t. Antti
--
Regards,
Mauro
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] usb id addition for Terratec Cinergy T Stick Dual rev. 2
2012-10-06 15:40 ` Mauro Carvalho Chehab
@ 2013-01-07 19:37 ` Antti Palosaari
2013-01-07 20:13 ` Malcolm Priestley
0 siblings, 1 reply; 13+ messages in thread
From: Antti Palosaari @ 2013-01-07 19:37 UTC (permalink / raw)
To: Damien Bally; +Cc: Mauro Carvalho Chehab, Malcolm Priestley, linux-media
On 10/06/2012 06:40 PM, Mauro Carvalho Chehab wrote:
> Em Mon, 01 Oct 2012 14:21:34 +0300
> Antti Palosaari <crope@iki.fi> escreveu:
>
>> On 10/01/2012 02:15 PM, Mauro Carvalho Chehab wrote:
>>> Em Sun, 30 Sep 2012 19:36:50 +0200
>>> Damien Bally <biribi@free.fr> escreveu:
>>>
>>>>
>>>>
>>>> Le 29/09/2012 19:33, Mauro Carvalho Chehab a écrit :
>>>> It seems that the it931x variant has bcdDevice equal to 2.00,
>>>>> from Damien's email:
>>>>>
>>>>> idVendor 0x0ccd TerraTec Electronic GmbH
>>>>> idProduct 0x0099
>>>>> bcdDevice 2.00
>>>>> iManufacturer 1 ITE Technologies, Inc.
>>>>> iProduct 2 DVB-T TV Stick
>>>>> iSerial 0
>>>>>
>>>>> If the af9015 variant uses another bcdDevice, the fix should be simple.
>>>>
>>>> Alas, according to
>>>> http://www.linuxtv.org/wiki/index.php/TerraTec_Cinergy_T_USB_Dual_RC the
>>>> af9015 variant appears to have the same bcdDevice. I join both lsusb
>>>> outputs for comparison.
>>>
>>> Well, then the alternative is to let both drivers to handle this USB ID,
>>> and add a code there on each of them that will check if the device is the
>>> right one, perhaps by looking at iProduct string. If the driver doesn't
>>> recognize it, it should return -ENODEV at .probe() time. The USB core will
>>> call the second driver.
>>
>> It is the easiest solution, but there should be very careful. Those
>> strings could change from device to device. I used earlier af9015 eeprom
>> hash (those string as coming from the eeprom) to map TerraTec dual
>> remote controller and git bug report quite soon as it didn't worked.
>> After I looked the reason I found out they was changed some not
>> meaningful value.
>
> Yeah, those strings can change, especially when vendors don't care enough
> to use a different USB ID/bcdDevice for different models. Yet, seems to
> be the cleaner approach, among the alternatives.
Damien, care to test?
http://git.linuxtv.org/anttip/media_tree.git/shortlog/refs/heads/it9135_tuner
I split tuner out from IT9135 driver and due to that AF9035 driver
supports IT9135 too (difference between AF9035 and IT9135 is integrated
RF-tuner). I added iManufacturer based checks for both AF9015 and AF9035
drivers
regards
Antti
--
http://palosaari.fi/
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] usb id addition for Terratec Cinergy T Stick Dual rev. 2
2013-01-07 19:37 ` Antti Palosaari
@ 2013-01-07 20:13 ` Malcolm Priestley
2013-01-07 21:06 ` Antti Palosaari
0 siblings, 1 reply; 13+ messages in thread
From: Malcolm Priestley @ 2013-01-07 20:13 UTC (permalink / raw)
To: Antti Palosaari; +Cc: Damien Bally, Mauro Carvalho Chehab, linux-media
On Mon, 2013-01-07 at 21:37 +0200, Antti Palosaari wrote:
> On 10/06/2012 06:40 PM, Mauro Carvalho Chehab wrote:
> > Em Mon, 01 Oct 2012 14:21:34 +0300
> > Antti Palosaari <crope@iki.fi> escreveu:
> >
> >> On 10/01/2012 02:15 PM, Mauro Carvalho Chehab wrote:
> >>> Em Sun, 30 Sep 2012 19:36:50 +0200
> >>> Damien Bally <biribi@free.fr> escreveu:
> >>>
> >>>>
> >>>>
> >>>> Le 29/09/2012 19:33, Mauro Carvalho Chehab a écrit :
> >>>> It seems that the it931x variant has bcdDevice equal to 2.00,
> >>>>> from Damien's email:
> >>>>>
> >>>>> idVendor 0x0ccd TerraTec Electronic GmbH
> >>>>> idProduct 0x0099
> >>>>> bcdDevice 2.00
> >>>>> iManufacturer 1 ITE Technologies, Inc.
> >>>>> iProduct 2 DVB-T TV Stick
> >>>>> iSerial 0
> >>>>>
> >>>>> If the af9015 variant uses another bcdDevice, the fix should be simple.
> >>>>
> >>>> Alas, according to
> >>>> http://www.linuxtv.org/wiki/index.php/TerraTec_Cinergy_T_USB_Dual_RC the
> >>>> af9015 variant appears to have the same bcdDevice. I join both lsusb
> >>>> outputs for comparison.
> >>>
> >>> Well, then the alternative is to let both drivers to handle this USB ID,
> >>> and add a code there on each of them that will check if the device is the
> >>> right one, perhaps by looking at iProduct string. If the driver doesn't
> >>> recognize it, it should return -ENODEV at .probe() time. The USB core will
> >>> call the second driver.
> >>
> >> It is the easiest solution, but there should be very careful. Those
> >> strings could change from device to device. I used earlier af9015 eeprom
> >> hash (those string as coming from the eeprom) to map TerraTec dual
> >> remote controller and git bug report quite soon as it didn't worked.
> >> After I looked the reason I found out they was changed some not
> >> meaningful value.
> >
> > Yeah, those strings can change, especially when vendors don't care enough
> > to use a different USB ID/bcdDevice for different models. Yet, seems to
> > be the cleaner approach, among the alternatives.
>
> Damien, care to test?
> http://git.linuxtv.org/anttip/media_tree.git/shortlog/refs/heads/it9135_tuner
>
> I split tuner out from IT9135 driver and due to that AF9035 driver
> supports IT9135 too (difference between AF9035 and IT9135 is integrated
> RF-tuner). I added iManufacturer based checks for both AF9015 and AF9035
> drivers
I can't see the point of adding this to the af9035/af9033 driver. It is
going to turn into one enormous blob.
The it913x is a stable driver and has it own entity moving forward.
The only thing that needs to happen is the id is added to it913x driver
and if it doesn't apply drop it.
Nack.
Regards
Malcolm
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] usb id addition for Terratec Cinergy T Stick Dual rev. 2
2013-01-07 20:13 ` Malcolm Priestley
@ 2013-01-07 21:06 ` Antti Palosaari
0 siblings, 0 replies; 13+ messages in thread
From: Antti Palosaari @ 2013-01-07 21:06 UTC (permalink / raw)
To: Malcolm Priestley; +Cc: Damien Bally, Mauro Carvalho Chehab, linux-media
On 01/07/2013 10:13 PM, Malcolm Priestley wrote:
> On Mon, 2013-01-07 at 21:37 +0200, Antti Palosaari wrote:
>> On 10/06/2012 06:40 PM, Mauro Carvalho Chehab wrote:
>>> Em Mon, 01 Oct 2012 14:21:34 +0300
>>> Antti Palosaari <crope@iki.fi> escreveu:
>>>
>>>> On 10/01/2012 02:15 PM, Mauro Carvalho Chehab wrote:
>>>>> Em Sun, 30 Sep 2012 19:36:50 +0200
>>>>> Damien Bally <biribi@free.fr> escreveu:
>>>>>
>>>>>>
>>>>>>
>>>>>> Le 29/09/2012 19:33, Mauro Carvalho Chehab a écrit :
>>>>>> It seems that the it931x variant has bcdDevice equal to 2.00,
>>>>>>> from Damien's email:
>>>>>>>
>>>>>>> idVendor 0x0ccd TerraTec Electronic GmbH
>>>>>>> idProduct 0x0099
>>>>>>> bcdDevice 2.00
>>>>>>> iManufacturer 1 ITE Technologies, Inc.
>>>>>>> iProduct 2 DVB-T TV Stick
>>>>>>> iSerial 0
>>>>>>>
>>>>>>> If the af9015 variant uses another bcdDevice, the fix should be simple.
>>>>>>
>>>>>> Alas, according to
>>>>>> http://www.linuxtv.org/wiki/index.php/TerraTec_Cinergy_T_USB_Dual_RC the
>>>>>> af9015 variant appears to have the same bcdDevice. I join both lsusb
>>>>>> outputs for comparison.
>>>>>
>>>>> Well, then the alternative is to let both drivers to handle this USB ID,
>>>>> and add a code there on each of them that will check if the device is the
>>>>> right one, perhaps by looking at iProduct string. If the driver doesn't
>>>>> recognize it, it should return -ENODEV at .probe() time. The USB core will
>>>>> call the second driver.
>>>>
>>>> It is the easiest solution, but there should be very careful. Those
>>>> strings could change from device to device. I used earlier af9015 eeprom
>>>> hash (those string as coming from the eeprom) to map TerraTec dual
>>>> remote controller and git bug report quite soon as it didn't worked.
>>>> After I looked the reason I found out they was changed some not
>>>> meaningful value.
>>>
>>> Yeah, those strings can change, especially when vendors don't care enough
>>> to use a different USB ID/bcdDevice for different models. Yet, seems to
>>> be the cleaner approach, among the alternatives.
>>
>> Damien, care to test?
>> http://git.linuxtv.org/anttip/media_tree.git/shortlog/refs/heads/it9135_tuner
>>
>> I split tuner out from IT9135 driver and due to that AF9035 driver
>> supports IT9135 too (difference between AF9035 and IT9135 is integrated
>> RF-tuner). I added iManufacturer based checks for both AF9015 and AF9035
>> drivers
>
> I can't see the point of adding this to the af9035/af9033 driver. It is
> going to turn into one enormous blob.
Stop speaking bullshit! It increases AF9035/AF9033 driver size
marginally. I could guess total binary size of AF9035+AF9033+IT913X
(IT913X == that new tuner driver) is smaller than size of your IT9135 blob.
> The it913x is a stable driver and has it own entity moving forward.
>
> The only thing that needs to happen is the id is added to it913x driver
> and if it doesn't apply drop it.
>
> Nack.
If you remember about year back when IT9135 was mainlined I reviewed
that stuff and criticized it wasn't split correctly. I also pointed out
all what is needed is new RF-tuner driver as AF9035/AF9033 is just same
silicon, but without a integrated tuner. You said on one mail something
like it is too much different than AF9035 family, I didn't cared to
start stupid yes/no discussion on that time.
You don't have any technical reason to NACK it. Last time I was
"discussing" with DS3000+TS2020 split with you and you were against.
Same happens now. I ask you to look driver guidelines [1] and stop
speaking BS, instead start fixing your own stuff and you will never end
up situation like that.
[1] http://lwn.net/Articles/529490/
Antti
--
http://palosaari.fi/
^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2013-01-07 21:07 UTC | newest]
Thread overview: 13+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <5064A3AD.70009@free.fr>
2012-09-27 19:41 ` [PATCH] usb id addition for Terratec Cinergy T Stick Dual rev. 2 Antti Palosaari
2012-09-28 16:34 ` Damien Bally
2012-09-28 17:55 ` Antti Palosaari
2012-09-28 19:00 ` Damien Bally
2012-09-28 19:30 ` Malcolm Priestley
2012-09-29 17:33 ` Mauro Carvalho Chehab
2012-09-30 17:36 ` Damien Bally
2012-10-01 11:15 ` Mauro Carvalho Chehab
2012-10-01 11:21 ` Antti Palosaari
2012-10-06 15:40 ` Mauro Carvalho Chehab
2013-01-07 19:37 ` Antti Palosaari
2013-01-07 20:13 ` Malcolm Priestley
2013-01-07 21:06 ` Antti Palosaari
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).