tools: Deprecate some input parameters to align with inclusive language

Some input parameter names are not appropriate and needs to be
updated. However, doing so might cause friction with the current
workflow and documents.

This patch deprecates the inappropriate names and replaces them with
the more appropriate substitutes, while still keeping the old terms
usable to not break the workflow of users.

Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Signed-off-by: Ayush Garg <ayush.garg@samsung.com>

diff --git a/tools/btiotest.c b/tools/btiotest.c
index 42b290f..803a528 100755 (executable)
--- a/tools/btiotest.c
+++ b/tools/btiotest.c
@@ -576,8 +576,10 @@ static GOptionEntry options[] = {
                                "Disconnect connection after N seconds" },
        { "accept", 'a', 0, G_OPTION_ARG_INT, &opt_accept,
                                "Accept connection after N seconds" },
+       { "central", 'C', 0, G_OPTION_ARG_NONE, &opt_central,
+                               "Central role switch (incoming connections)" },
        { "master", 'm', 0, G_OPTION_ARG_NONE, &opt_central,
-                               "Master role switch (incoming connections)" },
+                               "Deprecated. Use central instead." },
        { "priority", 'P', 0, G_OPTION_ARG_INT, &opt_priority,
                                "Transmission priority: Setting a priority "
                                "outside the range 0 to 6 requires the"

diff --git a/tools/hcitool.c b/tools/hcitool.c
index 54c23ea..771662b 100755 (executable)
--- a/tools/hcitool.c
+++ b/tools/hcitool.c
@@ -1269,10 +1269,10 @@ static struct option cc_options[] = {
 
 static const char *cc_help =
        "Usage:\n"
-       "\tcc [--role=m|s] [--ptype=pkt_types] <bdaddr>\n"
+       "\tcc [--role=c|p] [--ptype=pkt_types] <bdaddr>\n"
        "Example:\n"
        "\tcc --ptype=dm1,dh3,dh5 01:02:03:04:05:06\n"
-       "\tcc --role=m 01:02:03:04:05:06\n";
+       "\tcc --role=c 01:02:03:04:05:06\n";
 
 static void cmd_cc(int dev_id, int argc, char **argv)
 {
@@ -1292,7 +1292,7 @@ static void cmd_cc(int dev_id, int argc, char **argv)
                        break;
 
                case 'r':
-                       role = optarg[0] == 'm' ? 0 : 1;
+                       role = optarg[0] == 'm' || optarg[0] == 'c' ? 0 : 1;
                        break;
 
                default:
@@ -1419,10 +1419,12 @@ static void cmd_sr(int dev_id, int argc, char **argv)
 
        str2ba(argv[0], &bdaddr);
        switch (argv[1][0]) {
-       case 'm':
+       case 'm': /* Deprecated. Kept for compatibility. */
+       case 'c':
                role = 0;
                break;
-       case 's':
+       case 's': /* Deprecated. Kept for compatibility. */
+       case 'p':
                role = 1;
                break;
        default:
@@ -2533,7 +2535,8 @@ static struct option lescan_options[] = {
        { "static",     0, 0, 's' },
        { "privacy",    0, 0, 'p' },
        { "passive",    0, 0, 'P' },
-       { "whitelist",  0, 0, 'w' },
+       { "whitelist",  0, 0, 'w' }, /* Deprecated. Kept for compatibility. */
+       { "acceptlist", 0, 0, 'a' },
        { "discovery",  1, 0, 'd' },
        { "duplicates", 0, 0, 'D' },
        { 0, 0, 0, 0 }
@@ -2543,7 +2546,7 @@ static const char *lescan_help =
        "Usage:\n"
        "\tlescan [--privacy] enable privacy\n"
        "\tlescan [--passive] set scan type passive (default active)\n"
-       "\tlescan [--whitelist] scan for address in the whitelist only\n"
+       "\tlescan [--acceptlist] scan for address in the accept list only\n"
        "\tlescan [--discovery=g|l] enable general or limited discovery"
                "procedure\n"
        "\tlescan [--duplicates] don't filter duplicates\n";
@@ -2570,7 +2573,8 @@ static void cmd_lescan(int dev_id, int argc, char **argv)
                case 'P':
                        scan_type = 0x00; /* Passive */
                        break;
-               case 'w':
+               case 'w': /* Deprecated. Kept for compatibility. */
+               case 'a':
                        filter_policy = 0x01; /* Accept list */
                        break;
                case 'd':
@@ -2737,14 +2741,15 @@ static struct option lecc_options[] = {
        { "help",       0, 0, 'h' },
        { "static",     0, 0, 's' },
        { "random",     0, 0, 'r' },
-       { "whitelist",  0, 0, 'w' },
+       { "whitelist",  0, 0, 'w' }, /* Deprecated. Kept for compatibility. */
+       { "acceptlist", 0, 0, 'a' },
        { 0, 0, 0, 0 }
 };
 
 static const char *lecc_help =
        "Usage:\n"
        "\tlecc [--static] [--random] <bdaddr>\n"
-       "\tlecc --whitelist\n";
+       "\tlecc --acceptlist\n";
 
 static void cmd_lecc(int dev_id, int argc, char **argv)
 {
@@ -2766,7 +2771,8 @@ static void cmd_lecc(int dev_id, int argc, char **argv)
                case 'r':
                        peer_bdaddr_type = LE_RANDOM_ADDRESS;
                        break;
-               case 'w':
+               case 'w': /* Deprecated. Kept for compatibility. */
+               case 'a':
                        initiator_filter = 0x01; /* Use accept list */
                        break;
                default:
@@ -2812,34 +2818,34 @@ static void cmd_lecc(int dev_id, int argc, char **argv)
        hci_close_dev(dd);
 }
 
-static struct option lewladd_options[] = {
+static struct option lealadd_options[] = {
        { "help",       0, 0, 'h' },
        { "random",     0, 0, 'r' },
        { 0, 0, 0, 0 }
 };
 
-static const char *lewladd_help =
+static const char *lealadd_help =
        "Usage:\n"
-       "\tlewladd [--random] <bdaddr>\n";
+       "\tlealadd [--random] <bdaddr>\n";
 
-static void cmd_lewladd(int dev_id, int argc, char **argv)
+static void cmd_lealadd(int dev_id, int argc, char **argv)
 {
        int err, opt, dd;
        bdaddr_t bdaddr;
        uint8_t bdaddr_type = LE_PUBLIC_ADDRESS;
 
-       for_each_opt(opt, lewladd_options, NULL) {
+       for_each_opt(opt, lealadd_options, NULL) {
                switch (opt) {
                case 'r':
                        bdaddr_type = LE_RANDOM_ADDRESS;
                        break;
                default:
-                       printf("%s", lewladd_help);
+                       printf("%s", lealadd_help);
                        return;
                }
        }
 
-       helper_arg(1, 1, &argc, &argv, lewladd_help);
+       helper_arg(1, 1, &argc, &argv, lealadd_help);
 
        if (dev_id < 0)
                dev_id = hci_get_route(NULL);
@@ -2863,29 +2869,29 @@ static void cmd_lewladd(int dev_id, int argc, char **argv)
        }
 }
 
-static struct option lewlrm_options[] = {
+static struct option lealrm_options[] = {
        { "help",       0, 0, 'h' },
        { 0, 0, 0, 0 }
 };
 
-static const char *lewlrm_help =
+static const char *lealrm_help =
        "Usage:\n"
-       "\tlewlrm <bdaddr>\n";
+       "\tlealrm <bdaddr>\n";
 
-static void cmd_lewlrm(int dev_id, int argc, char **argv)
+static void cmd_lealrm(int dev_id, int argc, char **argv)
 {
        int err, opt, dd;
        bdaddr_t bdaddr;
 
-       for_each_opt(opt, lewlrm_options, NULL) {
+       for_each_opt(opt, lealrm_options, NULL) {
                switch (opt) {
                default:
-                       printf("%s", lewlrm_help);
+                       printf("%s", lealrm_help);
                        return;
                }
        }
 
-       helper_arg(1, 1, &argc, &argv, lewlrm_help);
+       helper_arg(1, 1, &argc, &argv, lealrm_help);
 
        if (dev_id < 0)
                dev_id = hci_get_route(NULL);
@@ -2909,29 +2915,29 @@ static void cmd_lewlrm(int dev_id, int argc, char **argv)
        }
 }
 
-static struct option lewlsz_options[] = {
+static struct option lealsz_options[] = {
        { "help",       0, 0, 'h' },
        { 0, 0, 0, 0 }
 };
 
-static const char *lewlsz_help =
+static const char *lealsz_help =
        "Usage:\n"
-       "\tlewlsz\n";
+       "\tlealsz\n";
 
-static void cmd_lewlsz(int dev_id, int argc, char **argv)
+static void cmd_lealsz(int dev_id, int argc, char **argv)
 {
        int err, dd, opt;
        uint8_t size;
 
-       for_each_opt(opt, lewlsz_options, NULL) {
+       for_each_opt(opt, lealsz_options, NULL) {
                switch (opt) {
                default:
-                       printf("%s", lewlsz_help);
+                       printf("%s", lealsz_help);
                        return;
                }
        }
 
-       helper_arg(0, 0, &argc, &argv, lewlsz_help);
+       helper_arg(0, 0, &argc, &argv, lealsz_help);
 
        if (dev_id < 0)
                dev_id = hci_get_route(NULL);
@@ -2955,28 +2961,28 @@ static void cmd_lewlsz(int dev_id, int argc, char **argv)
        printf("Accept list size: %d\n", size);
 }
 
-static struct option lewlclr_options[] = {
+static struct option lealclr_options[] = {
        { "help",       0, 0, 'h' },
        { 0, 0, 0, 0 }
 };
 
-static const char *lewlclr_help =
+static const char *lealclr_help =
        "Usage:\n"
-       "\tlewlclr\n";
+       "\tlealclr\n";
 
-static void cmd_lewlclr(int dev_id, int argc, char **argv)
+static void cmd_lealclr(int dev_id, int argc, char **argv)
 {
        int err, dd, opt;
 
-       for_each_opt(opt, lewlclr_options, NULL) {
+       for_each_opt(opt, lealclr_options, NULL) {
                switch (opt) {
                default:
-                       printf("%s", lewlclr_help);
+                       printf("%s", lealclr_help);
                        return;
                }
        }
 
-       helper_arg(0, 0, &argc, &argv, lewlclr_help);
+       helper_arg(0, 0, &argc, &argv, lealclr_help);
 
        if (dev_id < 0)
                dev_id = hci_get_route(NULL);
@@ -3441,7 +3447,7 @@ static struct {
        { "con",      cmd_con,     "Display active connections"           },
        { "cc",       cmd_cc,      "Create connection to remote device"   },
        { "dc",       cmd_dc,      "Disconnect from remote device"        },
-       { "sr",       cmd_sr,      "Switch master/slave role"             },
+       { "sr",       cmd_sr,      "Switch central/peripheral role"       },
        { "cpt",      cmd_cpt,     "Change connection packet type"        },
        { "rssi",     cmd_rssi,    "Display connection RSSI"              },
        { "lq",       cmd_lq,      "Display link quality"                 },
@@ -3456,10 +3462,14 @@ static struct {
        { "clock",    cmd_clock,   "Read local or remote clock"           },
        { "lescan",   cmd_lescan,  "Start LE scan"                        },
        { "leinfo",   cmd_leinfo,  "Get LE remote information"            },
-       { "lewladd",  cmd_lewladd, "Add device to LE White List"          },
-       { "lewlrm",   cmd_lewlrm,  "Remove device from LE White List"     },
-       { "lewlsz",   cmd_lewlsz,  "Read size of LE White List"           },
-       { "lewlclr",  cmd_lewlclr, "Clear LE White List"                  },
+       { "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"                  },
+       { "lewladd",  cmd_lealadd, "Deprecated. Use lealadd instead."     },
+       { "lewlrm",   cmd_lealrm,  "Deprecated. Use lealrm instead."      },
+       { "lewlsz",   cmd_lealsz,  "Deprecated. Use lealsz instead."      },
+       { "lewlclr",  cmd_lealclr, "Deprecated. Use lealclr instead."     },
        { "lerladd",  cmd_lerladd, "Add device to LE Resolving List"      },
        { "lerlrm",   cmd_lerlrm,  "Remove device from LE Resolving List" },
        { "lerlclr",  cmd_lerlclr, "Clear LE Resolving List"              },

diff --git a/tools/hcitool.rst b/tools/hcitool.rst
index f59d694..88b1263 100644 (file)
--- a/tools/hcitool.rst
+++ b/tools/hcitool.rst
@@ -74,7 +74,7 @@ cmd <*ogf*> <*ocf*> [*parameters*]
 con
     Display active baseband connections
 
-cc [--*role*\=m|s] [--*pkt-type*\=<*ptype*>] <*bdaddr*>
+cc [--*role*\=c|p] [--*pkt-type*\=<*ptype*>] <*bdaddr*>
     Create baseband connection to remote device with Bluetooth address *bdaddr*.
 
     Option **--pkt-type** specifies a list  of  allowed packet types.
@@ -82,9 +82,9 @@ cc [--*role*\=m|s] [--*pkt-type*\=<*ptype*>] <*bdaddr*>
     packet types are **DM1**, **DM3**, **DM5**, **DH1**, **DH3**, **DH5**,
     **HV1**, **HV2**, **HV3**. Default is to allow all packet types.
 
-    Option  **--role** can have value **m** (do not allow role switch, stay
-    master) or **s** (allow role switch, become slave if the peer asks to become
-    master). Default is **m**.
+    Option  **--role** can have value **c** (do not allow role switch, stay
+    central) or **p** (allow role switch, become peripheral if the peer asks to
+    become central). Default is **c**.
 
 dc <*bdaddr*> [*reason*]
     Delete baseband connection from remote device with Bluetooth address
@@ -96,7 +96,7 @@ dc <*bdaddr*> [*reason*]
 
 sr <*bdaddr*> <*role*>
     Switch role for the baseband connection from the remote device to
-    **master** or **slave**.
+    **central** or **peripheral**.
 
 cpt <*bdaddr*> <*ptypes*>
     Change packet types for baseband connection to device with Bluetooth
@@ -157,23 +157,23 @@ clock [*bdaddr*] [*clock*]
     The *clock* can be **0** for the local clock or **1** for the piconet
     clock (which is default).
 
-lescan [--*privacy*] [--*passive*] [--*whitelist*] [--*discovery*\=g|l] [--*duplicates*]
+lescan [--*privacy*] [--*passive*] [--*acceptlist*] [--*discovery*\=g|l] [--*duplicates*]
     Start LE scan
 
 leinfo [--*static*] [--*random*] <*bdaddr*>
     Get LE remote information
 
-lewladd [--*random*] <*bdaddr*>
-    Add device to LE White List
+lealadd [--*random*] <*bdaddr*>
+    Add device to LE Accept List
 
-lewlrm <*bdaddr*>
-    Remove device from LE White List
+lealrm <*bdaddr*>
+    Remove device from LE Accept List
 
-lewlsz
-    Read size of LE White List
+lealsz
+    Read size of LE Accept List
 
-lewlclr
-    Clear LE White List
+lealclr
+    Clear LE Accept List
 
 lerladd [--*local_irk*] [--*peer_irk*] [--*random*] <*bdaddr*>
     Add device to LE Resolving List
@@ -193,7 +193,7 @@ lerlon
 lerloff
     Disable LE Address Resolution
 
-lecc [--*static*] [--*random*] <*bdaddr*> | [--*whitelist*]
+lecc [--*static*] [--*random*] <*bdaddr*> | [--*acceptlist*]
     Create a LE Connection
 
 ledc <*handle*> [*reason*]

diff --git a/tools/rfcomm.c b/tools/rfcomm.c
index 7370ffe..c376395 100755 (executable)
--- a/tools/rfcomm.c
+++ b/tools/rfcomm.c
@@ -645,7 +645,7 @@ static void usage(void)
                "\t-A, --auth                     Enable authentication\n"
                "\t-E, --encrypt                  Enable encryption\n"
                "\t-S, --secure                   Secure connection\n"
-               "\t-M, --master                   Become the master of a piconet\n"
+               "\t-C, --central                  Become the central of a piconet\n"
                "\t-L, --linger [seconds]         Set linger timeout\n"
                "\t-a                             Show all devices (default)\n"
                "\n");
@@ -667,7 +667,8 @@ static struct option main_options[] = {
        { "auth",       0, 0, 'A' },
        { "encrypt",    0, 0, 'E' },
        { "secure",     0, 0, 'S' },
-       { "master",     0, 0, 'M' },
+       { "master",     0, 0, 'M' }, /* Deprecated. Kept for compatibility. */
+       { "central",    0, 0, 'C' },
        { "linger",     1, 0, 'L' },
        { 0, 0, 0, 0 }
 };
@@ -679,7 +680,7 @@ int main(int argc, char *argv[])
 
        bacpy(&bdaddr, BDADDR_ANY);
 
-       while ((opt = getopt_long(argc, argv, "+i:rahAESML:", main_options,
+       while ((opt = getopt_long(argc, argv, "+i:rahAESMCL:", main_options,
                                                                NULL)) != -1) {
                switch(opt) {
                case 'i':
@@ -713,7 +714,8 @@ int main(int argc, char *argv[])
                        secure = 1;
                        break;
 
-               case 'M':
+               case 'M': /* Deprecated. Kept for compatibility. */
+               case 'C':
                        central = 1;
                        break;