Hi Istanv,

On Thu, Feb 11, 2010 at 2:16 PM, istva...@mailbox.hu
<istva...@mailbox.hu> wrote:
> Update: the following patch, which should be applied after the previous
> ones, makes a few additional changes to the XC4000 driver:
>  - adds support for DTV7
>  - implements power management
>  - adds a mutex and locking for tuner operations
>  - some unused or unneeded code has been removed

Is the DTV7 support actually tested?  Or are you just blindly adding
the code in the hope that it works?  I'm just asking because the last
time I spoke to you, you actually didn't have access to a DVB-T signal
source.

Also, I'm not sure I'm comfortable with the way the mutex is
implemented here.  Is this logic copied from some other driver (and if
so, which one), or did you come up with it yourself?

Devin

-- 
Devin J. Heitmueller - Kernel Labs
http://www.kernellabs.com
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to