* [PATCH v2] Bluetooth: le_supported_roles experimental feature @ 2020-07-01 22:08 Alain Michaud 2020-07-02 13:18 ` Marcel Holtmann 0 siblings, 1 reply; 6+ messages in thread From: Alain Michaud @ 2020-07-01 22:08 UTC (permalink / raw) To: linux-bluetooth; +Cc: Alain Michaud This patch adds an le_supported_roles features which allows a clients to determine if the controller is able to support peripheral and central connections separately and at the same time. Signed-off-by: Alain Michaud <alainm@chromium.org> --- Changes in v2: - Slight change of design based on offline feedback net/bluetooth/mgmt.c | 36 +++++++++++++++++++++++++++++++++++- 1 file changed, 35 insertions(+), 1 deletion(-) diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c index 5e9b9728eeac..c13fcc21745f 100644 --- a/net/bluetooth/mgmt.c +++ b/net/bluetooth/mgmt.c @@ -3753,10 +3753,36 @@ static const u8 debug_uuid[16] = { }; #endif +/* 671b10b5-42c0-4696-9227-eb28d1b049d6 */ +static const u8 le_supported_roles[16] = { + 0xd6, 0x49, 0xb0, 0xd1, 0x28, 0xeb, 0x27, 0x92, + 0x96, 0x46, 0xc0, 0x42, 0xb5, 0x10, 0x1b, 0x67, +}; + +static u32 get_le_roles_flags(struct hci_dev *hdev) +{ + u32 flags = 0; + + /* Central connections supported */ + if (hdev->le_states[4] & 0x08) + flags |= BIT(0); + + /* Peripheral connections supported */ + if (hdev->le_states[4] & 0x40) + flags |= BIT(1); + + /* Simult central and peripheral connections supported */ + if (test_bit(HCI_QUIRK_VALID_LE_STATES, &hdev->quirks) && + (hdev->le_states[3] & 0x10)) + flags |= BIT(2); + + return flags; +} + static int read_exp_features_info(struct sock *sk, struct hci_dev *hdev, void *data, u16 data_len) { - char buf[42]; + char buf[44]; struct mgmt_rp_read_exp_features_info *rp = (void *)buf; u16 idx = 0; @@ -3774,6 +3800,14 @@ static int read_exp_features_info(struct sock *sk, struct hci_dev *hdev, } #endif + if (hdev) { + memcpy(rp->features[idx].uuid, le_supported_roles, + sizeof(le_supported_roles)); + + rp->features[idx].flags = cpu_to_le32(get_le_roles_flags(hdev)); + ++idx; + } + rp->feature_count = cpu_to_le16(idx); /* After reading the experimental features information, enable -- 2.27.0.212.ge8ba1cc988-goog ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2] Bluetooth: le_supported_roles experimental feature 2020-07-01 22:08 [PATCH v2] Bluetooth: le_supported_roles experimental feature Alain Michaud @ 2020-07-02 13:18 ` Marcel Holtmann [not found] ` <CALWDO_XT=O4NiqMur+_u1z4o0868ZzBr4gpUikgmgw2U4zqMzw@mail.gmail.com> 0 siblings, 1 reply; 6+ messages in thread From: Marcel Holtmann @ 2020-07-02 13:18 UTC (permalink / raw) To: Alain Michaud; +Cc: linux-bluetooth Hi Alain, > This patch adds an le_supported_roles features which allows a > clients to determine if the controller is able to support peripheral and > central connections separately and at the same time. > > Signed-off-by: Alain Michaud <alainm@chromium.org> > --- > > Changes in v2: > - Slight change of design based on offline feedback > > net/bluetooth/mgmt.c | 36 +++++++++++++++++++++++++++++++++++- > 1 file changed, 35 insertions(+), 1 deletion(-) > > diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c > index 5e9b9728eeac..c13fcc21745f 100644 > --- a/net/bluetooth/mgmt.c > +++ b/net/bluetooth/mgmt.c > @@ -3753,10 +3753,36 @@ static const u8 debug_uuid[16] = { > }; > #endif > > +/* 671b10b5-42c0-4696-9227-eb28d1b049d6 */ > +static const u8 le_supported_roles[16] = { > + 0xd6, 0x49, 0xb0, 0xd1, 0x28, 0xeb, 0x27, 0x92, > + 0x96, 0x46, 0xc0, 0x42, 0xb5, 0x10, 0x1b, 0x67, > +}; > + > +static u32 get_le_roles_flags(struct hci_dev *hdev) > +{ > + u32 flags = 0; > + > + /* Central connections supported */ > + if (hdev->le_states[4] & 0x08) > + flags |= BIT(0); > + > + /* Peripheral connections supported */ > + if (hdev->le_states[4] & 0x40) > + flags |= BIT(1); > + > + /* Simult central and peripheral connections supported */ > + if (test_bit(HCI_QUIRK_VALID_LE_STATES, &hdev->quirks) && > + (hdev->le_states[3] & 0x10)) > + flags |= BIT(2); > + > + return flags; > +} this is not what we can do here. The flags are defined like this. The following bits are defined for the Flags parameter: 0 Feature active 1 Causes change in supported settings And I want these flags for generic handling of experimental features. Individual features can not overwrite it. So if you only want to support a the “read" functionality, then something like this please. if ((hdev->le_states[4] & 0x08) && /* Central */ (hdev->le_states[4] & 0x40) && /* Peripheral */ (hdev->le_states[3] & 0x10) && /* Simultaneous */ test_bit(HCI_QUIRK_VALID_LE_STATES, &hdev->quirks)) flags |= BIT(0); > + > static int read_exp_features_info(struct sock *sk, struct hci_dev *hdev, > void *data, u16 data_len) > { > - char buf[42]; > + char buf[44]; > struct mgmt_rp_read_exp_features_info *rp = (void *)buf; > u16 idx = 0; > > @@ -3774,6 +3800,14 @@ static int read_exp_features_info(struct sock *sk, struct hci_dev *hdev, > } > #endif > > + if (hdev) { > + memcpy(rp->features[idx].uuid, le_supported_roles, > + sizeof(le_supported_roles)); > + And I would put it all here instead of a separate function. > + rp->features[idx].flags = cpu_to_le32(get_le_roles_flags(hdev)); > + ++idx; > + } > + > rp->feature_count = cpu_to_le16(idx); > > /* After reading the experimental features information, enable Regards Marcel ^ permalink raw reply [flat|nested] 6+ messages in thread
[parent not found: <CALWDO_XT=O4NiqMur+_u1z4o0868ZzBr4gpUikgmgw2U4zqMzw@mail.gmail.com>]
* Re: [PATCH v2] Bluetooth: le_supported_roles experimental feature [not found] ` <CALWDO_XT=O4NiqMur+_u1z4o0868ZzBr4gpUikgmgw2U4zqMzw@mail.gmail.com> @ 2020-07-02 15:01 ` Alain Michaud 2020-07-02 17:37 ` Alain Michaud 2020-07-03 7:24 ` Marcel Holtmann 1 sibling, 1 reply; 6+ messages in thread From: Alain Michaud @ 2020-07-02 15:01 UTC (permalink / raw) To: Marcel Holtmann; +Cc: Alain Michaud, BlueZ Resending as plaintext. On Thu, Jul 2, 2020 at 10:59 AM Alain Michaud <alainmichaud@google.com> wrote: > > Hi Marcel > > On Thu, Jul 2, 2020 at 9:18 AM Marcel Holtmann <marcel@holtmann.org> wrote: >> >> Hi Alain, >> >> > This patch adds an le_supported_roles features which allows a >> > clients to determine if the controller is able to support peripheral and >> > central connections separately and at the same time. >> > >> > Signed-off-by: Alain Michaud <alainm@chromium.org> >> > --- >> > >> > Changes in v2: >> > - Slight change of design based on offline feedback >> > >> > net/bluetooth/mgmt.c | 36 +++++++++++++++++++++++++++++++++++- >> > 1 file changed, 35 insertions(+), 1 deletion(-) >> > >> > diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c >> > index 5e9b9728eeac..c13fcc21745f 100644 >> > --- a/net/bluetooth/mgmt.c >> > +++ b/net/bluetooth/mgmt.c >> > @@ -3753,10 +3753,36 @@ static const u8 debug_uuid[16] = { >> > }; >> > #endif >> > >> > +/* 671b10b5-42c0-4696-9227-eb28d1b049d6 */ >> > +static const u8 le_supported_roles[16] = { >> > + 0xd6, 0x49, 0xb0, 0xd1, 0x28, 0xeb, 0x27, 0x92, >> > + 0x96, 0x46, 0xc0, 0x42, 0xb5, 0x10, 0x1b, 0x67, >> > +}; >> > + >> > +static u32 get_le_roles_flags(struct hci_dev *hdev) >> > +{ >> > + u32 flags = 0; >> > + >> > + /* Central connections supported */ >> > + if (hdev->le_states[4] & 0x08) >> > + flags |= BIT(0); >> > + >> > + /* Peripheral connections supported */ >> > + if (hdev->le_states[4] & 0x40) >> > + flags |= BIT(1); >> > + >> > + /* Simult central and peripheral connections supported */ >> > + if (test_bit(HCI_QUIRK_VALID_LE_STATES, &hdev->quirks) && >> > + (hdev->le_states[3] & 0x10)) >> > + flags |= BIT(2); >> > + >> > + return flags; >> > +} >> >> this is not what we can do here. The flags are defined like this. >> >> The following bits are defined for the Flags parameter: >> >> 0 Feature active >> 1 Causes change in supported settings >> >> And I want these flags for generic handling of experimental features. Individual features can not overwrite it. >> >> So if you only want to support a the “read" functionality, then something like this please. >> >> if ((hdev->le_states[4] & 0x08) && /* Central */ >> (hdev->le_states[4] & 0x40) && /* Peripheral */ >> (hdev->le_states[3] & 0x10) && /* Simultaneous */ >> test_bit(HCI_QUIRK_VALID_LE_STATES, &hdev->quirks)) >> flags |= BIT(0); >> > OK, Since the userspace Api we discussed reports individual states, would you suggest if LE is supported that the Central and Peripheral roles are supported and just use this to query the simultaneous support? > >> >> > + >> > static int read_exp_features_info(struct sock *sk, struct hci_dev *hdev, >> > void *data, u16 data_len) >> > { >> > - char buf[42]; >> > + char buf[44]; >> > struct mgmt_rp_read_exp_features_info *rp = (void *)buf; >> > u16 idx = 0; >> > >> > @@ -3774,6 +3800,14 @@ static int read_exp_features_info(struct sock *sk, struct hci_dev *hdev, >> > } >> > #endif >> > >> > + if (hdev) { >> > + memcpy(rp->features[idx].uuid, le_supported_roles, >> > + sizeof(le_supported_roles)); >> > + >> >> And I would put it all here instead of a separate function. >> >> > + rp->features[idx].flags = cpu_to_le32(get_le_roles_flags(hdev)); >> > + ++idx; >> > + } >> > + >> > rp->feature_count = cpu_to_le16(idx); >> > >> > /* After reading the experimental features information, enable >> >> Regards >> >> Marcel >> ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] Bluetooth: le_supported_roles experimental feature 2020-07-02 15:01 ` Alain Michaud @ 2020-07-02 17:37 ` Alain Michaud 0 siblings, 0 replies; 6+ messages in thread From: Alain Michaud @ 2020-07-02 17:37 UTC (permalink / raw) To: Marcel Holtmann; +Cc: Alain Michaud, BlueZ On Thu, Jul 2, 2020 at 11:01 AM Alain Michaud <alainmichaud@google.com> wrote: > > Resending as plaintext. > > > On Thu, Jul 2, 2020 at 10:59 AM Alain Michaud <alainmichaud@google.com> wrote: > > > > Hi Marcel > > > > On Thu, Jul 2, 2020 at 9:18 AM Marcel Holtmann <marcel@holtmann.org> wrote: > >> > >> Hi Alain, > >> > >> > This patch adds an le_supported_roles features which allows a > >> > clients to determine if the controller is able to support peripheral and > >> > central connections separately and at the same time. > >> > > >> > Signed-off-by: Alain Michaud <alainm@chromium.org> > >> > --- > >> > > >> > Changes in v2: > >> > - Slight change of design based on offline feedback > >> > > >> > net/bluetooth/mgmt.c | 36 +++++++++++++++++++++++++++++++++++- > >> > 1 file changed, 35 insertions(+), 1 deletion(-) > >> > > >> > diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c > >> > index 5e9b9728eeac..c13fcc21745f 100644 > >> > --- a/net/bluetooth/mgmt.c > >> > +++ b/net/bluetooth/mgmt.c > >> > @@ -3753,10 +3753,36 @@ static const u8 debug_uuid[16] = { > >> > }; > >> > #endif > >> > > >> > +/* 671b10b5-42c0-4696-9227-eb28d1b049d6 */ > >> > +static const u8 le_supported_roles[16] = { > >> > + 0xd6, 0x49, 0xb0, 0xd1, 0x28, 0xeb, 0x27, 0x92, > >> > + 0x96, 0x46, 0xc0, 0x42, 0xb5, 0x10, 0x1b, 0x67, > >> > +}; > >> > + > >> > +static u32 get_le_roles_flags(struct hci_dev *hdev) > >> > +{ > >> > + u32 flags = 0; > >> > + > >> > + /* Central connections supported */ > >> > + if (hdev->le_states[4] & 0x08) > >> > + flags |= BIT(0); > >> > + > >> > + /* Peripheral connections supported */ > >> > + if (hdev->le_states[4] & 0x40) > >> > + flags |= BIT(1); > >> > + > >> > + /* Simult central and peripheral connections supported */ > >> > + if (test_bit(HCI_QUIRK_VALID_LE_STATES, &hdev->quirks) && > >> > + (hdev->le_states[3] & 0x10)) > >> > + flags |= BIT(2); > >> > + > >> > + return flags; > >> > +} > >> > >> this is not what we can do here. The flags are defined like this. > >> > >> The following bits are defined for the Flags parameter: > >> > >> 0 Feature active > >> 1 Causes change in supported settings > >> > >> And I want these flags for generic handling of experimental features. Individual features can not overwrite it. > >> > >> So if you only want to support a the “read" functionality, then something like this please. > >> > >> if ((hdev->le_states[4] & 0x08) && /* Central */ > >> (hdev->le_states[4] & 0x40) && /* Peripheral */ > >> (hdev->le_states[3] & 0x10) && /* Simultaneous */ > >> test_bit(HCI_QUIRK_VALID_LE_STATES, &hdev->quirks)) > >> flags |= BIT(0); > >> > > OK, Since the userspace Api we discussed reports individual states, would you suggest if LE is supported that the Central and Peripheral roles are supported and just use this to query the simultaneous support? Also, seems like what you are suggesting looks a lot like v1. > > > >> > >> > + > >> > static int read_exp_features_info(struct sock *sk, struct hci_dev *hdev, > >> > void *data, u16 data_len) > >> > { > >> > - char buf[42]; > >> > + char buf[44]; > >> > struct mgmt_rp_read_exp_features_info *rp = (void *)buf; > >> > u16 idx = 0; > >> > > >> > @@ -3774,6 +3800,14 @@ static int read_exp_features_info(struct sock *sk, struct hci_dev *hdev, > >> > } > >> > #endif > >> > > >> > + if (hdev) { > >> > + memcpy(rp->features[idx].uuid, le_supported_roles, > >> > + sizeof(le_supported_roles)); > >> > + > >> > >> And I would put it all here instead of a separate function. > >> > >> > + rp->features[idx].flags = cpu_to_le32(get_le_roles_flags(hdev)); > >> > + ++idx; > >> > + } > >> > + > >> > rp->feature_count = cpu_to_le16(idx); > >> > > >> > /* After reading the experimental features information, enable > >> > >> Regards > >> > >> Marcel > >> ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] Bluetooth: le_supported_roles experimental feature [not found] ` <CALWDO_XT=O4NiqMur+_u1z4o0868ZzBr4gpUikgmgw2U4zqMzw@mail.gmail.com> 2020-07-02 15:01 ` Alain Michaud @ 2020-07-03 7:24 ` Marcel Holtmann 2020-07-03 13:03 ` Marcel Holtmann 1 sibling, 1 reply; 6+ messages in thread From: Marcel Holtmann @ 2020-07-03 7:24 UTC (permalink / raw) To: Alain Michaud; +Cc: Alain Michaud, BlueZ Hi Alain, > > This patch adds an le_supported_roles features which allows a > > clients to determine if the controller is able to support peripheral and > > central connections separately and at the same time. > > > > Signed-off-by: Alain Michaud <alainm@chromium.org> > > --- > > > > Changes in v2: > > - Slight change of design based on offline feedback > > > > net/bluetooth/mgmt.c | 36 +++++++++++++++++++++++++++++++++++- > > 1 file changed, 35 insertions(+), 1 deletion(-) > > > > diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c > > index 5e9b9728eeac..c13fcc21745f 100644 > > --- a/net/bluetooth/mgmt.c > > +++ b/net/bluetooth/mgmt.c > > @@ -3753,10 +3753,36 @@ static const u8 debug_uuid[16] = { > > }; > > #endif > > > > +/* 671b10b5-42c0-4696-9227-eb28d1b049d6 */ > > +static const u8 le_supported_roles[16] = { > > + 0xd6, 0x49, 0xb0, 0xd1, 0x28, 0xeb, 0x27, 0x92, > > + 0x96, 0x46, 0xc0, 0x42, 0xb5, 0x10, 0x1b, 0x67, > > +}; > > + > > +static u32 get_le_roles_flags(struct hci_dev *hdev) > > +{ > > + u32 flags = 0; > > + > > + /* Central connections supported */ > > + if (hdev->le_states[4] & 0x08) > > + flags |= BIT(0); > > + > > + /* Peripheral connections supported */ > > + if (hdev->le_states[4] & 0x40) > > + flags |= BIT(1); > > + > > + /* Simult central and peripheral connections supported */ > > + if (test_bit(HCI_QUIRK_VALID_LE_STATES, &hdev->quirks) && > > + (hdev->le_states[3] & 0x10)) > > + flags |= BIT(2); > > + > > + return flags; > > +} > > this is not what we can do here. The flags are defined like this. > > The following bits are defined for the Flags parameter: > > 0 Feature active > 1 Causes change in supported settings > > And I want these flags for generic handling of experimental features. Individual features can not overwrite it. > > So if you only want to support a the “read" functionality, then something like this please. > > if ((hdev->le_states[4] & 0x08) && /* Central */ > (hdev->le_states[4] & 0x40) && /* Peripheral */ > (hdev->le_states[3] & 0x10) && /* Simultaneous */ > test_bit(HCI_QUIRK_VALID_LE_STATES, &hdev->quirks)) > flags |= BIT(0); > > OK, Since the userspace Api we discussed reports individual states, would you suggest if LE is supported that the Central and Peripheral roles are supported and just use this to query the simultaneous support? you get the Central state support from the LE setting and Peripheral from the Advertising setting. So I think the only extra information you would need is the support for both at the same time. That said, I have been discussing with Daniel if we could extend (and with that rename) the Read Security Information command and allow it to report static information about the controller support. Maybe it fits better there. However for now, I would just export this via a simple experimental feature. Regards Marcel ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] Bluetooth: le_supported_roles experimental feature 2020-07-03 7:24 ` Marcel Holtmann @ 2020-07-03 13:03 ` Marcel Holtmann 0 siblings, 0 replies; 6+ messages in thread From: Marcel Holtmann @ 2020-07-03 13:03 UTC (permalink / raw) To: Alain Michaud; +Cc: Alain Michaud, BlueZ Hi Alain, >>> This patch adds an le_supported_roles features which allows a >>> clients to determine if the controller is able to support peripheral and >>> central connections separately and at the same time. >>> >>> Signed-off-by: Alain Michaud <alainm@chromium.org> >>> --- >>> >>> Changes in v2: >>> - Slight change of design based on offline feedback >>> >>> net/bluetooth/mgmt.c | 36 +++++++++++++++++++++++++++++++++++- >>> 1 file changed, 35 insertions(+), 1 deletion(-) >>> >>> diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c >>> index 5e9b9728eeac..c13fcc21745f 100644 >>> --- a/net/bluetooth/mgmt.c >>> +++ b/net/bluetooth/mgmt.c >>> @@ -3753,10 +3753,36 @@ static const u8 debug_uuid[16] = { >>> }; >>> #endif >>> >>> +/* 671b10b5-42c0-4696-9227-eb28d1b049d6 */ >>> +static const u8 le_supported_roles[16] = { >>> + 0xd6, 0x49, 0xb0, 0xd1, 0x28, 0xeb, 0x27, 0x92, >>> + 0x96, 0x46, 0xc0, 0x42, 0xb5, 0x10, 0x1b, 0x67, >>> +}; >>> + >>> +static u32 get_le_roles_flags(struct hci_dev *hdev) >>> +{ >>> + u32 flags = 0; >>> + >>> + /* Central connections supported */ >>> + if (hdev->le_states[4] & 0x08) >>> + flags |= BIT(0); >>> + >>> + /* Peripheral connections supported */ >>> + if (hdev->le_states[4] & 0x40) >>> + flags |= BIT(1); >>> + >>> + /* Simult central and peripheral connections supported */ >>> + if (test_bit(HCI_QUIRK_VALID_LE_STATES, &hdev->quirks) && >>> + (hdev->le_states[3] & 0x10)) >>> + flags |= BIT(2); >>> + >>> + return flags; >>> +} >> >> this is not what we can do here. The flags are defined like this. >> >> The following bits are defined for the Flags parameter: >> >> 0 Feature active >> 1 Causes change in supported settings >> >> And I want these flags for generic handling of experimental features. Individual features can not overwrite it. >> >> So if you only want to support a the “read" functionality, then something like this please. >> >> if ((hdev->le_states[4] & 0x08) && /* Central */ >> (hdev->le_states[4] & 0x40) && /* Peripheral */ >> (hdev->le_states[3] & 0x10) && /* Simultaneous */ >> test_bit(HCI_QUIRK_VALID_LE_STATES, &hdev->quirks)) >> flags |= BIT(0); >> >> OK, Since the userspace Api we discussed reports individual states, would you suggest if LE is supported that the Central and Peripheral roles are supported and just use this to query the simultaneous support? > > you get the Central state support from the LE setting and Peripheral from the Advertising setting. So I think the only extra information you would need is the support for both at the same time. > > That said, I have been discussing with Daniel if we could extend (and with that rename) the Read Security Information command and allow it to report static information about the controller support. Maybe it fits better there. However for now, I would just export this via a simple experimental feature. so something like this and then we can extend it. diff --git a/doc/mgmt-api.txt b/doc/mgmt-api.txt index ca0d38469e56..84a26183029d 100644 --- a/doc/mgmt-api.txt +++ b/doc/mgmt-api.txt @@ -3110,20 +3110,20 @@ Set Wideband Speech Command Invalid Index -Read Security Information Command -================================= +Read System Information Command +=============================== Command Code: 0x0048 Controller Index: <controller id> Command Parameters: - Return Parameters: Security_Data_Length (2 Octets) - Security_Data (0-65535 Octets) + Return Parameters: System_Data_Length (2 Octets) + System_Data (0-65535 Octets) - This command is used to retrieve the supported security features - by the controller or the host stack. + This command is used to retrieve the supported system and security + features by the controller or the host stack. - The Security_Data_Length and Security_Data parameters provide - a list of security settings, features and information. It uses + The System_Data_Length and System_Data parameters provide a list + of system and security settings, features and information. It uses the same format as EIR_Data, but with the namespace defined here. Data Type Name @@ -3131,6 +3131,8 @@ Read Security Information Command 0x01 Flags 0x02 Max Encryption Key Size (BR/EDR) 0x03 Max Encryption Key Size (LE) + 0x04 Transmit Power (BR/EDR) + 0x05 Transmit Power (LE) Flags (data type 0x01) @@ -3146,6 +3148,11 @@ Read Security Information Command present, then it is unknown what the max encryption key size of the controller or host is in use. + Transmit Power (data types 0x04 and 0x05) + + When this field is present, then it provides 4 Octets + indicating min and max transmit power values. + This command generates a Command Complete event on success or a Command Status event on failure. Regards Marcel ^ permalink raw reply related [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-07-03 13:03 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-01 22:08 [PATCH v2] Bluetooth: le_supported_roles experimental feature Alain Michaud
2020-07-02 13:18 ` Marcel Holtmann
[not found] ` <CALWDO_XT=O4NiqMur+_u1z4o0868ZzBr4gpUikgmgw2U4zqMzw@mail.gmail.com>
2020-07-02 15:01 ` Alain Michaud
2020-07-02 17:37 ` Alain Michaud
2020-07-03 7:24 ` Marcel Holtmann
2020-07-03 13:03 ` Marcel Holtmann
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).