[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 4143d61f79b1013b54156e7ee88a4d82234dc9ed..fe907f2e8f596e816786712bca6fe70ccf4f4b35 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 77f9c92186f4f4da362152a1716a75f0751a84dc..66c957a02ba7bfbfed0cf8a3939808de56f3ff88 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 7aea24bf3a37e7b37b955f35e6194bfd4981e897..4b9a256f506efa0b596f9ab8052bdf95bb68c9ab 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 613e5ae7d5caed9da4c38cd60da171097460bb4c..67b61cf3e03a32fdf6e8e61fc4c3152d5a84c634 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 173ccd204c1fea1c3393c31ecc217fc9a9b6bda0..254c50fabd736f1faf7ec55f24f6f2aa39e34a0a 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 aa9b44a611d3692b0b72e27fe1a87a3dd0aeb2f3..43433969d6331271969fc84477161eada3f540e2 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 812ec5ff593962dd62567076682366267550bb28..6dea11a7a858f08850887a1041f1f4af0f2f3944 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 2bbda43eb00e55225d875b86fe87196b16b9d549..ed08c89adde044ff22e6cbdca2efbea2d30a7c6a 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 75281c1530d831d0014a9791bb2a7a845fb68b39..93c7d576199b5bdc96f00be207ea51870edc194b 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 f0e70ffbedd4a3034449f0e6156a62186e3a8e7b..239f63c12f3330213ed5de38a5a24417530e1980 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 486ca8d5f2a0c6012c82922afc21d313a24354f4..150bb76e783905c5e85ab9ea4c58dc1c21575f59 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 ede6f139b5cc85eb0a2847b3681c9c908ef71899..ead5be58303d33beebd5ca8ae410d4899c333879 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 2928e7287da8b5559535ef224d6b290900d16017..07b8460953b6bfff7e05ecde46c4980ba2594438 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 68387d4369d66f5260f9582db11f0f76dc8b6ccc..ed73edd2bcd308e310197ac3ff9bff2559905bd5 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 852f11e0d636f3ef57c45f33b4706959f377694b..080f179070a61302340176bb0e99f0dd18ba52d5 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 34c6bc132ebd59955b21dd62328ceb171720acd3..75c397756116fff76953e0ab2eb0425b51874444 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 4df79c6569094311d6dc6cee081b72f62427cd2d..66a70814004c32ef5188ef6ff9a3d3fc56a638d4 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 1e23547531b92ce0ccba0366bd9843a1e5d00460..a6c69a4c7fbd3fc3cc140152cd650e67c0847bf9 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 71eefef5e324e86b0582fb0a5b8d84ea2984f40e..62169dd74c6af40b42df7d1d195ab34098c05edb 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 e7fbd036ba44faea97d572eb304ffb484b82c9d4..538de52b9cfd24b2b66c5b8bd8c3831157962aaf 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 bfc8cec110de4a1637a821259db52e4ed95d833f..da224eb39b95a002e8aa2c3c2b2c91e9a4848ef0 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 4b703fe8c9532dd35b4d00441f7dccb5d52ab7bd..7005695aa4bd26cfaffc4edabb33420412321ddb 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 2e8f518f298fd54b735afb538d54d6a4cace339d..1168a84a737dafae1dbf6e7e3bbe195b891815d6 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 476cfc88d1443a43ac142e388ee1281c0360cc39..96f6a4922d0ce3d3d936f327089786898687e63f 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 2a49f00c454337cadfa9ceee808fc48f1a92f9fe..33b9660b7f77b4b3128b6a3597c16f44ac9f752a 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 30976dc9f346f7023f3abc0e30c4bf25d58d38cd..bcf0ada3481d2fa976faf171c5fcef1b4e267f94 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 3d1af6704822bf14074b2aeee26acf699acc38d3..20db8a077990461a8e5373b28eb722d665968333 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 d869c8e868eec58761b08b52e8055486ece6275e..00b7f9c5ac04955499f7b75480768bd34b3bf0fd 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 3bb187c9402bc448422b0c2a1353da306f206ae8..f1dff9ec1ea25fd1479c41f44fa3f2c537ea8896 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 5f7db3f1f6f5005cacf757dd69346050db939392..3a6a0dcdc3e426390fa2b9ff2ab9edbcc9c47b0d 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 92a33f081852be7786ea0f099cbd4cd4b4f2f439..76a862318a7e8129c9f20aa3bd67c45666462f00 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 9961e13f683d420f0f34ca8437813afc705e6066..ef0efdf422fec8b9e1d447e853cfd04d5e590f8c 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 d853162586fa4ab2477bb14191a8890c2ab68812..8e3b95a6c29e91dd2c668fd116082a6e556e793d 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 8ec98051ea73151b95c9aec656f280a82a54bc13..3592968026cdc4e3197aa573c2c977c47e298580 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 2d91eeb1d2ae95b59b8e600fb62b82cb79dbff99..a2f7bdd5104f684324fb514600c7a02fbe367e75 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 b051c4a2855489d44b3e97e3212b4c6df7133514..a794cd6c44416fa2b57c6606175fff30204bc245 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 eb5d6f955709fbdbad6130f4b36cb74c59737608..c6af974c5b45c82eced9d54ee8d2337b2d276800 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 11316f2f2e421dc4b3c91f5ddfc987a6319ea9f3..e9dd01c4b6ed95930ffd6ef9679d1f769011e73b 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 49c842aa1f700797fa3a68acd622a3587cd65560..f548db8043d4bb1bfc52d7739348ac72d45d78f7 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 53020a7e01fe805c09fab3b1d812747ca274e211..4cff2f4075e19a25176298dc650deca1828570e9 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 3ef696d36c7e23dbb09750bd837f8c88429e1d92..96d84c22ef0004db11a337e0b1e6e7ef2a6bde90 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 6a15405360dcd51af1fef2fb89e5f000d3f3334f..2983bf0d5f0f04b2e524ec6db5e88e2ad1c4d859 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 0f729c92e3cf1fe63d9a50c0ed0fe99cc119c521..a7774e3836d4b01f2b47096c766cfb700296bbae 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 2bd86132d3e38069599c49bcd0cff361e4f01567..ab97e7d0b4f25e2cdd7adf239be95cecfa70e546 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 5307a63e378fc57e60b8f6652d181a36a7391b31..f6a6cdc8b28847bfb5c94d4e2cbc2911c205aeaf 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 9595309caed441d852774690864ebb0024557025..8df668d06552bb4323ab0213889d12105e09422f 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 b4618d7d9c4bddb8c3538199215e0dc86cc818ff..a78de1d1bc9eebf24597a198670268239a2a09df 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 874cfec0b8db4bd744f13f9f1b1bed94bd8817ac..041f19e1b05b47a43f1e823e3145cfc9576d8da5 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 8ec8abe0ffe79ebf241cfcbbd54b7b9f7dc95747..d80d8afba3fbaa7e1e0fbeb95ad5d4a19f425c40 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 19dc5b062d07ab3cf5cd392c81ad568e901cae4b..3670a7b0724eea2c5c9ff860b7b69960830f238c 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 e33b7f55d84e2a9aa8232c28463e622b0785c53c..073b3b35f8f42de2697dfe6e1e85a47ba019a000 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 4e477f3b78f3290d1b3652e2f8df5980218a192f..29fe94da5f717955d770efc6b829797e1fbcd64b 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 4977e869d5b78786a05f32c2db57dca6e3400882..310b126e128866bdade7a447a715314193e14afb 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 ca774cc57539f48a12760c13392d195e4fa32147..87f3c0452ca43f116b572a6450c03ce4074f546d 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 a7ca4884c46cc8a386a42a7d9f9eaae6c35dc8ba..57585c7004a4503d1a3a5f70973f53b29555926d 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 0da8682a048172092962ba8fcfc70617628d52a4..ee7b7c67c87ae3c30ffd70c6904c71bee8d95699 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 */