2011/8/16 Laurent Pinchart <[email protected]>: > Hi Bastian, > > On Tuesday 16 August 2011 14:58:58 Bastian Hecht wrote: >> The driver now supports arbitray resolutions (width up to 2592, height >> up to 720), automatic/manual gain, automatic/manual white balance, >> automatic/manual exposure control, vertical flip, brightness control, >> contrast control and saturation control. Additionally the following >> effects are available now: rotating the hue in the colorspace, gray >> scale image and solarize effect. > > That's a big patch, thus quite hard to review. What about splitting it in one > patch per feature (or group of features, at least separating format > configuration and controls) ? :-)
Hello Laurent, I have reposted the my code split into 2 patches. The first with changes related to the image sizes and the other with all the controls. I hope that this separation makes it easy enough to review it. If not, tell me and I can split it up further. Thanks for reviewing, Bastian Hecht > -- > Regards, > > Laurent Pinchart > -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
