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

Peripheral initiated connection parameter update is ignored #28870

Closed
thomasstenersen opened this issue Oct 2, 2020 · 0 comments · Fixed by #29154
Closed

Peripheral initiated connection parameter update is ignored #28870

thomasstenersen opened this issue Oct 2, 2020 · 0 comments · Fixed by #29154
Assignees
Labels
area: Bluetooth bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug

Comments

@thomasstenersen
Copy link
Collaborator

Describe the bug
If a peripheral initiates a connection parameter update before the first conn param update timer has expired, its requested parameters are ignored (no error code return from bt_conn_le_param_update()). If CONFIG_BT_GAP_AUTO_UPDATE_CONN_PARAMS=n nothing will happen when the timer eventually does expire

To Reproduce
Steps to reproduce the behavior:

  1. Checkout thomasstenersen@9a9e240
  2. Build and run the peripheral_hr sample (example west build -b nrf5284dk_nrf52840 -d build samples/bluetooth/peripheral_hr)
  3. Connect to the peripheral and observe that the first connection parameter update request only comes after >5 seconds.

Expected behavior
I would expect bt_conn_le_param_update() to either

  1. Return an error if it is not ready to handle the request yet, or
  2. Execute the connection parameter update request as possible.

Logs and console output
nRF Sniffer/Wireshark log: conn-update-delay.zip
Sample log output:

<snip>
Connected
Called conn update
Called conn update
Called conn update
Called conn update
Called conn update
Called conn update
Called conn update
</snip>
@thomasstenersen thomasstenersen added bug The issue is a bug, or the PR is fixing a bug area: Bluetooth labels Oct 2, 2020
@nashif nashif added the priority: medium Medium impact/importance bug label Oct 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Bluetooth bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants