2020-10-29 23:07:20

by Daniel Winkler

[permalink] [raw]
Subject: [Bluez PATCH v6 03/10] doc/mgmt-api: Update controller capabilities MGMT command in mgmt-api

This patch changes Read Security Info MGMT to be called Read Controller
Capabilities Command

---

Changes in v6: None
Changes in v5: None
Changes in v4:
- mgmt-api: Move Controller Capabilities MGMT doc into new patch
- mgmt-api: Tx Power range put into single capabilities entry

Changes in v3: None
Changes in v2: None

doc/mgmt-api.txt | 21 +++++++++++++++------
1 file changed, 15 insertions(+), 6 deletions(-)

diff --git a/doc/mgmt-api.txt b/doc/mgmt-api.txt
index 7c899a8fe..1aa43d6c3 100644
--- a/doc/mgmt-api.txt
+++ b/doc/mgmt-api.txt
@@ -3110,19 +3110,19 @@ Set Wideband Speech Command
Invalid Index


-Read Security Information Command
-=================================
+Read Controller Capabilities Command
+====================================

Command Code: 0x0048
Controller Index: <controller id>
Command Parameters:
- Return Parameters: Security_Data_Length (2 Octets)
- Security_Data (0-65535 Octets)
+ Return Parameters: Capabilities_Data_Length (2 Octets)
+ Capabilities_Data (0-65535 Octets)

- This command is used to retrieve the supported security features
+ This command is used to retrieve the supported capabilities
by the controller or the host stack.

- The Security_Data_Length and Security_Data parameters provide
+ The Capabilities_Data_Length and Capabilities_Data parameters provide
a list of security settings, features and information. It uses
the same format as EIR_Data, but with the namespace defined here.

@@ -3131,6 +3131,7 @@ Read Security Information Command
0x01 Flags
0x02 Max Encryption Key Size (BR/EDR)
0x03 Max Encryption Key Size (LE)
+ 0x04 Supported Tx Power (LE)

Flags (data type 0x01)

@@ -3146,6 +3147,14 @@ Read Security Information Command
present, then it is unknown what the max encryption key
size of the controller or host is in use.

+ Supported LE Tx Power (data type 0x04)
+
+ When present, this 2-octet field provides the min and max
+ LE Tx power supported by the controller, respectively, as
+ reported by the LE Read Transmit Power HCI command. If this
+ field is not available, it indicates that the LE Read
+ Transmit Power HCI command was not available.
+
This command generates a Command Complete event on success or
a Command Status event on failure.

--
2.29.1.341.ge80a0c044ae-goog


2020-11-09 12:50:32

by Marcel Holtmann

[permalink] [raw]
Subject: Re: [Bluez PATCH v6 03/10] doc/mgmt-api: Update controller capabilities MGMT command in mgmt-api

Hi Daniel,

> This patch changes Read Security Info MGMT to be called Read Controller
> Capabilities Command
>
> ---
>
> Changes in v6: None
> Changes in v5: None
> Changes in v4:
> - mgmt-api: Move Controller Capabilities MGMT doc into new patch
> - mgmt-api: Tx Power range put into single capabilities entry
>
> Changes in v3: None
> Changes in v2: None
>
> doc/mgmt-api.txt | 21 +++++++++++++++------
> 1 file changed, 15 insertions(+), 6 deletions(-)
>
> diff --git a/doc/mgmt-api.txt b/doc/mgmt-api.txt
> index 7c899a8fe..1aa43d6c3 100644
> --- a/doc/mgmt-api.txt
> +++ b/doc/mgmt-api.txt
> @@ -3110,19 +3110,19 @@ Set Wideband Speech Command
> Invalid Index
>
>
> -Read Security Information Command
> -=================================
> +Read Controller Capabilities Command
> +====================================
>
> Command Code: 0x0048
> Controller Index: <controller id>
> Command Parameters:
> - Return Parameters: Security_Data_Length (2 Octets)
> - Security_Data (0-65535 Octets)
> + Return Parameters: Capabilities_Data_Length (2 Octets)
> + Capabilities_Data (0-65535 Octets)
>
> - This command is used to retrieve the supported security features
> + This command is used to retrieve the supported capabilities
> by the controller or the host stack.
>
> - The Security_Data_Length and Security_Data parameters provide
> + The Capabilities_Data_Length and Capabilities_Data parameters provide
> a list of security settings, features and information. It uses
> the same format as EIR_Data, but with the namespace defined here.
>
> @@ -3131,6 +3131,7 @@ Read Security Information Command
> 0x01 Flags
> 0x02 Max Encryption Key Size (BR/EDR)
> 0x03 Max Encryption Key Size (LE)
> + 0x04 Supported Tx Power (LE)

make this “Supported Min/Max TX Power”. Otherwise looks good and ready for Luiz to apply.

Regards

Marcel

2020-11-24 18:18:48

by Daniel Winkler

[permalink] [raw]
Subject: Re: [Bluez PATCH v6 03/10] doc/mgmt-api: Update controller capabilities MGMT command in mgmt-api

Hello Marcel,

As I understand, Luiz accepted this series already. I will upload a
new patch to fix this wording, thanks for catching it!

Best,
Daniel

On Mon, Nov 9, 2020 at 4:49 AM Marcel Holtmann <[email protected]> wrote:
>
> Hi Daniel,
>
> > This patch changes Read Security Info MGMT to be called Read Controller
> > Capabilities Command
> >
> > ---
> >
> > Changes in v6: None
> > Changes in v5: None
> > Changes in v4:
> > - mgmt-api: Move Controller Capabilities MGMT doc into new patch
> > - mgmt-api: Tx Power range put into single capabilities entry
> >
> > Changes in v3: None
> > Changes in v2: None
> >
> > doc/mgmt-api.txt | 21 +++++++++++++++------
> > 1 file changed, 15 insertions(+), 6 deletions(-)
> >
> > diff --git a/doc/mgmt-api.txt b/doc/mgmt-api.txt
> > index 7c899a8fe..1aa43d6c3 100644
> > --- a/doc/mgmt-api.txt
> > +++ b/doc/mgmt-api.txt
> > @@ -3110,19 +3110,19 @@ Set Wideband Speech Command
> > Invalid Index
> >
> >
> > -Read Security Information Command
> > -=================================
> > +Read Controller Capabilities Command
> > +====================================
> >
> > Command Code: 0x0048
> > Controller Index: <controller id>
> > Command Parameters:
> > - Return Parameters: Security_Data_Length (2 Octets)
> > - Security_Data (0-65535 Octets)
> > + Return Parameters: Capabilities_Data_Length (2 Octets)
> > + Capabilities_Data (0-65535 Octets)
> >
> > - This command is used to retrieve the supported security features
> > + This command is used to retrieve the supported capabilities
> > by the controller or the host stack.
> >
> > - The Security_Data_Length and Security_Data parameters provide
> > + The Capabilities_Data_Length and Capabilities_Data parameters provide
> > a list of security settings, features and information. It uses
> > the same format as EIR_Data, but with the namespace defined here.
> >
> > @@ -3131,6 +3131,7 @@ Read Security Information Command
> > 0x01 Flags
> > 0x02 Max Encryption Key Size (BR/EDR)
> > 0x03 Max Encryption Key Size (LE)
> > + 0x04 Supported Tx Power (LE)
>
> make this “Supported Min/Max TX Power”. Otherwise looks good and ready for Luiz to apply.
>
> Regards
>
> Marcel
>