Skip to content

Conversation

@kruithofa
Copy link

@kruithofa kruithofa commented May 16, 2022

Upon collision either the error code for different procedure
collision or same procedure collision must be transmitted,
which is fixed in this PR. Previously always the error code
for same procedure collision was sent

This will fix the collision related EBQ failures from issue #43435

Signed-off-by: Andries Kruithof [email protected]

@kruithofa kruithofa marked this pull request as ready for review May 16, 2022 11:29
@github-actions github-actions bot added area: Bluetooth area: Bluetooth Controller area: Tests Issues related to a particular existing or missing test labels May 16, 2022
@kruithofa kruithofa requested review from erbr-ot and sjanc May 16, 2022 11:29
@zephyrbot zephyrbot requested review from mtpr-ot and wopu-ot May 16, 2022 12:08
Upon collision either the error code for different procedure
collision or same procedure collision must be transmitted,
which is fixed in this PR. Previously always the error code
for same procedure collision was sent

Signed-off-by: Andries Kruithof <[email protected]>
Copy link
Contributor

@erbr-ot erbr-ot left a comment

Choose a reason for hiding this comment

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

Passed run on EBQ at our end for the tests in question

@kruithofa kruithofa changed the title Bluetooth: controller: llcp: send correct collision code Bluetooth: controller: llcp: bugfix for send correct collision code May 19, 2022
@kruithofa kruithofa changed the title Bluetooth: controller: llcp: bugfix for send correct collision code Bluetooth: controller: llcp: fix for send correct collision code May 19, 2022
@kruithofa kruithofa added the bug The issue is a bug, or the PR is fixing a bug label May 19, 2022
@kruithofa
Copy link
Author

Passed run on EBQ at our end for the tests in question

Tests in question were run on EBQ at Nordic, and passed

@kruithofa kruithofa added this to the v3.1.0 milestone May 19, 2022
@carlescufi carlescufi merged commit c2e07a3 into zephyrproject-rtos:main May 20, 2022
@kruithofa kruithofa deleted the llcp_fix_coll branch September 20, 2022 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: Bluetooth Controller area: Bluetooth LLCP area: Bluetooth area: Tests Issues related to a particular existing or missing test bug The issue is a bug, or the PR is fixing a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants