diff mbox series

[-next] Bluetooth: btmtkuart: using pm_runtime_resume_and_get instead of pm_runtime_get_sync

Message ID 20210531142450.30779-1-zhangqilong3@huawei.com
State Accepted
Commit d6e15c04b70dec85b1f5d8d85b947a6758582246
Headers show
Series [-next] Bluetooth: btmtkuart: using pm_runtime_resume_and_get instead of pm_runtime_get_sync | expand

Commit Message

Zhang Qilong May 31, 2021, 2:24 p.m. UTC
Using pm_runtime_resume_and_get is more appropriate
for simplifing code

Signed-off-by: Zhang Qilong <zhangqilong3@huawei.com>
---
 drivers/bluetooth/btmtkuart.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

Comments

Marcel Holtmann June 2, 2021, 8:07 p.m. UTC | #1
Hi Zhang,

> Using pm_runtime_resume_and_get is more appropriate

> for simplifing code

> 

> Signed-off-by: Zhang Qilong <zhangqilong3@huawei.com>

> ---

> drivers/bluetooth/btmtkuart.c | 6 ++----

> 1 file changed, 2 insertions(+), 4 deletions(-)


patch has been applied to bluetooth-next tree.

Regards

Marcel
diff mbox series

Patch

diff --git a/drivers/bluetooth/btmtkuart.c b/drivers/bluetooth/btmtkuart.c
index 6c40bc75fb5b..e9d91d7c0db4 100644
--- a/drivers/bluetooth/btmtkuart.c
+++ b/drivers/bluetooth/btmtkuart.c
@@ -581,11 +581,9 @@  static int btmtkuart_open(struct hci_dev *hdev)
 
 	/* Enable the power domain and clock the device requires */
 	pm_runtime_enable(dev);
-	err = pm_runtime_get_sync(dev);
-	if (err < 0) {
-		pm_runtime_put_noidle(dev);
+	err = pm_runtime_resume_and_get(dev);
+	if (err < 0)
 		goto err_disable_rpm;
-	}
 
 	err = clk_prepare_enable(bdev->clk);
 	if (err < 0)