diff options
| author | Hans Verkuil <hans.verkuil@cisco.com> | 2012-06-09 09:54:19 -0400 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2012-07-06 16:27:23 -0400 |
| commit | dcaded7e5f7c66facfd0ced5b368e757cbf576ed (patch) | |
| tree | 97b1653d4eb5a3180b7d03675a3817cb7bc194ab | |
| parent | 074689d692e129082e1846e71870a0c95c1eb3eb (diff) | |
[media] cx18: don't mess with vfd->debug
This is now controlled by sysfs.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
| -rw-r--r-- | drivers/media/video/cx18/cx18-ioctl.c | 18 | ||||
| -rw-r--r-- | drivers/media/video/cx18/cx18-ioctl.h | 2 | ||||
| -rw-r--r-- | drivers/media/video/cx18/cx18-streams.c | 4 |
3 files changed, 2 insertions, 22 deletions
diff --git a/drivers/media/video/cx18/cx18-ioctl.c b/drivers/media/video/cx18/cx18-ioctl.c index 35fde4e931f5..e9912db3b496 100644 --- a/drivers/media/video/cx18/cx18-ioctl.c +++ b/drivers/media/video/cx18/cx18-ioctl.c | |||
| @@ -1142,24 +1142,6 @@ static long cx18_default(struct file *file, void *fh, bool valid_prio, | |||
| 1142 | return 0; | 1142 | return 0; |
| 1143 | } | 1143 | } |
| 1144 | 1144 | ||
| 1145 | long cx18_v4l2_ioctl(struct file *filp, unsigned int cmd, | ||
| 1146 | unsigned long arg) | ||
| 1147 | { | ||
| 1148 | struct video_device *vfd = video_devdata(filp); | ||
| 1149 | struct cx18_open_id *id = file2id(filp); | ||
| 1150 | struct cx18 *cx = id->cx; | ||
| 1151 | long res; | ||
| 1152 | |||
| 1153 | mutex_lock(&cx->serialize_lock); | ||
| 1154 | |||
| 1155 | if (cx18_debug & CX18_DBGFLG_IOCTL) | ||
| 1156 | vfd->debug = V4L2_DEBUG_IOCTL | V4L2_DEBUG_IOCTL_ARG; | ||
| 1157 | res = video_ioctl2(filp, cmd, arg); | ||
| 1158 | vfd->debug = 0; | ||
| 1159 | mutex_unlock(&cx->serialize_lock); | ||
| 1160 | return res; | ||
| 1161 | } | ||
| 1162 | |||
| 1163 | static const struct v4l2_ioctl_ops cx18_ioctl_ops = { | 1145 | static const struct v4l2_ioctl_ops cx18_ioctl_ops = { |
| 1164 | .vidioc_querycap = cx18_querycap, | 1146 | .vidioc_querycap = cx18_querycap, |
| 1165 | .vidioc_s_audio = cx18_s_audio, | 1147 | .vidioc_s_audio = cx18_s_audio, |
diff --git a/drivers/media/video/cx18/cx18-ioctl.h b/drivers/media/video/cx18/cx18-ioctl.h index dcb2559ad520..2f9dd591ee0f 100644 --- a/drivers/media/video/cx18/cx18-ioctl.h +++ b/drivers/media/video/cx18/cx18-ioctl.h | |||
| @@ -29,5 +29,3 @@ void cx18_set_funcs(struct video_device *vdev); | |||
| 29 | int cx18_s_std(struct file *file, void *fh, v4l2_std_id *std); | 29 | int cx18_s_std(struct file *file, void *fh, v4l2_std_id *std); |
| 30 | int cx18_s_frequency(struct file *file, void *fh, struct v4l2_frequency *vf); | 30 | int cx18_s_frequency(struct file *file, void *fh, struct v4l2_frequency *vf); |
| 31 | int cx18_s_input(struct file *file, void *fh, unsigned int inp); | 31 | int cx18_s_input(struct file *file, void *fh, unsigned int inp); |
| 32 | long cx18_v4l2_ioctl(struct file *filp, unsigned int cmd, | ||
| 33 | unsigned long arg); | ||
diff --git a/drivers/media/video/cx18/cx18-streams.c b/drivers/media/video/cx18/cx18-streams.c index 4185bcb80ca3..9d598ab88615 100644 --- a/drivers/media/video/cx18/cx18-streams.c +++ b/drivers/media/video/cx18/cx18-streams.c | |||
| @@ -40,8 +40,7 @@ static struct v4l2_file_operations cx18_v4l2_enc_fops = { | |||
| 40 | .owner = THIS_MODULE, | 40 | .owner = THIS_MODULE, |
| 41 | .read = cx18_v4l2_read, | 41 | .read = cx18_v4l2_read, |
| 42 | .open = cx18_v4l2_open, | 42 | .open = cx18_v4l2_open, |
| 43 | /* FIXME change to video_ioctl2 if serialization lock can be removed */ | 43 | .unlocked_ioctl = video_ioctl2, |
| 44 | .unlocked_ioctl = cx18_v4l2_ioctl, | ||
| 45 | .release = cx18_v4l2_close, | 44 | .release = cx18_v4l2_close, |
| 46 | .poll = cx18_v4l2_enc_poll, | 45 | .poll = cx18_v4l2_enc_poll, |
| 47 | .mmap = cx18_v4l2_mmap, | 46 | .mmap = cx18_v4l2_mmap, |
| @@ -376,6 +375,7 @@ static int cx18_prep_dev(struct cx18 *cx, int type) | |||
| 376 | s->video_dev->fops = &cx18_v4l2_enc_fops; | 375 | s->video_dev->fops = &cx18_v4l2_enc_fops; |
| 377 | s->video_dev->release = video_device_release; | 376 | s->video_dev->release = video_device_release; |
| 378 | s->video_dev->tvnorms = V4L2_STD_ALL; | 377 | s->video_dev->tvnorms = V4L2_STD_ALL; |
| 378 | s->video_dev->lock = &cx->serialize_lock; | ||
| 379 | set_bit(V4L2_FL_USE_FH_PRIO, &s->video_dev->flags); | 379 | set_bit(V4L2_FL_USE_FH_PRIO, &s->video_dev->flags); |
| 380 | cx18_set_funcs(s->video_dev); | 380 | cx18_set_funcs(s->video_dev); |
| 381 | return 0; | 381 | return 0; |
