[0/3] Bluetooth: btmrvl_sdio: Refactor remote wakeup support

Message

Abhishek Pandit-Subedi June 11, 2020, 1:53 a.m. UTC

Hi linux-bluetooth,

This patch series is refactoring the btmrvl driver to add better support
for controlling remote wakeup during suspend. Previously, the hci device
was getting created as /sys/devices/virtual/bluetooth/hci0 and there
wasn't a way to control wakeup for the device from userspace. This
series of changes adds the proper parent for the hci device, allows irq
configuration via power/wakeup and properly disables scanning during
suspend if wakeup isn't enabled.

This was tested on a Veyron Mighty Chromebook running the 4.19 kernel
with these patches. It passes both automated and manual suspend+resume
tests that make sure wakeup is working properly.

Thanks
Abhishek



Abhishek Pandit-Subedi (3):
  Bluetooth: btmrvl_sdio: Set parent dev to hdev
  Bluetooth: btmrvl_sdio: Implement prevent_wake
  Bluetooth: btmrvl_sdio: Refactor irq wakeup

 drivers/bluetooth/btmrvl_main.c | 11 +++++++++++
 drivers/bluetooth/btmrvl_sdio.c | 13 ++++++++++---
 2 files changed, 21 insertions(+), 3 deletions(-)

Comments

Marcel Holtmann June 12, 2020, 6:39 a.m. UTC | #1

Hi Abhishek,

> This patch series is refactoring the btmrvl driver to add better support
> for controlling remote wakeup during suspend. Previously, the hci device
> was getting created as /sys/devices/virtual/bluetooth/hci0 and there
> wasn't a way to control wakeup for the device from userspace. This
> series of changes adds the proper parent for the hci device, allows irq
> configuration via power/wakeup and properly disables scanning during
> suspend if wakeup isn't enabled.
> 
> This was tested on a Veyron Mighty Chromebook running the 4.19 kernel
> with these patches. It passes both automated and manual suspend+resume
> tests that make sure wakeup is working properly.
> 
> Thanks
> Abhishek
> 
> 
> 
> Abhishek Pandit-Subedi (3):
>  Bluetooth: btmrvl_sdio: Set parent dev to hdev
>  Bluetooth: btmrvl_sdio: Implement prevent_wake
>  Bluetooth: btmrvl_sdio: Refactor irq wakeup
> 
> drivers/bluetooth/btmrvl_main.c | 11 +++++++++++
> drivers/bluetooth/btmrvl_sdio.c | 13 ++++++++++---
> 2 files changed, 21 insertions(+), 3 deletions(-)

all 3 patches have been applied to bluetooth-next tree.

Regards

Marcel