Skip to content

soc: esp32s3: added initial support #53699

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 7 commits into from
Feb 27, 2023
Merged

soc: esp32s3: added initial support #53699

merged 7 commits into from
Feb 27, 2023

Conversation

@sylvioalves
Copy link
Contributor

@sylvioalves sylvioalves commented Jan 11, 2023

This brings initial support to ESP32-S3 SoC and ESP32-S3 DevKitM board.

@zephyrbot zephyrbot added area: Clock Control area: Interrupt Controller platform: ESP32 Espressif ESP32 area: Xtensa Xtensa Architecture area: UART Universal Asynchronous Receiver-Transmitter labels Jan 11, 2023
@zephyrbot
Copy link

zephyrbot commented Jan 11, 2023 •
edited
Loading

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
hal_espressif zephyrproject-rtos/hal_espressif@73e7af1 zephyrproject-rtos/hal_espressif@a06af24 zephyrproject-rtos/[email protected]

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@sylvioalves sylvioalves mentioned this pull request Jan 11, 2023
Merged
@sylvioalves sylvioalves force-pushed the feature/esp32s3_support branch from 16db706 to e05a6a5 Compare January 11, 2023 13:18
@sylvioalves sylvioalves marked this pull request as draft January 11, 2023 13:31
@sylvioalves sylvioalves force-pushed the feature/esp32s3_support branch 6 times, most recently from 422609f to 9008522 Compare January 11, 2023 16:44
marekmatej
marekmatej previously approved these changes Jan 13, 2023
View reviewed changes
@sylvioalves sylvioalves mentioned this pull request Jan 25, 2023
Merged
@sylvioalves
Copy link
Contributor Author

sylvioalves commented Jan 25, 2023

@stephanosio, would you check this and let me know how to proceed?

@sylvioalves
Copy link
Contributor Author

sylvioalves commented Feb 1, 2023 •
edited
Loading

@carlescufi, anyway you can check this PR and evaluate next action? ESP32-S3 toolchain is still pending as well: zephyrproject-rtos/sdk-ng#612

@stephanosio
Copy link
Member

stephanosio commented Feb 1, 2023

@stephanosio, would you check this and let me know how to proceed?

zephyrproject-rtos/sdk-ng#612 (comment)

We are currently in feature merge freeze for the upcoming v3.3 release. This PR can be merged after v3.3 is released and merge window is open again.

@stephanosio stephanosio added this to the v3.4.0 milestone Feb 1, 2023
@sylvioalves sylvioalves dismissed stale reviews from marekmatej, LucasTambor, and uLipe via 091db8b February 10, 2023 12:53
@sylvioalves sylvioalves force-pushed the feature/esp32s3_support branch from 9008522 to 091db8b Compare February 10, 2023 12:53
marekmatej
marekmatej previously approved these changes Feb 13, 2023
View reviewed changes
mniestroj
mniestroj previously approved these changes Feb 22, 2023
View reviewed changes
@carlescufi
Copy link
Member

carlescufi commented Feb 23, 2023

@sylvioalves can you please check why CI is failing?

@sylvioalves
Copy link
Contributor Author

sylvioalves commented Feb 23, 2023

@sylvioalves can you please check why CI is failing?

@carlescufi CI fails because ESP32-S3 toolchain isn't available yet. @stephanosio, any updates on this?

@sylvioalves
dt-bindings: add esp32s3 signals
77cacd7 
Adds esp32s3 related gpio sigmap, intmux and clock related.

Signed-off-by: Sylvio Alves <[email protected]>
@sylvioalves
soc: esp32s3: add base source content
10a3177 
This brings esp32s3 linker, DTS and all
necessary files to allow the soc support.

Signed-off-by: Sylvio Alves <[email protected]>
@sylvioalves
kconfig: add esp32s3 into configuration
c8344d4 
Add ESP32-S3 information into Espressif's Kconfig
definitions.

Signed-off-by: Sylvio Alves <[email protected]>
@sylvioalves
clock: esp32s3: add peripheral initialization
9638360 
Update clock control source to enable proper
ESP32S3 clock init.

Signed-off-by: Sylvio Alves <[email protected]>
@sylvioalves
driver: uart: esp32s3: enable ESP32S3 uart interface
9482748 
Includes additional SoC specific headers.

Signed-off-by: Sylvio Alves <[email protected]>
@sylvioalves
drivers: interrupt: add esp32s3 interrupt controller
066eab1 
Enables SoC specific interrupt controller.

Signed-off-by: Sylvio Alves <[email protected]>
@sylvioalves
boards: esp32s3_devkitm: initial soc board support
7d54852 
Adds support to ESP32-S3 devkitm as initial SoC board.

Signed-off-by: Sylvio Alves <[email protected]>
@sylvioalves sylvioalves dismissed stale reviews from mniestroj and marekmatej via 7d54852 February 27, 2023 13:27
@sylvioalves sylvioalves force-pushed the feature/esp32s3_support branch from 091db8b to 7d54852 Compare February 27, 2023 13:27
@zephyrbot zephyrbot requested a review from gmarull February 27, 2023 13:27
@sylvioalves
Copy link
Contributor Author

sylvioalves commented Feb 27, 2023

Last push was to change toolchain: espressif in order to make CI to pass and let this be merged. Once SDK 0.16.0 gets released, I submit another PR enabling S3 to work with Zephyr's troolchain.

@carlescufi carlescufi merged commit bc9b59a into zephyrproject-rtos:main Feb 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@uLipe uLipe uLipe approved these changes

@mniestroj mniestroj mniestroj approved these changes

@marekmatej marekmatej marekmatej left review comments

@LucasTambor LucasTambor LucasTambor left review comments

@nashif nashif Awaiting requested review from nashif

@glaubermaroto glaubermaroto Awaiting requested review from glaubermaroto

@carlescufi carlescufi Awaiting requested review from carlescufi

@galak galak Awaiting requested review from galak

@MaureenHelm MaureenHelm Awaiting requested review from MaureenHelm

@dcpleung dcpleung Awaiting requested review from dcpleung

@andyross andyross Awaiting requested review from andyross

@stephanosio stephanosio Awaiting requested review from stephanosio

@nordic-krch nordic-krch Awaiting requested review from nordic-krch

@gmarull gmarull Awaiting requested review from gmarull

Assignees

@sylvioalves sylvioalves

Labels

area: Clock Control area: Interrupt Controller area: Pinctrl area: UART Universal Asynchronous Receiver-Transmitter area: Xtensa Xtensa Architecture manifest manifest-hal_espressif platform: ESP32 Espressif ESP32

Projects

None yet

Milestone

v3.4.0

Development

Successfully merging this pull request may close these issues.

8 participants

@sylvioalves @zephyrbot @stephanosio @carlescufi @uLipe @mniestroj @marekmatej @LucasTambor