-
Notifications
You must be signed in to change notification settings - Fork 0
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
Wifi + Bluetooth Coexistance #15
Comments
commit 9b38f31 enabled bluetooth, but it still doesn't work. if bluetooth is enabled when wifi is enabled, it doesn't crash anymore, but it still breaks wifi. |
the return code in hex is 0x102, which is ESP_ERR_INVALID_ARG.
|
(note: i'm now using espidf 4.0.1)
i would be ok with ble and wifi being mutually exclusive (they can't be on at the same time), but enabling wifi successfully will mean that you can't enable ble without the esp crashing, and enabling ble successfully means that wifi won't work. |
i'm working on getting ble working after ble has been disabled. it seems like there's an issue setting up advertising the second time (after it's been disabled). BLEAdvertising.cpp, line 212, error 0x103 (invalid state), the function that returns this error is |
Commit 9612d23 added Bluetooth support, but weird issues arise when you try to enable Wifi and Bluetooth at the same time. I used a separate test sketch and verified that you can run Wifi and BT at the same time on the ESP32, so i think that if you are running the watch 2 system and Wifi and Bluetooth, then the internal RAM is filled up.
I can think of a few workarounds for this, but I don't really know what the best option is. For now, I'm going to disable bluetooth until I think of a good idea. Here are the ideas I've had:
This issue might take a while to solve. I'm going to do any Bluetooth stuff in the wifi_bt_coex branch, so new states and stuff can be worked on in the master branch (or another specialised branch).
aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa
Also, i'm an idiot, and the commit that adds bluetooth support removes wifi profile support so when merging that branch remember this you idiot
The text was updated successfully, but these errors were encountered: