samples: Bluetooth: Central plus Peripheral with notifications#110679
samples: Bluetooth: Central plus Peripheral with notifications#110679cvinayak wants to merge 1 commit into
Conversation
Add three new Bluetooth samples: - peripheral_gatt_notify: Peripheral that sends GATT notifications - central_notify_receive: Central that receives GATT notifications - central_peripheral_notify_send_receive: Central plus Peripheral Add bsim CI test tests/bsim/bluetooth/host/multirole_gatt_relay to verify the 3-device notification relay topology under simulation. Assisted-by: GitHub CoPilot: Claude Opus 4.6 Signed-off-by: Vinayak Kariappa Chettimada <vich@nordicsemi.no>
|
|
@cvinayak What are the purpose of these samples? We already have samples that perform these GATT operations, do we not? |
|
The sample is intended to cover multirole usecase similar to wireless split keyboards; analysis related to zmkfirmware/zmk#3370 (comment) I suspect regression in Zephyr Bluetooth Controller after v3.5.0. |
The multi-role response somewhat covers why the central_peripheral_notify sample is being added, but why the other 2? And if this is to find and fix a regression, wouldn't it be better with just a test rather than a sample that more or less just does the same as other samples we have? |
Thalley
left a comment
There was a problem hiding this comment.
There is a lack of rationale behind adding these samples IMO



Add three new Bluetooth samples:
Add bsim CI test tests/bsim/bluetooth/host/multirole_gatt_relay to verify the 3-device notification relay topology under simulation.
Assisted-by: GitHub CoPilot: Claude Opus 4.6
Steps to run:
Terminal 1:
Terminal 2:
Terminal 3:
Terminal 4:
Logs:
Terminal 1:
Terminal 2:
Terminal 3: