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.

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

nimble/host: Add HCI Commands/ events for connection subrating #1377

Conversation

rahult-github
Copy link
Contributor

This PR adds support for HCI commands and events in stack for Connection subrating feature.

@rahult-github
Copy link
Contributor Author

Hi @sjanc , this PR attempts to add host side support for Connection Subrating feature. Can you please take a look ?

@rahult-github rahult-github force-pushed the feature/host_commands_events_for_connection_subrating branch 4 times, most recently from 7127114 to cf8c925 Compare September 30, 2022 05:12
nimble/host/include/host/ble_gap.h Outdated Show resolved Hide resolved
nimble/host/include/host/ble_gap.h Outdated Show resolved Hide resolved
@rahult-github rahult-github force-pushed the feature/host_commands_events_for_connection_subrating branch from cf8c925 to a8161cb Compare October 4, 2022 09:14
Copy link
Contributor

@sjanc sjanc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

in general looks good to me but implementation should enabled only if enabled by config option nimble/syscfg.yml eg BLE_SUBRATE

similar to eg periodic sync transfer support (MYNEWT_VAL(BLE_PERIODIC_ADV_SYNC_TRANSFER))

@rahult-github rahult-github force-pushed the feature/host_commands_events_for_connection_subrating branch 2 times, most recently from cef1490 to e8fbcf9 Compare October 4, 2022 10:55
@rahult-github
Copy link
Contributor Author

in general looks good to me but implementation should enabled only if enabled by config option nimble/syscfg.yml eg BLE_SUBRATE

similar to eg periodic sync transfer support (MYNEWT_VAL(BLE_PERIODIC_ADV_SYNC_TRANSFER))

Thanks for this input. Yes, i missed making it configurable. Have added the change now.

@sjanc
Copy link
Contributor

sjanc commented Oct 4, 2022

looks like change to nimble/syscfg.yml is missing

@andrzej-kaczmarek
Copy link
Contributor

LL will use BLE_LL_CFG_FEAT_CONN_SUBRATING so syscfg flags should be called BLE_CONN_SUBRATING to be consistent

@rahult-github rahult-github force-pushed the feature/host_commands_events_for_connection_subrating branch from e8fbcf9 to cea7c54 Compare October 5, 2022 11:49
@rahult-github
Copy link
Contributor Author

The changes for macro name change and including in syscfg.xml are now pushed. Also changed the coding style issue reported in previous commit.

@rahult-github rahult-github force-pushed the feature/host_commands_events_for_connection_subrating branch from cea7c54 to 7b61e1f Compare October 5, 2022 12:44
@apache-mynewt-bot
Copy link

Style check summary

No suggestions at this time!

@andrzej-kaczmarek andrzej-kaczmarek merged commit 18203cc into apache:master Oct 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants