On 07/17/2015 05:06 PM, Philipp Zabel wrote: > This is useful to subscribe to HDMI hotplug events via the > V4L2_CID_DV_RX_POWER_PRESENT control. Very quick, but it doesn't apply. You need to combine the original "[PATCH 5/5] [media] tc358743: allow event subscription" together with this patch. Regards, Hans > > Signed-off-by: Philipp Zabel <p.zabel@xxxxxxxxxxxxxx> > --- > Changes since v1: > - Make use of v4l2_subdev_notify_event, v4l2_src_change_event_subdev_subscribe, > and v4l2_ctrl_subdev_subscribe_event. > --- > drivers/media/i2c/tc358743.c | 18 +++++++++++++++--- > 1 file changed, 15 insertions(+), 3 deletions(-) > > diff --git a/drivers/media/i2c/tc358743.c b/drivers/media/i2c/tc358743.c > index 0c3c8aa..0d31a4f 100644 > --- a/drivers/media/i2c/tc358743.c > +++ b/drivers/media/i2c/tc358743.c > @@ -860,8 +860,7 @@ static void tc358743_format_change(struct v4l2_subdev *sd) > &timings, false); > } > > - v4l2_subdev_notify(sd, V4L2_DEVICE_NOTIFY_EVENT, > - (void *)&tc358743_ev_fmt); > + v4l2_subdev_notify_event(sd, &tc358743_ev_fmt); > } > > static void tc358743_init_interrupts(struct v4l2_subdev *sd) > @@ -1318,6 +1317,19 @@ static irqreturn_t tc358743_irq_handler(int irq, void *dev_id) > return handled ? IRQ_HANDLED : IRQ_NONE; > } > > +static int tc358743_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; > + } > +} > + > /* --------------- VIDEO OPS --------------- */ > > static int tc358743_g_input_status(struct v4l2_subdev *sd, u32 *status) > @@ -1605,7 +1617,7 @@ static const struct v4l2_subdev_core_ops tc358743_core_ops = { > .s_register = tc358743_s_register, > #endif > .interrupt_service_routine = tc358743_isr, > - .subscribe_event = v4l2_ctrl_subdev_subscribe_event, > + .subscribe_event = tc358743_subscribe_event, > .unsubscribe_event = v4l2_event_subdev_unsubscribe, > }; > > -- 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