Skip to content

Added board - microS2 #3322

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 17 commits into from
Aug 26, 2020
Merged

Added board - microS2 #3322

merged 17 commits into from
Aug 26, 2020

Conversation

microdev1
Copy link
Collaborator

Added board specific config files for esp32s2 based microS2.

@microdev1 microdev1 marked this pull request as ready for review August 25, 2020 16:23
Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

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

One request about the board id. Otherwise it looks good. Thanks!

@microdev1 microdev1 requested review from tannewt and dhalbert August 26, 2020 03:56
@dhalbert
Copy link
Collaborator

The build was failing due to some trailing whitespace in a few files. I edited most of them in github, but making the sdkconfig file empty in github failed (some kind of github bug). So I cloned and pushed the changes.

@microdev1 If you can set your editor to automatically delete trailing whitespace on save, that will prevent these kinds of minor issues. Some editors have a setting, and some have a plugin to do this.

@microdev1
Copy link
Collaborator Author

@dhalbert Just added PSRAM support to sdkconfig

@dhalbert
Copy link
Collaborator

Hi, you need to pull my changes from your repo, since your push put back the trailing whitespace.

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.

All done. Thanks for your quick responses!

@dhalbert dhalbert removed the request for review from tannewt August 26, 2020 15:02
@dhalbert dhalbert dismissed tannewt’s stale review August 26, 2020 15:03

changes made

@dhalbert dhalbert merged commit 5755160 into adafruit:main Aug 26, 2020
@microdev1 microdev1 deleted the microS2 branch August 26, 2020 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants