* [PATCH 1/5] V4L2: I2C: ML86V7667 video decoder driver
@ 2013-04-20 20:16 ` Sergei Shtylyov
0 siblings, 0 replies; 28+ messages in thread
From: Sergei Shtylyov @ 2013-04-20 20:16 UTC (permalink / raw)
To: mchehab, linux-media; +Cc: linux-sh, matsu, vladimir.barinov
From: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
Add OKI Semiconductor ML86V7667 video decoder driver.
Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
[Sergei: added v4l2_device_unregister_subdev() call to the error cleanup path of
ml86v7667_probe(); some cleanup.]
Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
---
drivers/media/i2c/Kconfig | 9
drivers/media/i2c/Makefile | 1
drivers/media/i2c/ml86v7667.c | 504 ++++++++++++++++++++++++++++++++++++++++++
3 files changed, 514 insertions(+)
Index: renesas/drivers/media/i2c/Kconfig
===================================================================
--- renesas.orig/drivers/media/i2c/Kconfig
+++ renesas/drivers/media/i2c/Kconfig
@@ -227,6 +227,15 @@ config VIDEO_KS0127
To compile this driver as a module, choose M here: the
module will be called ks0127.
+config VIDEO_ML86V7667
+ tristate "OKI ML86V7667 video decoder"
+ depends on VIDEO_V4L2 && I2C
+ ---help---
+ Support for the OKI Semiconductor ML86V7667 video decoder.
+
+ To compile this driver as a module, choose M here: the
+ module will be called ml86v7667.
+
config VIDEO_SAA7110
tristate "Philips SAA7110 video decoder"
depends on VIDEO_V4L2 && I2C
Index: renesas/drivers/media/i2c/Makefile
===================================================================
--- renesas.orig/drivers/media/i2c/Makefile
+++ renesas/drivers/media/i2c/Makefile
@@ -64,3 +64,4 @@ obj-$(CONFIG_VIDEO_AS3645A) += as3645a.o
obj-$(CONFIG_VIDEO_SMIAPP_PLL) += smiapp-pll.o
obj-$(CONFIG_VIDEO_AK881X) += ak881x.o
obj-$(CONFIG_VIDEO_IR_I2C) += ir-kbd-i2c.o
+obj-$(CONFIG_VIDEO_ML86V7667) += ml86v7667.o
Index: renesas/drivers/media/i2c/ml86v7667.c
===================================================================
--- /dev/null
+++ renesas/drivers/media/i2c/ml86v7667.c
@@ -0,0 +1,504 @@
+/*
+ * OKI Semiconductor ML86V7667 video decoder driver
+ *
+ * Author: Vladimir Barinov <source@cogentembedded.com>
+ * Copyright (C) 2013 Cogent Embedded, Inc.
+ * Copyright (C) 2013 Renesas Solutions Corp.
+ *
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License as published by the
+ * Free Software Foundation; either version 2 of the License, or (at your
+ * option) any later version.
+ */
+
+#include <linux/init.h>
+#include <linux/module.h>
+#include <linux/i2c.h>
+#include <linux/slab.h>
+#include <linux/videodev2.h>
+#include <media/v4l2-chip-ident.h>
+#include <media/v4l2-subdev.h>
+#include <media/v4l2-device.h>
+#include <media/v4l2-ioctl.h>
+#include <media/v4l2-ctrls.h>
+
+#define DRV_NAME "ml86v7667"
+
+/* Subaddresses */
+#define MRA_REG 0x00 /* Mode Register A */
+#define MRC_REG 0x02 /* Mode Register C */
+#define LUMC_REG 0x0C /* Luminance Control */
+#define CLC_REG 0x10 /* Contrast level control */
+#define SSEPL_REG 0x11 /* Sync separation level */
+#define CHRCA_REG 0x12 /* Chrominance Control A */
+#define ACCC_REG 0x14 /* ACC Loop filter & Chrominance control */
+#define ACCRC_REG 0x15 /* ACC Reference level control */
+#define HUE_REG 0x16 /* Hue control */
+#define ADC2_REG 0x1F /* ADC Register 2 */
+#define PLLR1_REG 0x20 /* PLL Register 1 */
+#define STATUS_REG 0x2C /* STATUS Register */
+
+/* Mode Register A register bits */
+#define MRA_OUTPUT_MODE_MASK (3 << 6)
+#define MRA_ITUR_BT601 (1 << 6)
+#define MRA_ITUR_BT656 (0 << 6)
+#define MRA_INPUT_MODE_MASK (7 << 3)
+#define MRA_PAL_BT601 (4 << 3)
+#define MRA_NTSC_BT601 (0 << 3)
+#define MRA_REGISTER_MODE (1 << 0)
+
+/* Mode Register C register bits */
+#define MRC_AUTOSELECT (1 << 7)
+
+/* Luminance Control register bits */
+#define LUMC_ONOFF_SHIFT 7
+#define LUMC_ONOFF_MASK (1 << 7)
+
+/* Contrast level control register bits */
+#define CLC_CONTRAST_ONOFF (1 << 7)
+#define CLC_CONTRAST_MASK 0x0F
+
+/* Sync separation level register bits */
+#define SSEPL_LUMINANCE_ONOFF (1 << 7)
+#define SSEPL_LUMINANCE_MASK 0x7F
+
+/* Chrominance Control A register bits */
+#define CHRCA_MODE_SHIFT 6
+#define CHRCA_MODE_MASK (1 << 6)
+
+/* ACC Loop filter & Chrominance control register bits */
+#define ACCC_CHROMA_CR_SHIFT 3
+#define ACCC_CHROMA_CR_MASK (7 << 3)
+#define ACCC_CHROMA_CB_SHIFT 0
+#define ACCC_CHROMA_CB_MASK (7 << 3)
+
+/* ACC Reference level control register bits */
+#define ACCRC_CHROMA_MASK 0xfc
+#define ACCRC_CHROMA_SHIFT 2
+
+/* ADC Register 2 register bits */
+#define ADC2_CLAMP_VOLTAGE_MASK (7 << 1)
+#define ADC2_CLAMP_VOLTAGE(n) ((n & 7) << 1)
+
+/* PLL Register 1 register bits */
+#define PLLR1_FIXED_CLOCK (1 << 7)
+
+/* STATUS Register register bits */
+#define STATUS_HLOCK_DETECT (1 << 3)
+#define STATUS_NTSCPAL (1 << 2)
+
+struct ml86v7667_priv {
+ struct v4l2_subdev sd;
+ struct v4l2_ctrl_handler hdl;
+ struct v4l2_mbus_framefmt fmt;
+ v4l2_std_id std;
+ bool autodetect;
+};
+
+static inline struct ml86v7667_priv *to_ml86v7667(struct v4l2_subdev *subdev)
+{
+ return container_of(subdev, struct ml86v7667_priv, sd);
+}
+
+static inline struct v4l2_subdev *to_sd(struct v4l2_ctrl *ctrl)
+{
+ return &container_of(ctrl->handler, struct ml86v7667_priv, hdl)->sd;
+}
+
+static int ml86v7667_mask_set(struct i2c_client *client, const u8 reg,
+ const u8 mask, const u8 data)
+{
+ int val = i2c_smbus_read_byte_data(client, reg);
+ if (val < 0)
+ return val;
+
+ val = (val & ~mask) | (data & mask);
+ return i2c_smbus_write_byte_data(client, reg, val);
+}
+
+static int ml86v7667_s_ctrl(struct v4l2_ctrl *ctrl)
+{
+ struct v4l2_subdev *sd = to_sd(ctrl);
+ struct i2c_client *client = v4l2_get_subdevdata(sd);
+ int ret = 0;
+
+ switch (ctrl->id) {
+ case V4L2_CID_BRIGHTNESS:
+ ret = ml86v7667_mask_set(client, SSEPL_REG,
+ SSEPL_LUMINANCE_MASK, ctrl->val);
+ break;
+ case V4L2_CID_CONTRAST:
+ ret = ml86v7667_mask_set(client, CLC_REG,
+ CLC_CONTRAST_MASK, ctrl->val);
+ break;
+ case V4L2_CID_CHROMA_GAIN:
+ ret = ml86v7667_mask_set(client, ACCRC_REG, ACCRC_CHROMA_MASK,
+ ctrl->val << ACCRC_CHROMA_SHIFT);
+ break;
+ case V4L2_CID_HUE:
+ ret = ml86v7667_mask_set(client, HUE_REG, ~0, ctrl->val);
+ break;
+ case V4L2_CID_RED_BALANCE:
+ ret = ml86v7667_mask_set(client, ACCC_REG,
+ ACCC_CHROMA_CR_MASK,
+ ctrl->val << ACCC_CHROMA_CR_SHIFT);
+ break;
+ case V4L2_CID_BLUE_BALANCE:
+ ret = ml86v7667_mask_set(client, ACCC_REG,
+ ACCC_CHROMA_CB_MASK,
+ ctrl->val << ACCC_CHROMA_CB_SHIFT);
+ break;
+ case V4L2_CID_SHARPNESS:
+ ret = ml86v7667_mask_set(client, LUMC_REG,
+ LUMC_ONOFF_MASK,
+ ctrl->val << LUMC_ONOFF_SHIFT);
+ break;
+ case V4L2_CID_COLOR_KILLER:
+ ret = ml86v7667_mask_set(client, CHRCA_REG,
+ CHRCA_MODE_MASK,
+ ctrl->val << CHRCA_MODE_SHIFT);
+ break;
+ }
+
+ return 0;
+}
+
+static int ml86v7667_querystd(struct v4l2_subdev *sd, v4l2_std_id *std)
+{
+ struct ml86v7667_priv *priv = to_ml86v7667(sd);
+ struct i2c_client *client = v4l2_get_subdevdata(sd);
+ int status_reg;
+
+ if (!priv->autodetect) {
+ *std = priv->std;
+ return 0;
+ }
+
+ status_reg = i2c_smbus_read_byte_data(client, STATUS_REG);
+ if (status_reg < 0)
+ return status_reg;
+
+ if (status_reg & STATUS_HLOCK_DETECT)
+ *std = status_reg & STATUS_NTSCPAL ? V4L2_STD_PAL :
+ V4L2_STD_NTSC;
+ else
+ *std = V4L2_STD_UNKNOWN;
+
+ return 0;
+}
+
+static int ml86v7667_g_input_status(struct v4l2_subdev *sd, u32 *status)
+{
+ struct i2c_client *client = v4l2_get_subdevdata(sd);
+ int status_reg;
+
+ status_reg = i2c_smbus_read_byte_data(client, STATUS_REG);
+ if (status_reg < 0)
+ return status_reg;
+
+ *status = status_reg & STATUS_HLOCK_DETECT ? 0 : V4L2_IN_ST_NO_SIGNAL;
+
+ return 0;
+}
+
+static int ml86v7667_enum_mbus_fmt(struct v4l2_subdev *sd, unsigned int index,
+ enum v4l2_mbus_pixelcode *code)
+{
+ if (index > 0)
+ return -EINVAL;
+
+ *code = V4L2_MBUS_FMT_YUYV8_2X8;
+
+ return 0;
+}
+
+static int ml86v7667_try_mbus_fmt(struct v4l2_subdev *sd,
+ struct v4l2_mbus_framefmt *fmt)
+{
+ struct ml86v7667_priv *priv = to_ml86v7667(sd);
+
+ ml86v7667_querystd(sd, &priv->std);
+
+ fmt->code = V4L2_MBUS_FMT_YUYV8_2X8;
+ fmt->colorspace = V4L2_COLORSPACE_SMPTE170M;
+ fmt->field = V4L2_FIELD_INTERLACED;
+ fmt->width = 720;
+ fmt->height = priv->std & V4L2_STD_525_60 ? 480 : 576;
+
+ return 0;
+}
+
+static int ml86v7667_g_mbus_fmt(struct v4l2_subdev *sd,
+ struct v4l2_mbus_framefmt *fmt)
+{
+ struct ml86v7667_priv *priv = to_ml86v7667(sd);
+
+ *fmt = priv->fmt;
+
+ return 0;
+}
+
+static int ml86v7667_s_mbus_fmt(struct v4l2_subdev *sd,
+ struct v4l2_mbus_framefmt *fmt)
+{
+ struct ml86v7667_priv *priv = to_ml86v7667(sd);
+
+ ml86v7667_try_mbus_fmt(sd, fmt);
+ priv->fmt = *fmt;
+
+ return 0;
+}
+
+static int ml86v7667_cropcap(struct v4l2_subdev *sd, struct v4l2_cropcap *a)
+{
+ struct ml86v7667_priv *priv = to_ml86v7667(sd);
+
+ ml86v7667_querystd(sd, &priv->std);
+
+ a->bounds.left = 0;
+ a->bounds.top = 0;
+ a->bounds.width = 720;
+ a->bounds.height = priv->std & V4L2_STD_525_60 ? 480 : 576;
+ a->defrect = a->bounds;
+ a->type = V4L2_BUF_TYPE_VIDEO_CAPTURE;
+ a->pixelaspect.numerator = 1;
+ a->pixelaspect.denominator = 1;
+
+ return 0;
+}
+
+static int ml86v7667_g_mbus_config(struct v4l2_subdev *sd,
+ struct v4l2_mbus_config *cfg)
+{
+ cfg->flags = V4L2_MBUS_MASTER | V4L2_MBUS_PCLK_SAMPLE_RISING |
+ V4L2_MBUS_DATA_ACTIVE_HIGH;
+ cfg->type = V4L2_MBUS_BT656;
+
+ return 0;
+}
+
+static int ml86v7667_g_chip_ident(struct v4l2_subdev *sd,
+ struct v4l2_dbg_chip_ident *id)
+{
+ id->ident = V4L2_IDENT_UNKNOWN;
+ id->revision = 0;
+
+ return 0;
+}
+
+static int ml86v7667_s_std(struct v4l2_subdev *sd, v4l2_std_id std)
+{
+ struct ml86v7667_priv *priv = to_ml86v7667(sd);
+ struct i2c_client *client = v4l2_get_subdevdata(&priv->sd);
+ int ret;
+ u8 mode;
+
+ if (std == V4L2_STD_ALL) {
+ /* enable PAL/NTSC autodetection */
+ ret = ml86v7667_mask_set(client, MRC_REG,
+ MRC_AUTOSELECT, MRC_AUTOSELECT);
+ priv->autodetect = true;
+ } else {
+ /* PAL/NTSC ITU-R BT.601 input mode */
+ mode = std & V4L2_STD_525_60 ? MRA_NTSC_BT601 : MRA_PAL_BT601;
+ ret = ml86v7667_mask_set(client, MRA_REG,
+ MRA_INPUT_MODE_MASK, mode);
+ if (ret < 0)
+ return ret;
+
+ /* disable PAL/NTSC autodetection */
+ ret = ml86v7667_mask_set(client, MRC_REG, MRC_AUTOSELECT, 0);
+ if (ret < 0)
+ return ret;
+
+ priv->std = std;
+ priv->autodetect = false;
+ }
+
+ return 0;
+}
+
+#ifdef CONFIG_VIDEO_ADV_DEBUG
+static int ml86v7667_g_register(struct v4l2_subdev *sd,
+ struct v4l2_dbg_register *reg)
+{
+ struct i2c_client *client = v4l2_get_subdevdata(sd);
+ int ret;
+
+ if (!v4l2_chip_match_i2c_client(client, ®->match))
+ return -EINVAL;
+ if (!capable(CAP_SYS_ADMIN))
+ return -EPERM;
+
+ ret = i2c_smbus_read_byte_data(client, (u8)reg->reg);
+ if (ret < 0)
+ return ret;
+
+ reg->val = ret;
+ reg->size = sizeof(u8);
+
+ return 0;
+}
+
+static int ml86v7667_s_register(struct v4l2_subdev *sd,
+ struct v4l2_dbg_register *reg)
+{
+ struct i2c_client *client = v4l2_get_subdevdata(sd);
+
+ if (!v4l2_chip_match_i2c_client(client, ®->match))
+ return -EINVAL;
+ if (!capable(CAP_SYS_ADMIN))
+ return -EPERM;
+
+ return i2c_smbus_write_byte_data(client, (u8)reg->reg, (u8)reg->val);
+}
+#endif
+
+static const struct v4l2_ctrl_ops ml86v7667_ctrl_ops = {
+ .s_ctrl = ml86v7667_s_ctrl,
+};
+
+static struct v4l2_subdev_video_ops ml86v7667_subdev_video_ops = {
+ .querystd = ml86v7667_querystd,
+ .g_input_status = ml86v7667_g_input_status,
+ .enum_mbus_fmt = ml86v7667_enum_mbus_fmt,
+ .try_mbus_fmt = ml86v7667_try_mbus_fmt,
+ .g_mbus_fmt = ml86v7667_g_mbus_fmt,
+ .s_mbus_fmt = ml86v7667_s_mbus_fmt,
+ .cropcap = ml86v7667_cropcap,
+ .g_mbus_config = ml86v7667_g_mbus_config,
+};
+
+static struct v4l2_subdev_core_ops ml86v7667_subdev_core_ops = {
+ .g_chip_ident = ml86v7667_g_chip_ident,
+ .s_std = ml86v7667_s_std,
+#ifdef CONFIG_VIDEO_ADV_DEBUG
+ .g_register = ml86v7667_g_register,
+ .s_register = ml86v7667_s_register,
+#endif
+};
+
+static struct v4l2_subdev_ops ml86v7667_subdev_ops = {
+ .core = &ml86v7667_subdev_core_ops,
+ .video = &ml86v7667_subdev_video_ops,
+};
+
+static int ml86v7667_init(struct ml86v7667_priv *priv)
+{
+ struct i2c_client *client = v4l2_get_subdevdata(&priv->sd);
+ int ret;
+
+ /* BT.656-4 output mode, register mode */
+ ret = ml86v7667_mask_set(client, MRA_REG,
+ MRA_OUTPUT_MODE_MASK | MRA_REGISTER_MODE,
+ MRA_ITUR_BT656 | MRA_REGISTER_MODE);
+
+ /* PLL circuit fixed clock, 32MHz */
+ ret |= ml86v7667_mask_set(client, PLLR1_REG, PLLR1_FIXED_CLOCK,
+ PLLR1_FIXED_CLOCK);
+
+ /* ADC2 clamping voltage maximum */
+ ret |= ml86v7667_mask_set(client, ADC2_REG, ADC2_CLAMP_VOLTAGE_MASK,
+ ADC2_CLAMP_VOLTAGE(7));
+
+ /* enable luminance function */
+ ret |= ml86v7667_mask_set(client, SSEPL_REG, SSEPL_LUMINANCE_ONOFF,
+ SSEPL_LUMINANCE_ONOFF);
+
+ /* enable contrast function */
+ ret |= ml86v7667_mask_set(client, CLC_REG, CLC_CONTRAST_ONOFF, 0);
+
+ /* NTS/PAL autodetection enabled by default */
+ priv->autodetect = true;
+
+ return ret;
+}
+
+static int ml86v7667_probe(struct i2c_client *client,
+ const struct i2c_device_id *did)
+{
+ struct ml86v7667_priv *priv;
+ int ret;
+
+ if (!i2c_check_functionality(client->adapter, I2C_FUNC_SMBUS_BYTE_DATA))
+ return -EIO;
+
+ priv = devm_kzalloc(&client->dev, sizeof(*priv), GFP_KERNEL);
+ if (!priv)
+ return -ENOMEM;
+
+ v4l2_i2c_subdev_init(&priv->sd, client, &ml86v7667_subdev_ops);
+
+ v4l2_ctrl_handler_init(&priv->hdl, 8);
+ v4l2_ctrl_new_std(&priv->hdl, &ml86v7667_ctrl_ops,
+ V4L2_CID_BRIGHTNESS, -64, 63, 1, 0);
+ v4l2_ctrl_new_std(&priv->hdl, &ml86v7667_ctrl_ops,
+ V4L2_CID_CONTRAST, -8, 7, 1, 0);
+ v4l2_ctrl_new_std(&priv->hdl, &ml86v7667_ctrl_ops,
+ V4L2_CID_CHROMA_GAIN, -32, 31, 1, 0);
+ v4l2_ctrl_new_std(&priv->hdl, &ml86v7667_ctrl_ops,
+ V4L2_CID_HUE, -128, 127, 1, 0);
+ v4l2_ctrl_new_std(&priv->hdl, &ml86v7667_ctrl_ops,
+ V4L2_CID_RED_BALANCE, -4, 3, 1, 0);
+ v4l2_ctrl_new_std(&priv->hdl, &ml86v7667_ctrl_ops,
+ V4L2_CID_BLUE_BALANCE, -4, 3, 1, 0);
+ v4l2_ctrl_new_std(&priv->hdl, &ml86v7667_ctrl_ops,
+ V4L2_CID_SHARPNESS, 0, 1, 1, 0);
+ v4l2_ctrl_new_std(&priv->hdl, &ml86v7667_ctrl_ops,
+ V4L2_CID_COLOR_KILLER, 0, 1, 1, 0);
+ priv->sd.ctrl_handler = &priv->hdl;
+
+ ret = priv->hdl.error;
+ if (ret)
+ goto cleanup;
+
+ v4l2_ctrl_handler_setup(&priv->hdl);
+
+ ret = ml86v7667_init(priv);
+ if (ret)
+ goto cleanup;
+
+ v4l_info(client, "chip found @ 0x%02x (%s)\n",
+ client->addr, client->adapter->name);
+ return 0;
+
+cleanup:
+ v4l2_ctrl_handler_free(&priv->hdl);
+ v4l2_device_unregister_subdev(&priv->sd);
+ v4l_err(client, "failed to probe @ 0x%02x (%s)\n",
+ client->addr, client->adapter->name);
+ return ret;
+}
+
+static int ml86v7667_remove(struct i2c_client *client)
+{
+ struct v4l2_subdev *sd = i2c_get_clientdata(client);
+ struct ml86v7667_priv *priv = to_ml86v7667(sd);
+
+ v4l2_ctrl_handler_free(&priv->hdl);
+ v4l2_device_unregister_subdev(&priv->sd);
+
+ return 0;
+}
+
+static const struct i2c_device_id ml86v7667_id[] = {
+ {DRV_NAME, 0},
+ {},
+};
+MODULE_DEVICE_TABLE(i2c, ml86v7667_id);
+
+static struct i2c_driver ml86v7667_i2c_driver = {
+ .driver = {
+ .name = DRV_NAME,
+ .owner = THIS_MODULE,
+ },
+ .probe = ml86v7667_probe,
+ .remove = ml86v7667_remove,
+ .id_table = ml86v7667_id,
+};
+
+module_i2c_driver(ml86v7667_i2c_driver);
+
+MODULE_DESCRIPTION("OKI Semiconductor ML86V7667 video decoder driver");
+MODULE_AUTHOR("Vladimir Barinov");
+MODULE_LICENSE("GPL");
^ permalink raw reply [flat|nested] 28+ messages in thread* Re: [PATCH 1/5] V4L2: I2C: ML86V7667 video decoder driver
2013-04-20 20:16 ` Sergei Shtylyov
@ 2013-04-20 21:05 ` Sylwester Nawrocki
-1 siblings, 0 replies; 28+ messages in thread
From: Sylwester Nawrocki @ 2013-04-20 21:05 UTC (permalink / raw)
To: Sergei Shtylyov; +Cc: mchehab, linux-media, linux-sh, matsu, vladimir.barinov
Hi,
On 04/20/2013 10:16 PM, Sergei Shtylyov wrote:
> From: Vladimir Barinov<vladimir.barinov@cogentembedded.com>
>
> Add OKI Semiconductor ML86V7667 video decoder driver.
>
> Signed-off-by: Vladimir Barinov<vladimir.barinov@cogentembedded.com>
> [Sergei: added v4l2_device_unregister_subdev() call to the error cleanup path of
> ml86v7667_probe(); some cleanup.]
> Signed-off-by: Sergei Shtylyov<sergei.shtylyov@cogentembedded.com>
>
> ---
> drivers/media/i2c/Kconfig | 9
> drivers/media/i2c/Makefile | 1
> drivers/media/i2c/ml86v7667.c | 504 ++++++++++++++++++++++++++++++++++++++++++
> 3 files changed, 514 insertions(+)
> +static int ml86v7667_cropcap(struct v4l2_subdev *sd, struct v4l2_cropcap *a)
> +{
> + struct ml86v7667_priv *priv = to_ml86v7667(sd);
> +
> + ml86v7667_querystd(sd,&priv->std);
> +
> + a->bounds.left = 0;
> + a->bounds.top = 0;
> + a->bounds.width = 720;
> + a->bounds.height = priv->std& V4L2_STD_525_60 ? 480 : 576;
> + a->defrect = a->bounds;
> + a->type = V4L2_BUF_TYPE_VIDEO_CAPTURE;
> + a->pixelaspect.numerator = 1;
> + a->pixelaspect.denominator = 1;
> +
> + return 0;
> +}
> +static struct v4l2_subdev_video_ops ml86v7667_subdev_video_ops = {
> + .querystd = ml86v7667_querystd,
> + .g_input_status = ml86v7667_g_input_status,
> + .enum_mbus_fmt = ml86v7667_enum_mbus_fmt,
> + .try_mbus_fmt = ml86v7667_try_mbus_fmt,
> + .g_mbus_fmt = ml86v7667_g_mbus_fmt,
> + .s_mbus_fmt = ml86v7667_s_mbus_fmt,
> + .cropcap = ml86v7667_cropcap,
> + .g_mbus_config = ml86v7667_g_mbus_config,
> +};
Why do you need .cropcap when there is no s_crop/g_crop ops ? Is it
only for pixel aspect ratio ?
Also, new drivers are supposed to use the selections API instead
(set/get_selection ops). However this requires pad level ops support
in your host driver, hence might be a bigger issue.
Regards,
Sylwester
^ permalink raw reply [flat|nested] 28+ messages in thread* Re: [PATCH 1/5] V4L2: I2C: ML86V7667 video decoder driver
@ 2013-04-20 21:05 ` Sylwester Nawrocki
0 siblings, 0 replies; 28+ messages in thread
From: Sylwester Nawrocki @ 2013-04-20 21:05 UTC (permalink / raw)
To: Sergei Shtylyov; +Cc: mchehab, linux-media, linux-sh, matsu, vladimir.barinov
Hi,
On 04/20/2013 10:16 PM, Sergei Shtylyov wrote:
> From: Vladimir Barinov<vladimir.barinov@cogentembedded.com>
>
> Add OKI Semiconductor ML86V7667 video decoder driver.
>
> Signed-off-by: Vladimir Barinov<vladimir.barinov@cogentembedded.com>
> [Sergei: added v4l2_device_unregister_subdev() call to the error cleanup path of
> ml86v7667_probe(); some cleanup.]
> Signed-off-by: Sergei Shtylyov<sergei.shtylyov@cogentembedded.com>
>
> ---
> drivers/media/i2c/Kconfig | 9
> drivers/media/i2c/Makefile | 1
> drivers/media/i2c/ml86v7667.c | 504 ++++++++++++++++++++++++++++++++++++++++++
> 3 files changed, 514 insertions(+)
> +static int ml86v7667_cropcap(struct v4l2_subdev *sd, struct v4l2_cropcap *a)
> +{
> + struct ml86v7667_priv *priv = to_ml86v7667(sd);
> +
> + ml86v7667_querystd(sd,&priv->std);
> +
> + a->bounds.left = 0;
> + a->bounds.top = 0;
> + a->bounds.width = 720;
> + a->bounds.height = priv->std& V4L2_STD_525_60 ? 480 : 576;
> + a->defrect = a->bounds;
> + a->type = V4L2_BUF_TYPE_VIDEO_CAPTURE;
> + a->pixelaspect.numerator = 1;
> + a->pixelaspect.denominator = 1;
> +
> + return 0;
> +}
> +static struct v4l2_subdev_video_ops ml86v7667_subdev_video_ops = {
> + .querystd = ml86v7667_querystd,
> + .g_input_status = ml86v7667_g_input_status,
> + .enum_mbus_fmt = ml86v7667_enum_mbus_fmt,
> + .try_mbus_fmt = ml86v7667_try_mbus_fmt,
> + .g_mbus_fmt = ml86v7667_g_mbus_fmt,
> + .s_mbus_fmt = ml86v7667_s_mbus_fmt,
> + .cropcap = ml86v7667_cropcap,
> + .g_mbus_config = ml86v7667_g_mbus_config,
> +};
Why do you need .cropcap when there is no s_crop/g_crop ops ? Is it
only for pixel aspect ratio ?
Also, new drivers are supposed to use the selections API instead
(set/get_selection ops). However this requires pad level ops support
in your host driver, hence might be a bigger issue.
Regards,
Sylwester
^ permalink raw reply [flat|nested] 28+ messages in thread* Re: [PATCH 1/5] V4L2: I2C: ML86V7667 video decoder driver
2013-04-20 21:05 ` Sylwester Nawrocki
@ 2013-04-21 10:23 ` Vladimir Barinov
-1 siblings, 0 replies; 28+ messages in thread
From: Vladimir Barinov @ 2013-04-21 10:23 UTC (permalink / raw)
To: Sylwester Nawrocki; +Cc: Sergei Shtylyov, mchehab, linux-media, linux-sh, matsu
Hi Sylwester,
Thank you for the review.
Sylwester Nawrocki wrote
>
>> +static struct v4l2_subdev_video_ops ml86v7667_subdev_video_ops = {
>> + .querystd = ml86v7667_querystd,
>> + .g_input_status = ml86v7667_g_input_status,
>> + .enum_mbus_fmt = ml86v7667_enum_mbus_fmt,
>> + .try_mbus_fmt = ml86v7667_try_mbus_fmt,
>> + .g_mbus_fmt = ml86v7667_g_mbus_fmt,
>> + .s_mbus_fmt = ml86v7667_s_mbus_fmt,
>> + .cropcap = ml86v7667_cropcap,
>> + .g_mbus_config = ml86v7667_g_mbus_config,
>> +};
>
> Why do you need .cropcap when there is no s_crop/g_crop ops ? Is it
> only for pixel aspect ratio ?
Yes it is.
>
> Also, new drivers are supposed to use the selections API instead
> (set/get_selection ops). However this requires pad level ops support
> in your host driver, hence might be a bigger issue.
Yes, since the host driver (rcar_vin is already send for review) is
designed for soc-camera layer that does not use the pad level ops but
needs at least cropcap to get the pixel aspect ratio.
The soc-camera has it's own set/get_selection in soc_camera_host_ops and
it is possible to use it like all i2c/soc_camera sensors can/do.
But we would not like to design the ml86v7667 as soc-camera device since
it will not be available for all non soc-camera v4l2 hosts.
Regards,
Vladimir
^ permalink raw reply [flat|nested] 28+ messages in thread* Re: [PATCH 1/5] V4L2: I2C: ML86V7667 video decoder driver
@ 2013-04-21 10:23 ` Vladimir Barinov
0 siblings, 0 replies; 28+ messages in thread
From: Vladimir Barinov @ 2013-04-21 10:23 UTC (permalink / raw)
To: Sylwester Nawrocki; +Cc: Sergei Shtylyov, mchehab, linux-media, linux-sh, matsu
Hi Sylwester,
Thank you for the review.
Sylwester Nawrocki wrote
>
>> +static struct v4l2_subdev_video_ops ml86v7667_subdev_video_ops = {
>> + .querystd = ml86v7667_querystd,
>> + .g_input_status = ml86v7667_g_input_status,
>> + .enum_mbus_fmt = ml86v7667_enum_mbus_fmt,
>> + .try_mbus_fmt = ml86v7667_try_mbus_fmt,
>> + .g_mbus_fmt = ml86v7667_g_mbus_fmt,
>> + .s_mbus_fmt = ml86v7667_s_mbus_fmt,
>> + .cropcap = ml86v7667_cropcap,
>> + .g_mbus_config = ml86v7667_g_mbus_config,
>> +};
>
> Why do you need .cropcap when there is no s_crop/g_crop ops ? Is it
> only for pixel aspect ratio ?
Yes it is.
>
> Also, new drivers are supposed to use the selections API instead
> (set/get_selection ops). However this requires pad level ops support
> in your host driver, hence might be a bigger issue.
Yes, since the host driver (rcar_vin is already send for review) is
designed for soc-camera layer that does not use the pad level ops but
needs at least cropcap to get the pixel aspect ratio.
The soc-camera has it's own set/get_selection in soc_camera_host_ops and
it is possible to use it like all i2c/soc_camera sensors can/do.
But we would not like to design the ml86v7667 as soc-camera device since
it will not be available for all non soc-camera v4l2 hosts.
Regards,
Vladimir
^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [PATCH 1/5] V4L2: I2C: ML86V7667 video decoder driver
2013-04-20 20:16 ` Sergei Shtylyov
@ 2013-04-20 21:50 ` Hans Verkuil
-1 siblings, 0 replies; 28+ messages in thread
From: Hans Verkuil @ 2013-04-20 21:50 UTC (permalink / raw)
To: Sergei Shtylyov; +Cc: mchehab, linux-media, linux-sh, matsu, vladimir.barinov
Here's a quick review:
On Sat April 20 2013 22:16:33 Sergei Shtylyov wrote:
> From: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
>
> Add OKI Semiconductor ML86V7667 video decoder driver.
>
> Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
> [Sergei: added v4l2_device_unregister_subdev() call to the error cleanup path of
> ml86v7667_probe(); some cleanup.]
> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
>
> ---
> drivers/media/i2c/Kconfig | 9
> drivers/media/i2c/Makefile | 1
> drivers/media/i2c/ml86v7667.c | 504 ++++++++++++++++++++++++++++++++++++++++++
> 3 files changed, 514 insertions(+)
>
> Index: renesas/drivers/media/i2c/Kconfig
> =================================> --- renesas.orig/drivers/media/i2c/Kconfig
> +++ renesas/drivers/media/i2c/Kconfig
> @@ -227,6 +227,15 @@ config VIDEO_KS0127
> To compile this driver as a module, choose M here: the
> module will be called ks0127.
>
> +config VIDEO_ML86V7667
> + tristate "OKI ML86V7667 video decoder"
> + depends on VIDEO_V4L2 && I2C
> + ---help---
> + Support for the OKI Semiconductor ML86V7667 video decoder.
> +
> + To compile this driver as a module, choose M here: the
> + module will be called ml86v7667.
> +
> config VIDEO_SAA7110
> tristate "Philips SAA7110 video decoder"
> depends on VIDEO_V4L2 && I2C
> Index: renesas/drivers/media/i2c/Makefile
> =================================> --- renesas.orig/drivers/media/i2c/Makefile
> +++ renesas/drivers/media/i2c/Makefile
> @@ -64,3 +64,4 @@ obj-$(CONFIG_VIDEO_AS3645A) += as3645a.o
> obj-$(CONFIG_VIDEO_SMIAPP_PLL) += smiapp-pll.o
> obj-$(CONFIG_VIDEO_AK881X) += ak881x.o
> obj-$(CONFIG_VIDEO_IR_I2C) += ir-kbd-i2c.o
> +obj-$(CONFIG_VIDEO_ML86V7667) += ml86v7667.o
> Index: renesas/drivers/media/i2c/ml86v7667.c
> =================================> --- /dev/null
> +++ renesas/drivers/media/i2c/ml86v7667.c
> @@ -0,0 +1,504 @@
> +/*
> + * OKI Semiconductor ML86V7667 video decoder driver
> + *
> + * Author: Vladimir Barinov <source@cogentembedded.com>
> + * Copyright (C) 2013 Cogent Embedded, Inc.
> + * Copyright (C) 2013 Renesas Solutions Corp.
> + *
> + * This program is free software; you can redistribute it and/or modify it
> + * under the terms of the GNU General Public License as published by the
> + * Free Software Foundation; either version 2 of the License, or (at your
> + * option) any later version.
> + */
> +
> +#include <linux/init.h>
> +#include <linux/module.h>
> +#include <linux/i2c.h>
> +#include <linux/slab.h>
> +#include <linux/videodev2.h>
> +#include <media/v4l2-chip-ident.h>
> +#include <media/v4l2-subdev.h>
> +#include <media/v4l2-device.h>
> +#include <media/v4l2-ioctl.h>
> +#include <media/v4l2-ctrls.h>
> +
> +#define DRV_NAME "ml86v7667"
> +
> +/* Subaddresses */
> +#define MRA_REG 0x00 /* Mode Register A */
> +#define MRC_REG 0x02 /* Mode Register C */
> +#define LUMC_REG 0x0C /* Luminance Control */
> +#define CLC_REG 0x10 /* Contrast level control */
> +#define SSEPL_REG 0x11 /* Sync separation level */
> +#define CHRCA_REG 0x12 /* Chrominance Control A */
> +#define ACCC_REG 0x14 /* ACC Loop filter & Chrominance control */
> +#define ACCRC_REG 0x15 /* ACC Reference level control */
> +#define HUE_REG 0x16 /* Hue control */
> +#define ADC2_REG 0x1F /* ADC Register 2 */
> +#define PLLR1_REG 0x20 /* PLL Register 1 */
> +#define STATUS_REG 0x2C /* STATUS Register */
> +
> +/* Mode Register A register bits */
> +#define MRA_OUTPUT_MODE_MASK (3 << 6)
> +#define MRA_ITUR_BT601 (1 << 6)
> +#define MRA_ITUR_BT656 (0 << 6)
> +#define MRA_INPUT_MODE_MASK (7 << 3)
> +#define MRA_PAL_BT601 (4 << 3)
> +#define MRA_NTSC_BT601 (0 << 3)
> +#define MRA_REGISTER_MODE (1 << 0)
> +
> +/* Mode Register C register bits */
> +#define MRC_AUTOSELECT (1 << 7)
> +
> +/* Luminance Control register bits */
> +#define LUMC_ONOFF_SHIFT 7
> +#define LUMC_ONOFF_MASK (1 << 7)
> +
> +/* Contrast level control register bits */
> +#define CLC_CONTRAST_ONOFF (1 << 7)
> +#define CLC_CONTRAST_MASK 0x0F
> +
> +/* Sync separation level register bits */
> +#define SSEPL_LUMINANCE_ONOFF (1 << 7)
> +#define SSEPL_LUMINANCE_MASK 0x7F
> +
> +/* Chrominance Control A register bits */
> +#define CHRCA_MODE_SHIFT 6
> +#define CHRCA_MODE_MASK (1 << 6)
> +
> +/* ACC Loop filter & Chrominance control register bits */
> +#define ACCC_CHROMA_CR_SHIFT 3
> +#define ACCC_CHROMA_CR_MASK (7 << 3)
> +#define ACCC_CHROMA_CB_SHIFT 0
> +#define ACCC_CHROMA_CB_MASK (7 << 3)
> +
> +/* ACC Reference level control register bits */
> +#define ACCRC_CHROMA_MASK 0xfc
> +#define ACCRC_CHROMA_SHIFT 2
> +
> +/* ADC Register 2 register bits */
> +#define ADC2_CLAMP_VOLTAGE_MASK (7 << 1)
> +#define ADC2_CLAMP_VOLTAGE(n) ((n & 7) << 1)
> +
> +/* PLL Register 1 register bits */
> +#define PLLR1_FIXED_CLOCK (1 << 7)
> +
> +/* STATUS Register register bits */
> +#define STATUS_HLOCK_DETECT (1 << 3)
> +#define STATUS_NTSCPAL (1 << 2)
> +
> +struct ml86v7667_priv {
> + struct v4l2_subdev sd;
> + struct v4l2_ctrl_handler hdl;
> + struct v4l2_mbus_framefmt fmt;
> + v4l2_std_id std;
> + bool autodetect;
Generally autodetect is a bad idea: changing from pal to ntsc on the fly can
cause problems further down the pipeline since the it might affect the amount
of data being DMA-ed. It's also non-standard behavior.
I recommend dropping autodetect support.
> +};
> +
> +static inline struct ml86v7667_priv *to_ml86v7667(struct v4l2_subdev *subdev)
> +{
> + return container_of(subdev, struct ml86v7667_priv, sd);
> +}
> +
> +static inline struct v4l2_subdev *to_sd(struct v4l2_ctrl *ctrl)
> +{
> + return &container_of(ctrl->handler, struct ml86v7667_priv, hdl)->sd;
> +}
> +
> +static int ml86v7667_mask_set(struct i2c_client *client, const u8 reg,
> + const u8 mask, const u8 data)
> +{
> + int val = i2c_smbus_read_byte_data(client, reg);
> + if (val < 0)
> + return val;
> +
> + val = (val & ~mask) | (data & mask);
> + return i2c_smbus_write_byte_data(client, reg, val);
> +}
> +
> +static int ml86v7667_s_ctrl(struct v4l2_ctrl *ctrl)
> +{
> + struct v4l2_subdev *sd = to_sd(ctrl);
> + struct i2c_client *client = v4l2_get_subdevdata(sd);
> + int ret = 0;
> +
> + switch (ctrl->id) {
> + case V4L2_CID_BRIGHTNESS:
> + ret = ml86v7667_mask_set(client, SSEPL_REG,
> + SSEPL_LUMINANCE_MASK, ctrl->val);
> + break;
> + case V4L2_CID_CONTRAST:
> + ret = ml86v7667_mask_set(client, CLC_REG,
> + CLC_CONTRAST_MASK, ctrl->val);
> + break;
> + case V4L2_CID_CHROMA_GAIN:
> + ret = ml86v7667_mask_set(client, ACCRC_REG, ACCRC_CHROMA_MASK,
> + ctrl->val << ACCRC_CHROMA_SHIFT);
> + break;
> + case V4L2_CID_HUE:
> + ret = ml86v7667_mask_set(client, HUE_REG, ~0, ctrl->val);
> + break;
> + case V4L2_CID_RED_BALANCE:
> + ret = ml86v7667_mask_set(client, ACCC_REG,
> + ACCC_CHROMA_CR_MASK,
> + ctrl->val << ACCC_CHROMA_CR_SHIFT);
> + break;
> + case V4L2_CID_BLUE_BALANCE:
> + ret = ml86v7667_mask_set(client, ACCC_REG,
> + ACCC_CHROMA_CB_MASK,
> + ctrl->val << ACCC_CHROMA_CB_SHIFT);
> + break;
> + case V4L2_CID_SHARPNESS:
> + ret = ml86v7667_mask_set(client, LUMC_REG,
> + LUMC_ONOFF_MASK,
> + ctrl->val << LUMC_ONOFF_SHIFT);
> + break;
> + case V4L2_CID_COLOR_KILLER:
> + ret = ml86v7667_mask_set(client, CHRCA_REG,
> + CHRCA_MODE_MASK,
> + ctrl->val << CHRCA_MODE_SHIFT);
> + break;
> + }
> +
> + return 0;
> +}
> +
> +static int ml86v7667_querystd(struct v4l2_subdev *sd, v4l2_std_id *std)
> +{
> + struct ml86v7667_priv *priv = to_ml86v7667(sd);
> + struct i2c_client *client = v4l2_get_subdevdata(sd);
> + int status_reg;
> +
> + if (!priv->autodetect) {
> + *std = priv->std;
> + return 0;
> + }
> +
> + status_reg = i2c_smbus_read_byte_data(client, STATUS_REG);
> + if (status_reg < 0)
> + return status_reg;
> +
> + if (status_reg & STATUS_HLOCK_DETECT)
> + *std = status_reg & STATUS_NTSCPAL ? V4L2_STD_PAL :
> + V4L2_STD_NTSC;
> + else
> + *std = V4L2_STD_UNKNOWN;
> +
> + return 0;
> +}
> +
> +static int ml86v7667_g_input_status(struct v4l2_subdev *sd, u32 *status)
> +{
> + struct i2c_client *client = v4l2_get_subdevdata(sd);
> + int status_reg;
> +
> + status_reg = i2c_smbus_read_byte_data(client, STATUS_REG);
> + if (status_reg < 0)
> + return status_reg;
> +
> + *status = status_reg & STATUS_HLOCK_DETECT ? 0 : V4L2_IN_ST_NO_SIGNAL;
> +
> + return 0;
> +}
> +
> +static int ml86v7667_enum_mbus_fmt(struct v4l2_subdev *sd, unsigned int index,
> + enum v4l2_mbus_pixelcode *code)
> +{
> + if (index > 0)
> + return -EINVAL;
> +
> + *code = V4L2_MBUS_FMT_YUYV8_2X8;
> +
> + return 0;
> +}
> +
> +static int ml86v7667_try_mbus_fmt(struct v4l2_subdev *sd,
> + struct v4l2_mbus_framefmt *fmt)
> +{
> + struct ml86v7667_priv *priv = to_ml86v7667(sd);
> +
> + ml86v7667_querystd(sd, &priv->std);
> +
> + fmt->code = V4L2_MBUS_FMT_YUYV8_2X8;
> + fmt->colorspace = V4L2_COLORSPACE_SMPTE170M;
> + fmt->field = V4L2_FIELD_INTERLACED;
> + fmt->width = 720;
> + fmt->height = priv->std & V4L2_STD_525_60 ? 480 : 576;
> +
> + return 0;
> +}
> +
> +static int ml86v7667_g_mbus_fmt(struct v4l2_subdev *sd,
> + struct v4l2_mbus_framefmt *fmt)
> +{
> + struct ml86v7667_priv *priv = to_ml86v7667(sd);
> +
> + *fmt = priv->fmt;
> +
> + return 0;
> +}
> +
> +static int ml86v7667_s_mbus_fmt(struct v4l2_subdev *sd,
> + struct v4l2_mbus_framefmt *fmt)
> +{
> + struct ml86v7667_priv *priv = to_ml86v7667(sd);
> +
> + ml86v7667_try_mbus_fmt(sd, fmt);
> + priv->fmt = *fmt;
> +
> + return 0;
> +}
> +
> +static int ml86v7667_cropcap(struct v4l2_subdev *sd, struct v4l2_cropcap *a)
> +{
> + struct ml86v7667_priv *priv = to_ml86v7667(sd);
> +
> + ml86v7667_querystd(sd, &priv->std);
Always use the current standard, never query the standard inside functions
like this. It's up to userspace to call QUERYSTD and then S_STD.
> +
> + a->bounds.left = 0;
> + a->bounds.top = 0;
> + a->bounds.width = 720;
> + a->bounds.height = priv->std & V4L2_STD_525_60 ? 480 : 576;
> + a->defrect = a->bounds;
> + a->type = V4L2_BUF_TYPE_VIDEO_CAPTURE;
> + a->pixelaspect.numerator = 1;
> + a->pixelaspect.denominator = 1;
> +
> + return 0;
> +}
> +
> +static int ml86v7667_g_mbus_config(struct v4l2_subdev *sd,
> + struct v4l2_mbus_config *cfg)
> +{
> + cfg->flags = V4L2_MBUS_MASTER | V4L2_MBUS_PCLK_SAMPLE_RISING |
> + V4L2_MBUS_DATA_ACTIVE_HIGH;
> + cfg->type = V4L2_MBUS_BT656;
> +
> + return 0;
> +}
> +
> +static int ml86v7667_g_chip_ident(struct v4l2_subdev *sd,
> + struct v4l2_dbg_chip_ident *id)
> +{
> + id->ident = V4L2_IDENT_UNKNOWN;
> + id->revision = 0;
> +
> + return 0;
> +}
Just drop this function, it's no longer needed and this op will be removed
in 3.11.
> +
> +static int ml86v7667_s_std(struct v4l2_subdev *sd, v4l2_std_id std)
> +{
> + struct ml86v7667_priv *priv = to_ml86v7667(sd);
> + struct i2c_client *client = v4l2_get_subdevdata(&priv->sd);
> + int ret;
> + u8 mode;
> +
> + if (std = V4L2_STD_ALL) {
> + /* enable PAL/NTSC autodetection */
> + ret = ml86v7667_mask_set(client, MRC_REG,
> + MRC_AUTOSELECT, MRC_AUTOSELECT);
> + priv->autodetect = true;
> + } else {
> + /* PAL/NTSC ITU-R BT.601 input mode */
> + mode = std & V4L2_STD_525_60 ? MRA_NTSC_BT601 : MRA_PAL_BT601;
> + ret = ml86v7667_mask_set(client, MRA_REG,
> + MRA_INPUT_MODE_MASK, mode);
> + if (ret < 0)
> + return ret;
> +
> + /* disable PAL/NTSC autodetection */
> + ret = ml86v7667_mask_set(client, MRC_REG, MRC_AUTOSELECT, 0);
> + if (ret < 0)
> + return ret;
> +
> + priv->std = std;
> + priv->autodetect = false;
> + }
> +
> + return 0;
> +}
> +
> +#ifdef CONFIG_VIDEO_ADV_DEBUG
> +static int ml86v7667_g_register(struct v4l2_subdev *sd,
> + struct v4l2_dbg_register *reg)
> +{
> + struct i2c_client *client = v4l2_get_subdevdata(sd);
> + int ret;
> +
> + if (!v4l2_chip_match_i2c_client(client, ®->match))
> + return -EINVAL;
> + if (!capable(CAP_SYS_ADMIN))
> + return -EPERM;
> +
> + ret = i2c_smbus_read_byte_data(client, (u8)reg->reg);
> + if (ret < 0)
> + return ret;
> +
> + reg->val = ret;
> + reg->size = sizeof(u8);
> +
> + return 0;
> +}
> +
> +static int ml86v7667_s_register(struct v4l2_subdev *sd,
> + struct v4l2_dbg_register *reg)
> +{
> + struct i2c_client *client = v4l2_get_subdevdata(sd);
> +
> + if (!v4l2_chip_match_i2c_client(client, ®->match))
> + return -EINVAL;
> + if (!capable(CAP_SYS_ADMIN))
> + return -EPERM;
> +
> + return i2c_smbus_write_byte_data(client, (u8)reg->reg, (u8)reg->val);
> +}
> +#endif
> +
> +static const struct v4l2_ctrl_ops ml86v7667_ctrl_ops = {
> + .s_ctrl = ml86v7667_s_ctrl,
> +};
> +
> +static struct v4l2_subdev_video_ops ml86v7667_subdev_video_ops = {
> + .querystd = ml86v7667_querystd,
> + .g_input_status = ml86v7667_g_input_status,
> + .enum_mbus_fmt = ml86v7667_enum_mbus_fmt,
> + .try_mbus_fmt = ml86v7667_try_mbus_fmt,
> + .g_mbus_fmt = ml86v7667_g_mbus_fmt,
> + .s_mbus_fmt = ml86v7667_s_mbus_fmt,
> + .cropcap = ml86v7667_cropcap,
> + .g_mbus_config = ml86v7667_g_mbus_config,
> +};
> +
> +static struct v4l2_subdev_core_ops ml86v7667_subdev_core_ops = {
> + .g_chip_ident = ml86v7667_g_chip_ident,
> + .s_std = ml86v7667_s_std,
> +#ifdef CONFIG_VIDEO_ADV_DEBUG
> + .g_register = ml86v7667_g_register,
> + .s_register = ml86v7667_s_register,
> +#endif
> +};
> +
> +static struct v4l2_subdev_ops ml86v7667_subdev_ops = {
> + .core = &ml86v7667_subdev_core_ops,
> + .video = &ml86v7667_subdev_video_ops,
> +};
> +
> +static int ml86v7667_init(struct ml86v7667_priv *priv)
> +{
> + struct i2c_client *client = v4l2_get_subdevdata(&priv->sd);
> + int ret;
> +
> + /* BT.656-4 output mode, register mode */
> + ret = ml86v7667_mask_set(client, MRA_REG,
> + MRA_OUTPUT_MODE_MASK | MRA_REGISTER_MODE,
> + MRA_ITUR_BT656 | MRA_REGISTER_MODE);
> +
> + /* PLL circuit fixed clock, 32MHz */
> + ret |= ml86v7667_mask_set(client, PLLR1_REG, PLLR1_FIXED_CLOCK,
> + PLLR1_FIXED_CLOCK);
> +
> + /* ADC2 clamping voltage maximum */
> + ret |= ml86v7667_mask_set(client, ADC2_REG, ADC2_CLAMP_VOLTAGE_MASK,
> + ADC2_CLAMP_VOLTAGE(7));
> +
> + /* enable luminance function */
> + ret |= ml86v7667_mask_set(client, SSEPL_REG, SSEPL_LUMINANCE_ONOFF,
> + SSEPL_LUMINANCE_ONOFF);
> +
> + /* enable contrast function */
> + ret |= ml86v7667_mask_set(client, CLC_REG, CLC_CONTRAST_ONOFF, 0);
> +
> + /* NTS/PAL autodetection enabled by default */
> + priv->autodetect = true;
> +
> + return ret;
> +}
> +
> +static int ml86v7667_probe(struct i2c_client *client,
> + const struct i2c_device_id *did)
> +{
> + struct ml86v7667_priv *priv;
> + int ret;
> +
> + if (!i2c_check_functionality(client->adapter, I2C_FUNC_SMBUS_BYTE_DATA))
> + return -EIO;
> +
> + priv = devm_kzalloc(&client->dev, sizeof(*priv), GFP_KERNEL);
> + if (!priv)
> + return -ENOMEM;
> +
> + v4l2_i2c_subdev_init(&priv->sd, client, &ml86v7667_subdev_ops);
> +
> + v4l2_ctrl_handler_init(&priv->hdl, 8);
> + v4l2_ctrl_new_std(&priv->hdl, &ml86v7667_ctrl_ops,
> + V4L2_CID_BRIGHTNESS, -64, 63, 1, 0);
> + v4l2_ctrl_new_std(&priv->hdl, &ml86v7667_ctrl_ops,
> + V4L2_CID_CONTRAST, -8, 7, 1, 0);
> + v4l2_ctrl_new_std(&priv->hdl, &ml86v7667_ctrl_ops,
> + V4L2_CID_CHROMA_GAIN, -32, 31, 1, 0);
> + v4l2_ctrl_new_std(&priv->hdl, &ml86v7667_ctrl_ops,
> + V4L2_CID_HUE, -128, 127, 1, 0);
> + v4l2_ctrl_new_std(&priv->hdl, &ml86v7667_ctrl_ops,
> + V4L2_CID_RED_BALANCE, -4, 3, 1, 0);
> + v4l2_ctrl_new_std(&priv->hdl, &ml86v7667_ctrl_ops,
> + V4L2_CID_BLUE_BALANCE, -4, 3, 1, 0);
> + v4l2_ctrl_new_std(&priv->hdl, &ml86v7667_ctrl_ops,
> + V4L2_CID_SHARPNESS, 0, 1, 1, 0);
> + v4l2_ctrl_new_std(&priv->hdl, &ml86v7667_ctrl_ops,
> + V4L2_CID_COLOR_KILLER, 0, 1, 1, 0);
> + priv->sd.ctrl_handler = &priv->hdl;
> +
> + ret = priv->hdl.error;
> + if (ret)
> + goto cleanup;
> +
> + v4l2_ctrl_handler_setup(&priv->hdl);
> +
> + ret = ml86v7667_init(priv);
> + if (ret)
> + goto cleanup;
> +
> + v4l_info(client, "chip found @ 0x%02x (%s)\n",
> + client->addr, client->adapter->name);
> + return 0;
> +
> +cleanup:
> + v4l2_ctrl_handler_free(&priv->hdl);
> + v4l2_device_unregister_subdev(&priv->sd);
> + v4l_err(client, "failed to probe @ 0x%02x (%s)\n",
> + client->addr, client->adapter->name);
> + return ret;
> +}
> +
> +static int ml86v7667_remove(struct i2c_client *client)
> +{
> + struct v4l2_subdev *sd = i2c_get_clientdata(client);
> + struct ml86v7667_priv *priv = to_ml86v7667(sd);
> +
> + v4l2_ctrl_handler_free(&priv->hdl);
> + v4l2_device_unregister_subdev(&priv->sd);
> +
> + return 0;
> +}
> +
> +static const struct i2c_device_id ml86v7667_id[] = {
> + {DRV_NAME, 0},
> + {},
> +};
> +MODULE_DEVICE_TABLE(i2c, ml86v7667_id);
> +
> +static struct i2c_driver ml86v7667_i2c_driver = {
> + .driver = {
> + .name = DRV_NAME,
> + .owner = THIS_MODULE,
> + },
> + .probe = ml86v7667_probe,
> + .remove = ml86v7667_remove,
> + .id_table = ml86v7667_id,
> +};
> +
> +module_i2c_driver(ml86v7667_i2c_driver);
> +
> +MODULE_DESCRIPTION("OKI Semiconductor ML86V7667 video decoder driver");
> +MODULE_AUTHOR("Vladimir Barinov");
> +MODULE_LICENSE("GPL");
> --
> 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
>
Regards,
Hans
^ permalink raw reply [flat|nested] 28+ messages in thread* Re: [PATCH 1/5] V4L2: I2C: ML86V7667 video decoder driver
@ 2013-04-20 21:50 ` Hans Verkuil
0 siblings, 0 replies; 28+ messages in thread
From: Hans Verkuil @ 2013-04-20 21:50 UTC (permalink / raw)
To: Sergei Shtylyov; +Cc: mchehab, linux-media, linux-sh, matsu, vladimir.barinov
Here's a quick review:
On Sat April 20 2013 22:16:33 Sergei Shtylyov wrote:
> From: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
>
> Add OKI Semiconductor ML86V7667 video decoder driver.
>
> Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
> [Sergei: added v4l2_device_unregister_subdev() call to the error cleanup path of
> ml86v7667_probe(); some cleanup.]
> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
>
> ---
> drivers/media/i2c/Kconfig | 9
> drivers/media/i2c/Makefile | 1
> drivers/media/i2c/ml86v7667.c | 504 ++++++++++++++++++++++++++++++++++++++++++
> 3 files changed, 514 insertions(+)
>
> Index: renesas/drivers/media/i2c/Kconfig
> ===================================================================
> --- renesas.orig/drivers/media/i2c/Kconfig
> +++ renesas/drivers/media/i2c/Kconfig
> @@ -227,6 +227,15 @@ config VIDEO_KS0127
> To compile this driver as a module, choose M here: the
> module will be called ks0127.
>
> +config VIDEO_ML86V7667
> + tristate "OKI ML86V7667 video decoder"
> + depends on VIDEO_V4L2 && I2C
> + ---help---
> + Support for the OKI Semiconductor ML86V7667 video decoder.
> +
> + To compile this driver as a module, choose M here: the
> + module will be called ml86v7667.
> +
> config VIDEO_SAA7110
> tristate "Philips SAA7110 video decoder"
> depends on VIDEO_V4L2 && I2C
> Index: renesas/drivers/media/i2c/Makefile
> ===================================================================
> --- renesas.orig/drivers/media/i2c/Makefile
> +++ renesas/drivers/media/i2c/Makefile
> @@ -64,3 +64,4 @@ obj-$(CONFIG_VIDEO_AS3645A) += as3645a.o
> obj-$(CONFIG_VIDEO_SMIAPP_PLL) += smiapp-pll.o
> obj-$(CONFIG_VIDEO_AK881X) += ak881x.o
> obj-$(CONFIG_VIDEO_IR_I2C) += ir-kbd-i2c.o
> +obj-$(CONFIG_VIDEO_ML86V7667) += ml86v7667.o
> Index: renesas/drivers/media/i2c/ml86v7667.c
> ===================================================================
> --- /dev/null
> +++ renesas/drivers/media/i2c/ml86v7667.c
> @@ -0,0 +1,504 @@
> +/*
> + * OKI Semiconductor ML86V7667 video decoder driver
> + *
> + * Author: Vladimir Barinov <source@cogentembedded.com>
> + * Copyright (C) 2013 Cogent Embedded, Inc.
> + * Copyright (C) 2013 Renesas Solutions Corp.
> + *
> + * This program is free software; you can redistribute it and/or modify it
> + * under the terms of the GNU General Public License as published by the
> + * Free Software Foundation; either version 2 of the License, or (at your
> + * option) any later version.
> + */
> +
> +#include <linux/init.h>
> +#include <linux/module.h>
> +#include <linux/i2c.h>
> +#include <linux/slab.h>
> +#include <linux/videodev2.h>
> +#include <media/v4l2-chip-ident.h>
> +#include <media/v4l2-subdev.h>
> +#include <media/v4l2-device.h>
> +#include <media/v4l2-ioctl.h>
> +#include <media/v4l2-ctrls.h>
> +
> +#define DRV_NAME "ml86v7667"
> +
> +/* Subaddresses */
> +#define MRA_REG 0x00 /* Mode Register A */
> +#define MRC_REG 0x02 /* Mode Register C */
> +#define LUMC_REG 0x0C /* Luminance Control */
> +#define CLC_REG 0x10 /* Contrast level control */
> +#define SSEPL_REG 0x11 /* Sync separation level */
> +#define CHRCA_REG 0x12 /* Chrominance Control A */
> +#define ACCC_REG 0x14 /* ACC Loop filter & Chrominance control */
> +#define ACCRC_REG 0x15 /* ACC Reference level control */
> +#define HUE_REG 0x16 /* Hue control */
> +#define ADC2_REG 0x1F /* ADC Register 2 */
> +#define PLLR1_REG 0x20 /* PLL Register 1 */
> +#define STATUS_REG 0x2C /* STATUS Register */
> +
> +/* Mode Register A register bits */
> +#define MRA_OUTPUT_MODE_MASK (3 << 6)
> +#define MRA_ITUR_BT601 (1 << 6)
> +#define MRA_ITUR_BT656 (0 << 6)
> +#define MRA_INPUT_MODE_MASK (7 << 3)
> +#define MRA_PAL_BT601 (4 << 3)
> +#define MRA_NTSC_BT601 (0 << 3)
> +#define MRA_REGISTER_MODE (1 << 0)
> +
> +/* Mode Register C register bits */
> +#define MRC_AUTOSELECT (1 << 7)
> +
> +/* Luminance Control register bits */
> +#define LUMC_ONOFF_SHIFT 7
> +#define LUMC_ONOFF_MASK (1 << 7)
> +
> +/* Contrast level control register bits */
> +#define CLC_CONTRAST_ONOFF (1 << 7)
> +#define CLC_CONTRAST_MASK 0x0F
> +
> +/* Sync separation level register bits */
> +#define SSEPL_LUMINANCE_ONOFF (1 << 7)
> +#define SSEPL_LUMINANCE_MASK 0x7F
> +
> +/* Chrominance Control A register bits */
> +#define CHRCA_MODE_SHIFT 6
> +#define CHRCA_MODE_MASK (1 << 6)
> +
> +/* ACC Loop filter & Chrominance control register bits */
> +#define ACCC_CHROMA_CR_SHIFT 3
> +#define ACCC_CHROMA_CR_MASK (7 << 3)
> +#define ACCC_CHROMA_CB_SHIFT 0
> +#define ACCC_CHROMA_CB_MASK (7 << 3)
> +
> +/* ACC Reference level control register bits */
> +#define ACCRC_CHROMA_MASK 0xfc
> +#define ACCRC_CHROMA_SHIFT 2
> +
> +/* ADC Register 2 register bits */
> +#define ADC2_CLAMP_VOLTAGE_MASK (7 << 1)
> +#define ADC2_CLAMP_VOLTAGE(n) ((n & 7) << 1)
> +
> +/* PLL Register 1 register bits */
> +#define PLLR1_FIXED_CLOCK (1 << 7)
> +
> +/* STATUS Register register bits */
> +#define STATUS_HLOCK_DETECT (1 << 3)
> +#define STATUS_NTSCPAL (1 << 2)
> +
> +struct ml86v7667_priv {
> + struct v4l2_subdev sd;
> + struct v4l2_ctrl_handler hdl;
> + struct v4l2_mbus_framefmt fmt;
> + v4l2_std_id std;
> + bool autodetect;
Generally autodetect is a bad idea: changing from pal to ntsc on the fly can
cause problems further down the pipeline since the it might affect the amount
of data being DMA-ed. It's also non-standard behavior.
I recommend dropping autodetect support.
> +};
> +
> +static inline struct ml86v7667_priv *to_ml86v7667(struct v4l2_subdev *subdev)
> +{
> + return container_of(subdev, struct ml86v7667_priv, sd);
> +}
> +
> +static inline struct v4l2_subdev *to_sd(struct v4l2_ctrl *ctrl)
> +{
> + return &container_of(ctrl->handler, struct ml86v7667_priv, hdl)->sd;
> +}
> +
> +static int ml86v7667_mask_set(struct i2c_client *client, const u8 reg,
> + const u8 mask, const u8 data)
> +{
> + int val = i2c_smbus_read_byte_data(client, reg);
> + if (val < 0)
> + return val;
> +
> + val = (val & ~mask) | (data & mask);
> + return i2c_smbus_write_byte_data(client, reg, val);
> +}
> +
> +static int ml86v7667_s_ctrl(struct v4l2_ctrl *ctrl)
> +{
> + struct v4l2_subdev *sd = to_sd(ctrl);
> + struct i2c_client *client = v4l2_get_subdevdata(sd);
> + int ret = 0;
> +
> + switch (ctrl->id) {
> + case V4L2_CID_BRIGHTNESS:
> + ret = ml86v7667_mask_set(client, SSEPL_REG,
> + SSEPL_LUMINANCE_MASK, ctrl->val);
> + break;
> + case V4L2_CID_CONTRAST:
> + ret = ml86v7667_mask_set(client, CLC_REG,
> + CLC_CONTRAST_MASK, ctrl->val);
> + break;
> + case V4L2_CID_CHROMA_GAIN:
> + ret = ml86v7667_mask_set(client, ACCRC_REG, ACCRC_CHROMA_MASK,
> + ctrl->val << ACCRC_CHROMA_SHIFT);
> + break;
> + case V4L2_CID_HUE:
> + ret = ml86v7667_mask_set(client, HUE_REG, ~0, ctrl->val);
> + break;
> + case V4L2_CID_RED_BALANCE:
> + ret = ml86v7667_mask_set(client, ACCC_REG,
> + ACCC_CHROMA_CR_MASK,
> + ctrl->val << ACCC_CHROMA_CR_SHIFT);
> + break;
> + case V4L2_CID_BLUE_BALANCE:
> + ret = ml86v7667_mask_set(client, ACCC_REG,
> + ACCC_CHROMA_CB_MASK,
> + ctrl->val << ACCC_CHROMA_CB_SHIFT);
> + break;
> + case V4L2_CID_SHARPNESS:
> + ret = ml86v7667_mask_set(client, LUMC_REG,
> + LUMC_ONOFF_MASK,
> + ctrl->val << LUMC_ONOFF_SHIFT);
> + break;
> + case V4L2_CID_COLOR_KILLER:
> + ret = ml86v7667_mask_set(client, CHRCA_REG,
> + CHRCA_MODE_MASK,
> + ctrl->val << CHRCA_MODE_SHIFT);
> + break;
> + }
> +
> + return 0;
> +}
> +
> +static int ml86v7667_querystd(struct v4l2_subdev *sd, v4l2_std_id *std)
> +{
> + struct ml86v7667_priv *priv = to_ml86v7667(sd);
> + struct i2c_client *client = v4l2_get_subdevdata(sd);
> + int status_reg;
> +
> + if (!priv->autodetect) {
> + *std = priv->std;
> + return 0;
> + }
> +
> + status_reg = i2c_smbus_read_byte_data(client, STATUS_REG);
> + if (status_reg < 0)
> + return status_reg;
> +
> + if (status_reg & STATUS_HLOCK_DETECT)
> + *std = status_reg & STATUS_NTSCPAL ? V4L2_STD_PAL :
> + V4L2_STD_NTSC;
> + else
> + *std = V4L2_STD_UNKNOWN;
> +
> + return 0;
> +}
> +
> +static int ml86v7667_g_input_status(struct v4l2_subdev *sd, u32 *status)
> +{
> + struct i2c_client *client = v4l2_get_subdevdata(sd);
> + int status_reg;
> +
> + status_reg = i2c_smbus_read_byte_data(client, STATUS_REG);
> + if (status_reg < 0)
> + return status_reg;
> +
> + *status = status_reg & STATUS_HLOCK_DETECT ? 0 : V4L2_IN_ST_NO_SIGNAL;
> +
> + return 0;
> +}
> +
> +static int ml86v7667_enum_mbus_fmt(struct v4l2_subdev *sd, unsigned int index,
> + enum v4l2_mbus_pixelcode *code)
> +{
> + if (index > 0)
> + return -EINVAL;
> +
> + *code = V4L2_MBUS_FMT_YUYV8_2X8;
> +
> + return 0;
> +}
> +
> +static int ml86v7667_try_mbus_fmt(struct v4l2_subdev *sd,
> + struct v4l2_mbus_framefmt *fmt)
> +{
> + struct ml86v7667_priv *priv = to_ml86v7667(sd);
> +
> + ml86v7667_querystd(sd, &priv->std);
> +
> + fmt->code = V4L2_MBUS_FMT_YUYV8_2X8;
> + fmt->colorspace = V4L2_COLORSPACE_SMPTE170M;
> + fmt->field = V4L2_FIELD_INTERLACED;
> + fmt->width = 720;
> + fmt->height = priv->std & V4L2_STD_525_60 ? 480 : 576;
> +
> + return 0;
> +}
> +
> +static int ml86v7667_g_mbus_fmt(struct v4l2_subdev *sd,
> + struct v4l2_mbus_framefmt *fmt)
> +{
> + struct ml86v7667_priv *priv = to_ml86v7667(sd);
> +
> + *fmt = priv->fmt;
> +
> + return 0;
> +}
> +
> +static int ml86v7667_s_mbus_fmt(struct v4l2_subdev *sd,
> + struct v4l2_mbus_framefmt *fmt)
> +{
> + struct ml86v7667_priv *priv = to_ml86v7667(sd);
> +
> + ml86v7667_try_mbus_fmt(sd, fmt);
> + priv->fmt = *fmt;
> +
> + return 0;
> +}
> +
> +static int ml86v7667_cropcap(struct v4l2_subdev *sd, struct v4l2_cropcap *a)
> +{
> + struct ml86v7667_priv *priv = to_ml86v7667(sd);
> +
> + ml86v7667_querystd(sd, &priv->std);
Always use the current standard, never query the standard inside functions
like this. It's up to userspace to call QUERYSTD and then S_STD.
> +
> + a->bounds.left = 0;
> + a->bounds.top = 0;
> + a->bounds.width = 720;
> + a->bounds.height = priv->std & V4L2_STD_525_60 ? 480 : 576;
> + a->defrect = a->bounds;
> + a->type = V4L2_BUF_TYPE_VIDEO_CAPTURE;
> + a->pixelaspect.numerator = 1;
> + a->pixelaspect.denominator = 1;
> +
> + return 0;
> +}
> +
> +static int ml86v7667_g_mbus_config(struct v4l2_subdev *sd,
> + struct v4l2_mbus_config *cfg)
> +{
> + cfg->flags = V4L2_MBUS_MASTER | V4L2_MBUS_PCLK_SAMPLE_RISING |
> + V4L2_MBUS_DATA_ACTIVE_HIGH;
> + cfg->type = V4L2_MBUS_BT656;
> +
> + return 0;
> +}
> +
> +static int ml86v7667_g_chip_ident(struct v4l2_subdev *sd,
> + struct v4l2_dbg_chip_ident *id)
> +{
> + id->ident = V4L2_IDENT_UNKNOWN;
> + id->revision = 0;
> +
> + return 0;
> +}
Just drop this function, it's no longer needed and this op will be removed
in 3.11.
> +
> +static int ml86v7667_s_std(struct v4l2_subdev *sd, v4l2_std_id std)
> +{
> + struct ml86v7667_priv *priv = to_ml86v7667(sd);
> + struct i2c_client *client = v4l2_get_subdevdata(&priv->sd);
> + int ret;
> + u8 mode;
> +
> + if (std == V4L2_STD_ALL) {
> + /* enable PAL/NTSC autodetection */
> + ret = ml86v7667_mask_set(client, MRC_REG,
> + MRC_AUTOSELECT, MRC_AUTOSELECT);
> + priv->autodetect = true;
> + } else {
> + /* PAL/NTSC ITU-R BT.601 input mode */
> + mode = std & V4L2_STD_525_60 ? MRA_NTSC_BT601 : MRA_PAL_BT601;
> + ret = ml86v7667_mask_set(client, MRA_REG,
> + MRA_INPUT_MODE_MASK, mode);
> + if (ret < 0)
> + return ret;
> +
> + /* disable PAL/NTSC autodetection */
> + ret = ml86v7667_mask_set(client, MRC_REG, MRC_AUTOSELECT, 0);
> + if (ret < 0)
> + return ret;
> +
> + priv->std = std;
> + priv->autodetect = false;
> + }
> +
> + return 0;
> +}
> +
> +#ifdef CONFIG_VIDEO_ADV_DEBUG
> +static int ml86v7667_g_register(struct v4l2_subdev *sd,
> + struct v4l2_dbg_register *reg)
> +{
> + struct i2c_client *client = v4l2_get_subdevdata(sd);
> + int ret;
> +
> + if (!v4l2_chip_match_i2c_client(client, ®->match))
> + return -EINVAL;
> + if (!capable(CAP_SYS_ADMIN))
> + return -EPERM;
> +
> + ret = i2c_smbus_read_byte_data(client, (u8)reg->reg);
> + if (ret < 0)
> + return ret;
> +
> + reg->val = ret;
> + reg->size = sizeof(u8);
> +
> + return 0;
> +}
> +
> +static int ml86v7667_s_register(struct v4l2_subdev *sd,
> + struct v4l2_dbg_register *reg)
> +{
> + struct i2c_client *client = v4l2_get_subdevdata(sd);
> +
> + if (!v4l2_chip_match_i2c_client(client, ®->match))
> + return -EINVAL;
> + if (!capable(CAP_SYS_ADMIN))
> + return -EPERM;
> +
> + return i2c_smbus_write_byte_data(client, (u8)reg->reg, (u8)reg->val);
> +}
> +#endif
> +
> +static const struct v4l2_ctrl_ops ml86v7667_ctrl_ops = {
> + .s_ctrl = ml86v7667_s_ctrl,
> +};
> +
> +static struct v4l2_subdev_video_ops ml86v7667_subdev_video_ops = {
> + .querystd = ml86v7667_querystd,
> + .g_input_status = ml86v7667_g_input_status,
> + .enum_mbus_fmt = ml86v7667_enum_mbus_fmt,
> + .try_mbus_fmt = ml86v7667_try_mbus_fmt,
> + .g_mbus_fmt = ml86v7667_g_mbus_fmt,
> + .s_mbus_fmt = ml86v7667_s_mbus_fmt,
> + .cropcap = ml86v7667_cropcap,
> + .g_mbus_config = ml86v7667_g_mbus_config,
> +};
> +
> +static struct v4l2_subdev_core_ops ml86v7667_subdev_core_ops = {
> + .g_chip_ident = ml86v7667_g_chip_ident,
> + .s_std = ml86v7667_s_std,
> +#ifdef CONFIG_VIDEO_ADV_DEBUG
> + .g_register = ml86v7667_g_register,
> + .s_register = ml86v7667_s_register,
> +#endif
> +};
> +
> +static struct v4l2_subdev_ops ml86v7667_subdev_ops = {
> + .core = &ml86v7667_subdev_core_ops,
> + .video = &ml86v7667_subdev_video_ops,
> +};
> +
> +static int ml86v7667_init(struct ml86v7667_priv *priv)
> +{
> + struct i2c_client *client = v4l2_get_subdevdata(&priv->sd);
> + int ret;
> +
> + /* BT.656-4 output mode, register mode */
> + ret = ml86v7667_mask_set(client, MRA_REG,
> + MRA_OUTPUT_MODE_MASK | MRA_REGISTER_MODE,
> + MRA_ITUR_BT656 | MRA_REGISTER_MODE);
> +
> + /* PLL circuit fixed clock, 32MHz */
> + ret |= ml86v7667_mask_set(client, PLLR1_REG, PLLR1_FIXED_CLOCK,
> + PLLR1_FIXED_CLOCK);
> +
> + /* ADC2 clamping voltage maximum */
> + ret |= ml86v7667_mask_set(client, ADC2_REG, ADC2_CLAMP_VOLTAGE_MASK,
> + ADC2_CLAMP_VOLTAGE(7));
> +
> + /* enable luminance function */
> + ret |= ml86v7667_mask_set(client, SSEPL_REG, SSEPL_LUMINANCE_ONOFF,
> + SSEPL_LUMINANCE_ONOFF);
> +
> + /* enable contrast function */
> + ret |= ml86v7667_mask_set(client, CLC_REG, CLC_CONTRAST_ONOFF, 0);
> +
> + /* NTS/PAL autodetection enabled by default */
> + priv->autodetect = true;
> +
> + return ret;
> +}
> +
> +static int ml86v7667_probe(struct i2c_client *client,
> + const struct i2c_device_id *did)
> +{
> + struct ml86v7667_priv *priv;
> + int ret;
> +
> + if (!i2c_check_functionality(client->adapter, I2C_FUNC_SMBUS_BYTE_DATA))
> + return -EIO;
> +
> + priv = devm_kzalloc(&client->dev, sizeof(*priv), GFP_KERNEL);
> + if (!priv)
> + return -ENOMEM;
> +
> + v4l2_i2c_subdev_init(&priv->sd, client, &ml86v7667_subdev_ops);
> +
> + v4l2_ctrl_handler_init(&priv->hdl, 8);
> + v4l2_ctrl_new_std(&priv->hdl, &ml86v7667_ctrl_ops,
> + V4L2_CID_BRIGHTNESS, -64, 63, 1, 0);
> + v4l2_ctrl_new_std(&priv->hdl, &ml86v7667_ctrl_ops,
> + V4L2_CID_CONTRAST, -8, 7, 1, 0);
> + v4l2_ctrl_new_std(&priv->hdl, &ml86v7667_ctrl_ops,
> + V4L2_CID_CHROMA_GAIN, -32, 31, 1, 0);
> + v4l2_ctrl_new_std(&priv->hdl, &ml86v7667_ctrl_ops,
> + V4L2_CID_HUE, -128, 127, 1, 0);
> + v4l2_ctrl_new_std(&priv->hdl, &ml86v7667_ctrl_ops,
> + V4L2_CID_RED_BALANCE, -4, 3, 1, 0);
> + v4l2_ctrl_new_std(&priv->hdl, &ml86v7667_ctrl_ops,
> + V4L2_CID_BLUE_BALANCE, -4, 3, 1, 0);
> + v4l2_ctrl_new_std(&priv->hdl, &ml86v7667_ctrl_ops,
> + V4L2_CID_SHARPNESS, 0, 1, 1, 0);
> + v4l2_ctrl_new_std(&priv->hdl, &ml86v7667_ctrl_ops,
> + V4L2_CID_COLOR_KILLER, 0, 1, 1, 0);
> + priv->sd.ctrl_handler = &priv->hdl;
> +
> + ret = priv->hdl.error;
> + if (ret)
> + goto cleanup;
> +
> + v4l2_ctrl_handler_setup(&priv->hdl);
> +
> + ret = ml86v7667_init(priv);
> + if (ret)
> + goto cleanup;
> +
> + v4l_info(client, "chip found @ 0x%02x (%s)\n",
> + client->addr, client->adapter->name);
> + return 0;
> +
> +cleanup:
> + v4l2_ctrl_handler_free(&priv->hdl);
> + v4l2_device_unregister_subdev(&priv->sd);
> + v4l_err(client, "failed to probe @ 0x%02x (%s)\n",
> + client->addr, client->adapter->name);
> + return ret;
> +}
> +
> +static int ml86v7667_remove(struct i2c_client *client)
> +{
> + struct v4l2_subdev *sd = i2c_get_clientdata(client);
> + struct ml86v7667_priv *priv = to_ml86v7667(sd);
> +
> + v4l2_ctrl_handler_free(&priv->hdl);
> + v4l2_device_unregister_subdev(&priv->sd);
> +
> + return 0;
> +}
> +
> +static const struct i2c_device_id ml86v7667_id[] = {
> + {DRV_NAME, 0},
> + {},
> +};
> +MODULE_DEVICE_TABLE(i2c, ml86v7667_id);
> +
> +static struct i2c_driver ml86v7667_i2c_driver = {
> + .driver = {
> + .name = DRV_NAME,
> + .owner = THIS_MODULE,
> + },
> + .probe = ml86v7667_probe,
> + .remove = ml86v7667_remove,
> + .id_table = ml86v7667_id,
> +};
> +
> +module_i2c_driver(ml86v7667_i2c_driver);
> +
> +MODULE_DESCRIPTION("OKI Semiconductor ML86V7667 video decoder driver");
> +MODULE_AUTHOR("Vladimir Barinov");
> +MODULE_LICENSE("GPL");
> --
> 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
>
Regards,
Hans
^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [PATCH 1/5] V4L2: I2C: ML86V7667 video decoder driver
2013-04-20 20:16 ` Sergei Shtylyov
@ 2013-04-20 21:59 ` Sergei Shtylyov
-1 siblings, 0 replies; 28+ messages in thread
From: Sergei Shtylyov @ 2013-04-20 21:59 UTC (permalink / raw)
To: mchehab, linux-media; +Cc: linux-sh, matsu, vladimir.barinov
Hello.
On 04/21/2013 12:16 AM, Sergei Shtylyov wrote:
> From: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
>
> Add OKI Semiconductor ML86V7667 video decoder driver.
>
> Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
> [Sergei: added v4l2_device_unregister_subdev() call to the error cleanup path of
> ml86v7667_probe(); some cleanup.]
> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
[...]
> Index: renesas/drivers/media/i2c/ml86v7667.c
> =================================> --- /dev/null
> +++ renesas/drivers/media/i2c/ml86v7667.c
> @@ -0,0 +1,504 @@
[...]
> +/* ACC Loop filter & Chrominance control register bits */
> +#define ACCC_CHROMA_CR_SHIFT 3
> +#define ACCC_CHROMA_CR_MASK (7 << 3)
> +#define ACCC_CHROMA_CB_SHIFT 0
> +#define ACCC_CHROMA_CB_MASK (7 << 3)
Should be (7 << 0), of course. My fault. :-(
WBR, Sergei
^ permalink raw reply [flat|nested] 28+ messages in thread* Re: [PATCH 1/5] V4L2: I2C: ML86V7667 video decoder driver
@ 2013-04-20 21:59 ` Sergei Shtylyov
0 siblings, 0 replies; 28+ messages in thread
From: Sergei Shtylyov @ 2013-04-20 21:59 UTC (permalink / raw)
To: mchehab, linux-media; +Cc: linux-sh, matsu, vladimir.barinov
Hello.
On 04/21/2013 12:16 AM, Sergei Shtylyov wrote:
> From: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
>
> Add OKI Semiconductor ML86V7667 video decoder driver.
>
> Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
> [Sergei: added v4l2_device_unregister_subdev() call to the error cleanup path of
> ml86v7667_probe(); some cleanup.]
> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
[...]
> Index: renesas/drivers/media/i2c/ml86v7667.c
> ===================================================================
> --- /dev/null
> +++ renesas/drivers/media/i2c/ml86v7667.c
> @@ -0,0 +1,504 @@
[...]
> +/* ACC Loop filter & Chrominance control register bits */
> +#define ACCC_CHROMA_CR_SHIFT 3
> +#define ACCC_CHROMA_CR_MASK (7 << 3)
> +#define ACCC_CHROMA_CB_SHIFT 0
> +#define ACCC_CHROMA_CB_MASK (7 << 3)
Should be (7 << 0), of course. My fault. :-(
WBR, Sergei
^ permalink raw reply [flat|nested] 28+ messages in thread