Re: [PATCH v3 1/4] Bluetooth: hci_qca: use wait_until_sent() for power pulses

2018-12-14 Thread Balakrishna Godavarthi
Hi Johan, On 2018-12-14 18:46, Johan Hovold wrote: On Fri, Dec 14, 2018 at 06:02:40PM +0530, Balakrishna Godavarthi wrote: Hi Johan, On 2018-12-12 22:12, Johan Hovold wrote: > On Thu, Dec 06, 2018 at 04:10:07PM +0530, Balakrishna Godavarthi wrote: >>

Re: [PATCH v3 1/4] Bluetooth: hci_qca: use wait_until_sent() for power pulses

2018-12-14 Thread Johan Hovold
On Fri, Dec 14, 2018 at 06:02:40PM +0530, Balakrishna Godavarthi wrote: > Hi Johan, > > On 2018-12-12 22:12, Johan Hovold wrote: > > On Thu, Dec 06, 2018 at 04:10:07PM +0530, Balakrishna Godavarthi wrote: > >>

Re: [PATCH v3 1/4] Bluetooth: hci_qca: use wait_until_sent() for power pulses

2018-12-14 Thread Balakrishna Godavarthi
Hi Johan, On 2018-12-12 22:12, Johan Hovold wrote: On Thu, Dec 06, 2018 at 04:10:07PM +0530, Balakrishna Godavarthi wrote: Hi Johan, On 2018-12-05 11:55, Johan Hovold wrote: > On Fri, Nov 30, 2018 at 08:32:44PM +0530, Balakrishna Godavarthi wrote: >> + ret =

Re: [PATCH v3 1/4] Bluetooth: hci_qca: use wait_until_sent() for power pulses

2018-12-12 Thread Johan Hovold
On Thu, Dec 06, 2018 at 04:10:07PM +0530, Balakrishna Godavarthi wrote: > Hi Johan, > > On 2018-12-05 11:55, Johan Hovold wrote: > > On Fri, Nov 30, 2018 at 08:32:44PM +0530, Balakrishna Godavarthi wrote: > >> + ret = serdev_device_write(hu->serdev, , sizeof(cmd), 0); > > > > You're still

Re: [PATCH v3 1/4] Bluetooth: hci_qca: use wait_until_sent() for power pulses

2018-12-11 Thread Balakrishna Godavarthi
Hi Johan, On 2018-12-06 16:10, Balakrishna Godavarthi wrote: Hi Johan, On 2018-12-05 11:55, Johan Hovold wrote: On Fri, Nov 30, 2018 at 08:32:44PM +0530, Balakrishna Godavarthi wrote: wcn3990 requires a power pulse to turn ON/OFF along with regulators. Sometimes we are observing the power

Re: [PATCH v3 1/4] Bluetooth: hci_qca: use wait_until_sent() for power pulses

2018-12-06 Thread Balakrishna Godavarthi
Hi Johan, On 2018-12-05 11:55, Johan Hovold wrote: On Fri, Nov 30, 2018 at 08:32:44PM +0530, Balakrishna Godavarthi wrote: wcn3990 requires a power pulse to turn ON/OFF along with regulators. Sometimes we are observing the power pulses are sent out with some time delay, due to queuing these

Re: [PATCH v3 1/4] Bluetooth: hci_qca: use wait_until_sent() for power pulses

2018-12-06 Thread Balakrishna Godavarthi
Hi Johan, On 2018-12-05 11:55, Johan Hovold wrote: On Fri, Nov 30, 2018 at 08:32:44PM +0530, Balakrishna Godavarthi wrote: wcn3990 requires a power pulse to turn ON/OFF along with regulators. Sometimes we are observing the power pulses are sent out with some time delay, due to queuing these

Re: [PATCH v3 1/4] Bluetooth: hci_qca: use wait_until_sent() for power pulses

2018-12-04 Thread Johan Hovold
On Fri, Nov 30, 2018 at 08:32:44PM +0530, Balakrishna Godavarthi wrote: > wcn3990 requires a power pulse to turn ON/OFF along with > regulators. Sometimes we are observing the power pulses are sent > out with some time delay, due to queuing these commands. This is > causing synchronization issues

Re: [PATCH v3 1/4] Bluetooth: hci_qca: use wait_until_sent() for power pulses

2018-12-04 Thread Johan Hovold
On Fri, Nov 30, 2018 at 08:32:44PM +0530, Balakrishna Godavarthi wrote: > wcn3990 requires a power pulse to turn ON/OFF along with > regulators. Sometimes we are observing the power pulses are sent > out with some time delay, due to queuing these commands. This is > causing synchronization issues

[PATCH v3 1/4] Bluetooth: hci_qca: use wait_until_sent() for power pulses

2018-11-30 Thread Balakrishna Godavarthi
wcn3990 requires a power pulse to turn ON/OFF along with regulators. Sometimes we are observing the power pulses are sent out with some time delay, due to queuing these commands. This is causing synchronization issues with chip, which intern delay the chip setup or may end up with communication

[PATCH v3 1/4] Bluetooth: hci_qca: use wait_until_sent() for power pulses

2018-11-30 Thread Balakrishna Godavarthi
wcn3990 requires a power pulse to turn ON/OFF along with regulators. Sometimes we are observing the power pulses are sent out with some time delay, due to queuing these commands. This is causing synchronization issues with chip, which intern delay the chip setup or may end up with communication