From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: * X-Spam-Status: No, score=1.5 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, PDS_HP_HELO_NORDNS,RDNS_NONE,SPF_HELO_NONE,SPF_NONE,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail-vs1-f66.google.com ([209.85.217.66]:32935 "EHLO mail-vs1-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729646AbgCKPlX (ORCPT ); Wed, 11 Mar 2020 11:41:23 -0400 Received: by mail-vs1-f66.google.com with SMTP id n27so1641806vsa.0 for ; Wed, 11 Mar 2020 08:41:23 -0700 (PDT) MIME-Version: 1.0 References: <20200308212334.213841-1-abhishekpandit@chromium.org> In-Reply-To: From: Abhishek Pandit-Subedi Date: Wed, 11 Mar 2020 08:41:08 -0700 Message-ID: Subject: Re: [RFC PATCH v5 0/5] Bluetooth: Handle system suspend gracefully To: Marcel Holtmann Cc: Luiz Augusto von Dentz , Alain Michaud , Bluez mailing list , ChromeOS Bluetooth Upstreaming , "David S. Miller" , Johan Hedberg , netdev , LKML , Jakub Kicinski Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Sender: linux-kernel-owner@vger.kernel.org List-ID: Sure -- patch incoming in a few minutes. On Wed, Mar 11, 2020 at 8:03 AM Marcel Holtmann wrote= : > > Hi Abhishek, > > > This patch series prepares the Bluetooth controller for system suspend > > by disconnecting all devices and preparing the event filter and LE > > whitelist with devices that can wake the system from suspend. > > > > The main motivation for doing this is so we can enable Bluetooth as > > a wake up source during suspend without it being noisy. Bluetooth shoul= d > > wake the system when a HID device receives user input but otherwise not > > send any events to the host. > > > > This patch series was tested on several Chromebooks with both btusb and > > hci_serdev on kernel 4.19. The set of tests was basically the following= : > > * Reconnects after suspend succeed > > * HID devices can wake the system from suspend (needs some related blue= z > > changes to call the Set Wake Capable management command) > > * System properly pauses and unpauses discovery + advertising around > > suspend > > * System does not wake from any events from non wakeable devices > > > > Series 2 has refactored the change into multiple smaller commits as > > requested. I tried to simplify some of the whitelist filtering edge > > cases but unfortunately it remains quite complex. > > > > Series 3 has refactored it further and should have resolved the > > whitelisting complexity in series 2. > > > > Series 4 adds a fix to check for powered down and powering down adapter= s. > > > > Series 5 moves set_wake_capable to the last patch in the series and > > changes BT_DBG to bt_dev_dbg. > > > > Please review and provide any feedback. > > so I was planning to apply patches 1-4. The only thing that I noticed was= that patch 2 introduces the following warning. > > CC net/bluetooth/hci_request.o > net/bluetooth/hci_request.c: In function =E2=80=98hci_req_prepare_suspend= =E2=80=99: > net/bluetooth/hci_request.c:973:6: warning: unused variable =E2=80=98old_= state=E2=80=99 [-Wunused-variable] > 973 | int old_state; > | ^~~~~~~~~ > > I think this variable should only be introduced in patch 4. Are you able = to respin this series so that the variable moves to patch 4. If not, I can = try to fix this myself. > > Regards > > Marcel >