-
Notifications
You must be signed in to change notification settings - Fork 986
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
Adding XIAO ESP32C6 with BLE support #5421
base: master
Are you sure you want to change the base?
Conversation
I am testing a few corner cases with this. Especially with the other C6 board (TLora C6) we have. Is it ok to push minor corrections to your branch? |
Sure, I am fine with that! |
@caveman99 just FYI, I'm fixing the failed builds for other targets... 🤷 |
…e into xiao_esp32c6_with_ble
Bluetooth functionality is working again 🎉
Hi @caveman99 , I think we're good to go now. Could you take a look please? |
…_esp32c6_with_ble
…/firmware into xiao_esp32c6_with_ble
Status: it compiles again but i don't see any BLE advertising emitted from my TLora C6. I have Seeed Xiao C6 incoming in about 10 days to counter-check. Another thing i noticed when working with VSCode on Windows, due to the different file name handling compared to Linux the wrong RTC.h and CTR.h headers are picked up, presumably in ESP-IDF directories. I did remedy this for the RTC.h by prefixing the dir, but the crypto engine header is not so easy. I hope i can come up with a fix for this. |
Hi @caveman99 , Thank you for your effort! Great catch regarding the Windows environment. I didn’t encounter that issue on my Ubuntu setup. If you could share more details, I might be able to replicate it on my end as well. I built and flashed your commit 934096e d5bc is another xiao esp32c6 node flashed with 934096e commit that I can communicate just fine: Two things come to mind regarding BLE:
Mine is: sdkconfig.seeed-xiao-esp32c6
|
Bluetooth is indeed enabled per info dump. I was scanning with the Android "nRF52 connect" app for any bluetooth signal from the device and wasn't lucky. i'll compare the sdkconfig later when i am back home.
|
Related to #4692
The main purpose of this PR is to enhance the ESP32-C6 target by integrating BLE functionality