[media] v4l2: pass std by value to the write-only s_std ioctl
authorHans Verkuil <hans.verkuil@cisco.com>
Fri, 15 Mar 2013 09:10:40 +0000 (06:10 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Sun, 24 Mar 2013 09:47:00 +0000 (06:47 -0300)
This ioctl is defined as IOW, so pass the argument by value instead of by
reference. I could have chosen to add const instead, but this is 1) easier
to handle in drivers and 2) consistent with the s_std subdev operation.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Acked-by: Jonathan Corbet <corbet@lwn.net>
Acked-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Acked-by: Lad, Prabhakar <prabhakar.csengg@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
56 files changed:
drivers/media/common/saa7146/saa7146_video.c
drivers/media/parport/pms.c
drivers/media/pci/bt8xx/bttv-driver.c
drivers/media/pci/cx18/cx18-driver.c
drivers/media/pci/cx18/cx18-ioctl.c
drivers/media/pci/cx18/cx18-ioctl.h
drivers/media/pci/cx23885/cx23885-417.c
drivers/media/pci/cx23885/cx23885-video.c
drivers/media/pci/cx25821/cx25821-video.c
drivers/media/pci/cx25821/cx25821-video.h
drivers/media/pci/cx88/cx88-blackbird.c
drivers/media/pci/cx88/cx88-video.c
drivers/media/pci/ivtv/ivtv-driver.c
drivers/media/pci/ivtv/ivtv-firmware.c
drivers/media/pci/ivtv/ivtv-ioctl.c
drivers/media/pci/ivtv/ivtv-ioctl.h
drivers/media/pci/saa7134/saa7134-empress.c
drivers/media/pci/saa7134/saa7134-video.c
drivers/media/pci/saa7134/saa7134.h
drivers/media/pci/saa7164/saa7164-encoder.c
drivers/media/pci/saa7164/saa7164-vbi.c
drivers/media/pci/sta2x11/sta2x11_vip.c
drivers/media/pci/zoran/zoran_driver.c
drivers/media/platform/blackfin/bfin_capture.c
drivers/media/platform/davinci/vpbe.c
drivers/media/platform/davinci/vpbe_display.c
drivers/media/platform/davinci/vpfe_capture.c
drivers/media/platform/davinci/vpif_capture.c
drivers/media/platform/davinci/vpif_display.c
drivers/media/platform/fsl-viu.c
drivers/media/platform/marvell-ccic/mcam-core.c
drivers/media/platform/s5p-tv/mixer_video.c
drivers/media/platform/sh_vou.c
drivers/media/platform/soc_camera/soc_camera.c
drivers/media/platform/timblogiw.c
drivers/media/platform/via-camera.c
drivers/media/platform/vino.c
drivers/media/usb/au0828/au0828-video.c
drivers/media/usb/cx231xx/cx231xx-417.c
drivers/media/usb/cx231xx/cx231xx-video.c
drivers/media/usb/em28xx/em28xx-video.c
drivers/media/usb/hdpvr/hdpvr-video.c
drivers/media/usb/pvrusb2/pvrusb2-v4l2.c
drivers/media/usb/s2255/s2255drv.c
drivers/media/usb/stk1160/stk1160-v4l.c
drivers/media/usb/tlg2300/pd-video.c
drivers/media/usb/tm6000/tm6000-video.c
drivers/media/usb/usbvision/usbvision-video.c
drivers/media/v4l2-core/v4l2-ioctl.c
drivers/staging/media/davinci_vpfe/vpfe_video.c
drivers/staging/media/dt3155v4l/dt3155v4l.c
drivers/staging/media/go7007/go7007-v4l2.c
drivers/staging/media/solo6x10/v4l2-enc.c
drivers/staging/media/solo6x10/v4l2.c
include/media/davinci/vpbe.h
include/media/v4l2-ioctl.h

index 4143d61..fe907f2 100644 (file)
@@ -832,7 +832,7 @@ static int vidioc_g_std(struct file *file, void *fh, v4l2_std_id *norm)
        }
        */
 
-static int vidioc_s_std(struct file *file, void *fh, v4l2_std_id *id)
+static int vidioc_s_std(struct file *file, void *fh, v4l2_std_id id)
 {
        struct saa7146_dev *dev = ((struct saa7146_fh *)fh)->dev;
        struct saa7146_vv *vv = dev->vv_data;
@@ -856,7 +856,7 @@ static int vidioc_s_std(struct file *file, void *fh, v4l2_std_id *id)
        }
 
        for (i = 0; i < dev->ext_vv_data->num_stds; i++)
-               if (*id & dev->ext_vv_data->stds[i].id)
+               if (id & dev->ext_vv_data->stds[i].id)
                        break;
        if (i != dev->ext_vv_data->num_stds) {
                vv->standard = &dev->ext_vv_data->stds[i];
index 77f9c92..66c957a 100644 (file)
@@ -735,12 +735,12 @@ static int pms_g_std(struct file *file, void *fh, v4l2_std_id *std)
        return 0;
 }
 
-static int pms_s_std(struct file *file, void *fh, v4l2_std_id *std)
+static int pms_s_std(struct file *file, void *fh, v4l2_std_id std)
 {
        struct pms *dev = video_drvdata(file);
        int ret = 0;
 
-       dev->std = *std;
+       dev->std = std;
        if (dev->std & V4L2_STD_NTSC) {
                pms_framerate(dev, 30);
                pms_secamcross(dev, 0);
index 7aea24b..4b9a256 100644 (file)
@@ -1713,7 +1713,7 @@ static void radio_enable(struct bttv *btv)
        }
 }
 
-static int bttv_s_std(struct file *file, void *priv, v4l2_std_id *id)
+static int bttv_s_std(struct file *file, void *priv, v4l2_std_id id)
 {
        struct bttv_fh *fh  = priv;
        struct bttv *btv = fh->btv;
@@ -1721,14 +1721,14 @@ static int bttv_s_std(struct file *file, void *priv, v4l2_std_id *id)
        int err = 0;
 
        for (i = 0; i < BTTV_TVNORMS; i++)
-               if (*id & bttv_tvnorms[i].v4l2_id)
+               if (id & bttv_tvnorms[i].v4l2_id)
                        break;
        if (i == BTTV_TVNORMS) {
                err = -EINVAL;
                goto err;
        }
 
-       btv->std = *id;
+       btv->std = id;
        set_tvnorm(btv, i);
 
 err:
index 613e5ae..67b61cf 100644 (file)
@@ -1243,7 +1243,7 @@ int cx18_init_on_first_open(struct cx18 *cx)
           in one place. */
        cx->std++;              /* Force full standard initialization */
        std = (cx->tuner_std == V4L2_STD_ALL) ? V4L2_STD_NTSC_M : cx->tuner_std;
-       cx18_s_std(NULL, &fh, &std);
+       cx18_s_std(NULL, &fh, std);
        cx18_s_frequency(NULL, &fh, &vf);
        return 0;
 }
index 173ccd2..254c50f 100644 (file)
@@ -637,15 +637,15 @@ static int cx18_g_std(struct file *file, void *fh, v4l2_std_id *std)
        return 0;
 }
 
-int cx18_s_std(struct file *file, void *fh, v4l2_std_id *std)
+int cx18_s_std(struct file *file, void *fh, v4l2_std_id std)
 {
        struct cx18_open_id *id = fh2id(fh);
        struct cx18 *cx = id->cx;
 
-       if ((*std & V4L2_STD_ALL) == 0)
+       if ((std & V4L2_STD_ALL) == 0)
                return -EINVAL;
 
-       if (*std == cx->std)
+       if (std == cx->std)
                return 0;
 
        if (test_bit(CX18_F_I_RADIO_USER, &cx->i_flags) ||
@@ -656,8 +656,8 @@ int cx18_s_std(struct file *file, void *fh, v4l2_std_id *std)
                return -EBUSY;
        }
 
-       cx->std = *std;
-       cx->is_60hz = (*std & V4L2_STD_525_60) ? 1 : 0;
+       cx->std = std;
+       cx->is_60hz = (std & V4L2_STD_525_60) ? 1 : 0;
        cx->is_50hz = !cx->is_60hz;
        cx2341x_handler_set_50hz(&cx->cxhdl, cx->is_50hz);
        cx->cxhdl.width = 720;
index aa9b44a..4343396 100644 (file)
@@ -26,6 +26,6 @@ u16 cx18_service2vbi(int type);
 void cx18_expand_service_set(struct v4l2_sliced_vbi_format *fmt, int is_pal);
 u16 cx18_get_service_set(struct v4l2_sliced_vbi_format *fmt);
 void cx18_set_funcs(struct video_device *vdev);
-int cx18_s_std(struct file *file, void *fh, v4l2_std_id *std);
+int cx18_s_std(struct file *file, void *fh, v4l2_std_id std);
 int cx18_s_frequency(struct file *file, void *fh, const struct v4l2_frequency *vf);
 int cx18_s_input(struct file *file, void *fh, unsigned int inp);
index 812ec5f..6dea11a 100644 (file)
@@ -1222,14 +1222,14 @@ static int vidioc_g_std(struct file *file, void *priv, v4l2_std_id *id)
        return 0;
 }
 
-static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id *id)
+static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id id)
 {
        struct cx23885_fh  *fh  = file->private_data;
        struct cx23885_dev *dev = fh->dev;
        unsigned int i;
 
        for (i = 0; i < ARRAY_SIZE(cx23885_tvnorms); i++)
-               if (*id & cx23885_tvnorms[i].id)
+               if (id & cx23885_tvnorms[i].id)
                        break;
        if (i == ARRAY_SIZE(cx23885_tvnorms))
                return -EINVAL;
@@ -1237,7 +1237,7 @@ static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id *id)
 
        /* Have the drier core notify the subdevices */
        mutex_lock(&dev->lock);
-       cx23885_set_tvnorm(dev, *id);
+       cx23885_set_tvnorm(dev, id);
        mutex_unlock(&dev->lock);
 
        return 0;
index 2bbda43..ed08c89 100644 (file)
@@ -1259,13 +1259,13 @@ static int vidioc_g_std(struct file *file, void *priv, v4l2_std_id *id)
        return 0;
 }
 
-static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id *tvnorms)
+static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id tvnorms)
 {
        struct cx23885_dev *dev = ((struct cx23885_fh *)priv)->dev;
        dprintk(1, "%s()\n", __func__);
 
        mutex_lock(&dev->lock);
-       cx23885_set_tvnorm(dev, *tvnorms);
+       cx23885_set_tvnorm(dev, tvnorms);
        mutex_unlock(&dev->lock);
 
        return 0;
index 75281c1..93c7d57 100644 (file)
@@ -1203,7 +1203,7 @@ int cx25821_vidioc_s_priority(struct file *file, void *f,
 }
 
 #ifdef TUNER_FLAG
-int cx25821_vidioc_s_std(struct file *file, void *priv, v4l2_std_id tvnorms)
+int cx25821_vidioc_s_std(struct file *file, void *priv, v4l2_std_id tvnorms)
 {
        struct cx25821_fh *fh = priv;
        struct cx25821_dev *dev = ((struct cx25821_fh *)priv)->dev;
@@ -1218,11 +1218,11 @@ int cx25821_vidioc_s_std(struct file *file, void *priv, v4l2_std_id * tvnorms)
                        return err;
        }
 
-       if (dev->tvnorm == *tvnorms)
+       if (dev->tvnorm == tvnorms)
                return 0;
 
        mutex_lock(&dev->lock);
-       cx25821_set_tvnorm(dev, *tvnorms);
+       cx25821_set_tvnorm(dev, tvnorms);
        mutex_unlock(&dev->lock);
 
        medusa_set_videostandard(dev);
index f0e70ff..239f63c 100644 (file)
@@ -135,7 +135,7 @@ extern int cx25821_vidioc_querybuf(struct file *file, void *priv,
 extern int cx25821_vidioc_qbuf(struct file *file, void *priv,
                               struct v4l2_buffer *p);
 extern int cx25821_vidioc_s_std(struct file *file, void *priv,
-                               v4l2_std_id *tvnorms);
+                               v4l2_std_id tvnorms);
 extern int cx25821_enum_input(struct cx25821_dev *dev, struct v4l2_input *i);
 extern int cx25821_vidioc_enum_input(struct file *file, void *priv,
                                     struct v4l2_input *i);
index 486ca8d..150bb76 100644 (file)
@@ -939,12 +939,12 @@ static int vidioc_g_std(struct file *file, void *priv, v4l2_std_id *tvnorm)
        return 0;
 }
 
-static int vidioc_s_std (struct file *file, void *priv, v4l2_std_id *id)
+static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id id)
 {
        struct cx88_core  *core = ((struct cx8802_fh *)priv)->dev->core;
 
        mutex_lock(&core->lock);
-       cx88_set_tvnorm(core,*id);
+       cx88_set_tvnorm(core, id);
        mutex_unlock(&core->lock);
        return 0;
 }
index ede6f13..ead5be5 100644 (file)
@@ -1193,12 +1193,12 @@ static int vidioc_g_std(struct file *file, void *priv, v4l2_std_id *tvnorm)
        return 0;
 }
 
-static int vidioc_s_std (struct file *file, void *priv, v4l2_std_id *tvnorms)
+static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id tvnorms)
 {
        struct cx88_core  *core = ((struct cx8800_fh *)priv)->dev->core;
 
        mutex_lock(&core->lock);
-       cx88_set_tvnorm(core,*tvnorms);
+       cx88_set_tvnorm(core, tvnorms);
        mutex_unlock(&core->lock);
 
        return 0;
index 2928e72..07b8460 100644 (file)
@@ -1387,7 +1387,7 @@ int ivtv_init_on_first_open(struct ivtv *itv)
        if (!itv->has_cx23415)
                write_reg_sync(0x03, IVTV_REG_DMACONTROL);
 
-       ivtv_s_std_enc(itv, &itv->tuner_std);
+       ivtv_s_std_enc(itv, itv->tuner_std);
 
        /* Default interrupts enabled. For the PVR350 this includes the
           decoder VSYNC interrupt, which is always on. It is not only used
@@ -1397,7 +1397,7 @@ int ivtv_init_on_first_open(struct ivtv *itv)
        if (itv->v4l2_cap & V4L2_CAP_VIDEO_OUTPUT) {
                ivtv_clear_irq_mask(itv, IVTV_IRQ_MASK_INIT | IVTV_IRQ_DEC_VSYNC);
                ivtv_set_osd_alpha(itv);
-               ivtv_s_std_dec(itv, &itv->tuner_std);
+               ivtv_s_std_dec(itv, itv->tuner_std);
        } else {
                ivtv_clear_irq_mask(itv, IVTV_IRQ_MASK_INIT);
        }
index 68387d4..ed73edd 100644 (file)
@@ -302,7 +302,7 @@ static int ivtv_firmware_restart(struct ivtv *itv)
        /* Restore encoder video standard */
        std = itv->std;
        itv->std = 0;
-       ivtv_s_std_enc(itv, &std);
+       ivtv_s_std_enc(itv, std);
 
        if (itv->v4l2_cap & V4L2_CAP_VIDEO_OUTPUT) {
                ivtv_init_mpeg_decoder(itv);
@@ -310,7 +310,7 @@ static int ivtv_firmware_restart(struct ivtv *itv)
                /* Restore decoder video standard */
                std = itv->std_out;
                itv->std_out = 0;
-               ivtv_s_std_dec(itv, &std);
+               ivtv_s_std_dec(itv, std);
 
                /* Restore framebuffer if active */
                if (itv->ivtvfb_restore)
index 852f11e..080f179 100644 (file)
@@ -1103,10 +1103,10 @@ static int ivtv_g_std(struct file *file, void *fh, v4l2_std_id *std)
        return 0;
 }
 
-void ivtv_s_std_enc(struct ivtv *itv, v4l2_std_id *std)
+void ivtv_s_std_enc(struct ivtv *itv, v4l2_std_id std)
 {
-       itv->std = *std;
-       itv->is_60hz = (*std & V4L2_STD_525_60) ? 1 : 0;
+       itv->std = std;
+       itv->is_60hz = (std & V4L2_STD_525_60) ? 1 : 0;
        itv->is_50hz = !itv->is_60hz;
        cx2341x_handler_set_50hz(&itv->cxhdl, itv->is_50hz);
        itv->cxhdl.width = 720;
@@ -1122,15 +1122,15 @@ void ivtv_s_std_enc(struct ivtv *itv, v4l2_std_id *std)
        ivtv_call_all(itv, core, s_std, itv->std);
 }
 
-void ivtv_s_std_dec(struct ivtv *itv, v4l2_std_id *std)
+void ivtv_s_std_dec(struct ivtv *itv, v4l2_std_id std)
 {
        struct yuv_playback_info *yi = &itv->yuv_info;
        DEFINE_WAIT(wait);
        int f;
 
        /* set display standard */
-       itv->std_out = *std;
-       itv->is_out_60hz = (*std & V4L2_STD_525_60) ? 1 : 0;
+       itv->std_out = std;
+       itv->is_out_60hz = (std & V4L2_STD_525_60) ? 1 : 0;
        itv->is_out_50hz = !itv->is_out_60hz;
        ivtv_call_all(itv, video, s_std_output, itv->std_out);
 
@@ -1168,14 +1168,14 @@ void ivtv_s_std_dec(struct ivtv *itv, v4l2_std_id *std)
        }
 }
 
-static int ivtv_s_std(struct file *file, void *fh, v4l2_std_id *std)
+static int ivtv_s_std(struct file *file, void *fh, v4l2_std_id std)
 {
        struct ivtv *itv = fh2id(fh)->itv;
 
-       if ((*std & V4L2_STD_ALL) == 0)
+       if ((std & V4L2_STD_ALL) == 0)
                return -EINVAL;
 
-       if (*std == itv->std)
+       if (std == itv->std)
                return 0;
 
        if (test_bit(IVTV_F_I_RADIO_USER, &itv->i_flags) ||
index 34c6bc1..75c3977 100644 (file)
@@ -27,8 +27,8 @@ u16 ivtv_get_service_set(struct v4l2_sliced_vbi_format *fmt);
 void ivtv_set_osd_alpha(struct ivtv *itv);
 int ivtv_set_speed(struct ivtv *itv, int speed);
 void ivtv_set_funcs(struct video_device *vdev);
-void ivtv_s_std_enc(struct ivtv *itv, v4l2_std_id *std);
-void ivtv_s_std_dec(struct ivtv *itv, v4l2_std_id *std);
+void ivtv_s_std_enc(struct ivtv *itv, v4l2_std_id std);
+void ivtv_s_std_dec(struct ivtv *itv, v4l2_std_id std);
 int ivtv_s_frequency(struct file *file, void *fh, const struct v4l2_frequency *vf);
 int ivtv_s_input(struct file *file, void *fh, unsigned int inp);
 
index 4df79c6..66a7081 100644 (file)
@@ -428,7 +428,7 @@ static int empress_g_chip_ident(struct file *file, void *fh,
        return -EINVAL;
 }
 
-static int empress_s_std(struct file *file, void *priv, v4l2_std_id *id)
+static int empress_s_std(struct file *file, void *priv, v4l2_std_id id)
 {
        struct saa7134_dev *dev = file->private_data;
 
index 1e23547..a6c69a4 100644 (file)
@@ -1828,7 +1828,7 @@ static int saa7134_querycap(struct file *file, void  *priv,
        return 0;
 }
 
-int saa7134_s_std_internal(struct saa7134_dev *dev, struct saa7134_fh *fh, v4l2_std_id *id)
+int saa7134_s_std_internal(struct saa7134_dev *dev, struct saa7134_fh *fh, v4l2_std_id id)
 {
        unsigned long flags;
        unsigned int i;
@@ -1849,17 +1849,17 @@ int saa7134_s_std_internal(struct saa7134_dev *dev, struct saa7134_fh *fh, v4l2_
        }
 
        for (i = 0; i < TVNORMS; i++)
-               if (*id == tvnorms[i].id)
+               if (id == tvnorms[i].id)
                        break;
 
        if (i == TVNORMS)
                for (i = 0; i < TVNORMS; i++)
-                       if (*id & tvnorms[i].id)
+                       if (id & tvnorms[i].id)
                                break;
        if (i == TVNORMS)
                return -EINVAL;
 
-       if ((*id & V4L2_STD_SECAM) && (secam[0] != '-')) {
+       if ((id & V4L2_STD_SECAM) && (secam[0] != '-')) {
                if (secam[0] == 'L' || secam[0] == 'l') {
                        if (secam[1] == 'C' || secam[1] == 'c')
                                fixup = V4L2_STD_SECAM_LC;
@@ -1879,7 +1879,7 @@ int saa7134_s_std_internal(struct saa7134_dev *dev, struct saa7134_fh *fh, v4l2_
                        return -EINVAL;
        }
 
-       *id = tvnorms[i].id;
+       id = tvnorms[i].id;
 
        mutex_lock(&dev->lock);
        if (fh && res_check(fh, RESOURCE_OVERLAY)) {
@@ -1901,7 +1901,7 @@ int saa7134_s_std_internal(struct saa7134_dev *dev, struct saa7134_fh *fh, v4l2_
 }
 EXPORT_SYMBOL_GPL(saa7134_s_std_internal);
 
-static int saa7134_s_std(struct file *file, void *priv, v4l2_std_id *id)
+static int saa7134_s_std(struct file *file, void *priv, v4l2_std_id id)
 {
        struct saa7134_fh *fh = priv;
 
@@ -2396,7 +2396,7 @@ static int radio_s_input(struct file *filp, void *priv, unsigned int i)
        return 0;
 }
 
-static int radio_s_std(struct file *file, void *fh, v4l2_std_id *norm)
+static int radio_s_std(struct file *file, void *fh, v4l2_std_id norm)
 {
        return 0;
 }
index 71eefef..62169dd 100644 (file)
@@ -766,7 +766,7 @@ extern struct video_device saa7134_radio_template;
 int saa7134_s_ctrl_internal(struct saa7134_dev *dev,  struct saa7134_fh *fh, struct v4l2_control *c);
 int saa7134_g_ctrl_internal(struct saa7134_dev *dev,  struct saa7134_fh *fh, struct v4l2_control *c);
 int saa7134_queryctrl(struct file *file, void *priv, struct v4l2_queryctrl *c);
-int saa7134_s_std_internal(struct saa7134_dev *dev,  struct saa7134_fh *fh, v4l2_std_id *id);
+int saa7134_s_std_internal(struct saa7134_dev *dev,  struct saa7134_fh *fh, v4l2_std_id id);
 
 int saa7134_videoport_init(struct saa7134_dev *dev);
 void saa7134_set_tvnorm_hw(struct saa7134_dev *dev);
index e7fbd03..538de52 100644 (file)
@@ -211,17 +211,17 @@ static int saa7164_encoder_initialize(struct saa7164_port *port)
 }
 
 /* -- V4L2 --------------------------------------------------------- */
-static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id *id)
+static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id id)
 {
        struct saa7164_encoder_fh *fh = file->private_data;
        struct saa7164_port *port = fh->port;
        struct saa7164_dev *dev = port->dev;
        unsigned int i;
 
-       dprintk(DBGLVL_ENC, "%s(id=0x%x)\n", __func__, (u32)*id);
+       dprintk(DBGLVL_ENC, "%s(id=0x%x)\n", __func__, (u32)id);
 
        for (i = 0; i < ARRAY_SIZE(saa7164_tvnorms); i++) {
-               if (*id & saa7164_tvnorms[i].id)
+               if (id & saa7164_tvnorms[i].id)
                        break;
        }
        if (i == ARRAY_SIZE(saa7164_tvnorms))
@@ -234,7 +234,7 @@ static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id *id)
         */
        saa7164_api_set_audio_std(port);
 
-       dprintk(DBGLVL_ENC, "%s(id=0x%x) OK\n", __func__, (u32)*id);
+       dprintk(DBGLVL_ENC, "%s(id=0x%x) OK\n", __func__, (u32)id);
 
        return 0;
 }
index bfc8cec..da224eb 100644 (file)
@@ -183,17 +183,17 @@ static int saa7164_vbi_initialize(struct saa7164_port *port)
 }
 
 /* -- V4L2 --------------------------------------------------------- */
-static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id *id)
+static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id id)
 {
        struct saa7164_vbi_fh *fh = file->private_data;
        struct saa7164_port *port = fh->port;
        struct saa7164_dev *dev = port->dev;
        unsigned int i;
 
-       dprintk(DBGLVL_VBI, "%s(id=0x%x)\n", __func__, (u32)*id);
+       dprintk(DBGLVL_VBI, "%s(id=0x%x)\n", __func__, (u32)id);
 
        for (i = 0; i < ARRAY_SIZE(saa7164_tvnorms); i++) {
-               if (*id & saa7164_tvnorms[i].id)
+               if (id & saa7164_tvnorms[i].id)
                        break;
        }
        if (i == ARRAY_SIZE(saa7164_tvnorms))
@@ -206,7 +206,7 @@ static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id *id)
         */
        saa7164_api_set_audio_std(port);
 
-       dprintk(DBGLVL_VBI, "%s(id=0x%x) OK\n", __func__, (u32)*id);
+       dprintk(DBGLVL_VBI, "%s(id=0x%x) OK\n", __func__, (u32)id);
 
        return 0;
 }
index 4b703fe..7005695 100644 (file)
@@ -439,22 +439,22 @@ static int vidioc_querycap(struct file *file, void *priv,
  *
  * other, returned from video DAC.
  */
-static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id *std)
+static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id std)
 {
        struct sta2x11_vip *vip = video_drvdata(file);
        v4l2_std_id oldstd = vip->std, newstd;
        int status;
 
-       if (V4L2_STD_ALL == *std) {
-               v4l2_subdev_call(vip->decoder, core, s_std, *std);
+       if (V4L2_STD_ALL == std) {
+               v4l2_subdev_call(vip->decoder, core, s_std, std);
                ssleep(2);
                v4l2_subdev_call(vip->decoder, video, querystd, &newstd);
                v4l2_subdev_call(vip->decoder, video, g_input_status, &status);
                if (status & V4L2_IN_ST_NO_SIGNAL)
                        return -EIO;
-               *std = vip->std = newstd;
-               if (oldstd != *std) {
-                       if (V4L2_STD_525_60 & (*std))
+               std = vip->std = newstd;
+               if (oldstd != std) {
+                       if (V4L2_STD_525_60 & std)
                                vip->format = formats_60[0];
                        else
                                vip->format = formats_50[0];
@@ -462,14 +462,14 @@ static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id *std)
                return 0;
        }
 
-       if (oldstd != *std) {
-               if (V4L2_STD_525_60 & (*std))
+       if (oldstd != std) {
+               if (V4L2_STD_525_60 & std)
                        vip->format = formats_60[0];
                else
                        vip->format = formats_50[0];
        }
 
-       return v4l2_subdev_call(vip->decoder, core, s_std, *std);
+       return v4l2_subdev_call(vip->decoder, core, s_std, std);
 }
 
 /**
index 2e8f518..1168a84 100644 (file)
@@ -2435,14 +2435,14 @@ static int zoran_g_std(struct file *file, void *__fh, v4l2_std_id *std)
        return 0;
 }
 
-static int zoran_s_std(struct file *file, void *__fh, v4l2_std_id *std)
+static int zoran_s_std(struct file *file, void *__fh, v4l2_std_id std)
 {
        struct zoran_fh *fh = __fh;
        struct zoran *zr = fh->zr;
        int res = 0;
 
        mutex_lock(&zr->resource_lock);
-       res = zoran_set_norm(zr, *std);
+       res = zoran_set_norm(zr, std);
        if (res)
                goto sstd_unlock_and_return;
 
index 476cfc8..96f6a49 100644 (file)
@@ -633,7 +633,7 @@ static int bcap_g_std(struct file *file, void *priv, v4l2_std_id *std)
        return 0;
 }
 
-static int bcap_s_std(struct file *file, void *priv, v4l2_std_id *std)
+static int bcap_s_std(struct file *file, void *priv, v4l2_std_id std)
 {
        struct bcap_device *bcap_dev = video_drvdata(file);
        int ret;
@@ -641,11 +641,11 @@ static int bcap_s_std(struct file *file, void *priv, v4l2_std_id *std)
        if (vb2_is_busy(&bcap_dev->buffer_queue))
                return -EBUSY;
 
-       ret = v4l2_subdev_call(bcap_dev->sd, core, s_std, *std);
+       ret = v4l2_subdev_call(bcap_dev->sd, core, s_std, std);
        if (ret < 0)
                return ret;
 
-       bcap_dev->std = *std;
+       bcap_dev->std = std;
        return 0;
 }
 
index 2a49f00..33b9660 100644 (file)
@@ -431,7 +431,7 @@ static int vpbe_enum_dv_timings(struct vpbe_device *vpbe_dev,
  * Sets the standard if supported by the current encoder. Return the status.
  * 0 - success & -EINVAL on error
  */
-static int vpbe_s_std(struct vpbe_device *vpbe_dev, v4l2_std_id *std_id)
+static int vpbe_s_std(struct vpbe_device *vpbe_dev, v4l2_std_id std_id)
 {
        struct vpbe_config *cfg = vpbe_dev->cfg;
        int out_index = vpbe_dev->current_out_index;
@@ -442,14 +442,14 @@ static int vpbe_s_std(struct vpbe_device *vpbe_dev, v4l2_std_id *std_id)
                V4L2_OUT_CAP_STD))
                return -EINVAL;
 
-       ret = vpbe_get_std_info(vpbe_dev, *std_id);
+       ret = vpbe_get_std_info(vpbe_dev, std_id);
        if (ret)
                return ret;
 
        mutex_lock(&vpbe_dev->lock);
 
        ret = v4l2_subdev_call(vpbe_dev->encoders[sd_index], video,
-                              s_std_output, *std_id);
+                              s_std_output, std_id);
        /* set the lcd controller output for the given mode */
        if (!ret) {
                struct osd_state *osd_device = vpbe_dev->osd_device;
@@ -513,7 +513,7 @@ static int vpbe_set_mode(struct vpbe_device *vpbe_dev,
                         */
                        if (preset_mode->timings_type & VPBE_ENC_STD)
                                return vpbe_s_std(vpbe_dev,
-                                                &preset_mode->std_id);
+                                                preset_mode->std_id);
                        if (preset_mode->timings_type &
                                                VPBE_ENC_DV_TIMINGS) {
                                dv_timings =
index 30976dc..bcf0ada 100644 (file)
@@ -983,7 +983,7 @@ static int vpbe_display_try_fmt(struct file *file, void *priv,
  * 0 - success & -EINVAL on error
  */
 static int vpbe_display_s_std(struct file *file, void *priv,
-                               v4l2_std_id *std_id)
+                               v4l2_std_id std_id)
 {
        struct vpbe_fh *fh = priv;
        struct vpbe_layer *layer = fh->layer;
index 3d1af67..20db8a0 100644 (file)
@@ -376,7 +376,7 @@ static int vpfe_config_ccdc_image_format(struct vpfe_device *vpfe_dev)
  * values in ccdc
  */
 static int vpfe_config_image_format(struct vpfe_device *vpfe_dev,
-                                   const v4l2_std_id *std_id)
+                                   v4l2_std_id std_id)
 {
        struct vpfe_subdev_info *sdinfo = vpfe_dev->current_subdev;
        struct v4l2_mbus_framefmt mbus_fmt;
@@ -384,7 +384,7 @@ static int vpfe_config_image_format(struct vpfe_device *vpfe_dev,
        int i, ret = 0;
 
        for (i = 0; i < ARRAY_SIZE(vpfe_standards); i++) {
-               if (vpfe_standards[i].std_id & *std_id) {
+               if (vpfe_standards[i].std_id & std_id) {
                        vpfe_dev->std_info.active_pixels =
                                        vpfe_standards[i].width;
                        vpfe_dev->std_info.active_lines =
@@ -461,7 +461,7 @@ static int vpfe_initialize_device(struct vpfe_device *vpfe_dev)
 
        /* Configure the default format information */
        ret = vpfe_config_image_format(vpfe_dev,
-                               &vpfe_standards[vpfe_dev->std_index].std_id);
+                               vpfe_standards[vpfe_dev->std_index].std_id);
        if (ret)
                return ret;
 
@@ -1168,7 +1168,7 @@ static int vpfe_s_input(struct file *file, void *priv, unsigned int index)
 
        /* set the default image parameters in the device */
        ret = vpfe_config_image_format(vpfe_dev,
-                               &vpfe_standards[vpfe_dev->std_index].std_id);
+                               vpfe_standards[vpfe_dev->std_index].std_id);
 unlock_out:
        mutex_unlock(&vpfe_dev->lock);
        return ret;
@@ -1193,7 +1193,7 @@ static int vpfe_querystd(struct file *file, void *priv, v4l2_std_id *std_id)
        return ret;
 }
 
-static int vpfe_s_std(struct file *file, void *priv, v4l2_std_id *std_id)
+static int vpfe_s_std(struct file *file, void *priv, v4l2_std_id std_id)
 {
        struct vpfe_device *vpfe_dev = video_drvdata(file);
        struct vpfe_subdev_info *sdinfo;
@@ -1215,7 +1215,7 @@ static int vpfe_s_std(struct file *file, void *priv, v4l2_std_id *std_id)
        }
 
        ret = v4l2_device_call_until_err(&vpfe_dev->v4l2_dev, sdinfo->grp_id,
-                                        core, s_std, *std_id);
+                                        core, s_std, std_id);
        if (ret < 0) {
                v4l2_err(&vpfe_dev->v4l2_dev, "Failed to set standard\n");
                goto unlock_out;
index d869c8e..00b7f9c 100644 (file)
@@ -1395,7 +1395,7 @@ static int vpif_g_std(struct file *file, void *priv, v4l2_std_id *std)
  * @priv: file handle
  * @std_id: ptr to std id
  */
-static int vpif_s_std(struct file *file, void *priv, v4l2_std_id *std_id)
+static int vpif_s_std(struct file *file, void *priv, v4l2_std_id std_id)
 {
        struct vpif_fh *fh = priv;
        struct channel_obj *ch = fh->channel;
@@ -1424,7 +1424,7 @@ static int vpif_s_std(struct file *file, void *priv, v4l2_std_id *std_id)
        fh->initialized = 1;
 
        /* Call encoder subdevice function to set the standard */
-       ch->video.stdid = *std_id;
+       ch->video.stdid = std_id;
        memset(&ch->video.dv_timings, 0, sizeof(ch->video.dv_timings));
 
        /* Get the information about the standard */
@@ -1437,7 +1437,7 @@ static int vpif_s_std(struct file *file, void *priv, v4l2_std_id *std_id)
        vpif_config_format(ch);
 
        /* set standard in the sub device */
-       ret = v4l2_subdev_call(ch->sd, core, s_std, *std_id);
+       ret = v4l2_subdev_call(ch->sd, core, s_std, std_id);
        if (ret && ret != -ENOIOCTLCMD && ret != -ENODEV) {
                vpif_dbg(1, debug, "Failed to set standard for sub devices\n");
                return ret;
index 3bb187c..f1dff9e 100644 (file)
@@ -1059,14 +1059,14 @@ static int vpif_qbuf(struct file *file, void *priv, struct v4l2_buffer *buf)
        return vb2_qbuf(&common->buffer_queue, buf);
 }
 
-static int vpif_s_std(struct file *file, void *priv, v4l2_std_id *std_id)
+static int vpif_s_std(struct file *file, void *priv, v4l2_std_id std_id)
 {
        struct vpif_fh *fh = priv;
        struct channel_obj *ch = fh->channel;
        struct common_obj *common = &ch->common[VPIF_VIDEO_INDEX];
        int ret = 0;
 
-       if (!(*std_id & VPIF_V4L2_STD))
+       if (!(std_id & VPIF_V4L2_STD))
                return -EINVAL;
 
        if (common->started) {
@@ -1075,7 +1075,7 @@ static int vpif_s_std(struct file *file, void *priv, v4l2_std_id *std_id)
        }
 
        /* Call encoder subdevice function to set the standard */
-       ch->video.stdid = *std_id;
+       ch->video.stdid = std_id;
        memset(&ch->video.dv_timings, 0, sizeof(ch->video.dv_timings));
        /* Get the information about the standard */
        if (vpif_update_resolution(ch))
@@ -1093,14 +1093,14 @@ static int vpif_s_std(struct file *file, void *priv, v4l2_std_id *std_id)
        vpif_config_format(ch);
 
        ret = v4l2_device_call_until_err(&vpif_obj.v4l2_dev, 1, video,
-                                               s_std_output, *std_id);
+                                               s_std_output, std_id);
        if (ret < 0) {
                vpif_err("Failed to set output standard\n");
                return ret;
        }
 
        ret = v4l2_device_call_until_err(&vpif_obj.v4l2_dev, 1, core,
-                                                       s_std, *std_id);
+                                                       s_std, std_id);
        if (ret < 0)
                vpif_err("Failed to set standard for sub devices\n");
        return ret;
index 5f7db3f..3a6a0dc 100644 (file)
@@ -957,12 +957,12 @@ static int vidioc_querystd(struct file *file, void *priv, v4l2_std_id *std_id)
        return 0;
 }
 
-static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id *id)
+static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id id)
 {
        struct viu_fh *fh = priv;
 
-       fh->dev->std = *id;
-       decoder_call(fh->dev, core, s_std, *id);
+       fh->dev->std = id;
+       decoder_call(fh->dev, core, s_std, id);
        return 0;
 }
 
index 92a33f0..76a8623 100644 (file)
@@ -1357,7 +1357,7 @@ static int mcam_vidioc_s_input(struct file *filp, void *priv, unsigned int i)
 }
 
 /* from vivi.c */
-static int mcam_vidioc_s_std(struct file *filp, void *priv, v4l2_std_id *a)
+static int mcam_vidioc_s_std(struct file *filp, void *priv, v4l2_std_id a)
 {
        return 0;
 }
index 9961e13..ef0efdf 100644 (file)
@@ -574,7 +574,7 @@ static int mxr_dv_timings_cap(struct file *file, void *fh,
        return ret ? -EINVAL : 0;
 }
 
-static int mxr_s_std(struct file *file, void *fh, v4l2_std_id *norm)
+static int mxr_s_std(struct file *file, void *fh, v4l2_std_id norm)
 {
        struct mxr_layer *layer = video_drvdata(file);
        struct mxr_device *mdev = layer->mdev;
@@ -591,7 +591,7 @@ static int mxr_s_std(struct file *file, void *fh, v4l2_std_id *norm)
                return -EBUSY;
        }
 
-       ret = v4l2_subdev_call(to_outsd(mdev), video, s_std_output, *norm);
+       ret = v4l2_subdev_call(to_outsd(mdev), video, s_std_output, norm);
 
        mutex_unlock(&mdev->mutex);
 
index d853162..8e3b95a 100644 (file)
@@ -881,29 +881,29 @@ static u32 sh_vou_ntsc_mode(enum sh_vou_bus_fmt bus_fmt)
        }
 }
 
