Re: [PATCH] usb id addition for Terratec Cinergy T Stick Dual rev. 2

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 10/01/2012 02:15 PM, Mauro Carvalho Chehab wrote:
Em Sun, 30 Sep 2012 19:36:50 +0200
Damien Bally <biribi@xxxxxxx> 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/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux