Re: [PATCH] em28xx: Add Terratec Cinergy HTC Stick

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

 



On 29.11.2011 13:04, Jens Erdmann wrote:
> Hi Andreas,
> 
> On Monday, November 28, 2011 05:57:35 PM you wrote:
>> Hello Jens,
>>
>> On 28.11.2011 16:41, Jens Erdmann wrote:
>>> Hi folks,
>>>
>>> just a few quetstions:
>>> 1. Why is the device named EM2884_BOARD_CINERGY_HTC_STICK and not
>>>
>>>     EM2884_BOARD_TERRATEC_HTC_STICK like all the other devices from that
>>>     vendor? Looks inconsistent to me.
>>
>> that's because it's the product name. Even though TERRATEC is the
>> vendor, the TERRATEC series of devices is different from the Cinergy
>> series (mostly due to software bundles, IMHO). See their homepage for
>> details.
>>
>> So, TERRATEC_HTC_STICK would be wrong. You could change it to
>> TERRATEC_CINERGY_HTC_STICK, if it's important to you. However, following
>> the same logic, the TERRATEC H5 should then be called
>> TERRATEC_TERRATEC_H5, which seems rather odd.
>>
>> Btw.: The em28xx driver wrongly lists the H5 as "Terratec Cinergy H5".
>>
> 
> Don't get me wrong on this. I am just asking. You have made the desicion
> and if you are fine with it its ok.

No problem. I was just explaining the rationale. I don't really care
whether the constant's name gets changed or not.

>>> 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.

Regards,
Andreas
--
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