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/ll: Add more HCI commands for fake dual-mode #1244

Merged

Conversation

andrzej-kaczmarek
Copy link
Contributor

This allows NimBLE to be initialized as "dual-mode" controller by BlueZ.
Also inquiry "works" a bit better since Inquiry Complete event is
simulated after inquiry duration has elapsed.

@utzig
Copy link
Member

utzig commented Apr 22, 2022

Style check summary

No suggestions at this time!

@kasjer
Copy link
Contributor

kasjer commented Apr 22, 2022

With this PR Read Class of Device passes but later on something fails and interface is still not usable on bluez 5.5

< HCI Command: Set Event Filter (0x03|0x0005) plen 1                                                                                                                                                                                  #907 [hci0] 72416.495216
        Type: Clear All Filters (0x00)
> HCI Event: Command Status (0x0f) plen 4                                                                                                                                                                                             #908 [hci0] 72416.496167
      Set Event Filter (0x03|0x0005) ncmd 1
        Status: Unknown HCI Command (0x01)

This allows NimBLE to be initialized as "dual-mode" controller by BlueZ.
Also inquiry "works" a bit better since Inquiry Complete event is
simulated after inquiry duration has elapsed.
@kasjer
Copy link
Contributor

kasjer commented May 10, 2022

Tested on Windows and Linux without problems

@andrzej-kaczmarek andrzej-kaczmarek merged commit fc66099 into apache:master Jun 9, 2022
@andrzej-kaczmarek andrzej-kaczmarek deleted the ll-fake-dual-mode branch June 9, 2022 09:24
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