Skip to content

bleio_user_reset(): check if BLE running before doing any BLE ops#9453

Merged
dhalbert merged 1 commit into
adafruit:9.1.xfrom
dhalbert:9.1.x-fix-nordic-restart-crash
Jul 24, 2024
Merged

bleio_user_reset(): check if BLE running before doing any BLE ops#9453
dhalbert merged 1 commit into
adafruit:9.1.xfrom
dhalbert:9.1.x-fix-nordic-restart-crash

Conversation

@dhalbert

Copy link
Copy Markdown
Collaborator

bleio_user_reset() was trying to do SoftDevice operations after the Adapter was disabled, causing NRF_ERROR_SOFTDEVICE_NOT_ENABLED, which tried to raise an exception after the VM was shut down. Check before trying to do these operations.

@dhalbert
dhalbert requested review from jepler and tannewt July 23, 2024 22:57

@jepler jepler left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks fine, didn't test.

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.

nordic: restarting BLE advertising causes USB disconnect/reconnect or hard reset

2 participants