| Message ID | 1639373496-28009-1-git-send-email-quic_pharish@quicinc.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [v2] Bluetooth: hci_qca: Stop IBS timer during BT OFF | expand |
| Context | Check | Description |
|---|---|---|
| tedd_an/checkpatch | success | Checkpatch PASS |
| tedd_an/gitlint | success | Gitlint PASS |
| tedd_an/buildkernel | success | Build Kernel PASS |
| tedd_an/incremental_build | success | Pass |
| tedd_an/testrunnersetup | success | Test Runner Setup PASS |
| tedd_an/testrunnerl2cap-tester | success | Total: 40, Passed: 40 (100.0%), Failed: 0, Not Run: 0 |
| tedd_an/testrunnerbnep-tester | success | Total: 1, Passed: 1 (100.0%), Failed: 0, Not Run: 0 |
| tedd_an/testrunnermgmt-tester | success | Total: 493, Passed: 493 (100.0%), Failed: 0, Not Run: 0 |
| tedd_an/testrunnerrfcomm-tester | success | Total: 9, Passed: 9 (100.0%), Failed: 0, Not Run: 0 |
| tedd_an/testrunnersco-tester | success | Total: 12, Passed: 12 (100.0%), Failed: 0, Not Run: 0 |
| tedd_an/testrunnersmp-tester | success | Total: 8, Passed: 8 (100.0%), Failed: 0, Not Run: 0 |
| tedd_an/testrunneruserchan-tester | success | Total: 4, Passed: 4 (100.0%), Failed: 0, Not Run: 0 |
On Mon, Dec 13, 2021 at 11:01:36AM +0530, Panicker Harish wrote: > This change stops IBS timers during BT OFF. nit: avoid thing like 'this patch ...' in commit messages, you could just say 'Stop IBS timers while Bluetooth is off. > Signed-off-by: Panicker Harish <quic_pharish@quicinc.com> Looks like this should have a ´Fixes' tag to make sure it lands in the relevant -stable trees. Commit 3e4be65eb82c ("Bluetooth: hci_qca: Add poweroff support during hci down for wcn3990") would probably be a suitable commit for that.
-----Original Message----- From: Matthias Kaehlcke <mka@chromium.org> Sent: Monday, December 13, 2021 8:37 PM To: PANICKER HARISH (Temp) (QUIC) <quic_pharish@quicinc.com> Cc: marcel@holtmann.org; johan.hedberg@gmail.com; linux-kernel@vger.kernel.org; linux-bluetooth@vger.kernel.org; Hemant Gupta (QUIC) <quic_hemantg@quicinc.com>; linux-arm-msm@vger.kernel.org; quic_bgodavar <quic_bgodavar@quicinc.com>; rjliao@codeaurora.org; hbandi@codeaurora.org; abhishekpandit@chromium.org; mcchou@chromium.org; Sai Teja Aluvala (Temp) (QUIC) <quic_saluvala@quicinc.com> Subject: Re: [PATCH v2] Bluetooth: hci_qca: Stop IBS timer during BT OFF On Mon, Dec 13, 2021 at 11:01:36AM +0530, Panicker Harish wrote: > This change stops IBS timers during BT OFF. nit: avoid thing like 'this patch ...' in commit messages, you could just say 'Stop IBS timers while Bluetooth is off. [Harish] : I will update > Signed-off-by: Panicker Harish <quic_pharish@quicinc.com> Looks like this should have a ´Fixes' tag to make sure it lands in the relevant -stable trees. Commit 3e4be65eb82c ("Bluetooth: hci_qca: Add poweroff support during hci down for wcn3990") would probably be a suitable commit for that. [Harish] : I will update with Fixes tag.
diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c index dd768a8..6f44b26 100644 --- a/drivers/bluetooth/hci_qca.c +++ b/drivers/bluetooth/hci_qca.c @@ -1928,6 +1928,9 @@ static int qca_power_off(struct hci_dev *hdev) hu->hdev->hw_error = NULL; hu->hdev->cmd_timeout = NULL; + mod_timer(&qca->tx_idle_timer, 0); + mod_timer(&qca->wake_retrans_timer, 0); + /* Stop sending shutdown command if soc crashes. */ if (soc_type != QCA_ROME && qca->memdump_state == QCA_MEMDUMP_IDLE) {
This change stops IBS timers during BT OFF. Signed-off-by: Panicker Harish <quic_pharish@quicinc.com> v2: * Addressed the username * The full implementation of IBS is based on timers to that reason I have used timers. v1: initial patch --- drivers/bluetooth/hci_qca.c | 3 +++ 1 file changed, 3 insertions(+)