-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Add initial ESP32S2 support #2910
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
Conversation
Basic blinky works but doesn't check pins.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because I don't have the hardware or toolchain to test on, I mostly looked at the changes introduced outside of ports/esp32s2. They seem sensible. I asked Scott about a few on Discord for more information.
There's sure to be more work to do in the port, but this is a great step and looks like a low chance of disrupting other ports.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! ... Though of course I skipped over all the config stuff.
ports/atmel-samd/boards/circuitplayground_express_crickit/mpconfigboard.h
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
This adds to the ESP8266 tutorial instructions explaining which pins to pull low to enter programming mode. Commit made originally by @ARF1 in adafruit#2910. Signed-off-by: Damien George <[email protected]>
No description provided.