Bluetooth: hci_bcm: Replace spinlock by mutex
authorFrederic Danis <frederic.danis@linux.intel.com>
Tue, 1 Sep 2015 10:13:35 +0000 (12:13 +0200)
committerMarcel Holtmann <marcel@holtmann.org>
Thu, 17 Sep 2015 11:20:01 +0000 (13:20 +0200)
Replace spinlock by mutex to be able to use bcm_device_lock in
sleepable context like devm_request_threaded_irq or upcomming PM support.

Signed-off-by: Frederic Danis <frederic.danis@linux.intel.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
drivers/bluetooth/hci_bcm.c

index 835bfab88ef56acecbf9887d2e13ea912af750cf..42ab91d52a8225bda385193976a467a0a8610de7 100644 (file)
@@ -66,7 +66,7 @@ struct bcm_data {
 };
 
 /* List of BCM BT UART devices */
-static DEFINE_SPINLOCK(bcm_device_lock);
+static DEFINE_MUTEX(bcm_device_lock);
 static LIST_HEAD(bcm_device_list);
 
 static int bcm_set_baudrate(struct hci_uart *hu, unsigned int speed)
@@ -164,7 +164,7 @@ static int bcm_open(struct hci_uart *hu)
 
        hu->priv = bcm;
 
-       spin_lock(&bcm_device_lock);
+       mutex_lock(&bcm_device_lock);
        list_for_each(p, &bcm_device_list) {
                struct bcm_device *dev = list_entry(p, struct bcm_device, list);
 
@@ -185,7 +185,7 @@ static int bcm_open(struct hci_uart *hu)
        if (bcm->dev)
                bcm_gpio_set_power(bcm->dev, true);
 
-       spin_unlock(&bcm_device_lock);
+       mutex_unlock(&bcm_device_lock);
 
        return 0;
 }
@@ -197,14 +197,14 @@ static int bcm_close(struct hci_uart *hu)
        BT_DBG("hu %p", hu);
 
        /* Protect bcm->dev against removal of the device or driver */
-       spin_lock(&bcm_device_lock);
+       mutex_lock(&bcm_device_lock);
        if (bcm_device_exists(bcm->dev)) {
                bcm_gpio_set_power(bcm->dev, false);
 #ifdef CONFIG_PM_SLEEP
                bcm->dev->hu = NULL;
 #endif
        }
-       spin_unlock(&bcm_device_lock);
+       mutex_unlock(&bcm_device_lock);
 
        skb_queue_purge(&bcm->txq);
        kfree_skb(bcm->rx_skb);
@@ -338,7 +338,7 @@ static int bcm_suspend(struct device *dev)
 
        BT_DBG("suspend (%p): is_suspended %d", bdev, bdev->is_suspended);
 
-       spin_lock(&bcm_device_lock);
+       mutex_lock(&bcm_device_lock);
 
        if (!bdev->hu)
                goto unlock;
@@ -358,7 +358,7 @@ static int bcm_suspend(struct device *dev)
        }
 
 unlock:
-       spin_unlock(&bcm_device_lock);
+       mutex_unlock(&bcm_device_lock);
 
        return 0;
 }
@@ -370,7 +370,7 @@ static int bcm_resume(struct device *dev)
 
        BT_DBG("resume (%p): is_suspended %d", bdev, bdev->is_suspended);
 
-       spin_lock(&bcm_device_lock);
+       mutex_lock(&bcm_device_lock);
 
        if (!bdev->hu)
                goto unlock;
@@ -389,7 +389,7 @@ static int bcm_resume(struct device *dev)
        }
 
 unlock:
-       spin_unlock(&bcm_device_lock);
+       mutex_unlock(&bcm_device_lock);
 
        return 0;
 }
@@ -504,9 +504,9 @@ static int bcm_probe(struct platform_device *pdev)
        dev_info(&pdev->dev, "%s device registered.\n", dev->name);
 
        /* Place this instance on the device list */
-       spin_lock(&bcm_device_lock);
+       mutex_lock(&bcm_device_lock);
        list_add_tail(&dev->list, &bcm_device_list);
-       spin_unlock(&bcm_device_lock);
+       mutex_unlock(&bcm_device_lock);
 
        bcm_gpio_set_power(dev, false);
 
@@ -517,9 +517,9 @@ static int bcm_remove(struct platform_device *pdev)
 {
        struct bcm_device *dev = platform_get_drvdata(pdev);
 
-       spin_lock(&bcm_device_lock);
+       mutex_lock(&bcm_device_lock);
        list_del(&dev->list);
-       spin_unlock(&bcm_device_lock);
+       mutex_unlock(&bcm_device_lock);
 
        acpi_dev_remove_driver_gpios(ACPI_COMPANION(&pdev->dev));