Hi Istanv, On Thu, Feb 11, 2010 at 2:16 PM, istvan_v@xxxxxxxxxx <istvan_v@xxxxxxxxxx> 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html