lib: hw_id: change name "BLE" to "Bluetooth"/"BT" #25576

noahp · 2025-11-13T16:56:32Z

Rename the option and the name string to adhere to Bluetooth naming
convention.

See original suggestion here:

Signed-off-by: Noah Pendleton [email protected]

Copilot

Pull Request Overview

This PR renames the hardware ID library's BLE-related configuration option from CONFIG_HW_ID_LIBRARY_SOURCE_BLE_MAC to CONFIG_HW_ID_LIBRARY_SOURCE_BT_MAC to align with Bluetooth naming conventions. The Kconfig option description is also updated from "BLE MAC address" to "Bluetooth Device Address".

Key changes:

Kconfig option renamed from CONFIG_HW_ID_LIBRARY_SOURCE_BLE_MAC to CONFIG_HW_ID_LIBRARY_SOURCE_BT_MAC
Kconfig option description updated to "Bluetooth Device Address"
All references to the old config name updated across code, tests, samples, and documentation

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
lib/hw_id/Kconfig	Renames Kconfig option and updates description to follow Bluetooth naming convention
lib/hw_id/hw_id.c	Updates preprocessor directives to use new BT_MAC config name
include/hw_id.h	Updates preprocessor directive for HW_ID_LEN definition
tests/lib/hw_id/src/kconfig_override_ble.h	Updates test configuration override to use new config name
samples/hw_id/overlay-ble-mac.conf	Updates sample configuration to use new config name
doc/nrf/libraries/others/hw_id.rst	Updates documentation reference to new Kconfig option
doc/nrf/releases_and_maturity/releases/release-notes-changelog.rst	Adds release note documenting the config rename

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

NordicBuilder · 2025-11-13T16:59:09Z

CI Information

^{To view the history of this post, click the 'edited' button above}
Build number: 6

Inputs:

Sources:

sdk-nrf: PR head: 192369fb7385b2e1d3766910c1717abef5bf0b2e

more details

sdk-nrf:

PR head: 192369fb7385b2e1d3766910c1717abef5bf0b2e
merge base: 7a1b536a18daac0e53ef09332728a56125601f4c
target head (main): 7a1b536a18daac0e53ef09332728a56125601f4c
Diff

Github labels

Enabled	Name	Description
	ci-disabled	Disable the ci execution
	ci-all-test	Run all of ci, no test spec filtering will be done
	ci-force-downstream	Force execution of downstream even if twister fails
	ci-run-twister	Force run twister
	ci-run-zephyr-twister	Force run zephyr twister

List of changed files detected by CI (7)

doc
│  ├── nrf
│  │  ├── libraries
│  │  │  ├── others
│  │  │  │  │ hw_id.rst
│  │  ├── releases_and_maturity
│  │  │  ├── releases
│  │  │  │  │ release-notes-changelog.rst
include
│  │ hw_id.h
lib
│  ├── hw_id
│  │  ├── Kconfig
│  │  │ hw_id.c
samples
│  ├── hw_id
│  │  │ overlay-ble-mac.conf
tests
│  ├── lib
│  │  ├── hw_id
│  │  │  ├── src
│  │  │  │  │ kconfig_override_ble.h

Outputs:

Toolchain

Version: df3cc9d822
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:df3cc9d822_e595b21c39

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

◻️ Toolchain - Skipped: existing toolchain is used
✅ Build twister
- sdk-nrf test count: 140
✅ Integration tests

Disabled integration tests

- test-fw-nrfconnect-nrf_lrcs_positioning
- desktop52_verification
- test_ble_nrf_config
- test-fw-nrfconnect-apps
- test-fw-nrfconnect-ble_mesh
- test-fw-nrfconnect-ble_samples
- test-fw-nrfconnect-chip
- test-fw-nrfconnect-fem
- test-fw-nrfconnect-nfc
- test-fw-nrfconnect-nrf-iot_libmodem-nrf
- test-fw-nrfconnect-nrf-iot_lwm2m
- test-fw-nrfconnect-nrf-iot_samples
- test-fw-nrfconnect-nrf-iot_thingy91
- test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
- test-fw-nrfconnect-nrf_crypto
- test-fw-nrfconnect-ps-main
- test-fw-nrfconnect-rpc
- test-fw-nrfconnect-rs
- test-fw-nrfconnect-tfm
- test-fw-nrfconnect-thread-main
- test-low-level
- test-sdk-audio
- test-sdk-dfu
- test-sdk-find-my
- test-sdk-mcuboot
- test-sdk-wifi
- test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

Copilot

Pull Request Overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

doc/nrf/libraries/others/hw_id.rst

lib/hw_id/hw_id.c

github-actions · 2025-11-13T18:37:57Z

You can find the documentation preview for this PR here.

Preview links for modified nRF Connect SDK documents:

https://ncsdoc.z6.web.core.windows.net/PR-25576/nrf/libraries/others/hw_id.html
https://ncsdoc.z6.web.core.windows.net/PR-25576/nrf/releases_and_maturity/releases/release-notes-changelog.html

Copilot

Pull Request Overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated no new comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

doc/nrf/releases_and_maturity/releases/release-notes-changelog.rst

nordicjm · 2025-11-14T11:20:28Z

lib/hw_id/Kconfig


-config HW_ID_LIBRARY_SOURCE_BLE_MAC
-	bool "BLE MAC address"
+config HW_ID_LIBRARY_SOURCE_BT_MAC


MAC is the problem here

🤦 apologies, fixed now! CONFIG_HW_ID_LIBRARY_SOURCE_BT_DEVICE_ADDRESS

Rename the option and the name string to adhere to Bluetooth naming convention. See original suggestion here: nrfconnect#25313 (comment) Signed-off-by: Noah Pendleton <[email protected]>

Copilot

Pull Request Overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated no new comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings November 13, 2025 16:56

NordicBuilder added the doc-required PR must not be merged without tech writer approval. label Nov 13, 2025

Copilot started reviewing on behalf of noahp November 13, 2025 16:57 View session

Copilot finished reviewing on behalf of noahp November 13, 2025 16:58

Copilot AI reviewed Nov 13, 2025

View reviewed changes

noahp force-pushed the noahp/hw_id-bluetooth-naming branch from dab6f2c to 5aa2635 Compare November 13, 2025 17:54

noahp marked this pull request as ready for review November 13, 2025 18:09

noahp requested review from a team as code owners November 13, 2025 18:09

Copilot AI review requested due to automatic review settings November 13, 2025 18:09

noahp requested review from a team as code owners November 13, 2025 18:09

noahp requested review from b-gent, maxd-nordic and umapraseeda November 13, 2025 18:09

noahp mentioned this pull request Nov 13, 2025

modules: memfault: Use hw_id for default Memfault device id #25313

Open

Copilot started reviewing on behalf of noahp November 13, 2025 18:09 View session

Copilot finished reviewing on behalf of noahp November 13, 2025 18:11

Copilot AI reviewed Nov 13, 2025

View reviewed changes

doc/nrf/libraries/others/hw_id.rst Outdated Show resolved Hide resolved

lib/hw_id/hw_id.c Outdated Show resolved Hide resolved

Copilot AI review requested due to automatic review settings November 13, 2025 18:39

noahp force-pushed the noahp/hw_id-bluetooth-naming branch from 95e575f to 7ed6ca6 Compare November 13, 2025 18:40

Copilot started reviewing on behalf of noahp November 13, 2025 18:40 View session

Copilot finished reviewing on behalf of noahp November 13, 2025 18:42

Copilot AI reviewed Nov 13, 2025

View reviewed changes

umapraseeda approved these changes Nov 14, 2025

View reviewed changes

doc/nrf/releases_and_maturity/releases/release-notes-changelog.rst Outdated Show resolved Hide resolved

grochu approved these changes Nov 14, 2025

View reviewed changes

nordicjm requested changes Nov 14, 2025

View reviewed changes

Copilot AI review requested due to automatic review settings November 14, 2025 17:17

Copilot started reviewing on behalf of noahp November 14, 2025 17:17 View session

lib: hw_id: change name "BLE" to "Bluetooth"/"BT"

192369f

Rename the option and the name string to adhere to Bluetooth naming convention. See original suggestion here: nrfconnect#25313 (comment) Signed-off-by: Noah Pendleton <[email protected]>

Copilot finished reviewing on behalf of noahp November 14, 2025 17:19

noahp force-pushed the noahp/hw_id-bluetooth-naming branch from ee9c285 to 192369f Compare November 14, 2025 17:19

Copilot AI reviewed Nov 14, 2025

View reviewed changes

noahp requested a review from nordicjm November 14, 2025 17:20

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

lib: hw_id: change name "BLE" to "Bluetooth"/"BT" #25576

lib: hw_id: change name "BLE" to "Bluetooth"/"BT" #25576

Uh oh!

noahp commented Nov 13, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

NordicBuilder commented Nov 13, 2025 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

github-actions bot commented Nov 13, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

nordicjm Nov 14, 2025

Uh oh!

noahp Nov 14, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

lib: hw_id: change name "BLE" to "Bluetooth"/"BT" #25576

Are you sure you want to change the base?

lib: hw_id: change name "BLE" to "Bluetooth"/"BT" #25576

Uh oh!

Conversation

noahp commented Nov 13, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

NordicBuilder commented Nov 13, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

CI Information

Inputs:

Sources:

Github labels

Outputs:

Toolchain

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

Uh oh!

github-actions bot commented Nov 13, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

nordicjm Nov 14, 2025

Choose a reason for hiding this comment

Uh oh!

noahp Nov 14, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

NordicBuilder commented Nov 13, 2025 •

edited

Loading