Thread (3 messages) flat view 3 messages, 3 authors, 2021-09-10

Re: [PATCH v6] Bluetooth: Keep MSFT ext info throughout a hci_dev's life cycle

From: Marcel Holtmann <marcel@holtmann.org>
Date: 2021-09-10 07:27:56
Also in: lkml, netdev

Hi Manish,
This splits the msft_do_{open/close} to msft_do_{open/close} and
msft_{register/unregister}. With this change it is possible to retain
the MSFT extension info irrespective of controller power on/off state.
This helps bluetoothd to report correct 'supported features' of the
controller to the D-Bus clients event if the controller is off. It also
re-reads the MSFT info upon every msft_do_open().

The following test steps were performed.
1. Boot the test device and verify the MSFT support debug log in syslog.
2. Power off the controller and read the 'supported features', power on
  and read again.
3. Restart the bluetoothd and verify the 'supported features' value.

Signed-off-by: Miao-chen Chou <redacted>
Reviewed-by: Abhishek Pandit-Subedi <abhishekpandit@chromium.org>
Reviewed-by: Archie Pusaka <redacted>
Reviewed-by: Alain Michaud <redacted>
Signed-off-by: Manish Mandlik <redacted>
---

Changes in v6:
- Split msft_do_{open/close} into msft_do_{open/close} and
 msft_{register/unregister}

Changes in v5:
- Rebase on ToT and remove extra blank line

Changes in v4:
- Re-read the MSFT data instead of skipping if it's initiated already

Changes in v3:
- Remove the accepted commits from the series

net/bluetooth/hci_core.c |  3 +++
net/bluetooth/msft.c     | 55 +++++++++++++++++++++++++++++++++-------
net/bluetooth/msft.h     |  4 +++
3 files changed, 53 insertions(+), 9 deletions(-)
patch has been applied to bluetooth-next tree.

Regards

Marcel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help