Hi Laurent, Thanks for the patch. On Tue, Mar 11, 2014 at 4:45 AM, Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> wrote: > The video-level enum_dv_timings and dv_timings_cap operations are > deprecated in favor of the pad-level versions. All subdev drivers > implement the pad-level versions, switch to them. > > Cc: Prabhakar Lad <prabhakar.csengg@xxxxxxxxx> > Signed-off-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > Reviewed-by: Hans Verkuil <hans.verkuil@xxxxxxxxx> Acked-by: Lad, Prabhakar <prabhakar.csengg@xxxxxxxxx> Regards, --Prabhakar lad > --- > drivers/media/platform/davinci/vpif_capture.c | 4 +++- > drivers/media/platform/davinci/vpif_display.c | 4 +++- > 2 files changed, 6 insertions(+), 2 deletions(-) > > diff --git a/drivers/media/platform/davinci/vpif_capture.c b/drivers/media/platform/davinci/vpif_capture.c > index cd6da8b..16a1958 100644 > --- a/drivers/media/platform/davinci/vpif_capture.c > +++ b/drivers/media/platform/davinci/vpif_capture.c > @@ -1723,7 +1723,9 @@ vpif_enum_dv_timings(struct file *file, void *priv, > struct channel_obj *ch = fh->channel; > int ret; > > - ret = v4l2_subdev_call(ch->sd, video, enum_dv_timings, timings); > + timings->pad = 0; > + > + ret = v4l2_subdev_call(ch->sd, pad, enum_dv_timings, timings); > if (ret == -ENOIOCTLCMD || ret == -ENODEV) > return -EINVAL; > return ret; > diff --git a/drivers/media/platform/davinci/vpif_display.c b/drivers/media/platform/davinci/vpif_display.c > index fd68236..e1edefe 100644 > --- a/drivers/media/platform/davinci/vpif_display.c > +++ b/drivers/media/platform/davinci/vpif_display.c > @@ -1380,7 +1380,9 @@ vpif_enum_dv_timings(struct file *file, void *priv, > struct channel_obj *ch = fh->channel; > int ret; > > - ret = v4l2_subdev_call(ch->sd, video, enum_dv_timings, timings); > + timings->pad = 0; > + > + ret = v4l2_subdev_call(ch->sd, pad, enum_dv_timings, timings); > if (ret == -ENOIOCTLCMD || ret == -ENODEV) > return -EINVAL; > return ret; > -- > 1.8.3.2 > -- 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