V4L/DVB (6220): TCM825x: Add support for vidioc_int_g_ifparm call, small cleanups

vidioc_int_g_ifparm returns platform-specific information about the
interface settings used by the sensor. Support for [gs]_ext_clk has
been removed.

Fix indentation and remove useless & characters.

Remove experiment for typechecking slave callback function arguments.

Signed-off-by: Sakari Ailus <sakari.ailus@nokia.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>

diff --git a/drivers/media/video/tcm825x.c b/drivers/media/video/tcm825x.c
index 3a139f288a9d8a25e393281b5317b1204ee19dee..41cd6a0b0485a29c44f3821d7bcc41338f6a1d89 100644 (file)
--- a/drivers/media/video/tcm825x.c
+++ b/drivers/media/video/tcm825x.c
@@ -470,46 +470,6 @@ static int tcm825x_configure(struct v4l2_int_device *s)
        return 0;
 }
 
-/*
- * Given the image capture format in pix, the nominal frame period in
- * timeperframe, calculate the required xclk frequency.
- *
- * TCM825X input frequency characteristics are:
- *     Minimum 11.9 MHz, Typical 24.57 MHz and maximum 25/27 MHz
- */
-#define XCLK_MIN 11900000
-#define XCLK_MAX 25000000
-
-static int ioctl_g_ext_clk(struct v4l2_int_device *s, u32 *xclk)
-{
-       struct tcm825x_sensor *sensor = s->priv;
-       struct v4l2_fract *timeperframe = &sensor->timeperframe;
-       u32 tgt_xclk;   /* target xclk */
-       u32 tgt_fps;    /* target frames per secound */
-
-       tgt_fps = timeperframe->denominator / timeperframe->numerator;
-
-       tgt_xclk = (tgt_fps <= HIGH_FPS_MODE_LOWER_LIMIT) ?
-               (2457 * tgt_fps) / MAX_HALF_FPS :
-               (2457 * tgt_fps) / MAX_FPS;
-       tgt_xclk *= 10000;
-
-       tgt_xclk = min(tgt_xclk, (u32)XCLK_MAX);
-       tgt_xclk = max(tgt_xclk, (u32)XCLK_MIN);
-
-       *xclk = tgt_xclk;
-
-       return 0;
-}
-
-static int ioctl_s_ext_clk(struct v4l2_int_device *s, u32 xclk)
-{
-       if (xclk > XCLK_MAX || xclk < XCLK_MIN)
-               return -EINVAL;
-
-       return 0;
-}
-
 static int ioctl_queryctrl(struct v4l2_int_device *s,
                                struct v4l2_queryctrl *qc)
 {
@@ -756,6 +716,41 @@ static int ioctl_s_power(struct v4l2_int_device *s, int on)
        return sensor->platform_data->power_set(on);
 }
 
+/*
+ * Given the image capture format in pix, the nominal frame period in
+ * timeperframe, calculate the required xclk frequency.
+ *
+ * TCM825X input frequency characteristics are:
+ *     Minimum 11.9 MHz, Typical 24.57 MHz and maximum 25/27 MHz
+ */
+
+static int ioctl_g_ifparm(struct v4l2_int_device *s, struct v4l2_ifparm *p)
+{
+       struct tcm825x_sensor *sensor = s->priv;
+       struct v4l2_fract *timeperframe = &sensor->timeperframe;
+       u32 tgt_xclk;   /* target xclk */
+       u32 tgt_fps;    /* target frames per secound */
+       int rval;
+
+       rval = sensor->platform_data->ifparm(p);
+       if (rval)
+               return rval;
+
+       tgt_fps = timeperframe->denominator / timeperframe->numerator;
+
+       tgt_xclk = (tgt_fps <= HIGH_FPS_MODE_LOWER_LIMIT) ?
+               (2457 * tgt_fps) / MAX_HALF_FPS :
+               (2457 * tgt_fps) / MAX_FPS;
+       tgt_xclk *= 10000;
+
+       tgt_xclk = min(tgt_xclk, (u32)TCM825X_XCLK_MAX);
+       tgt_xclk = max(tgt_xclk, (u32)TCM825X_XCLK_MIN);
+
+       p->u.bt656.clock_curr = tgt_xclk;
+
+       return 0;
+}
+
 static int ioctl_g_needs_reset(struct v4l2_int_device *s, void *buf)
 {
        struct tcm825x_sensor *sensor = s->priv;
@@ -793,43 +788,39 @@ static int ioctl_dev_init(struct v4l2_int_device *s)
        return 0;
 }
 
