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

Add GATT busy checks #69

Closed
tannewt opened this issue May 30, 2024 · 2 comments
Closed

Add GATT busy checks #69

tannewt opened this issue May 30, 2024 · 2 comments

Comments

@tannewt
Copy link

tannewt commented May 30, 2024

I've triggered a bug where I'm sending multiple GATT operations at once. I'm doing an MTU exchange request and a discover services request at once. Other BLE stacks I've used will return a BUSY error for the second request. Nimble doesn't have these checks. Is this by design or would a PR be accepted to add these?

@rahult-github
Copy link
Collaborator

Hi @tannewt ,

Is this by design or would a PR be accepted to add these?

Since esp-nimble is taken from upstream mynewt nimble, request to please raise one PR upstream , so it can be taken up in all upcoming releases too going forward.

@tannewt
Copy link
Author

tannewt commented May 31, 2024

Done! apache#1790

@tannewt tannewt closed this as completed May 31, 2024
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

No branches or pull requests

2 participants