On Friday 25 November 2011 13:58:27 Laurent Pinchart wrote:
> Hi Sakari,
>
> On Friday 25 November 2011 13:56:50 Sakari Ailus wrote:
> > On Fri, Nov 25, 2011 at 01:43:12PM +0100, Laurent Pinchart wrote:
> > > On Friday 25 November 2011 13:02:02 Sakari Ailus wrote:
> > > > On Fri, Nov 25, 2011 at 11:28:46AM +0100, Laurent Pinchart wrote:
> > > > > 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?
> > >
> > > I was just wondering if we will ever have a standard menu control with
> > > standard integer items. If that never happens, v4l2_ctrl_new_std_menu()
> > > needs to either take a qmenu_int array, or reject integer menu controls
> > > completely. I would thus delay adding the V4L2_CTRL_TYPE_INTEGER_MENU
> > > check to the function as it wouldn't work anyway (or, alternatively, we
> > > would add the qmenu_int argument now).
> >
> > Either one, yes. I think I'll add a separate patch adding standard
> > integer menus and remove the check from this one.
> >
> > There'll definitely be a need for them. For example, there are bit rate
> > menus in the standard menu type controls that ideally should be integers.
>
> Sure, but I doubt that the bit rates themselves will be standard.
Actually, they are. MPEG audio level 1, 2, 3 and AC3 audio all have their own
standardized set of possible bitrates. If I had an integer menu at the time
I'm sure I would have used it.
Regards,
Hans
>
> > We won't change them but there will be others. Or I'd be very surprised
> > if there were not!
--
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
