-
Notifications
You must be signed in to change notification settings - Fork 8.3k
Bluetooth: BAP: Add bt_bap_ep_get_conn #96365
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
Open
Thalley
wants to merge
3
commits into
zephyrproject-rtos:main
Choose a base branch
from
Thalley:ep_get_conn
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
+140
−2
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
Contributor
Author
|
Depends on #95605 |
5986ef0 to
749ef62
Compare
749ef62 to
bdaaf65
Compare
ad34ce1 to
30963c9
Compare
Add a new function, bt_bap_ep_get_conn, which returns the ACL connection for the endpoint. This works because endpoints are specific to an ACL in BAP. The function returns a pointer with a new reference similar to the lookup functions from conn.h The conn pointer was not added to the bt_bap_ep_info struct, as doing so would be more likely to cause reference leaks if the caller did not care about the conn pointer when using bt_bap_ep_get_info. It can be added later if that is requested. Signed-off-by: Emil Gydesen <[email protected]>
Add entry for the new function bt_bap_ep_get_conn. Signed-off-by: Emil Gydesen <[email protected]>
Add steps in the BAP tests to verify the correctness of bt_bap_ep_get_conn. Signed-off-by: Emil Gydesen <[email protected]>
|
Contributor
Author
|
Will fix the unittest failiure once #96736 is merged |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
area: Bluetooth Audio
area: Bluetooth
area: Tests
Issues related to a particular existing or missing test
Release Notes
To be mentioned in the release notes
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.



Add a new function, bt_bap_ep_get_conn, which returns
the ACL connection for the endpoint. This works because
endpoints are specific to an ACL in BAP. The function
returns a pointer with a new reference similar to the
lookup functions from conn.h
The conn pointer was not added to the bt_bap_ep_info
struct, as doing so would be more likely to cause
reference leaks if the caller did not care about the
conn pointer when using bt_bap_ep_get_info. It can be
added later if that is requested.