Re: [Bluez PATCH 00/62] Inclusive language changes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Archie,

On Fri, Aug 13, 2021 at 5:19 AM Archie Pusaka <apusaka@xxxxxxxxxx> wrote:
>
> From: Archie Pusaka <apusaka@xxxxxxxxxxxx>
>
>
> Hi BlueZ maintainers,
>
> This series of patches promotes the usage of the more inclusive terms
> such as central/peripheral, accept list/reject list, or their
> equivalent where appropriate.
>
> This is also reflected on the changes to Core spec v5.3 and the
> appropriate language mapping table by Bluetooth SIG
> https://specificationrefs.bluetooth.com/language-mapping/Appropriate_Language_Mapping_Table.pdf
>
> I tried to make the changes for each patch as local as possible,
> hopefully this could ease the adoption for downstream. As a side
> effect, I ended up with more patches than I had hoped before.
>
> Do take note that some patches, most notably the ones ending with
> "* storing LTK", will break the stored data, thus perhaps are not
> suitable to be merged as is. I don't have the plan to write patch to
> support both the old and the replacement version since that would
> still left us the less inclusive terms on the code base, but with
> additional complexity.
>
> Some other patches, mainly those on tools/* change the output or
> input parameter to tools, this might come as a surprise for users who
> uses another program that depends on the i/o parameter of those tools.
>
> I leave the discretion of choosing which patches are better to be left
> out to you.
>
> Thanks,
> Archie
>
>
> Archie Pusaka (62):
>   lib: Inclusive language in HCI commands and events
>   lib: Inclusive language in consts and strings
>   tools/hciconfig: Inclusive language update
>   core: Inclusive language for l2cap
>   core: Inclusive language for rfcomm
>   lib: Inclusive language for filtering devices
>   lib/mgmt: Inclusive language for LTK
>   doc/mgmt: Inclusive language update
>   btio: Inclusive language changes
>   shared/ad: Inclusive language changes
>   shared/hfp: Inclusive language changes
>   monitor: Inclusive language in struct member
>   monitor: Inclusive language in SMP related things
>   monitor: Inclusive language in LE states
>   monitor: Inclusive language in LL feature
>   monitor: Inclusive language in link key transaction
>   monitor: Inclusive language in peripheral broadcast
>   monitor: Inclusive language in peripheral page response timeout
>   monitor: Inclusive language in consts and strings
>   monitor: Inclusive language in filtering devices
>   emulator: Inclusive language in naming
>   emulator: Inclusive language in strings
>   emulator: Inclusive language in filtering device
>   tools: Inclusive language in referring the central address
>   tools/parser/hci: Inclusive language changes
>   tools/parser/smp: Inclusive language changes
>   tools/3dsp: Inclusive language changes
>   tools/mgmt-tester: Inclusive language changes
>   tools/bdaddr: Inclusive language changes
>   tools/hciconfig: Inclusive language changes
>   tools/meshctl: Inclusive language changes
>   tools/mgmt-tester: Inclusive language changes
>   tools/parser: Inclusive language in struct member
>   tools/parser/lmp: Inclusive language changes
>   tools/parser/avdtp: Inclusive language changes
>   tools/parser/csr: Inclusive language changes
>   tools/btpclientctl: Inclusive language changes
>   tools/l2cap-tester: Inclusive language changes
>   tools/hci-tester: Inclusive language changes
>   tools/btiotest: Inclusive language changes
>   tools/hcitool: Inclusive language changes, central peripheral
>   tools/hcitool: Inclusive language changes, accept list
>   tools/l2test: Inclusive language changes
>   tools/rctest: Inclusive language changes
>   tools/rfcomm: Inclusive language changes
>   plugins/sixaxis: Inclusive language changes
>   profiles/audio: Inclusive language changes
>   profiles/health: Inclusive language changes
>   sdp: Inclusive language changes
>   adapter: Inclusive language for central and peripheral
>   adapter: Inclusive language for storing LTK
>   adapter: Inclusive language for device filtering
>   client: Inclusive language changes
>   mesh: Inclusive language changes
>   tools/mesh: Inclusive language changes
>   tools/mesh-gatt: Inclusive language changes
>   unit/mesh: Inclusive language changes
>   doc/mesh: Inclusive language changes
>   android: Inclusive language for volume control
>   android: Inclusive language for describing relation
>   android: Inclusive language for filtering devices
>   android: Inclusive language in storing LTK

Can we perhaps group these changes on a one per directory? Also I
wouldn't bother with Android it already considered deprecated and
should be removed at some point, so only change things if it no longer
builds but otherwise leave it alone.

>  android/a2dp.c                  |   2 +-
>  android/bluetooth.c             |  62 ++---
>  android/client/if-hf-client.c   |   2 +-
>  android/hal-audio.c             |  12 +-
>  android/hal-ipc-api.txt         |   2 +-
>  android/hal-sco.c               |  12 +-
>  android/handsfree-client.c      |   2 +-
>  android/hardware/audio.h        |  20 +-
>  android/hardware/bt_hf_client.h |   2 +-
>  android/tester-main.c           |   8 +-
>  btio/btio.c                     |  62 ++---
>  btio/btio.h                     |   2 +-
>  client/main.c                   |   6 +-
>  doc/mesh-api.txt                |   2 +-
>  doc/mgmt-api.txt                |   6 +-
>  doc/settings-storage.txt        |   4 +-
>  emulator/btdev.c                | 198 ++++++++--------
>  emulator/hciemu.c               |  21 +-
>  emulator/hciemu.h               |  12 +-
>  emulator/le.c                   | 116 ++++-----
>  emulator/serial.c               |   6 +-
>  emulator/smp.c                  |   8 +-
>  lib/hci.c                       |  52 ++--
>  lib/hci.h                       |  44 ++--
>  lib/hci_lib.h                   |   8 +-
>  lib/l2cap.h                     |   2 +-
>  lib/mgmt.h                      |   2 +-
>  lib/rfcomm.h                    |   2 +-
>  mesh/net-keys.c                 |  38 +--
>  mesh/net-keys.h                 |   8 +-
>  mesh/net.h                      |   4 +-
>  monitor/broadcom.c              |   8 +-
>  monitor/bt.h                    | 150 ++++++------
>  monitor/control.c               |  14 +-
>  monitor/l2cap.c                 |  10 +-
>  monitor/ll.c                    |  60 ++---
>  monitor/lmp.c                   |   2 +-
>  monitor/packet.c                | 407 ++++++++++++++++----------------
>  plugins/sixaxis.c               |  44 ++--
>  profiles/audio/a2dp.c           |   2 +-
>  profiles/audio/avctp.c          |  10 +-
>  profiles/audio/avctp.h          |   2 +-
>  profiles/health/mcap.c          |  22 +-
>  profiles/health/mcap.h          |   2 +-
>  profiles/sap/server.c           |   2 +-
>  src/adapter.c                   | 117 ++++-----
>  src/adapter.h                   |   4 +-
>  src/device.c                    |   6 +-
>  src/sdpd-server.c               |  10 +-
>  src/sdpd.h                      |   2 +-
>  src/shared/ad.c                 |   8 +-
>  src/shared/ad.h                 |   2 +-
>  src/shared/hfp.c                |   4 +-
>  src/shared/hfp.h                |   2 +-
>  tools/3dsp.c                    |  62 ++---
>  tools/bdaddr.rst                |   2 +-
>  tools/btiotest.c                |  22 +-
>  tools/btpclientctl.c            |   2 +-
>  tools/hci-tester.c              |  16 +-
>  tools/hciconfig.c               |  26 +-
>  tools/hciconfig.rst             |  24 +-
>  tools/hcitool.c                 | 110 ++++-----
>  tools/hcitool.rst               |  30 +--
>  tools/l2cap-tester.c            |  36 +--
>  tools/l2test.c                  |  22 +-
>  tools/mesh-cfgclient.c          |   4 +-
>  tools/mesh-gatt/mesh-net.h      |   4 +-
>  tools/mesh-gatt/net.c           |  60 ++---
>  tools/mesh/mesh-db.c            |  28 +--
>  tools/mesh/mesh-db.h            |   4 +-
>  tools/mesh/remote.c             |  53 ++---
>  tools/mesh/remote.h             |   5 +-
>  tools/meshctl.c                 |   6 +-
>  tools/mgmt-tester.c             | 138 +++++------
>  tools/oobtest.c                 |  12 +-
>  tools/parser/avdtp.c            |   4 +-
>  tools/parser/csr.c              |  17 +-
>  tools/parser/ericsson.c         |   2 +-
>  tools/parser/hci.c              |  49 ++--
>  tools/parser/lmp.c              | 112 ++++-----
>  tools/parser/parser.h           |   2 +-
>  tools/parser/smp.c              |  12 +-
>  tools/rctest.c                  |  22 +-
>  tools/rctest.rst                |   4 +-
>  tools/rfcomm-tester.c           |  14 +-
>  tools/rfcomm.c                  |  16 +-
>  tools/rfcomm.rst                |   2 +-
>  tools/sco-tester.c              |  10 +-
>  tools/smp-tester.c              |  16 +-
>  unit/test-mesh-crypto.c         |   4 +-
>  90 files changed, 1293 insertions(+), 1276 deletions(-)
>
> --
> 2.33.0.rc1.237.g0d66db33f3-goog
>


-- 
Luiz Augusto von Dentz



[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux