Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[nrfconnect] Changed BT device names from Chip to Matter. #7747

Merged
merged 1 commit into from
Jun 18, 2021

Conversation

kkasperczyk-no
Copy link
Contributor

Change overview

Changed BT device names for lighting-app and lock-app from ChipLock/Light to MatterLock/Light.

Testing

Small change and not much to test, so verified only that after changing device name commissioning using Python CHIP controller is still working fine.

Changed BT device names for lighting-app and lock-app
from ChipLock/Light to MatterLock/Light.
@github-actions
Copy link

Size increase report for "nrfconnect-example-build" from 01ee80e

File Section File VM
chip-lock.elf rodata 0 4
Full report output
BLOAT REPORT

Files found only in the build output:
    report.csv

Comparing ./master_artifact/chip-shell.elf and ./pull_artifact/chip-shell.elf:

sections,vmsize,filesize

Comparing ./master_artifact/chip-lock.elf and ./pull_artifact/chip-lock.elf:

sections,vmsize,filesize
rodata,4,0


@@ -29,7 +29,7 @@ CONFIG_OPENTHREAD_NETWORK_NAME="OpenThread"
CONFIG_OPENTHREAD_XPANID="11:11:11:11:22:22:22:22"

# Bluetooth overrides
CONFIG_BT_DEVICE_NAME="ChipLight"
CONFIG_BT_DEVICE_NAME="MatterLight"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quick question: why the device name on any advertisement from Matter reference implementation samples? For debugging?

The spec doesn't call for any device name in adv, and it calls for the entire Matter announcement to fit in the advertisement without needing a scan response:

In order to reduce 2.4 GHz spectrum congestion due to active BLE scanning, and to extend battery life in battery-powered devices, all critical data used for commissionable CHIP device discovery is contained in the Advertising Data rather than the Scan Response Data. This allows a BLE Commissioner to passively scan (i.e., not issue Scan Requests upon receiving scannable advertisements) and still be able to receive all information needed to commission a device.

Note that if additional vendor-specific information is to be conveyed and does not fit within the Advertising Data, it may be included in the Scan Response Data. See <<Manufacturer-specific data>> for details on including vendor-specific information.

Showing the advertisement of device name in examples, especially telling what kind it is, can give the impression it is needed, when it was designed specifically so that privacy was preserved about product types (e.g. VID/PID is not mandatory either)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, for debugging. It's much easier to analyze the traffic with the BLE sniffer etc. when you can easily filter by the device name. We don't use the scan response data. The BLE name fits in the advertising data if it's short enough. It's a good point, however, that we should guard the code for adding the device name to the advertising data with some compile time constant.

@andy31415 andy31415 merged commit 1d9927a into project-chip:master Jun 18, 2021
nikita-s-wrk pushed a commit to nikita-s-wrk/connectedhomeip that referenced this pull request Sep 23, 2021
…ip#7747)

Changed BT device names for lighting-app and lock-app
from ChipLock/Light to MatterLock/Light.
@kkasperczyk-no kkasperczyk-no deleted the ble_matter_name branch December 22, 2021 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation examples review - approved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants