From patchwork Thu Dec 14 09:22:55 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: 15013537245@163.com X-Patchwork-Id: 13492648 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=163.com header.i=@163.com header.b="eAMdi4I+" Received: from m15.mail.163.com (m15.mail.163.com [45.254.50.219]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id D87B9A6; Thu, 14 Dec 2023 01:23:22 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=163.com; s=s110527; h=From:Subject:Date:Message-Id:MIME-Version; bh=qooKP 2HsiHgDqnQG/MiDCCu3acZz5StldDyLPzc3OCc=; b=eAMdi4I+FliAsC7aQqgrF rVEomt0hrSJSwAj187zTqQ/ynpKPConXaCaHFHps1oQ2GffVjg8IjDk46LGATtRw 6OATL5B3CmBPD/WxSqy4dO5AvW5z6YsfAvw1jZvdPbLKRxJtJJTQ7vpKlkTgidUp +3AiJno6CUFGe/OCEJuo6U= Received: from WH-D-007635B.QUECTEL.COM (unknown [223.76.229.213]) by zwqz-smtp-mta-g2-1 (Coremail) with SMTP id _____wDXH_9xyXpl+qqDFg--.26059S2; Thu, 14 Dec 2023 17:22:58 +0800 (CST) From: 15013537245@163.com To: marcel@holtmann.org, johan.hedberg@gmail.com, luiz.dentz@gmail.com Cc: linux-bluetooth@vger.kernel.org, linux-kernel@vger.kernel.org, zhongjun.yu@quectel.com, Clancy Shang Subject: [PATCH] Bluetooth: hci_sync: fix BR/EDR wakeup bug Date: Thu, 14 Dec 2023 17:22:55 +0800 Message-Id: <20231214092255.1018859-1-15013537245@163.com> X-Mailer: git-send-email 2.25.1 Precedence: bulk X-Mailing-List: linux-bluetooth@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-CM-TRANSID: _____wDXH_9xyXpl+qqDFg--.26059S2 X-Coremail-Antispam: 1Uf129KBjvJXoWrKFWrJFWfArW3Cry8WFW7urg_yoW8JrWxpF y2kFZ3trZ5JrWaka43A3W0gFyUAF9YgFs3CFWDt345X3yaqr48trWjkr17WF18CrZ2kF1f ZF4UtwsxW34kCa7anT9S1TB71UUUUUUqnTZGkaVYY2UrUUUUjbIjqfuFe4nvWSU5nxnvy2 9KBjDUYxBIdaVFxhVjvjDU0xZFpf9x07jvLvtUUUUU= X-CM-SenderInfo: rprviiitvtljiuv6il2tof0z/xtbBEhVG12VOA8l3BQAAsF From: Clancy Shang when Bluetooth set the event mask and enter suspend, the controller has hci mode change event comming,it cause controller can not enter sleep mode. so it should to set the hci mode change event mask before enter suspend Signed-off-by: Clancy Shang --- net/bluetooth/hci_sync.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/net/bluetooth/hci_sync.c b/net/bluetooth/hci_sync.c index a15ab0b874a9..a26a58cb2c38 100644 --- a/net/bluetooth/hci_sync.c +++ b/net/bluetooth/hci_sync.c @@ -3800,12 +3800,14 @@ static int hci_set_event_mask_sync(struct hci_dev *hdev) if (lmp_bredr_capable(hdev)) { events[4] |= 0x01; /* Flow Specification Complete */ - /* Don't set Disconnect Complete when suspended as that - * would wakeup the host when disconnecting due to - * suspend. + /* Don't set Disconnect Complete and mode change when + * suspended as that would wakeup the host when disconnecting + * due to suspend. */ - if (hdev->suspended) + if (hdev->suspended) { events[0] &= 0xef; + events[2] &= 0xf7; + } } else { /* Use a different default for LE-only devices */ memset(events, 0, sizeof(events));