[media] tm6000: don't use BKL at the driver
authorMauro Carvalho Chehab <mchehab@redhat.com>
Sat, 9 Oct 2010 20:05:14 +0000 (17:05 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Thu, 21 Oct 2010 13:35:25 +0000 (11:35 -0200)
Instead, use core lock handling.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/staging/tm6000/tm6000-cards.c
drivers/staging/tm6000/tm6000-core.c
drivers/staging/tm6000/tm6000-video.c

index 9d091c34991b24986db43c51bd79dbc4bef302e9..4106ae009f9d12e0740a502fb93bd6f26c657a3f 100644 (file)
@@ -909,8 +909,6 @@ static void tm6000_usb_disconnect(struct usb_interface *interface)
 
        printk(KERN_INFO "tm6000: disconnecting %s\n", dev->name);
 
-       mutex_lock(&dev->lock);
-
        tm6000_ir_fini(dev);
 
        if (dev->gpio.power_led) {
@@ -945,7 +943,6 @@ static void tm6000_usb_disconnect(struct usb_interface *interface)
        tm6000_close_extension(dev);
        tm6000_remove_from_devlist(dev);
 
-       mutex_unlock(&dev->lock);
        kfree(dev);
 }
 
index 0912639369d95e8acf4652606b50a747eda82ff1..af41bbe2d2365626df5e7d1273b954b31db3b14e 100644 (file)
@@ -313,10 +313,8 @@ int tm6000_init_analog_mode(struct tm6000_core *dev)
         * beginning, we needed to add this hack. The better would be to
         * discover some way to make tm6000 to wake up without this hack.
         */
-       mutex_lock(&dev->lock);
        f.frequency = dev->freq;
        v4l2_device_call_all(&dev->v4l2_dev, 0, tuner, s_frequency, &f);
-       mutex_unlock(&dev->lock);
 
        msleep(100);
        tm6000_set_standard(dev, &dev->norm);
index 84ab49d1618c93dfa974a28d3d92a1d6c7d341ab..4a5fe6670fb4153ee83b66a49acb0cd61949ef6d 100644 (file)
@@ -791,16 +791,11 @@ static struct videobuf_queue_ops tm6000_video_qops = {
 static int res_get(struct tm6000_core *dev, struct tm6000_fh *fh)
 {
        /* is it free? */
-       mutex_lock(&dev->lock);
-       if (dev->resources) {
-               /* no, someone else uses it */
-               mutex_unlock(&dev->lock);
+       if (dev->resources)
                return 0;
-       }
        /* it's free, grab it */
        dev->resources =1;
        dprintk(dev, V4L2_DEBUG_RES_LOCK, "res: get\n");
-       mutex_unlock(&dev->lock);
        return 1;
 }
 
@@ -811,10 +806,8 @@ static int res_locked(struct tm6000_core *dev)
 
 static void res_free(struct tm6000_core *dev, struct tm6000_fh *fh)
 {
-       mutex_lock(&dev->lock);
        dev->resources = 0;
        dprintk(dev, V4L2_DEBUG_RES_LOCK, "res: put\n");
-       mutex_unlock(&dev->lock);
 }
 
 /* ------------------------------------------------------------------
@@ -1229,10 +1222,8 @@ static int vidioc_s_frequency (struct file *file, void *priv,
        if (unlikely(f->tuner != 0))
                return -EINVAL;
 
-//     mutex_lock(&dev->lock);
        dev->freq = f->frequency;
        v4l2_device_call_all(&dev->v4l2_dev, 0, tuner, s_frequency, f);
-//     mutex_unlock(&dev->lock);
 
        return 0;
 }
@@ -1308,7 +1299,7 @@ static int tm6000_open(struct file *file)
                        NULL, &dev->slock,
                        fh->type,
                        V4L2_FIELD_INTERLACED,
-                       sizeof(struct tm6000_buffer), fh, NULL);
+                       sizeof(struct tm6000_buffer), fh, &dev->lock);
 
        return 0;
 }
@@ -1389,7 +1380,7 @@ static struct v4l2_file_operations tm6000_fops = {
        .owner          = THIS_MODULE,
        .open           = tm6000_open,
        .release        = tm6000_release,
-       .ioctl          = video_ioctl2, /* V4L2 ioctl handler */
+       .unlocked_ioctl = video_ioctl2, /* V4L2 ioctl handler */
        .read           = tm6000_read,
        .poll           = tm6000_poll,
        .mmap           = tm6000_mmap,
@@ -1451,8 +1442,10 @@ int tm6000_v4l2_register(struct tm6000_core *dev)
        INIT_LIST_HEAD(&dev->vidq.active);
        INIT_LIST_HEAD(&dev->vidq.queued);
 
-       memcpy (dev->vfd, &tm6000_template, sizeof(*(dev->vfd)));
-       dev->vfd->debug=tm6000_debug;
+       memcpy(dev->vfd, &tm6000_template, sizeof(*(dev->vfd)));
+       dev->vfd->debug = tm6000_debug;
+       dev->vfd->lock = &dev->lock;
+
        vfd->v4l2_dev = &dev->v4l2_dev;
        video_set_drvdata(vfd, dev);