-#define NUM_IOCTLS 17
-
-static struct v4l2_int_ioctl_desc tcm825x_ioctl_desc[NUM_IOCTLS] = {
+static struct v4l2_int_ioctl_desc tcm825x_ioctl_desc[] = {
        { vidioc_int_dev_init_num,
-         (v4l2_int_ioctl_func *)&ioctl_dev_init },
+         (v4l2_int_ioctl_func *)ioctl_dev_init },
        { vidioc_int_dev_exit_num,
-         (v4l2_int_ioctl_func *)&ioctl_dev_exit },
+         (v4l2_int_ioctl_func *)ioctl_dev_exit },
        { vidioc_int_s_power_num,
-         (v4l2_int_ioctl_func *)&ioctl_s_power },
-       { vidioc_int_g_ext_clk_num,
-         (v4l2_int_ioctl_func *)&ioctl_g_ext_clk },
-       { vidioc_int_s_ext_clk_num,
-         (v4l2_int_ioctl_func *)&ioctl_s_ext_clk },
+         (v4l2_int_ioctl_func *)ioctl_s_power },
+       { vidioc_int_g_ifparm_num,
+         (v4l2_int_ioctl_func *)ioctl_g_ifparm },
        { vidioc_int_g_needs_reset_num,
-         (v4l2_int_ioctl_func *)&ioctl_g_needs_reset },
+         (v4l2_int_ioctl_func *)ioctl_g_needs_reset },
        { vidioc_int_reset_num,
-         (v4l2_int_ioctl_func *)&ioctl_reset },
+         (v4l2_int_ioctl_func *)ioctl_reset },
        { vidioc_int_init_num,
-         (v4l2_int_ioctl_func *)&ioctl_init },
+         (v4l2_int_ioctl_func *)ioctl_init },
        { vidioc_int_enum_fmt_cap_num,
-         (v4l2_int_ioctl_func *)&ioctl_enum_fmt_cap },
+         (v4l2_int_ioctl_func *)ioctl_enum_fmt_cap },
        { vidioc_int_try_fmt_cap_num,
-         (v4l2_int_ioctl_func *)&ioctl_try_fmt_cap },
+         (v4l2_int_ioctl_func *)ioctl_try_fmt_cap },
        { vidioc_int_g_fmt_cap_num,
-         (v4l2_int_ioctl_func *)&ioctl_g_fmt_cap },
+         (v4l2_int_ioctl_func *)ioctl_g_fmt_cap },
        { vidioc_int_s_fmt_cap_num,
-         (v4l2_int_ioctl_func *)&ioctl_s_fmt_cap },
+         (v4l2_int_ioctl_func *)ioctl_s_fmt_cap },
        { vidioc_int_g_parm_num,
-         (v4l2_int_ioctl_func *)&ioctl_g_parm },
+         (v4l2_int_ioctl_func *)ioctl_g_parm },
        { vidioc_int_s_parm_num,
-         (v4l2_int_ioctl_func *)&ioctl_s_parm },
+         (v4l2_int_ioctl_func *)ioctl_s_parm },
        { vidioc_int_queryctrl_num,
-         (v4l2_int_ioctl_func *)&ioctl_queryctrl },
+         (v4l2_int_ioctl_func *)ioctl_queryctrl },
        { vidioc_int_g_ctrl_num,
-         (v4l2_int_ioctl_func *)&ioctl_g_ctrl },
+         (v4l2_int_ioctl_func *)ioctl_g_ctrl },
        { vidioc_int_s_ctrl_num,
-         (v4l2_int_ioctl_func *)&ioctl_s_ctrl },
+         (v4l2_int_ioctl_func *)ioctl_s_ctrl },
 };
 
 static struct v4l2_int_slave tcm825x_slave = {
@@ -894,16 +885,16 @@ static int __exit tcm825x_remove(struct i2c_client *client)
 }
 
 static struct i2c_driver tcm825x_i2c_driver = {
-       .driver = {
+       .driver = {
                .name = TCM825X_NAME,
        },
-       .probe = &tcm825x_probe,
-       .remove = __exit_p(&tcm825x_remove),
+       .probe  = tcm825x_probe,
+       .remove = __exit_p(tcm825x_remove),
 };
 
 static struct tcm825x_sensor tcm825x = {
        .timeperframe = {
-               .numerator = 1,
+               .numerator   = 1,
                .denominator = DEFAULT_FPS,
        },
 };
@@ -911,11 +902,6 @@ static struct tcm825x_sensor tcm825x = {
 static int __init tcm825x_init(void)
 {
        int rval;
-       int i = 0;
-
-       /* Just an experiment --- don't use *_cb functions yet. */
-       tcm825x_ioctl_desc[i++] = vidioc_int_dev_init_cb(&ioctl_dev_init);
-       BUG_ON(i >= NUM_IOCTLS);
 
        rval = i2c_add_driver(&tcm825x_i2c_driver);
        if (rval)

diff --git a/drivers/media/video/tcm825x.h b/drivers/media/video/tcm825x.h
index d6471ec40bc51f010d89f1551de92c89b7097c9a..966765b66b3ad89de2913a6469ed9f9cc944e163 100644 (file)
--- a/drivers/media/video/tcm825x.h
+++ b/drivers/media/video/tcm825x.h
@@ -163,6 +163,9 @@ enum pixel_format { YUV422 = 0, RGB565 };
 #define NUM_IMAGE_SIZES 6
 #define NUM_PIXEL_FORMATS 2
 
+#define TCM825X_XCLK_MIN       11900000
+#define TCM825X_XCLK_MAX       25000000
+
 struct capture_size {
        unsigned long width;
        unsigned long height;
@@ -178,6 +181,7 @@ struct tcm825x_platform_data {
        const struct tcm825x_reg *(*default_regs)(void);
        int (*needs_reset)(struct v4l2_int_device *s, void *buf,
                           struct v4l2_pix_format *fmt);
+       int (*ifparm)(struct v4l2_ifparm *p);
 };
 
 /* Array of image sizes supported by TCM825X.  These must be ordered from