Skip to content

ESP32-S3 Octal SPIRAM Support - #610

Merged
jessebraham merged 2 commits into
esp-rs:mainfrom
bjoernQ:octal-psram-esp32s3
Jun 22, 2023
Merged

ESP32-S3 Octal SPIRAM Support#610
jessebraham merged 2 commits into
esp-rs:mainfrom
bjoernQ:octal-psram-esp32s3

Conversation

@bjoernQ

@bjoernQ bjoernQ commented Jun 21, 2023

Copy link
Copy Markdown
Contributor

This adds support for octal PSRAM on ESP32-S3. Tested with S3Box

Adds new features (opsram_2m, opsram_4m etc.) to activate support.

Fixes #547

@bjoernQ
bjoernQ force-pushed the octal-psram-esp32s3 branch from 6b653d2 to 36b84cb Compare June 22, 2023 06:40
@bjoernQ
bjoernQ marked this pull request as ready for review June 22, 2023 07:03
@bjoernQ
bjoernQ force-pushed the octal-psram-esp32s3 branch from 36b84cb to ceeab6e Compare June 22, 2023 07:47

@JurajSadel JurajSadel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, I tested the example and it works fine for me.
I left a few notes regarding commented C code (sorry for nitpicking)

Comment thread esp-hal-common/src/soc/esp32s3/psram.rs Outdated
Comment thread esp-hal-common/src/soc/esp32s3/psram.rs Outdated
Comment thread esp-hal-common/src/soc/esp32s3/psram.rs Outdated
Comment thread esp-hal-common/src/soc/esp32s3/psram.rs Outdated
Comment thread esp-hal-common/src/soc/esp32s3/psram.rs Outdated
Comment thread esp-hal-common/src/soc/esp32s3/psram.rs Outdated

@jessebraham jessebraham left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thanks for this!

@jessebraham
jessebraham merged commit 9052a39 into esp-rs:main Jun 22, 2023
MabezDev pushed a commit that referenced this pull request Jul 12, 2023
* ESP32-S3 Octal SPIRAM Support

* Adjust some code comments
@bjoernQ
bjoernQ deleted the octal-psram-esp32s3 branch November 26, 2024 08:44
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.

Add support for OCTAL PSRAM used in ESP32-S3-BOX

3 participants