2019-04-01 09:49:30

by Balakrishna Godavarthi

Subject: [PATCH v2] Bluetooth: hci_qca: Give enough time to ROME controller to bootup.

This patch enables enough time to ROME controller to bootup
after we bring the enable pin out of reset.

Fixes: 05ba533c5c11 ("Bluetooth: hci_qca: Add serdev support").
Signed-off-by: Balakrishna Godavarthi <[email protected]>
Reviewed-by: Rocky Liao <[email protected]>
Tested-by: Rocky Liao <[email protected]>
Tested-by: Claire Chang <[email protected]>
---
Changes in v2:
* added fixes tag in the commit text.
---
drivers/bluetooth/hci_qca.c | 2 ++
1 file changed, 2 insertions(+)

diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c
index 4ea995d610d2..a80c3bc90904 100644
--- a/drivers/bluetooth/hci_qca.c
+++ b/drivers/bluetooth/hci_qca.c
@@ -523,6 +523,8 @@ static int qca_open(struct hci_uart *hu)
qcadev = serdev_device_get_drvdata(hu->serdev);
if (qcadev->btsoc_type != QCA_WCN3990) {
gpiod_set_value_cansleep(qcadev->bt_en, 1);
+ /* Controller needs time to bootup. */
+ msleep(150);
} else {
hu->init_speed = qcadev->init_speed;
hu->oper_speed = qcadev->oper_speed;
--
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project

2019-04-23 17:30:19

by Marcel Holtmann

Subject: Re: [PATCH v2] Bluetooth: hci_qca: Give enough time to ROME controller to bootup.

Hi Balakrishna,

> This patch enables enough time to ROME controller to bootup
> after we bring the enable pin out of reset.
>
> Fixes: 05ba533c5c11 ("Bluetooth: hci_qca: Add serdev support").
> Signed-off-by: Balakrishna Godavarthi <[email protected]>
> Reviewed-by: Rocky Liao <[email protected]>
> Tested-by: Rocky Liao <[email protected]>
> Tested-by: Claire Chang <[email protected]>
> ---
> Changes in v2:
> * added fixes tag in the commit text.
> ---
> drivers/bluetooth/hci_qca.c | 2 ++
> 1 file changed, 2 insertions(+)

patch has been applied to bluetooth-next tree.

Regards

Marcel