* [PATCH 0/4] Bluetooth: Support for LE Set Preferred PHY [not found] <CGME20210722053858epcas5p2ecff386b264d4d8578a0ee454f898880@epcas5p2.samsung.com> @ 2021-07-22 5:38 ` Ayush Garg [not found] ` <CGME20210722053859epcas5p4b9441d43196ee1111b33b7c8d4827262@epcas5p4.samsung.com> ` (3 more replies) 0 siblings, 4 replies; 7+ messages in thread From: Ayush Garg @ 2021-07-22 5:38 UTC (permalink / raw) To: linux-bluetooth; +Cc: anupam.r, nitin.j Hello Maintainers, This patch set fixes some minor bugs and adds support for BT 5.0 PHY based HCI command "HCI_OP_LE_SET_PHY" to request controller to change connection specific PHY settings (2M PHY, coded PHY etc.). Ayush Garg (4): Bluetooth: Fix incorrect parsing of LE_PHY params Bluetooth: Implement BT_PHY setsocket option Bluetooth: Add support for sending LE PHY Update event over mgmt Bluetooth: Update LE TX & RX PHYs when connection established include/net/bluetooth/bluetooth.h | 7 +++ include/net/bluetooth/hci.h | 28 +++++++++++ include/net/bluetooth/hci_core.h | 2 + include/net/bluetooth/mgmt.h | 7 +++ net/bluetooth/hci_conn.c | 77 ++++++++++++++++++++++++++----- net/bluetooth/hci_event.c | 62 +++++++++++++++++++++++-- net/bluetooth/l2cap_sock.c | 21 ++++++++- net/bluetooth/mgmt.c | 34 ++++++++++++++ 8 files changed, 220 insertions(+), 18 deletions(-) -- 2.17.1 ^ permalink raw reply [flat|nested] 7+ messages in thread
[parent not found: <CGME20210722053859epcas5p4b9441d43196ee1111b33b7c8d4827262@epcas5p4.samsung.com>]
* [PATCH 1/4] Bluetooth: Fix incorrect parsing of LE_PHY params [not found] ` <CGME20210722053859epcas5p4b9441d43196ee1111b33b7c8d4827262@epcas5p4.samsung.com> @ 2021-07-22 5:38 ` Ayush Garg 2021-07-22 7:20 ` Bluetooth: Support for LE Set Preferred PHY bluez.test.bot 0 siblings, 1 reply; 7+ messages in thread From: Ayush Garg @ 2021-07-22 5:38 UTC (permalink / raw) To: linux-bluetooth; +Cc: anupam.r, nitin.j This change is fixing the parsing of PHY LE CODED which is represented with value 0x03 instead of 0x04 in LE PHY Update Complete event. > HCI Event: LE Meta Event (0x3e) plen 6 LE PHY Update Complete (0x0c) Status: Success (0x00) Handle: 0 TX PHY: LE Coded (0x03) RX PHY: LE Coded (0x03) Reviewed-by: Anupam Roy <anupam.r@samsung.com> Signed-off-by: Ayush Garg <ayush.garg@samsung.com> --- include/net/bluetooth/hci.h | 4 ++++ net/bluetooth/hci_conn.c | 21 +++++++++------------ 2 files changed, 13 insertions(+), 12 deletions(-) diff --git a/include/net/bluetooth/hci.h b/include/net/bluetooth/hci.h index b80415011dcd..56542a09ec43 100644 --- a/include/net/bluetooth/hci.h +++ b/include/net/bluetooth/hci.h @@ -1681,6 +1681,10 @@ struct hci_cp_le_set_default_phy { #define HCI_LE_SET_PHY_2M 0x02 #define HCI_LE_SET_PHY_CODED 0x04 +#define HCI_LE_READ_PHY_1M 0x01 +#define HCI_LE_READ_PHY_2M 0x02 +#define HCI_LE_READ_PHY_CODED 0x03 + #define HCI_OP_LE_SET_EXT_SCAN_PARAMS 0x2041 struct hci_cp_le_set_ext_scan_params { __u8 own_addr_type; diff --git a/net/bluetooth/hci_conn.c b/net/bluetooth/hci_conn.c index 2b5059a56cda..383efd969840 100644 --- a/net/bluetooth/hci_conn.c +++ b/net/bluetooth/hci_conn.c @@ -1915,22 +1915,19 @@ u32 hci_conn_get_phy(struct hci_conn *conn) break; case LE_LINK: - if (conn->le_tx_phy & HCI_LE_SET_PHY_1M) - phys |= BT_PHY_LE_1M_TX; - - if (conn->le_rx_phy & HCI_LE_SET_PHY_1M) - phys |= BT_PHY_LE_1M_RX; - if (conn->le_tx_phy & HCI_LE_SET_PHY_2M) + if (conn->le_tx_phy == HCI_LE_READ_PHY_1M) + phys |= BT_PHY_LE_1M_TX; + else if (conn->le_tx_phy == HCI_LE_READ_PHY_2M) phys |= BT_PHY_LE_2M_TX; - - if (conn->le_rx_phy & HCI_LE_SET_PHY_2M) - phys |= BT_PHY_LE_2M_RX; - - if (conn->le_tx_phy & HCI_LE_SET_PHY_CODED) + else if (conn->le_tx_phy == HCI_LE_READ_PHY_CODED) phys |= BT_PHY_LE_CODED_TX; - if (conn->le_rx_phy & HCI_LE_SET_PHY_CODED) + if (conn->le_rx_phy == HCI_LE_READ_PHY_1M) + phys |= BT_PHY_LE_1M_RX; + else if (conn->le_rx_phy == HCI_LE_READ_PHY_2M) + phys |= BT_PHY_LE_2M_RX; + else if (conn->le_rx_phy == HCI_LE_READ_PHY_CODED) phys |= BT_PHY_LE_CODED_RX; break; -- 2.17.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* RE: Bluetooth: Support for LE Set Preferred PHY 2021-07-22 5:38 ` [PATCH 1/4] Bluetooth: Fix incorrect parsing of LE_PHY params Ayush Garg @ 2021-07-22 7:20 ` bluez.test.bot 0 siblings, 0 replies; 7+ messages in thread From: bluez.test.bot @ 2021-07-22 7:20 UTC (permalink / raw) To: linux-bluetooth, ayush.garg [-- Attachment #1: Type: text/plain, Size: 5848 bytes --] This is automated email and please do not reply to this email! Dear submitter, Thank you for submitting the patches to the linux bluetooth mailing list. This is a CI test results with your patch series: PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=519481 ---Test result--- Test Summary: CheckPatch PASS 3.62 seconds GitLint FAIL 0.49 seconds BuildKernel PASS 675.81 seconds TestRunner: Setup PASS 439.04 seconds TestRunner: l2cap-tester PASS 3.18 seconds TestRunner: bnep-tester PASS 2.18 seconds TestRunner: mgmt-tester PASS 34.44 seconds TestRunner: rfcomm-tester PASS 2.46 seconds TestRunner: sco-tester PASS 2.34 seconds TestRunner: smp-tester FAIL 2.54 seconds TestRunner: userchan-tester PASS 2.24 seconds Details ############################## Test: CheckPatch - PASS - 3.62 seconds Run checkpatch.pl script with rule in .checkpatch.conf ############################## Test: GitLint - FAIL - 0.49 seconds Run gitlint with rule in .gitlint Bluetooth: Fix incorrect parsing of LE_PHY params 8: B3 Line contains hard tab characters (\t): " LE PHY Update Complete (0x0c)" 9: B3 Line contains hard tab characters (\t): " Status: Success (0x00)" 10: B3 Line contains hard tab characters (\t): " Handle: 0" 11: B3 Line contains hard tab characters (\t): " TX PHY: LE Coded (0x03)" 12: B3 Line contains hard tab characters (\t): " RX PHY: LE Coded (0x03)" Bluetooth: Implement BT_PHY setsocket option 7: B3 Line contains hard tab characters (\t): " Handle: 0" 8: B3 Line contains hard tab characters (\t): " All PHYs preference: 0x00" 9: B3 Line contains hard tab characters (\t): " TX PHYs preference: 0x02" 10: B3 Line contains hard tab characters (\t): " LE 2M" 11: B3 Line contains hard tab characters (\t): " RX PHYs preference: 0x02" 12: B3 Line contains hard tab characters (\t): " LE 2M" 13: B3 Line contains hard tab characters (\t): " PHY options preference: Reserved (0x0000)" 15: B3 Line contains hard tab characters (\t): " LE Set PHY (0x08|0x0032) ncmd 1" 16: B3 Line contains hard tab characters (\t): " Status: Success (0x00)" 18: B3 Line contains hard tab characters (\t): " LE PHY Update Complete (0x0c)" 19: B3 Line contains hard tab characters (\t): " Status: Success (0x00)" 20: B3 Line contains hard tab characters (\t): " Handle: 0" 21: B3 Line contains hard tab characters (\t): " TX PHY: LE 2M (0x02)" 22: B3 Line contains hard tab characters (\t): " RX PHY: LE 2M (0x02)" Bluetooth: Add support for sending LE PHY Update event over mgmt 10: B3 Line contains hard tab characters (\t): " LE PHY Update Complete (0x0c)" 11: B3 Line contains hard tab characters (\t): " Status: Success (0x00)" 12: B3 Line contains hard tab characters (\t): " Handle: 0" 13: B3 Line contains hard tab characters (\t): " TX PHY: LE 2M (0x02)" 14: B3 Line contains hard tab characters (\t): " RX PHY: LE 2M (0x02)" 16: B3 Line contains hard tab characters (\t): " LE Address: 45:18:F8:CF:23:7E (Resolvable)" 17: B3 Line contains hard tab characters (\t): " Status: Success (0x00)" 18: B3 Line contains hard tab characters (\t): " Updated PHYs: 0x1800" 19: B3 Line contains hard tab characters (\t): " LE 2M TX" 20: B3 Line contains hard tab characters (\t): " LE 2M RX" Bluetooth: Update LE TX & RX PHYs when connection established 11: B3 Line contains hard tab characters (\t): " Handle: 0" 13: B3 Line contains hard tab characters (\t): " LE Read PHY (0x08|0x0030) ncmd 1" 14: B3 Line contains hard tab characters (\t): " Status: Success (0x00)" 15: B3 Line contains hard tab characters (\t): " Handle: 0" 16: B3 Line contains hard tab characters (\t): " TX PHY: LE 1M (0x01)" 17: B3 Line contains hard tab characters (\t): " RX PHY: LE 1M (0x01)" 19: B3 Line contains hard tab characters (\t): " LE Address: 45:18:F8:CF:23:7E (Resolvable)" 20: B3 Line contains hard tab characters (\t): " Status: Success (0x00)" 21: B3 Line contains hard tab characters (\t): " Updated PHYs: 0x0600" 22: B3 Line contains hard tab characters (\t): " LE 1M TX" 23: B3 Line contains hard tab characters (\t): " LE 1M RX" ############################## Test: BuildKernel - PASS - 675.81 seconds Build Kernel with minimal configuration supports Bluetooth ############################## Test: TestRunner: Setup - PASS - 439.04 seconds Setup environment for running Test Runner ############################## Test: TestRunner: l2cap-tester - PASS - 3.18 seconds Run test-runner with l2cap-tester Total: 40, Passed: 40 (100.0%), Failed: 0, Not Run: 0 ############################## Test: TestRunner: bnep-tester - PASS - 2.18 seconds Run test-runner with bnep-tester Total: 1, Passed: 1 (100.0%), Failed: 0, Not Run: 0 ############################## Test: TestRunner: mgmt-tester - PASS - 34.44 seconds Run test-runner with mgmt-tester Total: 448, Passed: 445 (99.3%), Failed: 0, Not Run: 3 ############################## Test: TestRunner: rfcomm-tester - PASS - 2.46 seconds Run test-runner with rfcomm-tester Total: 9, Passed: 9 (100.0%), Failed: 0, Not Run: 0 ############################## Test: TestRunner: sco-tester - PASS - 2.34 seconds Run test-runner with sco-tester Total: 8, Passed: 8 (100.0%), Failed: 0, Not Run: 0 ############################## Test: TestRunner: smp-tester - FAIL - 2.54 seconds Run test-runner with smp-tester Total: 8, Passed: 7 (87.5%), Failed: 1, Not Run: 0 Failed Test Cases SMP Client - SC Request 2 Failed 0.036 seconds ############################## Test: TestRunner: userchan-tester - PASS - 2.24 seconds Run test-runner with userchan-tester Total: 3, Passed: 3 (100.0%), Failed: 0, Not Run: 0 --- Regards, Linux Bluetooth [-- Attachment #2: l2cap-tester.log --] [-- Type: application/octet-stream, Size: 44349 bytes --] [-- Attachment #3: bnep-tester.log --] [-- Type: application/octet-stream, Size: 3557 bytes --] [-- Attachment #4: mgmt-tester.log --] [-- Type: application/octet-stream, Size: 616827 bytes --] [-- Attachment #5: rfcomm-tester.log --] [-- Type: application/octet-stream, Size: 11677 bytes --] [-- Attachment #6: sco-tester.log --] [-- Type: application/octet-stream, Size: 9912 bytes --] [-- Attachment #7: smp-tester.log --] [-- Type: application/octet-stream, Size: 11705 bytes --] [-- Attachment #8: userchan-tester.log --] [-- Type: application/octet-stream, Size: 5453 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
[parent not found: <CGME20210722053900epcas5p48430eb1335c3e566be02135e63749f33@epcas5p4.samsung.com>]
* [PATCH 2/4] Bluetooth: Implement BT_PHY setsocket option [not found] ` <CGME20210722053900epcas5p48430eb1335c3e566be02135e63749f33@epcas5p4.samsung.com> @ 2021-07-22 5:38 ` Ayush Garg 0 siblings, 0 replies; 7+ messages in thread From: Ayush Garg @ 2021-07-22 5:38 UTC (permalink / raw) To: linux-bluetooth; +Cc: anupam.r, nitin.j This adds BT_PHY setsocket option which can be used to set the PHYs for the underline LE connection. < HCI Command: LE Set PHY (0x08|0x0032) plen 7 Handle: 0 All PHYs preference: 0x00 TX PHYs preference: 0x02 LE 2M RX PHYs preference: 0x02 LE 2M PHY options preference: Reserved (0x0000) > HCI Event: Command Status (0x0f) plen 4 LE Set PHY (0x08|0x0032) ncmd 1 Status: Success (0x00) > HCI Event: LE Meta Event (0x3e) plen 6 LE PHY Update Complete (0x0c) Status: Success (0x00) Handle: 0 TX PHY: LE 2M (0x02) RX PHY: LE 2M (0x02) Reviewed-by: Anupam Roy <anupam.r@samsung.com> Signed-off-by: Ayush Garg <ayush.garg@samsung.com> --- include/net/bluetooth/bluetooth.h | 7 ++++ include/net/bluetooth/hci.h | 12 +++++++ include/net/bluetooth/hci_core.h | 1 + net/bluetooth/hci_conn.c | 56 +++++++++++++++++++++++++++++++ net/bluetooth/l2cap_sock.c | 21 +++++++++++- 5 files changed, 96 insertions(+), 1 deletion(-) diff --git a/include/net/bluetooth/bluetooth.h b/include/net/bluetooth/bluetooth.h index 9125effbf448..af53a98e4236 100644 --- a/include/net/bluetooth/bluetooth.h +++ b/include/net/bluetooth/bluetooth.h @@ -140,6 +140,13 @@ struct bt_voice { #define BT_PHY_LE_2M_RX 0x00001000 #define BT_PHY_LE_CODED_TX 0x00002000 #define BT_PHY_LE_CODED_RX 0x00004000 +#define BT_PHY_LE_CODED_S2 0x00008000 +#define BT_PHY_LE_CODED_S8 0x00010000 + +#define BT_PHY_LE_TX_MASK (BT_PHY_LE_1M_TX | BT_PHY_LE_2M_TX | \ + BT_PHY_LE_CODED_TX) +#define BT_PHY_LE_RX_MASK (BT_PHY_LE_1M_RX | BT_PHY_LE_2M_RX | \ + BT_PHY_LE_CODED_RX) #define BT_MODE 15 diff --git a/include/net/bluetooth/hci.h b/include/net/bluetooth/hci.h index 56542a09ec43..06e10ccf2a1c 100644 --- a/include/net/bluetooth/hci.h +++ b/include/net/bluetooth/hci.h @@ -1677,10 +1677,22 @@ struct hci_cp_le_set_default_phy { __u8 rx_phys; } __packed; +#define HCI_OP_LE_SET_PHY 0x2032 +struct hci_cp_le_set_phy { + __le16 handle; + __u8 all_phys; + __u8 tx_phys; + __u8 rx_phys; + __le16 phy_opt; +} __packed; + #define HCI_LE_SET_PHY_1M 0x01 #define HCI_LE_SET_PHY_2M 0x02 #define HCI_LE_SET_PHY_CODED 0x04 +#define HCI_LE_PHY_CODED_S2 0x01 +#define HCI_LE_PHY_CODED_S8 0x02 + #define HCI_LE_READ_PHY_1M 0x01 #define HCI_LE_READ_PHY_2M 0x02 #define HCI_LE_READ_PHY_CODED 0x03 diff --git a/include/net/bluetooth/hci_core.h b/include/net/bluetooth/hci_core.h index a53e94459ecd..257467f9d28d 100644 --- a/include/net/bluetooth/hci_core.h +++ b/include/net/bluetooth/hci_core.h @@ -1702,6 +1702,7 @@ struct sk_buff *hci_cmd_sync(struct hci_dev *hdev, u16 opcode, u32 plen, const void *param, u32 timeout); u32 hci_conn_get_phy(struct hci_conn *conn); +int hci_conn_set_phy(struct hci_conn *conn, u32 phys); /* ----- HCI Sockets ----- */ void hci_send_to_sock(struct hci_dev *hdev, struct sk_buff *skb); diff --git a/net/bluetooth/hci_conn.c b/net/bluetooth/hci_conn.c index 383efd969840..3b69c7cb9523 100644 --- a/net/bluetooth/hci_conn.c +++ b/net/bluetooth/hci_conn.c @@ -1935,3 +1935,59 @@ u32 hci_conn_get_phy(struct hci_conn *conn) return phys; } + +int hci_conn_set_phy(struct hci_conn *conn, u32 phys) +{ + struct hci_cp_le_set_phy cp_phy; + u16 phy_opt = 0; + + if (conn->type != LE_LINK) + return -ENOTCONN; + + /* Check whether HCI_LE_Set_PHY command is supported or not */ + if (!(conn->hdev->commands[35] & 0x40)) + return -EOPNOTSUPP; + + hci_dev_lock(conn->hdev); + + memset(&cp_phy, 0, sizeof(cp_phy)); + cp_phy.handle = cpu_to_le16(conn->handle); + + if (!(phys & BT_PHY_LE_TX_MASK)) + cp_phy.all_phys |= 0x01; + + if (!(phys & BT_PHY_LE_RX_MASK)) + cp_phy.all_phys |= 0x02; + + if (phys & BT_PHY_LE_1M_TX) + cp_phy.tx_phys |= HCI_LE_SET_PHY_1M; + + if (phys & BT_PHY_LE_2M_TX) + cp_phy.tx_phys |= HCI_LE_SET_PHY_2M; + + if (phys & BT_PHY_LE_CODED_TX) + cp_phy.tx_phys |= HCI_LE_SET_PHY_CODED; + + if (phys & BT_PHY_LE_1M_RX) + cp_phy.rx_phys |= HCI_LE_SET_PHY_1M; + + if (phys & BT_PHY_LE_2M_RX) + cp_phy.rx_phys |= HCI_LE_SET_PHY_2M; + + if (phys & BT_PHY_LE_CODED_RX) + cp_phy.rx_phys |= HCI_LE_SET_PHY_CODED; + + if (phys & BT_PHY_LE_CODED_S2) + phy_opt |= HCI_LE_PHY_CODED_S2; + + if (phys & BT_PHY_LE_CODED_S8) + phy_opt |= HCI_LE_PHY_CODED_S8; + + cp_phy.phy_opt = cpu_to_le16(phy_opt); + + hci_send_cmd(conn->hdev, HCI_OP_LE_SET_PHY, sizeof(cp_phy), &cp_phy); + + hci_dev_unlock(conn->hdev); + + return 0; +} diff --git a/net/bluetooth/l2cap_sock.c b/net/bluetooth/l2cap_sock.c index c99d65ef13b1..eda5d61e37be 100644 --- a/net/bluetooth/l2cap_sock.c +++ b/net/bluetooth/l2cap_sock.c @@ -875,7 +875,7 @@ static int l2cap_sock_setsockopt(struct socket *sock, int level, int optname, struct bt_power pwr; struct l2cap_conn *conn; int len, err = 0; - u32 opt; + u32 opt, phys; BT_DBG("sk %p", sk); @@ -1071,6 +1071,25 @@ static int l2cap_sock_setsockopt(struct socket *sock, int level, int optname, break; + case BT_PHY: + if (sk->sk_state != BT_CONNECTED) { + err = -ENOTCONN; + break; + } + + if (copy_from_sockptr(&phys, optval, sizeof(u32))) { + err = -EFAULT; + break; + } + + err = hci_conn_set_phy(chan->conn->hcon, phys); + + if (err) + break; + + BT_DBG("phys %u", phys); + break; + case BT_MODE: if (!enable_ecred) { err = -ENOPROTOOPT; -- 2.17.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
[parent not found: <CGME20210722053902epcas5p4a38e75d06dcebec336ac1b1743c1a540@epcas5p4.samsung.com>]
* [PATCH 3/4] Bluetooth: Add support for sending LE PHY Update event over mgmt [not found] ` <CGME20210722053902epcas5p4a38e75d06dcebec336ac1b1743c1a540@epcas5p4.samsung.com> @ 2021-07-22 5:38 ` Ayush Garg 2021-07-22 14:43 ` Marcel Holtmann 0 siblings, 1 reply; 7+ messages in thread From: Ayush Garg @ 2021-07-22 5:38 UTC (permalink / raw) To: linux-bluetooth; +Cc: anupam.r, nitin.j The HCI_LE_PHY_Update_Complete event should be sent to upper layers in case of autonomous PHY update made by the controller or HCI_LE_SET_PHY command succeeded. This change will let user know immediately whenever controller change LE PHYs for a particular connection. > HCI Event: LE Meta Event (0x3e) plen 6 LE PHY Update Complete (0x0c) Status: Success (0x00) Handle: 0 TX PHY: LE 2M (0x02) RX PHY: LE 2M (0x02) @ MGMT Event: LE PHY Update Complete (0x002f) plen 12 LE Address: 45:18:F8:CF:23:7E (Resolvable) Status: Success (0x00) Updated PHYs: 0x1800 LE 2M TX LE 2M RX Reviewed-by: Anupam Roy <anupam.r@samsung.com> Signed-off-by: Ayush Garg <ayush.garg@samsung.com> --- include/net/bluetooth/hci_core.h | 1 + include/net/bluetooth/mgmt.h | 7 +++++++ net/bluetooth/hci_event.c | 11 ++++++----- net/bluetooth/mgmt.c | 34 ++++++++++++++++++++++++++++++++ 4 files changed, 48 insertions(+), 5 deletions(-) diff --git a/include/net/bluetooth/hci_core.h b/include/net/bluetooth/hci_core.h index 257467f9d28d..3a5c310ec937 100644 --- a/include/net/bluetooth/hci_core.h +++ b/include/net/bluetooth/hci_core.h @@ -1835,6 +1835,7 @@ void mgmt_advertising_removed(struct sock *sk, struct hci_dev *hdev, u8 instance); void mgmt_adv_monitor_removed(struct hci_dev *hdev, u16 handle); int mgmt_phy_configuration_changed(struct hci_dev *hdev, struct sock *skip); +void mgmt_le_phy_update(struct hci_dev *hdev, struct hci_conn *conn, u8 status); int mgmt_add_adv_patterns_monitor_complete(struct hci_dev *hdev, u8 status); int mgmt_remove_adv_monitor_complete(struct hci_dev *hdev, u8 status); diff --git a/include/net/bluetooth/mgmt.h b/include/net/bluetooth/mgmt.h index 23a0524061b7..54800c4883fc 100644 --- a/include/net/bluetooth/mgmt.h +++ b/include/net/bluetooth/mgmt.h @@ -1100,6 +1100,13 @@ struct mgmt_ev_controller_resume { struct mgmt_addr_info addr; } __packed; +#define MGMT_EV_LE_PHY_UPDATE_COMPLETE 0x002f +struct mgmt_ev_le_phy_update_complete { + struct mgmt_addr_info addr; + __u8 status; + __le32 phys; +} __packed; + #define MGMT_WAKE_REASON_NON_BT_WAKE 0x0 #define MGMT_WAKE_REASON_UNEXPECTED 0x1 #define MGMT_WAKE_REASON_REMOTE_WAKE 0x2 diff --git a/net/bluetooth/hci_event.c b/net/bluetooth/hci_event.c index 1c3018202564..effe525e5272 100644 --- a/net/bluetooth/hci_event.c +++ b/net/bluetooth/hci_event.c @@ -5970,17 +5970,18 @@ static void hci_le_phy_update_evt(struct hci_dev *hdev, struct sk_buff *skb) BT_DBG("%s status 0x%2.2x", hdev->name, ev->status); - if (ev->status) - return; - hci_dev_lock(hdev); conn = hci_conn_hash_lookup_handle(hdev, __le16_to_cpu(ev->handle)); if (!conn) goto unlock; - conn->le_tx_phy = ev->tx_phy; - conn->le_rx_phy = ev->rx_phy; + if (!ev->status) { + conn->le_tx_phy = ev->tx_phy; + conn->le_rx_phy = ev->rx_phy; + } + + mgmt_le_phy_update(hdev, conn, ev->status); unlock: hci_dev_unlock(hdev); diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c index 3663f880df11..683e4b66f810 100644 --- a/net/bluetooth/mgmt.c +++ b/net/bluetooth/mgmt.c @@ -172,6 +172,7 @@ static const u16 mgmt_events[] = { MGMT_EV_ADV_MONITOR_REMOVED, MGMT_EV_CONTROLLER_SUSPEND, MGMT_EV_CONTROLLER_RESUME, + MGMT_EV_LE_PHY_UPDATE_COMPLETE, }; static const u16 mgmt_untrusted_commands[] = { @@ -3611,6 +3612,39 @@ static int set_phy_configuration(struct sock *sk, struct hci_dev *hdev, return err; } +void mgmt_le_phy_update(struct hci_dev *hdev, struct hci_conn *conn, + u8 status) +{ + struct mgmt_ev_le_phy_update_complete ev; + u32 phys = 0; + + memset(&ev, 0, sizeof(ev)); + + bacpy(&ev.addr.bdaddr, &conn->dst); + ev.addr.type = link_to_bdaddr(conn->type, conn->dst_type); + + ev.status = status; + + if (conn->le_tx_phy == HCI_LE_READ_PHY_1M) + phys |= MGMT_PHY_LE_1M_TX; + else if (conn->le_tx_phy == HCI_LE_READ_PHY_2M) + phys |= MGMT_PHY_LE_2M_TX; + else if (conn->le_tx_phy == HCI_LE_READ_PHY_CODED) + phys |= MGMT_PHY_LE_CODED_TX; + + if (conn->le_rx_phy == HCI_LE_READ_PHY_1M) + phys |= MGMT_PHY_LE_1M_RX; + else if (conn->le_rx_phy == HCI_LE_READ_PHY_2M) + phys |= MGMT_PHY_LE_2M_RX; + else if (conn->le_rx_phy == HCI_LE_READ_PHY_CODED) + phys |= MGMT_PHY_LE_CODED_RX; + + ev.phys = cpu_to_le32(phys); + + mgmt_event(MGMT_EV_LE_PHY_UPDATE_COMPLETE, hdev, &ev, sizeof(ev), + NULL); +} + static int set_blocked_keys(struct sock *sk, struct hci_dev *hdev, void *data, u16 len) { -- 2.17.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 3/4] Bluetooth: Add support for sending LE PHY Update event over mgmt 2021-07-22 5:38 ` [PATCH 3/4] Bluetooth: Add support for sending LE PHY Update event over mgmt Ayush Garg @ 2021-07-22 14:43 ` Marcel Holtmann 0 siblings, 0 replies; 7+ messages in thread From: Marcel Holtmann @ 2021-07-22 14:43 UTC (permalink / raw) To: Ayush Garg; +Cc: linux-bluetooth, anupam.r, nitin.j Hi Ayush, > The HCI_LE_PHY_Update_Complete event should be sent to > upper layers in case of autonomous PHY update made by > the controller or HCI_LE_SET_PHY command succeeded. > This change will let user know immediately whenever > controller change LE PHYs for a particular connection. > >> HCI Event: LE Meta Event (0x3e) plen 6 > LE PHY Update Complete (0x0c) > Status: Success (0x00) > Handle: 0 > TX PHY: LE 2M (0x02) > RX PHY: LE 2M (0x02) > @ MGMT Event: LE PHY Update Complete (0x002f) plen 12 > LE Address: 45:18:F8:CF:23:7E (Resolvable) > Status: Success (0x00) > Updated PHYs: 0x1800 > LE 2M TX > LE 2M RX I am not in favor of reporting this via mgmt. It is rather pointless to send this out here. I rather prefer that this is added via auxiliary CMSG data to the GATT and all other L2CAP sockets that opted in for wanting to know. Regards Marcel ^ permalink raw reply [flat|nested] 7+ messages in thread
[parent not found: <CGME20210722053903epcas5p4e25daf9341dca446a0fbfd7b5d54324c@epcas5p4.samsung.com>]
* [PATCH 4/4] Bluetooth: Update LE TX & RX PHYs when connection established [not found] ` <CGME20210722053903epcas5p4e25daf9341dca446a0fbfd7b5d54324c@epcas5p4.samsung.com> @ 2021-07-22 5:38 ` Ayush Garg 0 siblings, 0 replies; 7+ messages in thread From: Ayush Garg @ 2021-07-22 5:38 UTC (permalink / raw) To: linux-bluetooth; +Cc: anupam.r, nitin.j The TX and RX PHYs will always be 1M if the established LE connection is a legacy connection. These values may differ in case of LE Extended connection. This change will read and update the connection PHYs and autonomously send LE PHY Update event if the connection is LE Extended. < HCI Command: LE Read PHY (0x08|0x0030) plen 2 Handle: 0 > HCI Event: Command Complete (0x0e) plen 8 LE Read PHY (0x08|0x0030) ncmd 1 Status: Success (0x00) Handle: 0 TX PHY: LE 1M (0x01) RX PHY: LE 1M (0x01) @ MGMT Event: LE PHY Update Complete (0x002f) plen 12 LE Address: 45:18:F8:CF:23:7E (Resolvable) Status: Success (0x00) Updated PHYs: 0x0600 LE 1M TX LE 1M RX Reviewed-by: Anupam Roy <anupam.r@samsung.com> Signed-off-by: Ayush Garg <ayush.garg@samsung.com> --- include/net/bluetooth/hci.h | 12 +++++++++ net/bluetooth/hci_event.c | 51 +++++++++++++++++++++++++++++++++++++ 2 files changed, 63 insertions(+) diff --git a/include/net/bluetooth/hci.h b/include/net/bluetooth/hci.h index 06e10ccf2a1c..6073e8431ed5 100644 --- a/include/net/bluetooth/hci.h +++ b/include/net/bluetooth/hci.h @@ -1670,6 +1670,18 @@ struct hci_rp_le_read_max_data_len { __le16 rx_time; } __packed; +#define HCI_OP_LE_READ_PHY 0x2030 +struct hci_cp_le_read_phy { + __le16 handle; +} __packed; + +struct hci_rp_le_read_phy { + __u8 status; + __le16 handle; + __u8 tx_phy; + __u8 rx_phy; +} __packed; + #define HCI_OP_LE_SET_DEFAULT_PHY 0x2031 struct hci_cp_le_set_default_phy { __u8 all_phys; diff --git a/net/bluetooth/hci_event.c b/net/bluetooth/hci_event.c index effe525e5272..37ab2f2a1d06 100644 --- a/net/bluetooth/hci_event.c +++ b/net/bluetooth/hci_event.c @@ -1196,6 +1196,31 @@ static void hci_cc_le_set_default_phy(struct hci_dev *hdev, struct sk_buff *skb) hci_dev_unlock(hdev); } +static void hci_cc_le_read_phy(struct hci_dev *hdev, struct sk_buff *skb) +{ + struct hci_rp_le_read_phy *rp = (void *)skb->data; + struct hci_conn *conn; + + BT_DBG("%s status 0x%2.2x", hdev->name, rp->status); + + if (rp->status) + return; + + hci_dev_lock(hdev); + + conn = hci_conn_hash_lookup_handle(hdev, __le16_to_cpu(rp->handle)); + if (!conn) + goto unlock; + + conn->le_tx_phy = rp->tx_phy; + conn->le_rx_phy = rp->rx_phy; + + mgmt_le_phy_update(hdev, conn, 0); + +unlock: + hci_dev_unlock(hdev); +} + static void hci_cc_le_set_adv_set_random_addr(struct hci_dev *hdev, struct sk_buff *skb) { @@ -3642,6 +3667,10 @@ static void hci_cmd_complete_evt(struct hci_dev *hdev, struct sk_buff *skb, hci_cc_le_read_transmit_power(hdev, skb); break; + case HCI_OP_LE_READ_PHY: + hci_cc_le_read_phy(hdev, skb); + break; + default: BT_DBG("%s opcode 0x%4.4x", hdev->name, *opcode); break; @@ -5229,6 +5258,17 @@ static void le_conn_complete_evt(struct hci_dev *hdev, u8 status, goto unlock; } + /* If this LE connection is not an extended connection, then LE + * PHYs will always be 1M at the time of connection establishment. + * So, set 1M as initial LE PHY values. + * But if this is LE Extended connection, then the PHY values + * can be 1M, 2M or CODED. So, In this case, read and update + * the values after the conn->state becomes connected and + * then, send the LE_PHY_UPDATE_COMPLETE event. + */ + conn->le_tx_phy = HCI_LE_SET_PHY_1M; + conn->le_rx_phy = HCI_LE_SET_PHY_1M; + if (!test_and_set_bit(HCI_CONN_MGMT_CONNECTED, &conn->flags)) mgmt_device_connected(hdev, conn, NULL, 0); @@ -5267,6 +5307,17 @@ static void le_conn_complete_evt(struct hci_dev *hdev, u8 status, hci_connect_cfm(conn, status); } + /* If this is LE Extended connection and HCI_LE_READ_PHY + * command is supported, then update the LE PHYs. + */ + if (use_ext_conn(hdev) && hdev->commands[35] & 0x10) { + struct hci_cp_le_read_phy cp; + + cp.handle = __cpu_to_le16(conn->handle); + + hci_send_cmd(hdev, HCI_OP_LE_READ_PHY, sizeof(cp), &cp); + } + params = hci_pend_le_action_lookup(&hdev->pend_le_conns, &conn->dst, conn->dst_type); if (params) { -- 2.17.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
end of thread, other threads:[~2021-07-22 14:43 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <CGME20210722053858epcas5p2ecff386b264d4d8578a0ee454f898880@epcas5p2.samsung.com>
2021-07-22 5:38 ` [PATCH 0/4] Bluetooth: Support for LE Set Preferred PHY Ayush Garg
[not found] ` <CGME20210722053859epcas5p4b9441d43196ee1111b33b7c8d4827262@epcas5p4.samsung.com>
2021-07-22 5:38 ` [PATCH 1/4] Bluetooth: Fix incorrect parsing of LE_PHY params Ayush Garg
2021-07-22 7:20 ` Bluetooth: Support for LE Set Preferred PHY bluez.test.bot
[not found] ` <CGME20210722053900epcas5p48430eb1335c3e566be02135e63749f33@epcas5p4.samsung.com>
2021-07-22 5:38 ` [PATCH 2/4] Bluetooth: Implement BT_PHY setsocket option Ayush Garg
[not found] ` <CGME20210722053902epcas5p4a38e75d06dcebec336ac1b1743c1a540@epcas5p4.samsung.com>
2021-07-22 5:38 ` [PATCH 3/4] Bluetooth: Add support for sending LE PHY Update event over mgmt Ayush Garg
2021-07-22 14:43 ` Marcel Holtmann
[not found] ` <CGME20210722053903epcas5p4e25daf9341dca446a0fbfd7b5d54324c@epcas5p4.samsung.com>
2021-07-22 5:38 ` [PATCH 4/4] Bluetooth: Update LE TX & RX PHYs when connection established Ayush Garg
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).