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@xxxxxxxxxxxx> 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 >