Hi Hans, On Fri, Dec 30, 2011 at 07:56:39PM +0100, Hans de Goede wrote: > Hi, > > On 12/30/2011 07:42 PM, 'Sakari Ailus' wrote: > >Hi Hans, > > > >On Fri, Dec 30, 2011 at 09:41:57AM +0100, Hans de Goede wrote: > >... > >>Right, so the above is exactly why I ended up making the pwc whitebalance > >>control the way it is, the user can essentially choice between a number > >>of options: > >>1) auto whitebal > >>2) a number of preset whitebal values (seems your proposal has some more then the pwc > >> driver, which is fine) > >>3) manual whitebal, at which point the user may set whitebal through one of: > >> a) a color temperature control > >> b) red and blue balance controls > >> c) red, green and blue gains > > > >d) red, green, blue and... another green. This is how some raw bayer sensors > >can be controlled. > > > > Yes, but have you ever tried setting the 2 green gains to different values? > (Hint the result is not pretty) I strongly believe the 2 separate green gains are > only there as it is easier to do things that way (it keeps the sensor array symmetric) and > there is no value in controlling them separately. That's debatable. I agree that most of the time they're best kept same, but I don't think that'd still universally true. We could add controls for the five and depending on the capabilities of the hardware the driver would expose either three or four. In any case, I dont't think these controls are really something for the user interface for the regular user, even with three components. > >>Notice that we also need to add some standardized controls for the 3c case, but that > >>is a different discussion. > > > >I was planning to add the gain to low-level controls once the other sensor > >controls have been properly discussed, and hopefully approved, first. > > > >>Seeing how this discussion has evolved I believe that what I did in the pwc driver > >>is actually right from the user pov, the user gets one simple menu control which > >>allows the user to choice between auto / preset 1 - x / manual and since as > >>described above choosing one of the options excludes the other options from being > >>active I believe having this all in one control is the right thing to do. > > > >I still think automatic white balance should be separate from the rest, as > >it currently is (V4L2_CID_AUTO_WHITE_BALANCE). If such presets aren't > >available, the way the user would enable automatic white balance changes, > >which is bad. > > Well we can just put in the spec that the control can be either a boolean or > a menu control depending on if it has presets. I really believe that we should > not make this 2 separate controls, it does not match from a user pov. For a test program like yavta that's fine, but for an application that could be worse. Applications should be able to rely that the type of controls is constant. The control frame work also assumes this. Do you see adverse effects in providing two controls? > There are 2 things which the user wants to control here: > > 1) How the value of the white balance controls gets controlled, which is > one of: a) auto b) select values from preset 1 - # c) manaul > > 2) The white balance controls themselves (temperature, or color balances ...), > but only if 1) is set to manual > > There is no reason to separate 1) in 2 separate controls, that will only serve > to confuse the user. > > As for the theoretical automatic wb which takes some hints like the presets, > AFAIK no such device exists and it is unlikely one will show up in the near > future. IOW it is purely theoretical and thus not interesting. I beg to disagree. Remember that not all cameras that use V4L2 are webcams. The Nokia N9 does use similar configuration items as hints for the white balance algorithm. The algorithm is implemented in Linux user space, and to provide full functionality over V4L2 interface that kind of a control would eventually be required. That said, there are other issues in the way of supporting the N9 camera properly for the V4L2 applications. -- Sakari Ailus e-mail: sakari.ailus@xxxxxx jabber/XMPP/Gmail: sailus@xxxxxxxxxxxxxx -- 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
