Skip to content
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

ESP32-S3 in QEMU: PSRAM cannot work for esp32s3 device (QEMU-211) #103

Open
1 of 3 tasks
dongsheng28849455 opened this issue Sep 19, 2024 · 6 comments
Open
1 of 3 tasks
Assignees
Labels
Resolution: NA Status: Done Issue is done internally

Comments

@dongsheng28849455
Copy link

dongsheng28849455 commented Sep 19, 2024

Checklist

  • Checked the issue tracker for similar issues to ensure this is not a duplicate.
  • Described the feature in detail and justified the reason for the request.
  • Provided specific use cases and examples.

Feature description

Is it possible to support PSRAM for S3 also?

Use cases

In nuttx nsh, we expect the umem can be 32M size.
NuttShell (NSH) NuttX-12.6.0-RC1
nsh> free
total used free maxused maxfree nused nfree
Kmem: 359196 16612 342584 17000 342584 43 1
Umem: 33161212 10596 33150616 10984 33150616 9 1

Alternatives

No response

Additional context

When I try to enable the PSRAM, I met several issues:
1, Similar with #57, neither SPIRAM_SPEED_80M nor SPIRAM_SPEED_40M can work
2, The ESP32S3MMUEntry e value will be overwritten by e.reserved
4, dcache was created as rom device, so dbus address cannot be written
3, SSS_PSRAM does not attach to SPI1 for S3 machine
4, RAM size larger than 4 MB not supported, seems S3 can support 32M as maximum.

@github-actions github-actions bot changed the title ESP32-S3 in QEMU: PSRAM cannot work for esp32s3 device ESP32-S3 in QEMU: PSRAM cannot work for esp32s3 device (QEMU-211) Sep 19, 2024
@igrr
Copy link
Member

igrr commented Sep 19, 2024

Yes, as you found, PSRAM support is not implemented for S3 yet.

We'll update this issue when that gets done.

@dongsheng28849455
Copy link
Author

Yes, as you found, PSRAM support is not implemented for S3 yet.

We'll update this issue when that gets done.

Can I ask will you implement it recently?

@dongsheng28849455
Copy link
Author

I try to fix it by #104, please help review it, thanks.

@o-marshmallow
Copy link
Collaborator

Hello @dongsheng28849455 ,

Sorry for not giving more updates about this. A few days ago, we opened an internal MR to add support for the PSRAM on the S3. It is currently under review.

@igrr
Copy link
Member

igrr commented Nov 11, 2024

PSRAM support for ESP32-S3 has been merged in 39534c3 and 89fe574, will appear in the next release.

@dongsheng28849455
Copy link
Author

Thanks for your works, when i test it, seems the get_eid_by_size does not handle the case of size_mbytes equal to 16M or 32M?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: NA Status: Done Issue is done internally
Projects
None yet
Development

No branches or pull requests

4 participants