-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
ESP32:fix compilation errors for other esp examples on esp32c3 #11716
ESP32:fix compilation errors for other esp examples on esp32c3 #11716
Conversation
d262902
to
602446a
Compare
PR #11716: Size comparison from 3db50b1 to 602446a Decreases (17 builds for efr32, k32w, nrfconnect, qpg, telink)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
@wqx6 Could you add some more instructions in each example README, to indicate the ESP32C3 support? Similar instructions like: https://github.com/project-chip/connectedhomeip/blob/master/examples/all-clusters-app/esp32/README.md?plain=1#L82 |
602446a
to
3b2fe6a
Compare
3b2fe6a
to
603a4a2
Compare
PR #11716: Size comparison from 5623978 to 603a4a2 Decreases (17 builds for efr32, k32w, nrfconnect, qpg, telink)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Comments resolved and replied to, re-requesting separate review
This PR generate error when i try to use target ESP32S3, AppConfig.h is not prepared to use this target CONFIG_IDF_TARGET_ESP32S3 , only can use CONFIG_IDF_TARGET_ESP32 or CONFIG_IDF_TARGET_ESP32C3. Is it possible to develop ESP32S3 option similar this?
|
Problem
There are some compilation errors for ESP32C3.
Change overview
Fix those errors for ESP32C3 device.
Testing
Tested compiling for the examples manually