-
Notifications
You must be signed in to change notification settings - Fork 7.6k
Added new boards pinout images for ESP32, ESP32-S2 and ESP32-C3 #5133
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
Added new boards pinout images for ESP32, ESP32-S2 and ESP32-C3 #5133
Conversation
I can also add the SVG files if needed :) |
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 like docs/_old_esp32_pinmap.png and docs/_old_esp32s2_pinmap.png aren't used anywhere, can they be removed?
@pedrominatel is it possible to add the default pull-up/down and other indicators to the new images that are present on the current ones? |
Yes, it is @atanisoft. I'll update the images to include this information. |
pin46 on esp32-s2 is GPI, and really needs to be indicated somehow that it is input only. |
Can you review @atanisoft? |
|
Hi @atanisoft thanks for your feedback!
|
Fine with me, just needs to be easy to read and understand in the image. |
I'll change the color and add it to the legend :) |
@pedrominatel the latest images look really nice! One more rather minor change to the esp32 picture, if possible add a note about IO 6-11 being not usable by the application code. |
Board images updated and added warning on Flash/PSRAM pins
I've added the warning in the pins used by Flash/PSRAM on both boards (ESP32/S2). I also did some minor changes in some labels, according to the datasheet naming. |
…ssif#5133) Board images updated and added warning on Flash/PSRAM pins
Updated images (suggestion) for the ESP32 DevKitC, ESP32-S2 Saola and ESP32-C3-DevKitM.