On 01/15/2012 08:30 PM, Sakari Ailus wrote: > Hi Sylwester, > > Sylwester Nawrocki wrote: >> On 01/14/2012 09:51 PM, Sakari Ailus wrote: >>>>> diff --git a/drivers/media/video/v4l2-ctrls.c >>>>> b/drivers/media/video/v4l2-ctrls.c index 083bb79..da1ec52 100644 >>>>> --- a/drivers/media/video/v4l2-ctrls.c >>>>> +++ b/drivers/media/video/v4l2-ctrls.c >>>>> @@ -606,6 +606,12 @@ const char *v4l2_ctrl_get_name(u32 id) >>>>> case V4L2_CID_FLASH_CHARGE: return "Charge"; >>>>> case V4L2_CID_FLASH_READY: return "Ready to >>>>> strobe"; >>>>> >>>>> + case V4L2_CID_IMAGE_SOURCE_CLASS: return "Image source controls"; >>>>> + case V4L2_CID_IMAGE_SOURCE_VBLANK: return "Vertical blanking"; >>>>> + case V4L2_CID_IMAGE_SOURCE_HBLANK: return "Horizontal blanking"; >>>>> + case V4L2_CID_IMAGE_SOURCE_LINK_FREQ: return "Link frequency"; >>>>> + case V4L2_CID_IMAGE_SOURCE_ANALOGUE_GAIN: return "Analogue gain"; >>>> >>>> Please capitalize each word, as done for the other controls. >>> >>> This isn't done for the flash controls either, have you noticed that? >>> >>> Well, I guess I have to admit that they were added by myself. ;-) >>> >>> I can fix this for the next patchset. >> >> I don't want to be annoying (too much ;)) but the FLASH controls >> documentation >> is missing some<constant> tags in the text. Other classes use them for >> standard identifiers. > > Thanks for letting me know --- that could be fixed with the flash timing > control API, or unrelated to that. We should btw. continue discussion on > that one. :-)
Sure, I remember having a patch for that style correction somewhere around. Certainly the Flash topic needs continuation, I'll get back to it shortly. I just need to do some more research about it and it is not of really high priority on my side now. Plus I have temporarily been out of order for the last week.. I have just prepared an auto focus controls draft patch. I'll post it hopefully tonight to move things forward. > Speaking of things to do... The colour of the object to point and press > V4L2_CID_DO_WHITE_BALANCE (or what was it called) to fix the white > balance --- is that white, gray, something else or implementation > dependent? I vaguely remember having seen some grayish plates being used > for that but I'm definitely not sure. :-) Any idea? As far as I know it should be always an object which is perceived as white. Something else don't seem to be terribly useful to set up white balance. Unfortunately I don't have much of experience with that yet. -- Regards, Sylwester -- 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
