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

LL.TS Test LL/CON/SLA/BV-129-C fails (split) #23109

Closed
mtpr-ot opened this issue Feb 26, 2020 · 1 comment · Fixed by #23126
Closed

LL.TS Test LL/CON/SLA/BV-129-C fails (split) #23109

mtpr-ot opened this issue Feb 26, 2020 · 1 comment · Fixed by #23126
Assignees
Labels
area: Bluetooth Qualification Bluetooth Qualification -related issues and pull requests area: Bluetooth bug The issue is a bug, or the PR is fixing a bug priority: high High impact/importance bug
Milestone

Comments

@mtpr-ot
Copy link
Collaborator

mtpr-ot commented Feb 26, 2020

With the introduction of commit "Bluetooth: controller: split: fix LL/CON/MAS/BV-74-C tx timing" (0ac4433), the test LL/CON/SLA/BV-129-C fails on EBQ.

To Reproduce
Run LL/CON/SLA/BV-129-C on EBQ on "split" controller, without coded PHY.

Expected behavior
Without commit 0ac4433 the test passes - with it, it fails.

Impact
Qualification on Zephyr 2.2 will fail.

Additional context
Non-Nordic HW platform and link layer used.

@mtpr-ot mtpr-ot added the bug The issue is a bug, or the PR is fixing a bug label Feb 26, 2020
@aescolar aescolar added area: Bluetooth Qualification Bluetooth Qualification -related issues and pull requests priority: high High impact/importance bug labels Feb 26, 2020
@aescolar aescolar added this to the v2.2.0 milestone Feb 26, 2020
@aescolar
Copy link
Member

CC @kruithofa @cvinayak

cvinayak added a commit to cvinayak/zephyr that referenced this issue Mar 6, 2020
Fix to return the max tx/rx time back to set default time
after using mandatory minimum PDU length and time while
switching back from Coded PHY to 1M PHY.

Also fixes zephyrproject-rtos#23109.

Signed-off-by: Vinayak Kariappa Chettimada <vich@nordicsemi.no>
jhedberg pushed a commit that referenced this issue Mar 10, 2020
Fix to return the max tx/rx time back to set default time
after using mandatory minimum PDU length and time while
switching back from Coded PHY to 1M PHY.

Also fixes #23109.

Signed-off-by: Vinayak Kariappa Chettimada <vich@nordicsemi.no>
hakehuang pushed a commit to hakehuang/zephyr that referenced this issue Mar 18, 2020
Fix to return the max tx/rx time back to set default time
after using mandatory minimum PDU length and time while
switching back from Coded PHY to 1M PHY.

Also fixes zephyrproject-rtos#23109.

Signed-off-by: Vinayak Kariappa Chettimada <vich@nordicsemi.no>
cvinayak added a commit to cvinayak/zephyr that referenced this issue Sep 23, 2020
Fix to return the max tx/rx time back to set default time
after using mandatory minimum PDU length and time while
switching back from Coded PHY to 1M PHY.

Also fixes zephyrproject-rtos#23109.

Signed-off-by: Vinayak Kariappa Chettimada <vich@nordicsemi.no>
nashif pushed a commit that referenced this issue Nov 17, 2020
Fix to return the max tx/rx time back to set default time
after using mandatory minimum PDU length and time while
switching back from Coded PHY to 1M PHY.

Also fixes #23109.

Signed-off-by: Vinayak Kariappa Chettimada <vich@nordicsemi.no>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Bluetooth Qualification Bluetooth Qualification -related issues and pull requests area: Bluetooth bug The issue is a bug, or the PR is fixing a bug priority: high High impact/importance bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants