Re: [RFC/PATCH 1/3] v4l: Introduce integer menu controls

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Nov 25, 2011 at 11:28:46AM +0100, Laurent Pinchart wrote:
> Hi Sakari,
> 
> Thanks for the patch.

Hi Laurent,

Thanks for the comments!

> On Thursday 24 November 2011 17:12:50 Sakari Ailus wrote:
...
> > @@ -1440,12 +1458,13 @@ struct v4l2_ctrl *v4l2_ctrl_new_std_menu(struct
> > v4l2_ctrl_handler *hdl, u32 flags;
> > 
> >  	v4l2_ctrl_fill(id, &name, &type, &min, &max, &step, &def, &flags);
> > -	if (type != V4L2_CTRL_TYPE_MENU) {
> > +	if (type != V4L2_CTRL_TYPE_MENU
> > +	    && type != V4L2_CTRL_TYPE_INTEGER_MENU) {
> >  		handler_set_err(hdl, -EINVAL);
> >  		return NULL;
> >  	}
> >  	return v4l2_ctrl_new(hdl, ops, id, name, type,
> > -				    0, max, mask, def, flags, qmenu, NULL);
> > +			     0, max, mask, def, flags, qmenu, NULL, NULL);
> 
> You pass NULL to the v4l2_ctrl_new() qmenu_int argument, which will make the 
> function fail for integer menu controls. Do you expect standard integer menu 
> controls to share a list of values ? If not, what about modifying 
> v4l2_ctrl_new_std_menu() to take a list of values (or alternatively forbidding 
> the function from being used for integer menu controls) ?

We currently have no integer menu controls, let alone one which would have a
set of standard values. We need same functionality as in
v4l2_ctrl_get_menu() for integer menus when we add the first standardised
integer menu control. I think it could be added at that time, or I could
implement a v4l2_ctrl_get_integer_menu() which would do nothing.

What do you think?

-- 
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


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux