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

Timeout #34

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Timeout #34

wants to merge 4 commits into from

Conversation

mweal-ed
Copy link
Contributor

Add the following commands

  • gattlib_connect_timeout
  • gattlib_connect_async_timeout
  • gattlib_write_cmd_by_handle

note: these were only tested on Bluez 5

@oliviermartin
Copy link
Contributor

I am happy with this patch but there is the commit 38ba174 in it which is not strictly related to it. If you remove this commit for the time being I will be happy to merge this patch.

@mweal-ed
Copy link
Contributor Author

mweal-ed commented Jul 18, 2017

I believe these changes are dependent on the of the changes in PR #33 (38ba174). I will check in then next few days and see if I can pull them out as separate changes

@wojciiii
Copy link

wojciiii commented Oct 15, 2017

What is the status of these changes?

I tested with and without these changes and it looks like:

  • with the changes - connection to my device, but no data received from it.
  • without the changes - connection to my device and data received from it.

I really need gattlib_connect_timeout, as the ATMOTUBE LE device I have is slow to respond.
It looks like a connection attempt first succeeds after 2-3 attempts.

Is there anything that I could do to help?

I assume that someone should look at the output of hcidump with and without the changes and compare them.

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

3 participants