From: Antti Palosaari <crope@iki.fi>
To: Mauro Carvalho Chehab <mchehab@redhat.com>
Cc: Damien Bally <biribi@free.fr>,
Malcolm Priestley <tvboxspy@gmail.com>,
linux-media@vger.kernel.org
Subject: Re: [PATCH] usb id addition for Terratec Cinergy T Stick Dual rev. 2
Date: Mon, 01 Oct 2012 14:21:34 +0300 [thread overview]
Message-ID: <50697CBE.8060001@iki.fi> (raw)
In-Reply-To: <20121001081540.69bdae23@redhat.com>
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/
next prev parent reply other threads:[~2012-10-01 11:21 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
[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 [this message]
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
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=50697CBE.8060001@iki.fi \
--to=crope@iki.fi \
--cc=biribi@free.fr \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@redhat.com \
--cc=tvboxspy@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.