diff options
author | Matthias Kaehlcke <matthias.kaehlcke@gmail.com> | 2007-07-02 09:04:52 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@infradead.org> | 2007-07-18 13:24:28 -0400 |
commit | b9378fdbc334d1575b492108eac822a78c0c46d9 (patch) | |
tree | 9a78a121b72309a4471dee193ceefc5cb17ce520 /drivers/media/video/pwc/pwc-if.c | |
parent | 55c0d1005a0e5f590f71f918e49bdc81362f93a6 (diff) |
V4L/DVB (5809): Use mutex instead of semaphore in Philips webcam driver
The Philips webcam driver uses a semaphore as mutex. Use the mutex API
instead of the (binary) semaphore.
--
Signed-off-by: Matthias Kaehlcke <matthias.kaehlcke@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
Diffstat (limited to 'drivers/media/video/pwc/pwc-if.c')
-rw-r--r-- | drivers/media/video/pwc/pwc-if.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/drivers/media/video/pwc/pwc-if.c b/drivers/media/video/pwc/pwc-if.c index 085332a503de..9c0e8d18c2f6 100644 --- a/drivers/media/video/pwc/pwc-if.c +++ b/drivers/media/video/pwc/pwc-if.c | |||
@@ -1099,7 +1099,7 @@ static int pwc_video_open(struct inode *inode, struct file *file) | |||
1099 | return -EBUSY; | 1099 | return -EBUSY; |
1100 | } | 1100 | } |
1101 | 1101 | ||
1102 | down(&pdev->modlock); | 1102 | mutex_lock(&pdev->modlock); |
1103 | if (!pdev->usb_init) { | 1103 | if (!pdev->usb_init) { |
1104 | PWC_DEBUG_OPEN("Doing first time initialization.\n"); | 1104 | PWC_DEBUG_OPEN("Doing first time initialization.\n"); |
1105 | pdev->usb_init = 1; | 1105 | pdev->usb_init = 1; |
@@ -1131,7 +1131,7 @@ static int pwc_video_open(struct inode *inode, struct file *file) | |||
1131 | if (i < 0) { | 1131 | if (i < 0) { |
1132 | PWC_DEBUG_OPEN("Failed to allocate buffers memory.\n"); | 1132 | PWC_DEBUG_OPEN("Failed to allocate buffers memory.\n"); |
1133 | pwc_free_buffers(pdev); | 1133 | pwc_free_buffers(pdev); |
1134 | up(&pdev->modlock); | 1134 | mutex_unlock(&pdev->modlock); |
1135 | return i; | 1135 | return i; |
1136 | } | 1136 | } |
1137 | 1137 | ||
@@ -1172,7 +1172,7 @@ static int pwc_video_open(struct inode *inode, struct file *file) | |||
1172 | if (i) { | 1172 | if (i) { |
1173 | PWC_DEBUG_OPEN("Second attempt at set_video_mode failed.\n"); | 1173 | PWC_DEBUG_OPEN("Second attempt at set_video_mode failed.\n"); |
1174 | pwc_free_buffers(pdev); | 1174 | pwc_free_buffers(pdev); |
1175 | up(&pdev->modlock); | 1175 | mutex_unlock(&pdev->modlock); |
1176 | return i; | 1176 | return i; |
1177 | } | 1177 | } |
1178 | 1178 | ||
@@ -1181,7 +1181,7 @@ static int pwc_video_open(struct inode *inode, struct file *file) | |||
1181 | PWC_DEBUG_OPEN("Failed to init ISOC stuff = %d.\n", i); | 1181 | PWC_DEBUG_OPEN("Failed to init ISOC stuff = %d.\n", i); |
1182 | pwc_isoc_cleanup(pdev); | 1182 | pwc_isoc_cleanup(pdev); |
1183 | pwc_free_buffers(pdev); | 1183 | pwc_free_buffers(pdev); |
1184 | up(&pdev->modlock); | 1184 | mutex_unlock(&pdev->modlock); |
1185 | return i; | 1185 | return i; |
1186 | } | 1186 | } |
1187 | 1187 | ||
@@ -1191,7 +1191,7 @@ static int pwc_video_open(struct inode *inode, struct file *file) | |||
1191 | 1191 | ||
1192 | pdev->vopen++; | 1192 | pdev->vopen++; |
1193 | file->private_data = vdev; | 1193 | file->private_data = vdev; |
1194 | up(&pdev->modlock); | 1194 | mutex_unlock(&pdev->modlock); |
1195 | PWC_DEBUG_OPEN("<< video_open() returns 0.\n"); | 1195 | PWC_DEBUG_OPEN("<< video_open() returns 0.\n"); |
1196 | return 0; | 1196 | return 0; |
1197 | } | 1197 | } |
@@ -1685,7 +1685,7 @@ static int usb_pwc_probe(struct usb_interface *intf, const struct usb_device_id | |||
1685 | pdev->angle_range.tilt_max = 2500; | 1685 | pdev->angle_range.tilt_max = 2500; |
1686 | } | 1686 | } |
1687 | 1687 | ||
1688 | init_MUTEX(&pdev->modlock); | 1688 | mutex_init(&pdev->modlock); |
1689 | spin_lock_init(&pdev->ptrlock); | 1689 | spin_lock_init(&pdev->ptrlock); |
1690 | 1690 | ||
1691 | pdev->udev = udev; | 1691 | pdev->udev = udev; |