2017-07-07 Shuah Khan <[email protected]>:
> On 06/16/2017 01:39 AM, Gustavo Padovan wrote:
> > From: Gustavo Padovan <[email protected]>
> >
> > Call v4l2_ctrl_subscribe_event to subscribe to more events supported by
> > v4l.
> >
> > Signed-off-by: Gustavo Padovan <[email protected]>
> > ---
> > drivers/media/usb/uvc/uvc_v4l2.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/media/usb/uvc/uvc_v4l2.c
> > b/drivers/media/usb/uvc/uvc_v4l2.c
> > index 3e7e283..dfa0ccd 100644
> > --- a/drivers/media/usb/uvc/uvc_v4l2.c
> > +++ b/drivers/media/usb/uvc/uvc_v4l2.c
> > @@ -1240,7 +1240,7 @@ static int uvc_ioctl_subscribe_event(struct v4l2_fh
> > *fh,
> > case V4L2_EVENT_CTRL:
> > return v4l2_event_subscribe(fh, sub, 0, &uvc_ctrl_sub_ev_ops);
> > default:
> > - return -EINVAL;
> > + return v4l2_ctrl_subscribe_event(fh, sub);
>
> This looks incorrect. With this change driver will be subscribing to all
> v4l2 events? Is this the intent?
The intent was to enable this driver to subscribe to BUF_QUEUED events.
It is the only one who can't at the moment. I'll review this.
Gustavo
