Skip to content

Timeout - #34

Open
mweal-ed wants to merge 4 commits into
labapart:masterfrom
mweal-ed:timeout
Open

Timeout#34
mweal-ed wants to merge 4 commits into
labapart:masterfrom
mweal-ed:timeout

Conversation

@mweal-ed

Copy link
Copy Markdown
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
Copy Markdown
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

mweal-ed commented Jul 18, 2017

Copy link
Copy Markdown
Contributor Author

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

wojciiii commented Oct 15, 2017

Copy link
Copy Markdown

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.

3 participants