-static int sh_vou_s_std(struct file *file, void *priv, v4l2_std_id *std_id)
+static int sh_vou_s_std(struct file *file, void *priv, v4l2_std_id std_id)
 {
        struct sh_vou_device *vou_dev = video_drvdata(file);
        int ret;
 
-       dev_dbg(vou_dev->v4l2_dev.dev, "%s(): 0x%llx\n", __func__, *std_id);
+       dev_dbg(vou_dev->v4l2_dev.dev, "%s(): 0x%llx\n", __func__, std_id);
 
-       if (*std_id & ~vou_dev->vdev->tvnorms)
+       if (std_id & ~vou_dev->vdev->tvnorms)
                return -EINVAL;
 
        ret = v4l2_device_call_until_err(&vou_dev->v4l2_dev, 0, video,
-                                        s_std_output, *std_id);
+                                        s_std_output, std_id);
        /* Shall we continue, if the subdev doesn't support .s_std_output()? */
        if (ret < 0 && ret != -ENOIOCTLCMD)
                return ret;
 
-       if (*std_id & V4L2_STD_525_60)
+       if (std_id & V4L2_STD_525_60)
                sh_vou_reg_ab_set(vou_dev, VOUCR,
                        sh_vou_ntsc_mode(vou_dev->pdata->bus_fmt) << 29, 7 << 29);
        else
                sh_vou_reg_ab_set(vou_dev, VOUCR, 5 << 29, 7 << 29);
 
-       vou_dev->std = *std_id;
+       vou_dev->std = std_id;
 
        return 0;
 }
index 8ec9805..3592968 100644 (file)
@@ -256,12 +256,12 @@ static int soc_camera_s_input(struct file *file, void *priv, unsigned int i)
        return 0;
 }
 
-static int soc_camera_s_std(struct file *file, void *priv, v4l2_std_id *a)
+static int soc_camera_s_std(struct file *file, void *priv, v4l2_std_id a)
 {
        struct soc_camera_device *icd = file->private_data;
        struct v4l2_subdev *sd = soc_camera_to_subdev(icd);
 
-       return v4l2_subdev_call(sd, core, s_std, *a);
+       return v4l2_subdev_call(sd, core, s_std, a);
 }
 
 static int soc_camera_g_std(struct file *file, void *priv, v4l2_std_id *a)
index 2d91eeb..a2f7bdd 100644 (file)
@@ -336,7 +336,7 @@ static int timblogiw_g_std(struct file *file, void  *priv, v4l2_std_id *std)
        return 0;
 }
 
-static int timblogiw_s_std(struct file *file, void  *priv, v4l2_std_id *std)
+static int timblogiw_s_std(struct file *file, void  *priv, v4l2_std_id std)
 {
        struct video_device *vdev = video_devdata(file);
        struct timblogiw *lw = video_get_drvdata(vdev);
@@ -348,10 +348,10 @@ static int timblogiw_s_std(struct file *file, void  *priv, v4l2_std_id *std)
        mutex_lock(&lw->lock);
 
        if (TIMBLOGIW_HAS_DECODER(lw))
-               err = v4l2_subdev_call(lw->sd_enc, core, s_std, *std);
+               err = v4l2_subdev_call(lw->sd_enc, core, s_std, std);
 
        if (!err)
-               fh->cur_norm = timblogiw_get_norm(*std);
+               fh->cur_norm = timblogiw_get_norm(std);
 
        mutex_unlock(&lw->lock);
 
index b051c4a..a794cd6 100644 (file)
@@ -847,7 +847,7 @@ static int viacam_s_input(struct file *filp, void *priv, unsigned int i)
        return 0;
 }
 
-static int viacam_s_std(struct file *filp, void *priv, v4l2_std_id *std)
+static int viacam_s_std(struct file *filp, void *priv, v4l2_std_id std)
 {
        return 0;
 }
index eb5d6f9..c6af974 100644 (file)
@@ -3042,7 +3042,7 @@ static int vino_g_std(struct file *file, void *__fh,
 }
 
 static int vino_s_std(struct file *file, void *__fh,
-                          v4l2_std_id *std)
+                          v4l2_std_id std)
 {
        struct vino_channel_settings *vcs = video_drvdata(file);
        unsigned long flags;
@@ -3056,7 +3056,7 @@ static int vino_s_std(struct file *file, void *__fh,
        }
 
        /* check if the standard is valid for the current input */
-       if ((*std) & vino_inputs[vcs->input].std) {
+       if (std & vino_inputs[vcs->input].std) {
                dprintk("standard accepted\n");
 
                /* change the video norm for SAA7191
@@ -3065,13 +3065,13 @@ static int vino_s_std(struct file *file, void *__fh,
                if (vcs->input == VINO_INPUT_D1)
                        goto out;
 
-               if ((*std) & V4L2_STD_PAL) {
+               if (std & V4L2_STD_PAL) {
                        ret = vino_set_data_norm(vcs, VINO_DATA_NORM_PAL,
                                                 &flags);
-               } else if ((*std) & V4L2_STD_NTSC) {
+               } else if (std & V4L2_STD_NTSC) {
                        ret = vino_set_data_norm(vcs, VINO_DATA_NORM_NTSC,
                                                 &flags);
-               } else if ((*std) & V4L2_STD_SECAM) {
+               } else if (std & V4L2_STD_SECAM) {
                        ret = vino_set_data_norm(vcs, VINO_DATA_NORM_SECAM,
                                                 &flags);
                } else {
index 11316f2..e9dd01c 100644 (file)
@@ -1320,7 +1320,7 @@ out:
        return rc;
 }
 
-static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id norm)
+static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id norm)
 {
        struct au0828_fh *fh = priv;
        struct au0828_dev *dev = fh->dev;
@@ -1332,7 +1332,7 @@ static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id * norm)
           have to make the au0828 bridge adjust the size of its capture
           buffer, which is currently hardcoded at 720x480 */
 
-       v4l2_device_call_all(&dev->v4l2_dev, 0, core, s_std, *norm);
+       v4l2_device_call_all(&dev->v4l2_dev, 0, core, s_std, norm);
        dev->std_set_in_tuner_core = 1;
 
        if (dev->dvb.frontend && dev->dvb.frontend->ops.analog_ops.i2c_gate_ctrl)
index 49c842a..f548db8 100644 (file)
@@ -1492,14 +1492,14 @@ static int vidioc_g_std(struct file *file, void *fh0, v4l2_std_id *norm)
        return 0;
 }
 
-static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id *id)
+static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id id)
 {
        struct cx231xx_fh  *fh  = file->private_data;
        struct cx231xx *dev = fh->dev;
        unsigned int i;
 
        for (i = 0; i < ARRAY_SIZE(cx231xx_tvnorms); i++)
-               if (*id & cx231xx_tvnorms[i].id)
+               if (id & cx231xx_tvnorms[i].id)
                        break;
        if (i == ARRAY_SIZE(cx231xx_tvnorms))
                return -EINVAL;
index 53020a7..4cff2f4 100644 (file)
@@ -987,7 +987,7 @@ static int vidioc_g_std(struct file *file, void *priv, v4l2_std_id *id)
        return 0;
 }
 
-static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id *norm)
+static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id norm)
 {
        struct cx231xx_fh *fh = priv;
        struct cx231xx *dev = fh->dev;
@@ -998,13 +998,13 @@ static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id *norm)
        if (rc < 0)
                return rc;
 
-       if (dev->norm == *norm)
+       if (dev->norm == norm)
                return 0;
 
        if (videobuf_queue_is_busy(&fh->vb_vidq))
                return -EBUSY;
 
-       dev->norm = *norm;
+       dev->norm = norm;
 
        /* Adjusts width/height, if needed */
        dev->width = 720;
index 3ef696d..96d84c2 100644 (file)
@@ -996,23 +996,23 @@ static int vidioc_querystd(struct file *file, void *priv, v4l2_std_id *norm)
        return 0;
 }
 
-static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id *norm)
+static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id norm)
 {
        struct em28xx_fh   *fh  = priv;
        struct em28xx      *dev = fh->dev;
        struct v4l2_format f;
 
-       if (*norm == dev->norm)
+       if (norm == dev->norm)
                return 0;
 
        if (dev->streaming_users > 0)
                return -EBUSY;
 
-       dev->norm = *norm;
+       dev->norm = norm;
 
        /* Adjusts width/height, if needed */
        f.fmt.pix.width = 720;
-       f.fmt.pix.height = (*norm & V4L2_STD_525_60) ? 480 : 576;
+       f.fmt.pix.height = (norm & V4L2_STD_525_60) ? 480 : 576;
        vidioc_try_fmt_vid_cap(file, priv, &f);
 
        /* set new image size */
index 6a15405..2983bf0 100644 (file)
@@ -582,13 +582,13 @@ static int vidioc_querycap(struct file *file, void  *priv,
 }
 
 static int vidioc_s_std(struct file *file, void *private_data,
-                       v4l2_std_id *std)
+                       v4l2_std_id std)
 {
        struct hdpvr_fh *fh = file->private_data;
        struct hdpvr_device *dev = fh->dev;
        u8 std_type = 1;
 
-       if (*std & (V4L2_STD_NTSC | V4L2_STD_PAL_60))
+       if (std & (V4L2_STD_NTSC | V4L2_STD_PAL_60))
                std_type = 0;
 
        return hdpvr_config_call(dev, CTRL_VIDEO_STD_TYPE, std_type);
index 0f729c9..a7774e3 100644 (file)
@@ -196,13 +196,13 @@ static int pvr2_g_std(struct file *file, void *priv, v4l2_std_id *std)
        return ret;
 }
 
-static int pvr2_s_std(struct file *file, void *priv, v4l2_std_id *std)
+static int pvr2_s_std(struct file *file, void *priv, v4l2_std_id std)
 {
        struct pvr2_v4l2_fh *fh = file->private_data;
        struct pvr2_hdw *hdw = fh->channel.mc_head->hdw;
 
        return pvr2_ctrl_set_value(
-               pvr2_hdw_get_ctrl_by_id(hdw, PVR2_CID_STDCUR), *std);
+               pvr2_hdw_get_ctrl_by_id(hdw, PVR2_CID_STDCUR), std);
 }
 
 static int pvr2_querystd(struct file *file, void *priv, v4l2_std_id *std)
index 2bd8613..ab97e7d 100644 (file)
@@ -1294,7 +1294,7 @@ static int vidioc_streamoff(struct file *file, void *priv, enum v4l2_buf_type i)
        return 0;
 }
 
-static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id *i)
+static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id i)
 {
        struct s2255_fh *fh = priv;
        struct s2255_mode mode;
@@ -1314,7 +1314,7 @@ static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id *i)
                goto out_s_std;
        }
        mode = fh->channel->mode;
-       if (*i & V4L2_STD_525_60) {
+       if (i & V4L2_STD_525_60) {
                dprintk(4, "%s 60 Hz\n", __func__);
                /* if changing format, reset frame decimation/intervals */
                if (mode.format != FORMAT_NTSC) {
@@ -1324,7 +1324,7 @@ static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id *i)
                        channel->width = LINE_SZ_4CIFS_NTSC;
                        channel->height = NUM_LINES_4CIFS_NTSC * 2;
                }
-       } else if (*i & V4L2_STD_625_50) {
+       } else if (i & V4L2_STD_625_50) {
                dprintk(4, "%s 50 Hz\n", __func__);
                if (mode.format != FORMAT_PAL) {
                        mode.restart = 1;
@@ -1337,7 +1337,7 @@ static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id *i)
                ret = -EINVAL;
                goto out_s_std;
        }
-       fh->channel->std = *i;
+       fh->channel->std = i;
        if (mode.restart)
                s2255_set_mode(fh->channel, &mode);
 out_s_std:
index 5307a63..f6a6cdc 100644 (file)
@@ -375,7 +375,7 @@ static int vidioc_g_std(struct file *file, void *priv, v4l2_std_id *norm)
        return 0;
 }
 
-static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id *norm)
+static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id norm)
 {
        struct stk1160 *dev = video_drvdata(file);
        struct vb2_queue *q = &dev->vb_vidq;
@@ -388,7 +388,7 @@ static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id *norm)
                return -ENODEV;
 
        /* We need to set this now, before we call stk1160_set_std */
-       dev->norm = *norm;
+       dev->norm = norm;
 
        /* This is taken from saa7115 video decoder */
        if (dev->norm & V4L2_STD_525_60) {
index 9595309..8df668d 100644 (file)
@@ -801,7 +801,7 @@ static int vidioc_g_fmt_vbi(struct file *file, void *fh,
        return 0;
 }
 
-static int set_std(struct poseidon *pd, v4l2_std_id *norm)
+static int set_std(struct poseidon *pd, v4l2_std_id norm)
 {
        struct video_data *video = &pd->video_data;
        struct vbi_data *vbi    = &pd->vbi_data;
@@ -811,7 +811,7 @@ static int set_std(struct poseidon *pd, v4l2_std_id *norm)
        int height;
 
        for (i = 0; i < POSEIDON_TVNORMS; i++) {
-               if (*norm & poseidon_tvnorms[i].v4l2_id) {
+               if (norm & poseidon_tvnorms[i].v4l2_id) {
                        param = poseidon_tvnorms[i].tlg_tvnorm;
                        log("name : %s", poseidon_tvnorms[i].name);
                        goto found;
@@ -846,7 +846,7 @@ out:
        return ret;
 }
 
-static int vidioc_s_std(struct file *file, void *fh, v4l2_std_id *norm)
+static int vidioc_s_std(struct file *file, void *fh, v4l2_std_id norm)
 {
        struct front_face *front = fh;
 
@@ -1270,7 +1270,7 @@ static int restore_v4l2_context(struct poseidon *pd,
 
        pd_video_checkmode(pd);
 
-       set_std(pd, &context->tvnormid);
+       set_std(pd, context->tvnormid);
        vidioc_s_input(NULL, front, context->sig_index);
        pd_vidioc_s_tuner(pd, context->audio_idx);
        pd_vidioc_s_fmt(pd, &context->pix);
index b4618d7..a78de1d 100644 (file)
@@ -1056,13 +1056,13 @@ static int vidioc_streamoff(struct file *file, void *priv, enum v4l2_buf_type i)
        return 0;
 }
 
-static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id *norm)
+static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id norm)
 {
        int rc = 0;
        struct tm6000_fh *fh = priv;
        struct tm6000_core *dev = fh->dev;
 
-       dev->norm = *norm;
+       dev->norm = norm;
        rc = tm6000_init_analog_mode(dev);
 
        fh->width  = dev->width;
@@ -1134,7 +1134,7 @@ static int vidioc_s_input(struct file *file, void *priv, unsigned int i)
 
        dev->input = i;
 
-       rc = vidioc_s_std(file, priv, &dev->vfd->current_norm);
+       rc = vidioc_s_std(file, priv, dev->vfd->current_norm);
 
        return rc;
 }
index 874cfec..041f19e 100644 (file)
@@ -595,11 +595,11 @@ static int vidioc_s_input(struct file *file, void *priv, unsigned int input)
        return 0;
 }
 
-static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id *id)
+static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id id)
 {
        struct usb_usbvision *usbvision = video_drvdata(file);
 
-       usbvision->tvnorm_id = *id;
+       usbvision->tvnorm_id = id;
 
        call_all(usbvision, core, s_std, usbvision->tvnorm_id);
        /* propagate the change to the decoder */
index 8ec8abe..d80d8af 100644 (file)
@@ -1383,15 +1383,15 @@ static int v4l_s_std(const struct v4l2_ioctl_ops *ops,
                                struct file *file, void *fh, void *arg)
 {
        struct video_device *vfd = video_devdata(file);
-       v4l2_std_id *id = arg, norm;
+       v4l2_std_id id = *(v4l2_std_id *)arg, norm;
        int ret;
 
-       norm = (*id) & vfd->tvnorms;
+       norm = id & vfd->tvnorms;
        if (vfd->tvnorms && !norm)      /* Check if std is supported */
                return -EINVAL;
 
        /* Calls the specific handler */
-       ret = ops->vidioc_s_std(file, fh, &norm);
+       ret = ops->vidioc_s_std(file, fh, norm);
 
        /* Updates standard information */
        if (ret >= 0)
index 19dc5b0..3670a7b 100644 (file)
@@ -924,7 +924,7 @@ static int vpfe_querystd(struct file *file, void *priv, v4l2_std_id *std_id)
  *
  * Return 0 on success, error code otherwise
  */
-static int vpfe_s_std(struct file *file, void *priv, v4l2_std_id *std_id)
+static int vpfe_s_std(struct file *file, void *priv, v4l2_std_id std_id)
 {
        struct vpfe_video_device *video = video_drvdata(file);
        struct vpfe_device *vpfe_dev = video->vpfe_dev;
@@ -945,13 +945,13 @@ static int vpfe_s_std(struct file *file, void *priv, v4l2_std_id *std_id)
                goto unlock_out;
        }
        ret = v4l2_device_call_until_err(&vpfe_dev->v4l2_dev, sdinfo->grp_id,
-                                        core, s_std, *std_id);
+                                        core, s_std, std_id);
        if (ret < 0) {
                v4l2_err(&vpfe_dev->v4l2_dev, "Failed to set standard\n");
                video->stdid = V4L2_STD_UNKNOWN;
                goto unlock_out;
        }
-       video->stdid = *std_id;
+       video->stdid = std_id;
 unlock_out:
        mutex_unlock(&video->lock);
        return ret;
index e33b7f5..073b3b3 100644 (file)
@@ -612,9 +612,9 @@ dt3155_ioc_g_std(struct file *filp, void *p, v4l2_std_id *norm)
 }
 
 static int
-dt3155_ioc_s_std(struct file *filp, void *p, v4l2_std_id *norm)
+dt3155_ioc_s_std(struct file *filp, void *p, v4l2_std_id norm)
 {
-       if (*norm & DT3155_CURRENT_NORM)
+       if (norm & DT3155_CURRENT_NORM)
                return 0;
        return -EINVAL;
 }
index 4e477f3..29fe94d 100644 (file)
@@ -1117,40 +1117,40 @@ static int vidioc_g_std(struct file *file, void *priv, v4l2_std_id *std)
        return 0;
 }
 
-static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id *std)
+static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id std)
 {
        struct go7007 *go = ((struct go7007_file *) priv)->go;
 
        if (go->streaming)
                return -EBUSY;
 
-       if (!(go->board_info->sensor_flags & GO7007_SENSOR_TV) && *std != 0)
+       if (!(go->board_info->sensor_flags & GO7007_SENSOR_TV) && std != 0)
                return -EINVAL;
 
-       if (*std == 0)
+       if (std == 0)
                return -EINVAL;
 
        if ((go->board_info->flags & GO7007_BOARD_HAS_TUNER) &&
                        go->input == go->board_info->num_inputs - 1) {
                if (!go->i2c_adapter_online)
                        return -EIO;
-               if (call_all(&go->v4l2_dev, core, s_std, *std) < 0)
+               if (call_all(&go->v4l2_dev, core, s_std, std) < 0)
                        return -EINVAL;
        }
 
-       if (*std & V4L2_STD_NTSC) {
+       if (std & V4L2_STD_NTSC) {
                go->standard = GO7007_STD_NTSC;
                go->sensor_framerate = 30000;
-       } else if (*std & V4L2_STD_PAL) {
+       } else if (std & V4L2_STD_PAL) {
                go->standard = GO7007_STD_PAL;
                go->sensor_framerate = 25025;
-       } else if (*std & V4L2_STD_SECAM) {
+       } else if (std & V4L2_STD_SECAM) {
                go->standard = GO7007_STD_PAL;
                go->sensor_framerate = 25025;
        } else
                return -EINVAL;
 
-       call_all(&go->v4l2_dev, core, s_std, *std);
+       call_all(&go->v4l2_dev, core, s_std, std);
        set_capture_size(go, NULL, 0);
 
        return 0;
index 4977e86..310b126 100644 (file)
@@ -1326,7 +1326,7 @@ static int solo_enc_streamoff(struct file *file, void *priv,
        return videobuf_streamoff(&fh->vidq);
 }
 
-static int solo_enc_s_std(struct file *file, void *priv, v4l2_std_id *i)
+static int solo_enc_s_std(struct file *file, void *priv, v4l2_std_id i)
 {
        return 0;
 }
index ca774cc..87f3c04 100644 (file)
@@ -773,7 +773,7 @@ static int solo_streamoff(struct file *file, void *priv, enum v4l2_buf_type i)
        return videobuf_streamoff(&fh->vidq);
 }
 
-static int solo_s_std(struct file *file, void *priv, v4l2_std_id *i)
+static int solo_s_std(struct file *file, void *priv, v4l2_std_id i)
 {
        return 0;
 }
index a7ca488..57585c7 100644 (file)
@@ -132,7 +132,7 @@ struct vpbe_device_ops {
                               struct v4l2_enum_dv_timings *timings_info);
 
        /* Set std at the output */
-       int (*s_std)(struct vpbe_device *vpbe_dev, v4l2_std_id *std_id);
+       int (*s_std)(struct vpbe_device *vpbe_dev, v4l2_std_id std_id);
 
        /* Get the current std at the output */
        int (*g_std)(struct vpbe_device *vpbe_dev, v4l2_std_id *std_id);
index 0da8682..ee7b7c6 100644 (file)
@@ -132,7 +132,7 @@ struct v4l2_ioctl_ops {
                        ENUMSTD is handled by videodev.c
                 */
        int (*vidioc_g_std) (struct file *file, void *fh, v4l2_std_id *norm);
-       int (*vidioc_s_std) (struct file *file, void *fh, v4l2_std_id *norm);
+       int (*vidioc_s_std) (struct file *file, void *fh, v4l2_std_id norm);
        int (*vidioc_querystd) (struct file *file, void *fh, v4l2_std_id *a);
 
                /* Input handling */