[media] v4l2-ctrls: use ptrs for all but the s32 type
authorHans Verkuil <hans.verkuil@cisco.com>
Sun, 27 Apr 2014 06:38:13 +0000 (03:38 -0300)
committerMauro Carvalho Chehab <m.chehab@samsung.com>
Thu, 17 Jul 2014 14:57:17 +0000 (11:57 -0300)
Rather than having two unions for all types just keep 'val' and
'cur.val' and use the p_cur and p_new unions to access all others.

The only reason for keeping 'val' and 'cur.val' is that it is used
all over, so converting this as well would be a huge job.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Documentation/video4linux/v4l2-controls.txt
drivers/media/i2c/mt9v032.c
drivers/media/i2c/smiapp/smiapp-core.c
drivers/media/pci/ivtv/ivtv-controls.c
drivers/media/platform/vivi.c
drivers/media/radio/si4713/si4713.c
drivers/media/v4l2-core/v4l2-ctrls.c
drivers/staging/media/solo6x10/solo6x10-v4l2-enc.c
include/media/v4l2-ctrls.h

index 06cf3ac83631cc760de1ddd75776644314293eac..c9ee9a7e403da5f28459a71c0ed5fe3138310d78 100644 (file)
@@ -362,7 +362,7 @@ will result in a deadlock since these helpers lock the handler as well.
 You can also take the handler lock yourself:
 
        mutex_lock(&state->ctrl_handler.lock);
-       printk(KERN_INFO "String value is '%s'\n", ctrl1->cur.string);
+       pr_info("String value is '%s'\n", ctrl1->p_cur.p_char);
        printk(KERN_INFO "Integer value is '%s'\n", ctrl2->cur.val);
        mutex_unlock(&state->ctrl_handler.lock);
 
index 65e6e1a9d1049d03d833ed559f3ebbaef00eb18c..435e9e661bbb6929551bbcae028ea3984fe61f0a 100644 (file)
@@ -667,7 +667,7 @@ static int mt9v032_s_ctrl(struct v4l2_ctrl *ctrl)
                        break;
 
                freq = mt9v032->pdata->link_freqs[mt9v032->link_freq->val];
-               mt9v032->pixel_rate->val64 = freq;
+               *mt9v032->pixel_rate->p_new.p_s64 = freq;
                mt9v032->sysclk = freq;
                break;
 
index 72350f0befc509c3e97c96ea42f1b69578662976..13411e9a0e25d3e5bda7a29aa4183c45966e4300 100644 (file)
@@ -297,8 +297,8 @@ static int smiapp_pll_update(struct smiapp_sensor *sensor)
        if (rval < 0)
                return rval;
 
-       sensor->pixel_rate_parray->cur.val64 = pll->vt_pix_clk_freq_hz;
-       sensor->pixel_rate_csi->cur.val64 = pll->pixel_rate_csi;
+       *sensor->pixel_rate_parray->p_cur.p_s64 = pll->vt_pix_clk_freq_hz;
+       *sensor->pixel_rate_csi->p_cur.p_s64 = pll->pixel_rate_csi;
 
        return 0;
 }
index c60424601cb91e3e38d9140a566967c346f34db9..2b0ab26e11e8e6a03d9d9505108ea55a5e0a0ee2 100644 (file)
@@ -135,8 +135,8 @@ static int ivtv_g_volatile_ctrl(struct v4l2_ctrl *ctrl)
        /* V4L2_CID_MPEG_VIDEO_DEC_PTS and V4L2_CID_MPEG_VIDEO_DEC_FRAME
           control cluster */
        case V4L2_CID_MPEG_VIDEO_DEC_PTS:
-               return ivtv_g_pts_frame(itv, &itv->ctrl_pts->val64,
-                                            &itv->ctrl_frame->val64);
+               return ivtv_g_pts_frame(itv, itv->ctrl_pts->p_new.p_s64,
+                                            itv->ctrl_frame->p_new.p_s64);
        }
        return 0;
 }
