Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Bluetooth: Controller: Enable Connection Subrating support in the SDC #15500

Merged
merged 2 commits into from
Jul 12, 2024
Merged

Bluetooth: Controller: Enable Connection Subrating support in the SDC #15500

merged 2 commits into from
Jul 12, 2024

Conversation

weeTike
Copy link
Contributor

@weeTike weeTike commented May 24, 2024

This enables experimental support for Connection Subrating.

@weeTike weeTike requested a review from a team as a code owner May 24, 2024 13:17
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label May 24, 2024
@weeTike weeTike marked this pull request as draft May 24, 2024 13:21
@weeTike weeTike marked this pull request as draft May 24, 2024 13:21
@weeTike weeTike removed changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels May 24, 2024
@NordicBuilder
Copy link
Contributor

NordicBuilder commented May 24, 2024 •
edited
Loading

Test specification

CI/Jenkins/NRF

  • Integration Platforms

CI/Jenkins/integration

Test Module File based changes Manually selected West overwrite
test-fw-nrfconnect-ble_samples X
test-fw-nrfconnect-chip X
test-fw-nrfconnect-fem X
test-fw-nrfconnect-rs X
test-fw-nrfconnect-zigbee X
test-sdk-find-my X
test-sdk-sidewalk X
test_ble_commit X

Detailed information of selected test modules

Note: This message is automatically posted and updated by the CI

@weeTike weeTike force-pushed the add_subrating_api branch from 9797d3b to 0e42d05 Compare May 24, 2024 13:35
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label May 24, 2024
@NordicBuilder
Copy link
Contributor

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publishing GitHub Action.

@weeTike weeTike force-pushed the add_subrating_api branch from 0e42d05 to accbebf Compare May 24, 2024 15:16
@weeTike weeTike force-pushed the add_subrating_api branch from accbebf to 7ba0ac8 Compare June 26, 2024 09:59
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Jun 26, 2024 •
edited
Loading

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@36fbc3f nrfconnect/sdk-zephyr@f6325a5 nrfconnect/[email protected]

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@weeTike weeTike force-pushed the add_subrating_api branch 2 times, most recently from 0905ab7 to 6bbb16c Compare July 2, 2024 13:10
@weeTike weeTike marked this pull request as ready for review July 9, 2024 08:14
@weeTike weeTike force-pushed the add_subrating_api branch 3 times, most recently from 63827aa to 7a23a0a Compare July 9, 2024 11:53
@weeTike weeTike force-pushed the add_subrating_api branch from 48390dd to b499654 Compare July 10, 2024 09:53
@NordicBuilder NordicBuilder removed the DNM label Jul 10, 2024
@weeTike weeTike force-pushed the add_subrating_api branch from a716a1f to b499654 Compare July 10, 2024 10:10
@NordicBuilder NordicBuilder removed the DNM label Jul 10, 2024
@adamfc2000 adamfc2000 force-pushed the add_subrating_api branch 2 times, most recently from f51e759 to 67a387f Compare July 10, 2024 10:21
@weeTike weeTike force-pushed the add_subrating_api branch from 67a387f to 27fafcb Compare July 10, 2024 10:35
@weeTike weeTike requested a review from sean-madigan July 10, 2024 10:50
@bugo-nordic bugo-nordic self-requested a review July 10, 2024 12:38
@weeTike weeTike force-pushed the add_subrating_api branch from 27fafcb to 0b3ee7e Compare July 10, 2024 14:42
weeTike and others added 2 commits July 11, 2024 16:26
@weeTike
Bluetooth: Controller: Enable Connection Subrating support in the SDC
7cecedd 
This enables experimental support for Connection Subrating.

Signed-off-by: Timothy Keys <[email protected]>
@adamfc2000 @weeTike
Bluetooth: Controller: Add memory macro for subrating
aa9ab97 
This includes memory used for subrating in the MEMPOOL_SIZE
macro.

Signed-off-by: Adam Cavender <[email protected]>
@weeTike weeTike force-pushed the add_subrating_api branch from 0b3ee7e to aa9ab97 Compare July 11, 2024 15:28
@github-actions github-actions bot removed the manifest label Jul 11, 2024
@rlubos rlubos merged commit 046cca6 into nrfconnect:main Jul 12, 2024
22 checks passed
@weeTike weeTike deleted the add_subrating_api branch July 24, 2024 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bugo-nordic bugo-nordic bugo-nordic approved these changes

@sean-madigan sean-madigan sean-madigan approved these changes

@rugeGerritsen rugeGerritsen rugeGerritsen approved these changes

@carlescufi carlescufi Awaiting requested review from carlescufi

@tejlmand tejlmand Awaiting requested review from tejlmand

Assignees
No one assigned
Labels
changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@weeTike @NordicBuilder @rugeGerritsen @sean-madigan @bugo-nordic @rlubos @adamfc2000