V4L/DVB (8731): zr364xx: remove BKL
authorAntoine Jacquet <royale@zerezo.com>
Mon, 18 Aug 2008 20:09:53 +0000 (17:09 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Sun, 12 Oct 2008 11:36:51 +0000 (09:36 -0200)
Remove the Big Kernel Lock from zr364xx driver after pushdown.
Now using an internal locking mecanism on open().

Signed-off-by: Antoine Jacquet <royale@zerezo.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/video/zr364xx.c

index 4e1ef10d22df0e2fd00c8fe00e42ecf0d9925a3d..9d5d5985a7ea6ba2fb638145240ae8c9a7a50ca9 100644 (file)
@@ -641,25 +641,23 @@ static int zr364xx_open(struct inode *inode, struct file *file)
 
        DBG("zr364xx_open");
 
+       mutex_lock(&cam->lock);
+
        cam->skip = 2;
 
-       lock_kernel();
        err = video_exclusive_open(inode, file);
-       if (err < 0) {
-               unlock_kernel();
-               return err;
-       }
+       if (err < 0)
+               goto out;
 
        if (!cam->framebuf) {
                cam->framebuf = vmalloc_32(MAX_FRAME_SIZE * FRAMES);
                if (!cam->framebuf) {
                        info("vmalloc_32 failed!");
-                       unlock_kernel();
-                       return -ENOMEM;
+                       err = -ENOMEM;
+                       goto out;
                }
        }
 
-       mutex_lock(&cam->lock);
        for (i = 0; init[cam->method][i].size != -1; i++) {
                err =
                    send_control_msg(udev, 1, init[cam->method][i].value,
@@ -667,9 +665,7 @@ static int zr364xx_open(struct inode *inode, struct file *file)
                                     init[cam->method][i].size);
                if (err < 0) {
                        info("error during open sequence: %d", i);
-                       mutex_unlock(&cam->lock);
-                       unlock_kernel();
-                       return err;
+                       goto out;
                }
        }
 
@@ -679,10 +675,11 @@ static int zr364xx_open(struct inode *inode, struct file *file)
         * like Ekiga does during its startup, can crash the webcam
         */
        mdelay(100);
+       err = 0;
 
+out:
        mutex_unlock(&cam->lock);
-       unlock_kernel();
-       return 0;
+       return err;
 }