[media] adv7604: Deliver resolution change events to userspace
authorLars-Peter Clausen <lars@metafoo.de>
Wed, 24 Jun 2015 16:50:30 +0000 (13:50 -0300)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Fri, 17 Jul 2015 12:48:07 +0000 (09:48 -0300)
Use the new v4l2_subdev_notify_event() helper function to deliver the
resolution change event to userspace via the v4l2 subdev event queue as
well as to the bridge driver using the callback notify mechanism.

This allows userspace applications to react to changes in resolution. This
is useful and often necessary for video pipelines where there is no direct
1-to-1 relationship between the subdevice converter and the video capture
device and hence it does not make sense to directly forward the event to
the video capture device node.

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
[hans.verkuil@cisco.com: fix obvious mistake: v4l2_event_subdev_unsubscribe -> v4l2_ctrl_subdev_subscribe_event]
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
drivers/media/i2c/adv7604.c

index a7d47e07934393a77bca85bfc0ab45c592265d67..bfb0b6a72bbc475680dc5479fe4274ab5b8b4fba 100644 (file)
@@ -1767,8 +1767,8 @@ static int adv76xx_s_routing(struct v4l2_subdev *sd,
        select_input(sd);
        enable_input(sd);
 
-       v4l2_subdev_notify(sd, V4L2_DEVICE_NOTIFY_EVENT,
-                          (void *)&adv76xx_ev_fmt);
+       v4l2_subdev_notify_event(sd, &adv76xx_ev_fmt);
+
        return 0;
 }
 
@@ -1935,8 +1935,7 @@ static int adv76xx_isr(struct v4l2_subdev *sd, u32 status, bool *handled)
                        "%s: fmt_change = 0x%x, fmt_change_digital = 0x%x\n",
                        __func__, fmt_change, fmt_change_digital);
 
-               v4l2_subdev_notify(sd, V4L2_DEVICE_NOTIFY_EVENT,
-                                  (void *)&adv76xx_ev_fmt);
+               v4l2_subdev_notify_event(sd, &adv76xx_ev_fmt);
 
                if (handled)
                        *handled = true;
@@ -2354,6 +2353,20 @@ static int adv76xx_log_status(struct v4l2_subdev *sd)
        return 0;
 }
 
+static int adv76xx_subscribe_event(struct v4l2_subdev *sd,
+                                  struct v4l2_fh *fh,
+                                  struct v4l2_event_subscription *sub)
+{
+       switch (sub->type) {
+       case V4L2_EVENT_SOURCE_CHANGE:
+               return v4l2_src_change_event_subdev_subscribe(sd, fh, sub);
+       case V4L2_EVENT_CTRL:
+               return v4l2_ctrl_subdev_subscribe_event(sd, fh, sub);
+       default:
+               return -EINVAL;
+       }
+}
+
 /* ----------------------------------------------------------------------- */
 
 static const struct v4l2_ctrl_ops adv76xx_ctrl_ops = {
@@ -2363,7 +2376,7 @@ static const struct v4l2_ctrl_ops adv76xx_ctrl_ops = {
 static const struct v4l2_subdev_core_ops adv76xx_core_ops = {
        .log_status = adv76xx_log_status,
        .interrupt_service_routine = adv76xx_isr,
-       .subscribe_event = v4l2_ctrl_subdev_subscribe_event,
+       .subscribe_event = adv76xx_subscribe_event,
        .unsubscribe_event = v4l2_event_subdev_unsubscribe,
 #ifdef CONFIG_VIDEO_ADV_DEBUG
        .g_register = adv76xx_g_register,