2021-09-24 11:11:34

by Archie Pusaka

[permalink] [raw]
Subject: [Bluez PATCH] tools/hcitool: Fix the descriptions of leal* commands

From: Archie Pusaka <[email protected]>

The descriptions of leal* commands are copied from lewl* commands.
However they should use "accept list" instead.
---

tools/hcitool.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/tools/hcitool.c b/tools/hcitool.c
index b6f4a4e665..639ee6a511 100644
--- a/tools/hcitool.c
+++ b/tools/hcitool.c
@@ -3400,10 +3400,10 @@ static struct {
{ "clock", cmd_clock, "Read local or remote clock" },
{ "lescan", cmd_lescan, "Start LE scan" },
{ "leinfo", cmd_leinfo, "Get LE remote information" },
- { "lealadd", cmd_lealadd, "Add device to LE White List" },
- { "lealrm", cmd_lealrm, "Remove device from LE White List" },
- { "lealsz", cmd_lealsz, "Read size of LE White List" },
- { "lealclr", cmd_lealclr, "Clear LE White List" },
+ { "lealadd", cmd_lealadd, "Add device to LE Accept List" },
+ { "lealrm", cmd_lealrm, "Remove device from LE Accept List" },
+ { "lealsz", cmd_lealsz, "Read size of LE Accept List" },
+ { "lealclr", cmd_lealclr, "Clear LE Accept List" },
{ "lewladd", cmd_lealadd, "Deprecated. Use lealadd instead." },
{ "lewlrm", cmd_lealrm, "Deprecated. Use lealrm instead." },
{ "lewlsz", cmd_lealsz, "Deprecated. Use lealsz instead." },
--
2.33.0.685.g46640cef36-goog


2021-10-21 20:34:13

by Luiz Augusto von Dentz

[permalink] [raw]
Subject: Re: [Bluez PATCH] tools/hcitool: Fix the descriptions of leal* commands

Hi Archie,

On Fri, Sep 24, 2021 at 2:29 AM Archie Pusaka <[email protected]> wrote:
>
> From: Archie Pusaka <[email protected]>
>
> The descriptions of leal* commands are copied from lewl* commands.
> However they should use "accept list" instead.
> ---
>
> tools/hcitool.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/tools/hcitool.c b/tools/hcitool.c
> index b6f4a4e665..639ee6a511 100644
> --- a/tools/hcitool.c
> +++ b/tools/hcitool.c
> @@ -3400,10 +3400,10 @@ static struct {
> { "clock", cmd_clock, "Read local or remote clock" },
> { "lescan", cmd_lescan, "Start LE scan" },
> { "leinfo", cmd_leinfo, "Get LE remote information" },
> - { "lealadd", cmd_lealadd, "Add device to LE White List" },
> - { "lealrm", cmd_lealrm, "Remove device from LE White List" },
> - { "lealsz", cmd_lealsz, "Read size of LE White List" },
> - { "lealclr", cmd_lealclr, "Clear LE White List" },
> + { "lealadd", cmd_lealadd, "Add device to LE Accept List" },
> + { "lealrm", cmd_lealrm, "Remove device from LE Accept List" },
> + { "lealsz", cmd_lealsz, "Read size of LE Accept List" },
> + { "lealclr", cmd_lealclr, "Clear LE Accept List" },
> { "lewladd", cmd_lealadd, "Deprecated. Use lealadd instead." },
> { "lewlrm", cmd_lealrm, "Deprecated. Use lealrm instead." },
> { "lewlsz", cmd_lealsz, "Deprecated. Use lealsz instead." },
> --
> 2.33.0.685.g46640cef36-goog

Applied, thanks.

--
Luiz Augusto von Dentz