|[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Hi, Thanks for the patches. I've added them all to my tree, so they will be included in my next pull-req. In the mean time you can find them (unmodified) here: http://git.linuxtv.org/hgoede/gspca.git/shortlog/refs/heads/media-for_v3.5-wip On 05/16/2012 11:42 PM, Antonio Ospite wrote:
Hi, the first patch just removes traces of the gspca control handling mechanism from the kinect driver; this driver does not have any controls. The change is trivial and can be applied right away, or postponed to when the gspca_main code is removed, you decide. The second patch removes the dependency between auto gain and auto white balance, I'd like to hear Jean-Francois on this, the webcam (the ov772x sensor) is able to set the two parameters independently and the user can see the difference of either, is there a reason why we were preventing the user from doing so before? The third patch is the conversion of the ov534 subdriver to the v4l2 control framework, I tested the code with a PS3 Eye (ov772x sensor) and it works fine (now disabling automatic exposure works too, yay), maybe someone else can give it a run on a webcam with OV767x. NOTE: in patch 3, in sd_init_controls(), I left multiple checks if (sd->sensor == SENSOR_OV772x) just to preserve the order the controls were declared in "struct sd", if you feel the order is not that important I can aggregate the checks, just let me know, it just looked neater to me this way. From a purely aesthetic point of view maybe the gspca mechanism of defining controls was prettier, more declarative, but the control framework really looks "more correct" even from userspace, qv4l2 can now display labels of control classes in tabs automatically while before we had empty labels, disabled controls in clusters work beautifully, and disabled controls with associated automatic settings can show the value calculated by the hardware on every update, very instructive if not super-useful.
I'm glad to hear you like the control framework. Regards, Hans
Thanks, Antonio Antonio Ospite (3): gspca - kinect: remove traces of gspca control handling gspca - ov534: make AGC and AWB controls independent gspca - ov534: convert to v4l2 control framework drivers/media/video/gspca/kinect.c | 9 - drivers/media/video/gspca/ov534.c | 590 ++++++++++++++++-------------------- 2 files changed, 261 insertions(+), 338 deletions(-)
-- 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