Hi Hans,
On Tuesday 11 March 2014 00:32:48 Hans Verkuil wrote:
> On 03/11/2014 12:23 AM, Guennadi Liakhovetski wrote:
> > Hi Hans,
> >
> > Thanks for taking care about this problem. I'm not sure it would be ok for
> > me to pull this specific patch via my tree, because it's for the V4L2
> > core, and the other 2 patches in this series depend on this one.
>
> It's OK by me to pull this through your tree.
>
> > But anyway I've got a question to this patch:
> >
> > On Mon, 17 Feb 2014, Hans Verkuil wrote:
> >> From: Hans Verkuil <hans.verkuil@xxxxxxxxx>
> >>
> >> While there was already a g_tvnorms_output video op, it's counterpart for
> >> video capture was missing. Add it.
> >>
> >> This is necessary for generic bridge drivers like soc-camera to set the
> >> video_device tvnorms field correctly. Otherwise ENUMSTD cannot work.
> >>
> >> Signed-off-by: Hans Verkuil <hans.verkuil@xxxxxxxxx>
> >> ---
> >>
> >> include/media/v4l2-subdev.h | 8 ++++++--
> >> 1 file changed, 6 insertions(+), 2 deletions(-)
> >>
> >> diff --git a/include/media/v4l2-subdev.h b/include/media/v4l2-subdev.h
> >> index d67210a..787d078 100644
> >> --- a/include/media/v4l2-subdev.h
> >> +++ b/include/media/v4l2-subdev.h
> >> @@ -264,8 +264,11 @@ struct v4l2_mbus_frame_desc {
> >>
> >> g_std_output: get current standard for video OUTPUT devices. This is
> >> ignored by video input devices.
> >>
> >> - g_tvnorms_output: get v4l2_std_id with all standards supported by
> >> video
> >> - OUTPUT device. This is ignored by video input devices.
> >> + g_tvnorms: get v4l2_std_id with all standards supported by the video
> >> + CAPTURE device. This is ignored by video output devices.
> >> +
> >> + g_tvnorms_output: get v4l2_std_id with all standards supported by the
> >> video + OUTPUT device. This is ignored by video capture devices.
> >
> > Why do we need two separate operations with the same functionality - one
> > for capture and one for output? Can we have subdevices, that need to
> >
> > implement both? Besides, what about these two core ops:
> > int (*g_std)(struct v4l2_subdev *sd, v4l2_std_id *norm);
> > int (*s_std)(struct v4l2_subdev *sd, v4l2_std_id norm);
> >
> > ? Seems like a slightly different approach is needed? Or am I missing
> > anything?
>
> There are drivers (ivtv) that have both capture and output subdevices. Each
> can have its own standard. Such drivers use v4l2_device_call_all() to call
> the same op for all subdevs so any subdev that can handle that op gets
> called. So they use it to call the s_std op to change the capture standard
> and they call s_std_output to change the output standard.
>
> If you can't tell the difference between capture tvnorms and output tvnorms
> this becomes tricky to handle. Just keep the two separate and there is no
> confusion.
>
> Note that the video ops have the g/s_std_output ops. It's due to historical
> reasons that g/s_std ended up in the core ops. They probably should be moved
> to the video ops, but it's just not worth the effort.
It's not such a big effort, I can easily cook up a patch. However, it looks
like the s_std operation is implemented by the following subdev drivers that
don't implement video ops:
tuner-core
tvaudio
sony-btf-mpx
vp27smpx
cx18-gpio
It probably doesn't make much sense to add video ops to all of those.
--
Regards,
Laurent Pinchart
--
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
