Skip to content

Deinit everything when restarting bt standalone temp sensor#745

Merged
will-v-pi merged 1 commit intodevelopfrom
bt-standalone-restart-fix
Mar 18, 2026
Merged

Deinit everything when restarting bt standalone temp sensor#745
will-v-pi merged 1 commit intodevelopfrom
bt-standalone-restart-fix

Conversation

@will-v-pi
Copy link
Copy Markdown
Contributor

Without this, I get an error when attempting to reconnect after a restart by pressing the S key:

Connecting to device with addr DE:AB:01:06:C1:6F.
SERVICE_QUERY_RESULT, ATT Error 0x7f.
Disconnected DE:AB:01:06:C1:6F
Connecting to device with addr DE:AB:01:06:C1:6F.
SERVICE_QUERY_RESULT, ATT Error 0x7f.
Disconnected DE:AB:01:06:C1:6F
...

With this fix everything works correctly when starting & stopping using the S key

I'm not sure if everything needs to be deinit'ed, but it is probably safest to deinit all three of these as they are init'ed above

Without this, you get an error when attempting to reconnect after a restart
@will-v-pi will-v-pi requested a review from peterharperuk March 18, 2026 16:56
@will-v-pi
Copy link
Copy Markdown
Contributor Author

@peterharperuk Do you want to absorb this change into your larger PR #743 - or should I merge this separately?

@peterharperuk
Copy link
Copy Markdown
Contributor

It might be quicker if you merge this first. I can make sure this change is picked up in there

@will-v-pi
Copy link
Copy Markdown
Contributor Author

Sounds good - I'll merge this now then (once the CI is done)

@will-v-pi will-v-pi merged commit 49c413e into develop Mar 18, 2026
4 checks passed
@will-v-pi will-v-pi deleted the bt-standalone-restart-fix branch March 18, 2026 17:09
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