index cd535b38469d1fcbd15952f44a802acb4de8052f..7542b5dd99103319d0924f0b67482289b71e3748 100644 (file)
@@ -648,13 +648,13 @@ static void vivi_fillbuff(struct vivi_dev *dev, struct vivi_buffer *buf)
        gen_text(dev, vbuf, line++ * 16, 16, str);
        snprintf(str, sizeof(str), " int32 %d, int64 %lld, bitmask %08x ",
                        dev->int32->cur.val,
-                       dev->int64->cur.val64,
+                       *dev->int64->p_cur.p_s64,
                        dev->bitmask->cur.val);
        gen_text(dev, vbuf, line++ * 16, 16, str);
        snprintf(str, sizeof(str), " boolean %d, menu %s, string \"%s\" ",
                        dev->boolean->cur.val,
                        dev->menu->qmenu[dev->menu->cur.val],
-                       dev->string->cur.string);
+                       dev->string->p_cur.p_char);
        gen_text(dev, vbuf, line++ * 16, 16, str);
        snprintf(str, sizeof(str), " integer_menu %lld, value %d ",
                        dev->int_menu->qmenu_int[dev->int_menu->cur.val],
index 07d5153811e88b4fc3c12c2136bb46ea35cc42e4..dbe47261b5135d663293d4bba515baccd97d34c6 100644 (file)
@@ -1098,11 +1098,11 @@ static int si4713_s_ctrl(struct v4l2_ctrl *ctrl)
 
                switch (ctrl->id) {
                case V4L2_CID_RDS_TX_PS_NAME:
-                       ret = si4713_set_rds_ps_name(sdev, ctrl->string);
+                       ret = si4713_set_rds_ps_name(sdev, ctrl->p_new.p_char);
                        break;
 
                case V4L2_CID_RDS_TX_RADIO_TEXT:
-                       ret = si4713_set_rds_radio_text(sdev, ctrl->string);
+                       ret = si4713_set_rds_radio_text(sdev, ctrl->p_new.p_char);
                        break;
 
                case V4L2_CID_TUNE_ANTENNA_CAPACITOR:
index 1b4d37ce4afe84e92bf1827b83723e2f79e3f799..4b105713e54953766192faa6a081e4273ee2277f 100644 (file)
@@ -1135,7 +1135,7 @@ static void fill_event(struct v4l2_event *ev, struct v4l2_ctrl *ctrl, u32 change
        if (ctrl->is_ptr)
                ev->u.ctrl.value64 = 0;
        else
-               ev->u.ctrl.value64 = ctrl->cur.val64;
+               ev->u.ctrl.value64 = *ctrl->p_cur.p_s64;
        ev->u.ctrl.minimum = ctrl->minimum;
        ev->u.ctrl.maximum = ctrl->maximum;
        if (ctrl->type == V4L2_CTRL_TYPE_MENU
@@ -1801,7 +1801,9 @@ static struct v4l2_ctrl *v4l2_ctrl_new(struct v4l2_ctrl_handler *hdl,
                flags |= V4L2_CTRL_FLAG_WRITE_ONLY;
        else if (type == V4L2_CTRL_TYPE_CTRL_CLASS)
                flags |= V4L2_CTRL_FLAG_READ_ONLY;
-       else if (type == V4L2_CTRL_TYPE_STRING || type >= V4L2_CTRL_COMPOUND_TYPES)
+       else if (type == V4L2_CTRL_TYPE_INTEGER64 ||
+                type == V4L2_CTRL_TYPE_STRING ||
+                type >= V4L2_CTRL_COMPOUND_TYPES)
                sz_extra += 2 * elem_size;
 
        ctrl = kzalloc(sizeof(*ctrl) + sz_extra, GFP_KERNEL);
@@ -1833,10 +1835,10 @@ static struct v4l2_ctrl *v4l2_ctrl_new(struct v4l2_ctrl_handler *hdl,
                ctrl->qmenu_int = qmenu_int;
        ctrl->priv = priv;
        ctrl->cur.val = ctrl->val = def;
-       data = &ctrl->cur + 1;
+       data = &ctrl[1];
 
-       if (ctrl->is_ptr) {
-               ctrl->p = ctrl->p_new.p = data;
+       if (!ctrl->is_int) {
+               ctrl->p_new.p = data;
                ctrl->p_cur.p = data + elem_size;
        } else {
                ctrl->p_new.p = &ctrl->val;
@@ -3103,10 +3105,10 @@ int v4l2_ctrl_modify_range(struct v4l2_ctrl *ctrl,
        ctrl->maximum = max;
        ctrl->step = step;
        ctrl->default_value = def;
-       c.value = ctrl->cur.val;
+       c.value = *ctrl->p_cur.p_s32;
        if (validate_new(ctrl, &c))
                c.value = def;
-       if (c.value != ctrl->cur.val)
+       if (c.value != *ctrl->p_cur.p_s32)
                ret = set_ctrl(NULL, ctrl, &c, V4L2_EVENT_CTRL_CH_RANGE);
        else
                send_event(NULL, ctrl, V4L2_EVENT_CTRL_CH_RANGE);
index bb2604ead21679bc2601dead15cb40e0cf9f89a8..37f7fd090bc29e2c99e2aa73063baf805ecf5668 100644 (file)
@@ -1131,7 +1131,7 @@ static int solo_s_ctrl(struct v4l2_ctrl *ctrl)
                solo_motion_toggle(solo_enc, ctrl->val);
                return 0;
        case V4L2_CID_OSD_TEXT:
-               strcpy(solo_enc->osd_text, ctrl->string);
+               strcpy(solo_enc->osd_text, ctrl->p_new.p_char);
                err = solo_osd_print(solo_enc);
                return err;
        default:
index a38bd55909b2b16e7e8f4b7c3c00d2ceb0dbb64c..eb69c52e2f640631493ba8268947e2417c37ad74 100644 (file)
@@ -187,17 +187,9 @@ struct v4l2_ctrl {
        };
        unsigned long flags;
        void *priv;
-       union {
-               s32 val;
-               s64 val64;
-               char *string;
-               void *p;
-       };
-       union {
+       s32 val;
+       struct {
                s32 val;
-               s64 val64;
-               char *string;
-               void *p;
        } cur;
 
        union v4l2_ctrl_ptr p_new;