diff mbox series

[Bluez,v6,03/10] doc/mgmt-api: Update controller capabilities MGMT command in mgmt-api

Message ID 20201029160317.Bluez.v6.3.Iabfcf7ec8ac293130a7d903ee8094414256799b3@changeid
State New
Headers show
Series [Bluez,v6,01/10] doc/advertising-api: update API with new interface | expand

Commit Message

Daniel Winkler Oct. 29, 2020, 11:06 p.m. UTC
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(-)

Comments

Marcel Holtmann Nov. 9, 2020, 12:49 p.m. UTC | #1
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
Daniel Winkler Nov. 24, 2020, 6:16 p.m. UTC | #2
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 <marcel@holtmann.org> 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

>
diff mbox series

Patch

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.