-
Notifications
You must be signed in to change notification settings - Fork 734
Zephyr oct 3542 cherry pick of pts fixes from upstream zephyr part 2 #3541
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
Merged
rlubos
merged 9 commits into
nrfconnect:main
from
koffes:zephyr-OCT-3542-cherry-pick-of-pts-fixes-from-upstream-zephyr-part-2
Nov 27, 2025
Merged
Zephyr oct 3542 cherry pick of pts fixes from upstream zephyr part 2 #3541
rlubos
merged 9 commits into
nrfconnect:main
from
koffes:zephyr-OCT-3542-cherry-pick-of-pts-fixes-from-upstream-zephyr-part-2
Nov 27, 2025
+471
−281
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
6f4cbd0 to
f9e32c2
Compare
f9e32c2 to
cec48b4
Compare
Thalley
approved these changes
Nov 26, 2025
Contributor
Thalley
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks good to me. You can consider dropping the migration guide entry if you want to reduce the number of commits by 1
alexsven
approved these changes
Nov 26, 2025
Remove lookups in the Scan Delegator that relates to the advertiser address. The reason for this, is that the address is not considered a unique value for receive states, since the address may change over time in the case of (N)RPAs. Instead we shall rely exclusively on the address type, the sid and the broadcast ID. The implementation of the Scan Delegator and Broadcast Sink has been updated to not use addresses for lookups anymore, and there has been a minor API modification to set the PA sync state as part of bt_bap_scan_delegator_add_src as the higher layers are better suited to handle the PA Sync state. Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no> (cherry picked from commit 64ea533)
…state The Scan Delegator has been modified to not set the PA sync state automatically anymore, so the test needed to be updated. Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no> (cherry picked from commit 397c8ee)
…T BAP SD The Bluetooth BAP Scan Delegator had a minor behavioral change that may affect a small subset of devices. Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no> (cherry picked from commit 15abc32)
…A synced - BASS mandates that a remove_source operation is not done if we are synced to PA or BIS. This PR fixes that - Update BabbleSim tests to reflect this behavior - Fixes BASS/SR/SPE/BI-05-C Signed-off-by: Alexander Svensen <alexander.svensen@nordicsemi.no> (cherry picked from commit 87997afeb2b881fefd871aaabc6637626d3304e4)
…nnect A CIS may be connected in either the QoS Configured state or the enabling state. The QoS Configured state is the earliest state it is allowed, due to it being the first state where the CIS_ID and ASE_ID are paired. The enabling state is the "last" state it is allowed, as if the ASE is in the streaming, disabling or releasing state we should not allow/expect a CIS connection to happen. Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no> (cherry picked from commit 4c54eef0d0f00c32d075a7498ad33d41b2764cb8)
Add support for the ASCS CIS connection state events. Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no> (cherry picked from commit 29f4f24019ac5463d539364cc7c65961e28d71c0)
…state Add support for connecting the CIS in QoS state. To support this some checks were added to check the CIS state while entering the enabling state, as well as the ASE state when the CIS was connected to determine when to send the ASE start opcode. Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no> (cherry picked from commit b8e71e6098e48f5f2f95291825bcdc28f5b3c1e9)
Fixes a bug where encrypt state value was not updated based on given parameters. Signed-off-by: Omkar Kulkarni <omkar.kulkarni@nordicsemi.no> (cherry picked from commit b6e5d03ff4f74b02df05f70cc5c355b721e8beb6)
cec48b4 to
74bc861
Compare
This commit adds API to tester to allow exposing of Broadcast Receive State. Signed-off-by: Omkar Kulkarni <omkar.kulkarni@nordicsemi.no> (cherry picked from commit d8ed0f8e3de02642e6ccabbb144037688d107ad7)
74bc861 to
4cd53d1
Compare
NordicBuilder
added a commit
to NordicBuilder/sdk-nrf
that referenced
this pull request
Nov 27, 2025
Automatically created by action-manifest-pr GH action from PR: nrfconnect/sdk-zephyr#3541 Signed-off-by: Nordic Builder <pylon@nordicsemi.no>
rlubos
pushed a commit
to nrfconnect/sdk-nrf
that referenced
this pull request
Nov 27, 2025
Automatically created by action-manifest-pr GH action from PR: nrfconnect/sdk-zephyr#3541 Signed-off-by: Nordic Builder <pylon@nordicsemi.no>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.