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

Bluetooth: controller: split: Fix procedure complete event generation #21994

Closed
cvinayak opened this issue Jan 17, 2020 · 4 comments · Fixed by #44791
Closed

Bluetooth: controller: split: Fix procedure complete event generation #21994

cvinayak opened this issue Jan 17, 2020 · 4 comments · Fixed by #44791
Assignees
Labels
area: Bluetooth Controller 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: medium Medium impact/importance bug

Comments

@cvinayak
Copy link
Contributor

Describe the bug
Generate PHY update complete and Conn Update events after the on-air instant

To Reproduce
Steps to reproduce the behavior:

  1. mkdir build; cd build
  2. cmake -DBOARD=board_xyz
  3. make
  4. See error

Expected behavior
A clear and concise description of what you expected to happen.

Impact
What impact does this issue have on your progress (e.g., annoyance, showstopper)

showstopper

Screenshots or console output
If applicable, add a screenshot (drag-and-drop an image), or console logs
(cut-and-paste text and put a code fence (```) before and after, to help
explain the issue.

Environment (please complete the following information):

  • OS: (e.g. Linux, MacOS, Windows)
  • Toolchain (e.g Zephyr SDK, ...)
  • Commit SHA or Version used

Additional context
Add any other context about the problem here.

@cvinayak cvinayak added bug The issue is a bug, or the PR is fixing a bug area: Bluetooth area: Bluetooth Qualification Bluetooth Qualification -related issues and pull requests labels Jan 17, 2020
@cvinayak cvinayak self-assigned this Jan 17, 2020
@cvinayak
Copy link
Contributor Author

Use inspiration from legacy controller: 2bf9d49

@jhedberg jhedberg added the priority: medium Medium impact/importance bug label Jan 21, 2020
@carlescufi
Copy link
Member

@kruithofa @cvinayak is this still an issue on the current EBQ runs?

@cvinayak
Copy link
Contributor Author

cvinayak commented Feb 20, 2020

@carlescufi This is not an issue for the runs, we have workaround. Fix be postpone as a requirement to the new LLCP work.

@cvinayak
Copy link
Contributor Author

@carlescufi this depends on new LLCP work, not worth new effort to fix this in current split controller.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Bluetooth Controller 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: medium Medium impact/importance bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants