[2/2] media: tvp5150: Add mbus_fmt callbacks.
Commit Message
These callbacks allow a host video driver
to poll video supported video formats of tvp5150.
Signed-off-by: Javier Martin <javier.martin@vista-silicon.com>
---
drivers/media/video/tvp5150.c | 72 +++++++++++++++++++++++++++++++++++++++++
1 files changed, 72 insertions(+), 0 deletions(-)
Comments
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.martin@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 majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Mauro,
thank you for your review, I will prepare a new version with those fixes.
@@ -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;
+ }
+ 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 = {