Fix sid ble option fault by ktaborowski · Pull Request #741 · nrfconnect/sdk-sidewalk

ktaborowski · 2026-02-16T10:24:38Z

CI parameters

Jenkins:
  test-sdk-sidewalk: master
  # To reconfigure functional tests:
  # use GH labels func-* (default is func-commit)
  # or
  # Use YAML Filters. Helper side to set filters: https://tester-pc.nordicsemi.no:8080/test_mgmt
  # Filters (place copied YAML filters here):
  #  - filter1:
  #     board: nrf52

Description

KRKNWK-21514
Fix Sidewalk option SID_BLE_USER_CFG_ADV_AND_CONN fail (not implemented).
The aim of this PR is to avoid fault (device hangout). The option will return not implemented error code.

Self review

There is no commented code.
There are no TODO/FIXME comments without associated issue ticket.
Commits are properly organized.
Change has been tested.
Tests were updated (if applicable).

- Improved the return value of `sid_ble_bt_disable` to indicate the number of active connections. - Updated doc for `sid_ble_bt_disable` to clarify return values. Signed-off-by: Krzysztof Taborowski <krzysztof.taborowski@nordicsemi.no>

doc/known_issues.rst

github-actions · 2026-02-16T11:37:32Z

Sample		diff	used
thingy53/nrf5340/cpuapp:sample.sidewalk.demo.ble_only	RAM	0 B	114.22 KB
	ROM	244 B	374.66 KB
nrf5340dk/nrf5340/cpuapp:sample.sidewalk.demo.ble_only	RAM	0 B	104.38 KB
	ROM	248 B	334.29 KB
nrf5340dk/nrf5340/cpuapp:sample.sidewalk.demo.sx1262	RAM	0 B	121.17 KB
	ROM	244 B	418.35 KB
nrf5340dk/nrf5340/cpuapp:sample.sidewalk.dut.ble_only	RAM	0 B	128.8 KB
	ROM	1.18 KB	412.51 KB
nrf5340dk/nrf5340/cpuapp:sample.sidewalk.dut.no_secure	RAM	0 B	145.63 KB
	ROM	1.19 KB	490.98 KB
nrf52840dk/nrf52840:sample.sidewalk.demo.ble_only	RAM	0 B	113.52 KB
	ROM	244 B	414.17 KB
nrf52840dk/nrf52840:sample.sidewalk.demo.lr1110	RAM	0 B	131.02 KB
	ROM	244 B	503.84 KB
nrf52840dk/nrf52840:sample.sidewalk.demo.sx1262	RAM	0 B	130.62 KB
	ROM	244 B	497.35 KB
nrf52840dk/nrf52840:sample.sidewalk.dut.ble_only	RAM	0 B	137.77 KB
	ROM	1.19 KB	490.88 KB
nrf54l15dk/nrf54l10/cpuapp:sample.sidewalk.demo.ble_only	RAM	0 B	115.77 KB
	ROM	244 B	391.69 KB
nrf54l15dk/nrf54l10/cpuapp:sample.sidewalk.demo.lr1110	RAM	0 B	132.95 KB
	ROM	248 B	476.71 KB
nrf54l15dk/nrf54l10/cpuapp:sample.sidewalk.demo.sx1262	RAM	0 B	132.56 KB
	ROM	244 B	470.21 KB
nrf54l15dk/nrf54l10/cpuapp:sample.sidewalk.dut.ble_only	RAM	0 B	140.17 KB
	ROM	1.18 KB	469.55 KB
nrf54l15dk/nrf54l15/cpuapp/ns:sample.sidewalk.demo.ble_only	RAM	0 B	114.92 KB
	ROM	248 B	343.92 KB
nrf54l15dk/nrf54l15/cpuapp/ns:sample.sidewalk.demo.lr1110	RAM	0 B	132.1 KB
	ROM	244 B	428.94 KB
nrf54l15dk/nrf54l15/cpuapp/ns:sample.sidewalk.demo.sx1262	RAM	0 B	131.71 KB
	ROM	244 B	422.45 KB
nrf54l15dk/nrf54l15/cpuapp/ns:sample.sidewalk.dut.ble_only	RAM	0 B	139.32 KB
	ROM	1.18 KB	422.03 KB
nrf54l15dk/nrf54l15/cpuapp:sample.sidewalk.demo.ble_only	RAM	0 B	115.77 KB
	ROM	244 B	391.66 KB
nrf54l15dk/nrf54l15/cpuapp:sample.sidewalk.demo.lr1110	RAM	0 B	132.95 KB
	ROM	248 B	476.67 KB
nrf54l15dk/nrf54l15/cpuapp:sample.sidewalk.demo.sx1262	RAM	0 B	132.56 KB
	ROM	244 B	470.18 KB
nrf54l15dk/nrf54l15/cpuapp:sample.sidewalk.dut.ble_only	RAM	0 B	140.17 KB
	ROM	1.18 KB	469.52 KB
nrf5340dk/nrf5340/cpuapp:sample.sidewalk.dut.sx1262	RAM	0 B	145.65 KB
	ROM	1.18 KB	498.5 KB
nrf5340dk/nrf5340/cpuapp:sample.sidewalk.hello.ble_only	RAM	0 B	94.46 KB
	ROM	244 B	332.85 KB
nrf5340dk/nrf5340/cpuapp:sample.sidewalk.hello.ble_only.release	RAM	0 B	87.84 KB
	ROM	60 B	255.66 KB
nrf52840dk/nrf52840:sample.sidewalk.dut.lr1110	RAM	0 B	164.39 KB
	ROM	1.18 KB	675.32 KB
nrf52840dk/nrf52840:sample.sidewalk.dut.no_secure	RAM	0 B	154.91 KB
	ROM	1.18 KB	567.56 KB
nrf52840dk/nrf52840:sample.sidewalk.dut.sx1262	RAM	0 B	154.93 KB
	ROM	1.18 KB	576.17 KB
nrf52840dk/nrf52840:sample.sidewalk.hello.ble_only	RAM	0 B	103.35 KB
	ROM	244 B	412.5 KB
nrf54l15dk/nrf54l10/cpuapp:sample.sidewalk.dut.lr1110	RAM	0 B	166.46 KB
	ROM	1.18 KB	649.26 KB
nrf54l15dk/nrf54l10/cpuapp:sample.sidewalk.dut.no_secure	RAM	0 B	156.97 KB
	ROM	1.19 KB	542.23 KB
nrf54l15dk/nrf54l10/cpuapp:sample.sidewalk.dut.sx1262	RAM	0 B	157.02 KB
	ROM	1.18 KB	550.12 KB
nrf54l15dk/nrf54l10/cpuapp:sample.sidewalk.hello.ble_only	RAM	0 B	105.81 KB
	ROM	244 B	389.99 KB
nrf54l15dk/nrf54l15/cpuapp/ns:sample.sidewalk.dut.no_secure	RAM	0 B	156.16 KB
	ROM	1.18 KB	499.8 KB
nrf54l15dk/nrf54l15/cpuapp/ns:sample.sidewalk.dut.sx1262	RAM	0 B	156.17 KB
	ROM	1.18 KB	502.58 KB
nrf54l15dk/nrf54l15/cpuapp/ns:sample.sidewalk.hello.ble_only	RAM	0 B	104.96 KB
	ROM	248 B	342.24 KB
nrf54l15dk/nrf54l15/cpuapp:sample.sidewalk.dut.lr1110	RAM	0 B	166.46 KB
	ROM	1.18 KB	649.23 KB
nrf54l15dk/nrf54l15/cpuapp:sample.sidewalk.dut.no_secure	RAM	0 B	156.97 KB
	ROM	1.19 KB	542.2 KB
nrf54l15dk/nrf54l15/cpuapp:sample.sidewalk.dut.sx1262	RAM	0 B	157.02 KB
	ROM	1.18 KB	550.09 KB
nrf54l15dk/nrf54l15/cpuapp:sample.sidewalk.hello.ble_only	RAM	0 B	105.81 KB
	ROM	244 B	389.96 KB
nrf5340dk/nrf5340/cpuapp:sample.sidewalk.hello.release.sx1262	RAM	0 B	104.64 KB
	ROM	60 B	335.86 KB
nrf5340dk/nrf5340/cpuapp:sample.sidewalk.hello.sx1262	RAM	0 B	122.81 KB
	ROM	244 B	432.81 KB
nrf52840dk/nrf52840:sample.sidewalk.hello.ble_only.release	RAM	0 B	96.56 KB
	ROM	56 B	336.72 KB
nrf52840dk/nrf52840:sample.sidewalk.hello.lr1110	RAM	0 B	132.38 KB
	ROM	248 B	518.04 KB
nrf52840dk/nrf52840:sample.sidewalk.hello.release.lr1110	RAM	0 B	114.06 KB
	ROM	56 B	419.28 KB
nrf54l15dk/nrf54l10/cpuapp:sample.sidewalk.hello.ble_only.release	RAM	0 B	99.18 KB
	ROM	56 B	311.98 KB
nrf54l15dk/nrf54l10/cpuapp:sample.sidewalk.hello.lr1110	RAM	0 B	134.54 KB
	ROM	244 B	490.9 KB
nrf54l15dk/nrf54l10/cpuapp:sample.sidewalk.hello.release.lr1110	RAM	0 B	116.37 KB
	ROM	56 B	391.41 KB
nrf54l15dk/nrf54l15/cpuapp/ns:sample.sidewalk.hello.ble_only.release	RAM	0 B	98.33 KB
	ROM	60 B	264.99 KB
nrf54l15dk/nrf54l15/cpuapp/ns:sample.sidewalk.hello.lr1110	RAM	0 B	133.69 KB
	ROM	244 B	443.13 KB
nrf54l15dk/nrf54l15/cpuapp/ns:sample.sidewalk.hello.release.lr1110	RAM	0 B	115.52 KB
	ROM	60 B	344.42 KB
nrf54l15dk/nrf54l15/cpuapp:sample.sidewalk.hello.ble_only.release	RAM	0 B	99.18 KB
	ROM	56 B	311.95 KB
nrf54l15dk/nrf54l15/cpuapp:sample.sidewalk.hello.lr1110	RAM	0 B	134.54 KB
	ROM	244 B	490.87 KB
nrf54l15dk/nrf54l15/cpuapp:sample.sidewalk.hello.release.lr1110	RAM	0 B	116.37 KB
	ROM	56 B	391.38 KB
nrf54l15dk/nrf54l15/cpuapp:sample.sidewalk.hello.release.sx1262	RAM	0 B	115.98 KB
	ROM	60 B	388.54 KB
nrf52840dk/nrf52840:sample.sidewalk.hello.release.sx1262	RAM	0 B	113.67 KB
	ROM	56 B	416.44 KB
nrf52840dk/nrf52840:sample.sidewalk.hello.sx1262	RAM	0 B	131.99 KB
	ROM	244 B	511.55 KB
nrf54l15dk/nrf54l10/cpuapp:sample.sidewalk.hello.release.sx1262	RAM	0 B	115.98 KB
	ROM	60 B	388.57 KB
nrf54l15dk/nrf54l10/cpuapp:sample.sidewalk.hello.sx1262	RAM	0 B	134.14 KB
	ROM	248 B	484.41 KB
nrf54l15dk/nrf54l15/cpuapp/ns:sample.sidewalk.hello.release.sx1262	RAM	0 B	115.13 KB
	ROM	56 B	341.57 KB
nrf54l15dk/nrf54l15/cpuapp/ns:sample.sidewalk.hello.sx1262	RAM	0 B	133.29 KB
	ROM	244 B	436.64 KB
nrf54l15dk/nrf54l15/cpuapp:sample.sidewalk.hello.sx1262	RAM	0 B	134.14 KB
	ROM	248 B	484.39 KB
nrf54l15dk/nrf54l15/cpuapp:sample.sidewalk.helloPower.ble.release	RAM	0 B	99.17 KB
	ROM	60 B	311.33 KB
nrf54l15dk/nrf54l15/cpuapp:sample.sidewalk.helloPower.ble_only.release	RAM	0 B	99.17 KB
	ROM	60 B	311.33 KB
nrf54l15dk/nrf54l15/cpuapp:sample.sidewalk.helloPower.fsk.release.sx1262	RAM	0 B	115.97 KB
	ROM	56 B	387.78 KB
nrf54l15dk/nrf54l15/cpuapp:sample.sidewalk.helloPower.lora.release.sx1262	RAM	0 B	115.97 KB
	ROM	56 B	387.78 KB

- Updated the BLE adapter interface to include a user config. - Added a new known issue regarding unsupported Sidewalk option for BLE config in NCS. - Introduced a new command for setting BLE user config in dut. Signed-off-by: Krzysztof Taborowski <krzysztof.taborowski@nordicsemi.no>

ktaborowski requested a review from LuDuda February 16, 2026 10:24

github-actions bot added the source PR changing src files label Feb 16, 2026

ktaborowski force-pushed the ble-cfg-test branch from ef3e7a2 to 7a65c21 Compare February 16, 2026 11:13

github-actions bot added the doc-required PR must not be merged without tech writer approval. label Feb 16, 2026

ktaborowski requested a review from annwoj February 16, 2026 11:16

ktaborowski force-pushed the ble-cfg-test branch from 7a65c21 to b89451a Compare February 16, 2026 11:21

sal: sid_pal: Improve app callbacks return codes

c7dd8cc

- Improved the return value of `sid_ble_bt_disable` to indicate the number of active connections. - Updated doc for `sid_ble_bt_disable` to clarify return values. Signed-off-by: Krzysztof Taborowski <krzysztof.taborowski@nordicsemi.no>

ktaborowski force-pushed the ble-cfg-test branch from b89451a to 93c58e9 Compare February 16, 2026 11:26

annwoj reviewed Feb 16, 2026

View reviewed changes

doc/known_issues.rst Outdated Show resolved Hide resolved

ktaborowski force-pushed the ble-cfg-test branch from 93c58e9 to 3833ee0 Compare February 16, 2026 11:35

ktaborowski force-pushed the ble-cfg-test branch from 3833ee0 to d58b4d6 Compare February 16, 2026 12:02

github-actions bot added the tests label Feb 16, 2026

ktaborowski requested a review from annwoj February 16, 2026 13:25

annwoj approved these changes Feb 16, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix sid ble option fault #741

Fix sid ble option fault #741
ktaborowski wants to merge 2 commits intonrfconnect:mainfrom
ktaborowski:ble-cfg-test

ktaborowski commented Feb 16, 2026 •

edited

Loading

Uh oh!

Uh oh!

github-actions bot commented Feb 16, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Comments

Conversation

ktaborowski commented Feb 16, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

CI parameters

Description

Self review

Uh oh!

Uh oh!

github-actions bot commented Feb 16, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Comments

ktaborowski commented Feb 16, 2026 •

edited

Loading