Skip to content

Conversation

@leonrinkel
Copy link
Contributor

@leonrinkel leonrinkel commented Jul 13, 2024

This PR adds board support for the Adafruit Feather ESP32S3 development board. This is basically a copy of the Espressif ESP32-S3-DevKitC board, adapted to accommodate the board-specific features of the Adafruit board, i.e. pinout, feather connector, fuel gauge and on-board NeoPixel. There is an issue #68512 requesting support for this board.

@github-actions
Copy link

Hello @leonrinkel, and thank you very much for your first pull request to the Zephyr project!
Our Continuous Integration pipeline will execute a series of checks on your Pull Request commit messages and code, and you are expected to address any failures by updating the PR. Please take a look at our commit message guidelines to find out how to format your commit messages, and at our contribution workflow to understand how to update your Pull Request. If you haven't already, please make sure to review the project's Contributor Expectations and update (by amending and force-pushing the commits) your pull request if necessary.
If you are stuck or need help please join us on Discord and ask your question there. Additionally, you can escalate the review when applicable. 😊

@leonrinkel leonrinkel force-pushed the board/adafruit-feather-esp32s3 branch from 7169507 to a4de851 Compare July 15, 2024 14:12
@leonrinkel leonrinkel marked this pull request as ready for review July 15, 2024 14:14
@zephyrbot zephyrbot added area: LED Label to identify LED subsystem area: Samples Samples labels Jul 15, 2024
@leonrinkel leonrinkel force-pushed the board/adafruit-feather-esp32s3 branch from a4de851 to 2c7644b Compare July 30, 2024 19:33
@leonrinkel
Copy link
Contributor Author

Rebased on latest main and resized board image to please CI compliance checks.

@kartben
Copy link
Contributor

kartben commented Aug 1, 2024

@leonrinkel your name sounds familiar 😄

@kartben kartben added platform: ESP32 Espressif ESP32 and removed area: LED Label to identify LED subsystem area: Samples Samples labels Aug 1, 2024
@kartben kartben assigned sylvioalves and unassigned kartben Aug 1, 2024
uLipe
uLipe previously approved these changes Aug 1, 2024
marekmatej
marekmatej previously approved these changes Aug 6, 2024
@zpm1066
Copy link

zpm1066 commented Aug 12, 2024

@leonrinkel Hi,
Is there any chance of also adding support for the Adafruit ESP32S3 TFT Feather (https://www.adafruit.com/product/5483)?

This board is identical to the Feather ESP32S3 but has a built-in ST7789 135x240 display that has TFT/I2C power controlled by a GPIO.

I’ve been using both boards with esp32s3_devkitc (and esp32s3_devkitm) and external ST7789 but haven’t been able to get the built-in display to work by turning on the TFT power using GPIO hog.

@leonrinkel
Copy link
Contributor Author

@zpm1066 I ordered one and will have a look once it arrives 👍

@leonrinkel
Copy link
Contributor Author

Ok I managed to get the TFT working and will extend this PR in the coming days. Now that I'm at it, I also got the PSRAM and reverse TFT variants to make sure they'll work too.

@zpm1066
Copy link

zpm1066 commented Aug 30, 2024

@leonrinkel Were you able to test out the Adafruit Feather PSRAM and reverse TFT board variants?
Any chance of extending the PR in the next week or so? Thanks.

Copy link
Contributor

@sylvioalves sylvioalves Dec 17, 2024

Choose a reason for hiding this comment

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

ESP32 -> ESP32-S3
Please apply this change elsewhere accordingly.

Copy link
Contributor

Choose a reason for hiding this comment

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

#include <espressif/esp32s3/esp32s3_wroom_n8.dtsi>
#include <adafruit_qt_py_esp32s3-pinctrl.dtsi>

Comment on lines 23 to 26
Copy link
Contributor

Choose a reason for hiding this comment

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

As before, let this go.

@leonrinkel leonrinkel force-pushed the board/adafruit-feather-esp32s3 branch 4 times, most recently from be7c1ce to 6634bc7 Compare December 27, 2024 14:37
Copy link
Contributor

Choose a reason for hiding this comment

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

Make it sram1 instead: zephyr,sram = &sram1;

Copy link
Contributor

@sylvioalves sylvioalves Dec 27, 2024

Choose a reason for hiding this comment

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

Make it sram1 instead: zephyr,sram = &sram1;

Copy link
Contributor

Choose a reason for hiding this comment

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

Please move all the below to Kconfig.defconfig file instead.

Copy link
Contributor

Choose a reason for hiding this comment

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

if BOARD_ADAFRUIT_FEATHER_ESP32S3_TFT_ESP32S3_PROCPU

Copy link
Contributor

Choose a reason for hiding this comment

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

Same as before: sram1

Copy link
Contributor

Choose a reason for hiding this comment

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

Same as before: sram1

The Adafruit Feather ESP32S3 is a development board in the Feather
standard layout, sharing peripheral placement with other devices labeled
as Feathers or FeatherWings. The board is equipped with an ESP32-S3 mini
module, a lithium ion battery charger and a USB-C connector.

Signed-off-by: Leon Rinkel <[email protected]>
The Adafruit Feather ESP32S3 is a development board in the Feather
standard layout, sharing peripheral placement with other devices labeled
as Feathers or FeatherWings. The board is equipped with an ESP32-S3 mini
module, a lithium ion battery charger and a USB-C connector. Compared to
the base model, this TFT variant additionally comes with a 240x135 pixel
IPS TFT color display.

Signed-off-by: Leon Rinkel <[email protected]>
@leonrinkel leonrinkel force-pushed the board/adafruit-feather-esp32s3 branch from 6634bc7 to cbd9d61 Compare December 28, 2024 22:03
CONFIG_POWER_DOMAIN=y
CONFIG_GPIO=y
CONFIG_PM_DEVICE=y
CONFIG_PM_DEVICE_POWER_DOMAIN=y
Copy link
Contributor

Choose a reason for hiding this comment

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

Is all this necessary to demonstrate the led-strip driver on this board ? For example, I can see that CONFIG_GPIO is already enabled in the board defconfig and as far I can tell it is not a dependency of the WS2812 LED strip driver...

@github-actions
Copy link

github-actions bot commented Mar 8, 2025

This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time.

@github-actions github-actions bot added the Stale label Mar 8, 2025
@nashif nashif removed the Stale label Mar 13, 2025
@nashif nashif dismissed nordicjm’s stale review March 13, 2025 10:54

please revisit

@kartben kartben merged commit b5cd22e into zephyrproject-rtos:main Mar 14, 2025
21 checks passed
@github-actions
Copy link

Hi @leonrinkel!
Congratulations on getting your very first Zephyr pull request merged 🎉🥳. This is a fantastic achievement, and we're thrilled to have you as part of our community!

To celebrate this milestone and showcase your contribution, we'd love to award you the Zephyr Technical Contributor badge. If you're interested, please claim your badge by filling out this form: Claim Your Zephyr Badge.

Thank you for your valuable input, and we look forward to seeing more of your contributions in the future! 🪁

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: Display area: LED Label to identify LED subsystem area: Samples Samples platform: ESP32 Espressif ESP32

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Support for Adafruit ESP32-S3 and ESP32-S2 Feather Boards