Hi Hans,

Thanks for the patch.

On Monday 04 April 2011 13:51:53 Hans Verkuil wrote:
> Signed-off-by: Hans Verkuil <hans.verk...@cisco.com>
> ---
>  drivers/media/video/v4l2-event.c |    6 ++++--
>  drivers/media/video/vivi.c       |    8 ++++++--
>  2 files changed, 10 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/media/video/v4l2-event.c
> b/drivers/media/video/v4l2-event.c index c9251a5..9b503aa 100644
> --- a/drivers/media/video/v4l2-event.c
> +++ b/drivers/media/video/v4l2-event.c
> @@ -258,7 +258,8 @@ int v4l2_event_subscribe(struct v4l2_fh *fh,
>               return -ENOMEM;
>       }
> 
> -     if (sub->type == V4L2_EVENT_CTRL_CH_VALUE) {
> +     if (sub->type == V4L2_EVENT_CTRL_CH_VALUE ||
> +                     sub->type == V4L2_EVENT_CTRL_CH_STATE) {

Indentation looks weird here.

>               ctrl = v4l2_ctrl_find(fh->ctrl_handler, sub->id);
>               if (ctrl == NULL)
>                       return -EINVAL;
> @@ -341,7 +342,8 @@ int v4l2_event_unsubscribe(struct v4l2_fh *fh,
>               list_del(&sev->list);
> 
>       spin_unlock_irqrestore(&fh->vdev->fh_lock, flags);
> -     if (sev->type == V4L2_EVENT_CTRL_CH_VALUE) {
> +     if (sev->type == V4L2_EVENT_CTRL_CH_VALUE ||
> +                     sev->type == V4L2_EVENT_CTRL_CH_STATE) {

And here.

>               struct v4l2_ctrl *ctrl = v4l2_ctrl_find(fh->ctrl_handler, 
> sev->id);
> 
>               if (ctrl)

-- 
Regards,

Laurent Pinchart
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to