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

[Marcel Holtmann]

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 
> Reviewed-by: Rocky Liao 
> Tested-by: Rocky Liao 
> Tested-by: Claire Chang 
> ---
> 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

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

[Balakrishna Godavarthi]

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 
Reviewed-by: Rocky Liao 
Tested-by: Rocky Liao 
Tested-by: Claire Chang 
---
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