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

accept LE Set Extended commands in the controller #229

Closed
wants to merge 1 commit into from

Conversation

vChavezB
Copy link

@vChavezB vChavezB commented Jul 30, 2023

Following commands are now accepted:

  • hci_le_set_extended_advertising_parameters_command
  • hci_le_set_advertising_set_random_address_command
  • hci_le_set_extended_advertising_data_command
  • hci_le_set_extended_scan_response_data_command
  • hci_le_set_extended_advertising_enable_command

This should fix connection in case a BT Host uses extended advertisement commands. Seee #215 (comment)

Following commands are now accepted:

-hci_le_set_extended_advertising_parameters_command
-hci_le_set_advertising_set_random_address_command
-hci_le_set_extended_advertising_data_command
-hci_le_set_extended_scan_response_data_command
-hci_le_set_extended_advertising_enable_command
@google-cla
Copy link

google-cla bot commented Jul 30, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@barbibulle
Copy link
Collaborator

Did you try the extended advertising support in this branch? https://github.com/google/bumble/tree/gbg/controller-extended-advertising
it should have what you need.

@vChavezB
Copy link
Author

thanks for the info, did not know about this.

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.

2 participants