Hi all, We had an IRC discussion (Sakari, Laurent and me) regarding the control classes in general and new controls that are required by more advanced camera sensors [3] (usually running a customizable firmware). More details on the agenda can be found in the invitation message [1]. There was an agreement on selected topics however some issues remain more or less open. The full meeting log can be found at [2]. The conclusions from the meeting could be summarized as follows: - We are having multiple levels of controls that interact with each other, on the lowest level are controls that are directly mapped to hardware blocks - analogue gain for example; on the highest level are controls that influence complex software algorithms implemented in a firmware; The control classes are not much helpful in logical classification of the various level controls, it is often difficult to clearly associate particular control with specific class. Therefore the controls [3] documentation should talk about of some sort of profile indicating which controls to expose on which kind of device. - The selection API should be used for specifying window-of-interest for auto focus, exposure and white balance algorithms; - We're going to use separate selection target base for AF, AE, AWB and reserve a pull of targets for each of them, e.g. 256; - A bitmask control can be used to determine which of multiple rectangles are currently in use; - In order to associate the selection targets with specific controls (AF, AE, AWB) the queryctrl ioctl may be extended to return the selection target base with the above mentioned bitmask control. The number of bits in the control will indicate maximum number of selection targets for each algorithm; - Single point coordinates, where needed, should be specified by rectangle width and height set to 0. - Control names should not generally be prefixed with a control class name, as it is not always clear which class a control should belong to; - As there is no better alternative solution it has been initially agreed to keep two new control classes: the image source and image processing class [4]; [1] http://www.mail-archive.com/linux-media@xxxxxxxxxxxxxxx/msg42735.html [2] http://www.retiisi.org.uk/v4l2/notes/v4l2-control-classes-camera-2012-02-03.txt [3] http://git.infradead.org/users/kmpark/linux-samsung/shortlog/refs/heads/camera-controls [4] http://www.mail-archive.com/linux-media@xxxxxxxxxxxxxxx/msg42823.html http://www.mail-archive.com/linux-media@xxxxxxxxxxxxxxx/msg42802.html -- Regards, Sylwester -- 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