On 15-12-2011 07:39, Javier Martin wrote:
> These callbacks allow a host video driver
> to poll video supported video formats of tvp5150.
>
> Signed-off-by: Javier Martin <javier.mar...@vista-silicon.com>
> ---
> drivers/media/video/tvp5150.c | 72
> +++++++++++++++++++++++++++++++++++++++++
> 1 files changed, 72 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/media/video/tvp5150.c b/drivers/media/video/tvp5150.c
> index 26cc75b..8f01f08 100644
> --- a/drivers/media/video/tvp5150.c
> +++ b/drivers/media/video/tvp5150.c
> @@ -778,6 +778,75 @@ static int tvp5150_s_ctrl(struct v4l2_ctrl *ctrl)
> return -EINVAL;
> }
>
> +static v4l2_std_id tvp5150_read_std(struct v4l2_subdev *sd)
> +{
> + int val = tvp5150_read(sd, TVP5150_STATUS_REG_5);
> +
> + switch (val & 0x0F) {
> + case 0x01:
> + return V4L2_STD_NTSC;
> + case 0x03:
> + return V4L2_STD_PAL;
> + case 0x05:
> + return V4L2_STD_PAL_M;
> + case 0x07:
> + return V4L2_STD_PAL_N | V4L2_STD_PAL_Nc;
> + case 0x09:
> + return V4L2_STD_NTSC_443;
> + case 0xb:
> + return V4L2_STD_SECAM;
> + default:
> + return V4L2_STD_UNKNOWN;
> + }
> +}
> +
> +static int tvp5150_enum_mbus_fmt(struct v4l2_subdev *sd, unsigned index,
> + enum v4l2_mbus_pixelcode *code)
> +{
> + if (index)
> + return -EINVAL;
> +
> + *code = V4L2_MBUS_FMT_YUYV8_2X8;
> + return 0;
> +}
> +
> +static int tvp5150_mbus_fmt(struct v4l2_subdev *sd,
> + struct v4l2_mbus_framefmt *f)
> +{
> + struct tvp5150 *decoder = to_tvp5150(sd);
> + v4l2_std_id std;
> +
> + if (f == NULL)
> + return -EINVAL;
> +
> + tvp5150_reset(sd, 0);
> +
> + /* Calculate height and width based on current standard */
> + if (decoder->norm == V4L2_STD_ALL)
> + std = tvp5150_read_std(sd);
> + else
> + std = decoder->norm;
> +
> + if ((std == V4L2_STD_NTSC) || (std == V4L2_STD_NTSC_443) ||
> + (std == V4L2_STD_PAL_M)) {
> + f->width = 720;
> + f->height = 480;
> + }
> + if ((std == V4L2_STD_PAL) ||
> + (std == (V4L2_STD_PAL_N | V4L2_STD_PAL_Nc)) ||
> + (std == V4L2_STD_SECAM)) {
> + f->width = 720;
> + f->height = 576;
> + }
The above is wrong, as std is a bitmask. So, userspace can pass more than
one bit set there. It should be, instead:
f->width = 720;
if (std & V4L2_STD_525_60)
f->height = 480;
else
f->height = 576;
> + f->code = V4L2_MBUS_FMT_YUYV8_2X8;
> + f->field = V4L2_FIELD_SEQ_TB;
> + f->colorspace = V4L2_COLORSPACE_SMPTE170M;
> +
> + v4l2_dbg(1, debug, sd, "width = %d, height = %d\n", f->width,
> + f->height);
> + return 0;
> +}
> +
> /****************************************************************************
> I2C Command
>
> ****************************************************************************/
> @@ -930,6 +999,9 @@ static const struct v4l2_subdev_tuner_ops
> tvp5150_tuner_ops = {
>
> static const struct v4l2_subdev_video_ops tvp5150_video_ops = {
> .s_routing = tvp5150_s_routing,
> + .enum_mbus_fmt = tvp5150_enum_mbus_fmt,
> + .s_mbus_fmt = tvp5150_mbus_fmt,
> + .try_mbus_fmt = tvp5150_mbus_fmt,
> };
>
> static const struct v4l2_subdev_vbi_ops tvp5150_vbi_ops = {
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
