Skip to content

Conversation

@cjpeterson
Copy link

This is an update to an old PR #3937 . Since the old PR has not been touched in quite a while, I figured it was worth another shot.

This adds support for the ESP32-S3 chip. Currently, serial output is working through UART. I count 3 people that have gotten text to output successfully with this code. I have also finished setting up GPIO support, so blinky is working.

Some programs compile, but freeze during execution. This seems to happen when the compiler optimizes out dead code (e.g. code after infinite loops). Adding -opt=0 during compilation has fixed all the issues I've seen.

Most of the comments from the previous pull request have been addressed:

  • Built-in svd files have been removed
  • The startup assembly code has been fully merged with esp32.s. The extra bit in PS_OWB_MASK does not appear in the esp-idf source file, so I removed it.
  • The cpu frequency configuration stuff has been implemented
  • "esp32s3" has been added to builder_test

I did not move the .S files out of /src/device/esp. It may be a good idea, but it seems beyond the scope of this PR.

@cjpeterson
Copy link
Author

Updated to pass the checks and to correct a critical bug in the linker script. This eliminated the need for -opt=0 in the two test cases I had.

@deadprogram
Copy link
Member

I have a Xiao ESP32S3 board I can test with. Will take a look soon.

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.

4 participants