On Thu, 13 Jan 2011 12:38:04 +0100 Antonio Ospite <ospite@xxxxxxxxxxxxxxxxx> wrote: > > Jean-FranÃois Moine (9): > [...] > > gspca - ov534: Use the new video control mechanism > > In this commit, is there a reason why you didn't rename also > sd_setagc() into setagc() like for the other functions? > > I am going to test the changes and report back if there's anything > more, I like the cleanup tho. Hi Antonio, With the new video control mechanism, the '.set_control' function is called only when capture is active. Otherwise, the '.set' function is called in any case, and here, it activates/inactivates the auto white balance control... Oh, I forgot to disable the awb when the agc is disabled! Thank you for reporting any problem. BTW, the webcam 06f8:3002 which had been removed some time ago is being tested. I will add it to this subdriver as soon as it works correctly. Cheers. -- Ken ar c'hentaà | ** Breizh ha Linux atav! ** Jef | http://moinejf.free.fr/ -- 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
