Skip to content

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

Merged
merged 10 commits into from
May 19, 2020
Merged

Add initial ESP32S2 support #2910

merged 10 commits into from
May 19, 2020

Conversation

tannewt
Copy link
Member

@tannewt tannewt commented May 15, 2020

No description provided.

@tannewt tannewt added the espressif applies to multiple Espressif chips label May 15, 2020
@tannewt tannewt added this to the 5.x.0 - Features milestone May 15, 2020
@tannewt tannewt requested review from dhalbert and jepler May 19, 2020 18:08
jepler
jepler previously approved these changes May 19, 2020
Copy link

@jepler jepler left a 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.

dhalbert
dhalbert previously approved these changes May 19, 2020
Copy link
Collaborator

@dhalbert dhalbert left a 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.

@tannewt tannewt dismissed stale reviews from dhalbert and jepler via acf4b1b May 19, 2020 18:44
Copy link
Collaborator

@dhalbert dhalbert left a comment

Choose a reason for hiding this comment

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

👍

@tannewt tannewt merged commit 916ca9f into adafruit:master May 19, 2020
tyomitch pushed a commit to tyomitch/circuitpython that referenced this pull request May 6, 2021
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]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
espressif applies to multiple Espressif chips
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants