-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
Bluetooth: CCP: Add read bearer provider name for the CCP client #79462
Draft
Thalley
wants to merge
9
commits into
zephyrproject-rtos:main
Choose a base branch
from
Thalley:ccp_client_name
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.
+4,615
−306
Conversation
This file contains 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
Thalley
force-pushed
the
ccp_client_name
branch
8 times, most recently
from
October 8, 2024 21:19
911c28e
to
c298c01
Compare
alloc_value is more correct as it does allocate a new value, rather than just returning/getting a value. Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no>
Rather than throwing an assert, it will now do a better check for already-allocated CCIDs and return an error instead. Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no>
Move the CCID header to the include directory to make it public. This also compiles the ccid.c file whenver CONFIG_BT_AUDIO is enabled, rather than having an additional Kconfig for it, since the API is now public and may be used by other than our internal services. Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no>
Add unit tests of the CCID functions. To support these new tests, the GATT mocks need to support dynamic GATT DB so we can add and remove a service that contains CCIDs. Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no>
The CCP API for the Call Control Profile works on top of the TBS API, and will eventually replace parts of the TBS API. Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no>
Thalley
force-pushed
the
ccp_client_name
branch
2 times, most recently
from
October 9, 2024 13:08
b996153
to
e46a97f
Compare
Added initial CCP client implementation that simply does discovery of TBS on a remote CCP server. Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no>
Add support for setting and getting the bearer provider name. For now the name will be duplicated by the TBS implementation, but will be optimizied in the future so only one copy of the name exists. Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no>
Add bt_ccp_client_get_bearers that will return the bearers of a client so that the application can always retrieve them if they do not store them from the discovery callback. Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no>
Thalley
force-pushed
the
ccp_client_name
branch
from
October 9, 2024 15:10
e46a97f
to
9f46ae7
Compare
Add support for getting the remote bearer provider name. Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no>
Thalley
force-pushed
the
ccp_client_name
branch
from
October 9, 2024 15:26
9f46ae7
to
2cd92cd
Compare
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.
Add a function to read the bearer provider name as the CCP client.