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. 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?
Thanks
Guennadi
> s_crystal_freq: sets the frequency of the crystal used to generate the
> clocks in Hz. An extra flags field allows device specific configuration
> @@ -308,6 +311,7 @@ struct v4l2_subdev_video_ops {
> int (*s_std_output)(struct v4l2_subdev *sd, v4l2_std_id std);
> int (*g_std_output)(struct v4l2_subdev *sd, v4l2_std_id *std);
> int (*querystd)(struct v4l2_subdev *sd, v4l2_std_id *std);
> + int (*g_tvnorms)(struct v4l2_subdev *sd, v4l2_std_id *std);
> int (*g_tvnorms_output)(struct v4l2_subdev *sd, v4l2_std_id *std);
> int (*g_input_status)(struct v4l2_subdev *sd, u32 *status);
> int (*s_stream)(struct v4l2_subdev *sd, int enable);
> --
> 1.8.5.2
>
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
--
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
