From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: Andreas Oberritter <obi@linuxtv.org>
Cc: linux-media@vger.kernel.org, Jens.Erdmann@web.de
Subject: Re: [PATCH] em28xx: Add Terratec Cinergy HTC Stick
Date: Wed, 30 Nov 2011 19:22:16 -0200 [thread overview]
Message-ID: <4ED69E88.6020302@redhat.com> (raw)
In-Reply-To: <4ED694E3.6080500@linuxtv.org>
On 30-11-2011 18:41, Andreas Oberritter wrote:
> On 30.11.2011 20:39, Jens Erdmann wrote:
>> Dear Andreas,
>>
>> On Tuesday, November 29, 2011 01:56:35 PM you wrote:
>> <snip>
>>>
>>>>>> 2. I stumbled over http://linux.terratec.de/tv_en.html where they list
>>>>>> a NXP TDA18271
>>>>>>
>>>>>> as used tuner for H5 and HTC Stick devices. I dont have any
>>>>>> experience in this kind of stuff but i am just asking.
>>>>>
>>>>> That's right.
>>>>
>>>> So this should be made like the other devices which are using the
>>>> TDA18271? Or is there no driver for this tuner yet?
>>>
>>> I don't understand your question. Both TERRATEC H5 and Cinergy HTC Stick
>>> are already supported by Linux (at least for digital signals, the latter
>>> since the patch you're referring to), so a driver for every relevant
>>> chip, including TDA18271, is already involved.
>>>
>>
>> If i remember correctly there was used another tuner driver in the out
>> commended code. Is this just a coyp paste leftover?
>
> Hm. When Mauro committed the patch, the description got lost:
>
>> - Can receive DVB-C and DVB-T. No analogue television or radio yet.
>> - For now it's a copy of the Terratec H5 code with a different name.
Gah! sorry for that. Not sure what happened there. I'll try to remember
about it when sending upstream, in order to recover the original comment.
Feel free to ping me closer to the next merge window, in order to remind
me.
>
> So, to answer your question: The disabled code was copied from the H5
> right above the new code, in the hope that if someone is going to fix
> analogue TV for the H5, he or she might just fix it for the HTC Stick as
> well, assuming that there's only little or no difference.
Support for analog is not trivial, as it requires writing a driver for
tvf4910b. Not sure if is there any publicly released driver with some
code for it. Maybe this could be done via sniffing the USB traffic, but, if
this device is as complicated as DRX, then I doubt people would try to
do it.
> HTH,
> Andreas
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2011-11-30 21:22 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-28 15:41 [PATCH] em28xx: Add Terratec Cinergy HTC Stick Jens Erdmann
2011-11-28 16:02 ` Antti Palosaari
2011-11-28 16:57 ` Andreas Oberritter
2011-11-29 12:04 ` Jens Erdmann
2011-11-29 12:56 ` Andreas Oberritter
2011-11-30 19:39 ` Jens Erdmann
2011-11-30 20:41 ` Andreas Oberritter
2011-11-30 21:22 ` Mauro Carvalho Chehab [this message]
-- strict thread matches above, loose matches on Subject: below --
2011-11-16 15:06 Andreas Oberritter
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=4ED69E88.6020302@redhat.com \
--to=mchehab@redhat.com \
--cc=Jens.Erdmann@web.de \
--cc=linux-media@vger.kernel.org \
--cc=obi@linuxtv.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox