* [PATCH v2 0/3] Bluetooth: hci_qca: Add delay after power-off pulse
@ 2019-02-26 19:46 Matthias Kaehlcke
2019-02-26 19:46 ` [PATCH v2 1/3] Bluetooth: hci_qca: Pass boolean 'on/off' to qca_send_power_pulse() Matthias Kaehlcke
` (3 more replies)
0 siblings, 4 replies; 6+ messages in thread
From: Matthias Kaehlcke @ 2019-02-26 19:46 UTC (permalink / raw)
To: Marcel Holtmann, Johan Hedberg
Cc: linux-bluetooth, linux-kernel, Balakrishna Godavarthi, Hemantg,
Matthias Kaehlcke
Initialization of the QCA WCN3990 often fails at boot time:
[ 15.205224] Bluetooth: hci0: setting up wcn3990
[ 17.341062] Bluetooth: hci0: command 0xfc00 tx timeout
[ 22.101453] ERROR: Bluetooth initialization failed
[ 25.337740] Bluetooth: hci0: Reading QCA version information failed (-110)
A short delay after sending a power-off pulse (which is done immediately
before sending a power-on pulse) fixes this.
Also move the delays to qca_send_power_pulse(), since they are directly
related with sending the pulses. Change the interface of
qca_send_power_pulse() to receive a boolean instead of a power pulse
command, this improves readability of the function with the delays
added.
Matthias Kaehlcke (3):
Bluetooth: hci_qca: Pass boolean 'on/off' to qca_send_power_pulse()
Bluetooth: hci_qca: Move boot delay to qca_send_power_pulse()
Bluetooth: hci_qca: Add delay after power-off pulse
drivers/bluetooth/hci_qca.c | 21 +++++++++++----------
1 file changed, 11 insertions(+), 10 deletions(-)
--
2.21.0.rc2.261.ga7da99ff1b-goog
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2 1/3] Bluetooth: hci_qca: Pass boolean 'on/off' to qca_send_power_pulse()
2019-02-26 19:46 [PATCH v2 0/3] Bluetooth: hci_qca: Add delay after power-off pulse Matthias Kaehlcke
@ 2019-02-26 19:46 ` Matthias Kaehlcke
2019-02-27 7:48 ` Marcel Holtmann
2019-02-26 19:46 ` [PATCH v2 2/3] Bluetooth: hci_qca: Move boot delay " Matthias Kaehlcke
` (2 subsequent siblings)
3 siblings, 1 reply; 6+ messages in thread
From: Matthias Kaehlcke @ 2019-02-26 19:46 UTC (permalink / raw)
To: Marcel Holtmann, Johan Hedberg
Cc: linux-bluetooth, linux-kernel, Balakrishna Godavarthi, Hemantg,
Matthias Kaehlcke
There are only two types of power pulses 'on' or 'off', pass a boolean
instead of the power pulse 'command'.
Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
Reviewed-by: Balakrishna Godavarthi <bgodavar@codeaurora.org>
---
Changes in v2:
- added 'Reviewed-by: Balakrishna Godavarthi <bgodavar@codeaurora.org>' tag
---
drivers/bluetooth/hci_qca.c | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c
index 5e03504c4e0ca..e4128774e9686 100644
--- a/drivers/bluetooth/hci_qca.c
+++ b/drivers/bluetooth/hci_qca.c
@@ -1004,10 +1004,11 @@ static inline void host_set_baudrate(struct hci_uart *hu, unsigned int speed)
hci_uart_set_baudrate(hu, speed);
}
-static int qca_send_power_pulse(struct hci_uart *hu, u8 cmd)
+static int qca_send_power_pulse(struct hci_uart *hu, bool on)
{
int ret;
int timeout = msecs_to_jiffies(POWER_PULSE_TRANS_TIMEOUT_MS);
+ u8 cmd = on ? QCA_WCN3990_POWERON_PULSE : QCA_WCN3990_POWEROFF_PULSE;
/* These power pulses are single byte command which are sent
* at required baudrate to wcn3990. On wcn3990, we have an external
@@ -1138,12 +1139,12 @@ static int qca_wcn3990_init(struct hci_uart *hu)
/* Forcefully enable wcn3990 to enter in to boot mode. */
host_set_baudrate(hu, 2400);
- ret = qca_send_power_pulse(hu, QCA_WCN3990_POWEROFF_PULSE);
+ ret = qca_send_power_pulse(hu, false);
if (ret)
return ret;
qca_set_speed(hu, QCA_INIT_SPEED);
- ret = qca_send_power_pulse(hu, QCA_WCN3990_POWERON_PULSE);
+ ret = qca_send_power_pulse(hu, true);
if (ret)
return ret;
@@ -1289,7 +1290,7 @@ static void qca_power_shutdown(struct hci_uart *hu)
spin_unlock_irqrestore(&qca->hci_ibs_lock, flags);
host_set_baudrate(hu, 2400);
- qca_send_power_pulse(hu, QCA_WCN3990_POWEROFF_PULSE);
+ qca_send_power_pulse(hu, false);
qca_power_setup(hu, false);
}
--
2.21.0.rc2.261.ga7da99ff1b-goog
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH v2 2/3] Bluetooth: hci_qca: Move boot delay to qca_send_power_pulse()
2019-02-26 19:46 [PATCH v2 0/3] Bluetooth: hci_qca: Add delay after power-off pulse Matthias Kaehlcke
2019-02-26 19:46 ` [PATCH v2 1/3] Bluetooth: hci_qca: Pass boolean 'on/off' to qca_send_power_pulse() Matthias Kaehlcke
@ 2019-02-26 19:46 ` Matthias Kaehlcke
2019-02-26 19:46 ` [PATCH v2 3/3] Bluetooth: hci_qca: Add delay after power-off pulse Matthias Kaehlcke
2019-02-27 6:33 ` [PATCH v2 0/3] " Bjorn Andersson
3 siblings, 0 replies; 6+ messages in thread
From: Matthias Kaehlcke @ 2019-02-26 19:46 UTC (permalink / raw)
To: Marcel Holtmann, Johan Hedberg
Cc: linux-bluetooth, linux-kernel, Balakrishna Godavarthi, Hemantg,
Matthias Kaehlcke
After sending a power on pulse the driver has a delay of 100ms
to allow the host controller to boot. Move the delay into
qca_send_power_pulse(), since it is directly related with the
power-on pulse.
Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
Reviewed-by: Balakrishna Godavarthi <bgodavar@codeaurora.org>
---
Changes in v2:
- added 'Reviewed-by: Balakrishna Godavarthi <bgodavar@codeaurora.org>' tag
---
drivers/bluetooth/hci_qca.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c
index e4128774e9686..eacc108c422d0 100644
--- a/drivers/bluetooth/hci_qca.c
+++ b/drivers/bluetooth/hci_qca.c
@@ -1036,6 +1036,9 @@ static int qca_send_power_pulse(struct hci_uart *hu, bool on)
usleep_range(100, 200);
hci_uart_set_flow_control(hu, false);
+ if (on)
+ msleep(100);
+
return 0;
}
@@ -1148,9 +1151,6 @@ static int qca_wcn3990_init(struct hci_uart *hu)
if (ret)
return ret;
- /* Wait for 100 ms for SoC to boot */
- msleep(100);
-
/* Now the device is in ready state to communicate with host.
* To sync host with device we need to reopen port.
* Without this, we will have RTS and CTS synchronization
--
2.21.0.rc2.261.ga7da99ff1b-goog
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH v2 3/3] Bluetooth: hci_qca: Add delay after power-off pulse
2019-02-26 19:46 [PATCH v2 0/3] Bluetooth: hci_qca: Add delay after power-off pulse Matthias Kaehlcke
2019-02-26 19:46 ` [PATCH v2 1/3] Bluetooth: hci_qca: Pass boolean 'on/off' to qca_send_power_pulse() Matthias Kaehlcke
2019-02-26 19:46 ` [PATCH v2 2/3] Bluetooth: hci_qca: Move boot delay " Matthias Kaehlcke
@ 2019-02-26 19:46 ` Matthias Kaehlcke
2019-02-27 6:33 ` [PATCH v2 0/3] " Bjorn Andersson
3 siblings, 0 replies; 6+ messages in thread
From: Matthias Kaehlcke @ 2019-02-26 19:46 UTC (permalink / raw)
To: Marcel Holtmann, Johan Hedberg
Cc: linux-bluetooth, linux-kernel, Balakrishna Godavarthi, Hemantg,
Matthias Kaehlcke
During initialization the power-on pulse is currently sent inmediately
after the prior power-off pulse. With this initialization often fails
at boot time:
[ 15.205224] Bluetooth: hci0: setting up wcn3990
[ 17.341062] Bluetooth: hci0: command 0xfc00 tx timeout
[ 22.101453] ERROR: Bluetooth initialization failed
[ 25.337740] Bluetooth: hci0: Reading QCA version information failed (-110)
After a power-off pulse wait 10ms to give the controller time to power
off. Remove the previous short settling delay, it isn't needed anymore.
Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
Reviewed-by: Balakrishna Godavarthi <bgodavar@codeaurora.org>
---
Changes in v2:
- removed previous short setting delay
- updated commit message
- added 'Reviewed-by: Balakrishna Godavarthi <bgodavar@codeaurora.org>' tag
---
drivers/bluetooth/hci_qca.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c
index eacc108c422d0..3f77140798dd1 100644
--- a/drivers/bluetooth/hci_qca.c
+++ b/drivers/bluetooth/hci_qca.c
@@ -1031,13 +1031,13 @@ static int qca_send_power_pulse(struct hci_uart *hu, bool on)
}
serdev_device_wait_until_sent(hu->serdev, timeout);
-
- /* Wait for 100 uS for SoC to settle down */
- usleep_range(100, 200);
hci_uart_set_flow_control(hu, false);
+ /* Give to controller time to boot/shutdown */
if (on)
msleep(100);
+ else
+ msleep(10);
return 0;
}
--
2.21.0.rc2.261.ga7da99ff1b-goog
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2 0/3] Bluetooth: hci_qca: Add delay after power-off pulse
2019-02-26 19:46 [PATCH v2 0/3] Bluetooth: hci_qca: Add delay after power-off pulse Matthias Kaehlcke
` (2 preceding siblings ...)
2019-02-26 19:46 ` [PATCH v2 3/3] Bluetooth: hci_qca: Add delay after power-off pulse Matthias Kaehlcke
@ 2019-02-27 6:33 ` Bjorn Andersson
3 siblings, 0 replies; 6+ messages in thread
From: Bjorn Andersson @ 2019-02-27 6:33 UTC (permalink / raw)
To: Matthias Kaehlcke
Cc: Marcel Holtmann, Johan Hedberg, open list:BLUETOOTH DRIVERS,
lkml, Balakrishna Godavarthi, Hemantg
On Tue, Feb 26, 2019 at 11:47 AM Matthias Kaehlcke <mka@chromium.org> wrote:
>
> Initialization of the QCA WCN3990 often fails at boot time:
>
> [ 15.205224] Bluetooth: hci0: setting up wcn3990
> [ 17.341062] Bluetooth: hci0: command 0xfc00 tx timeout
> [ 22.101453] ERROR: Bluetooth initialization failed
> [ 25.337740] Bluetooth: hci0: Reading QCA version information failed (-110)
>
> A short delay after sending a power-off pulse (which is done immediately
> before sending a power-on pulse) fixes this.
>
> Also move the delays to qca_send_power_pulse(), since they are directly
> related with sending the pulses. Change the interface of
> qca_send_power_pulse() to receive a boolean instead of a power pulse
> command, this improves readability of the function with the delays
> added.
>
For the series:
Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Regards,
Bjorn
> Matthias Kaehlcke (3):
> Bluetooth: hci_qca: Pass boolean 'on/off' to qca_send_power_pulse()
> Bluetooth: hci_qca: Move boot delay to qca_send_power_pulse()
> Bluetooth: hci_qca: Add delay after power-off pulse
>
> drivers/bluetooth/hci_qca.c | 21 +++++++++++----------
> 1 file changed, 11 insertions(+), 10 deletions(-)
>
> --
> 2.21.0.rc2.261.ga7da99ff1b-goog
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2 1/3] Bluetooth: hci_qca: Pass boolean 'on/off' to qca_send_power_pulse()
2019-02-26 19:46 ` [PATCH v2 1/3] Bluetooth: hci_qca: Pass boolean 'on/off' to qca_send_power_pulse() Matthias Kaehlcke
@ 2019-02-27 7:48 ` Marcel Holtmann
0 siblings, 0 replies; 6+ messages in thread
From: Marcel Holtmann @ 2019-02-27 7:48 UTC (permalink / raw)
To: Matthias Kaehlcke
Cc: Johan Hedberg, linux-bluetooth, linux-kernel,
Balakrishna Godavarthi, Hemantg
Hi Matthias,
> There are only two types of power pulses 'on' or 'off', pass a boolean
> instead of the power pulse 'command'.
>
> Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
> Reviewed-by: Balakrishna Godavarthi <bgodavar@codeaurora.org>
> ---
> Changes in v2:
> - added 'Reviewed-by: Balakrishna Godavarthi <bgodavar@codeaurora.org>' tag
> ---
> drivers/bluetooth/hci_qca.c | 9 +++++----
> 1 file changed, 5 insertions(+), 4 deletions(-)
all 3 patches have been applied to bluetooth-next tree.
Regards
Marcel
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-02-27 7:48 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-02-26 19:46 [PATCH v2 0/3] Bluetooth: hci_qca: Add delay after power-off pulse Matthias Kaehlcke
2019-02-26 19:46 ` [PATCH v2 1/3] Bluetooth: hci_qca: Pass boolean 'on/off' to qca_send_power_pulse() Matthias Kaehlcke
2019-02-27 7:48 ` Marcel Holtmann
2019-02-26 19:46 ` [PATCH v2 2/3] Bluetooth: hci_qca: Move boot delay " Matthias Kaehlcke
2019-02-26 19:46 ` [PATCH v2 3/3] Bluetooth: hci_qca: Add delay after power-off pulse Matthias Kaehlcke
2019-02-27 6:33 ` [PATCH v2 0/3] " Bjorn Andersson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).