Skip to content

Fix PSRAM code and rodata on ESP32-S3 #88073

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

marekmatej
Copy link
Collaborator

@marekmatej marekmatej commented Apr 2, 2025

This PR.

  • fixes startup issues with CONFIG_SPIRAM_FETCH_INSTRUCTIONS=y and CONFIG_SPIRAM_RODATA=y

Copy link

github-actions bot commented Apr 2, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff

All manifest checks OK

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

@github-actions github-actions bot added manifest manifest-hal_espressif DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Apr 2, 2025
@marekmatej marekmatej force-pushed the feature/psram_code_run branch from 778e20c to 0ede76e Compare April 2, 2025 23:19
FacepalmMute added a commit to epneo/zephyr that referenced this pull request Apr 3, 2025
@marekmatej marekmatej force-pushed the feature/psram_code_run branch from 0ede76e to 894fef0 Compare April 4, 2025 13:28
@marekmatej marekmatej added the platform: ESP32 Espressif ESP32 label Apr 4, 2025
@marekmatej marekmatej marked this pull request as ready for review April 4, 2025 14:02
@Thalley Thalley removed their request for review April 4, 2025 14:46
@github-actions github-actions bot removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Apr 7, 2025
wmrsouza
wmrsouza previously approved these changes Apr 8, 2025
sylvioalves
sylvioalves previously approved these changes Apr 8, 2025
raffarost
raffarost previously approved these changes Apr 8, 2025
@marekmatej marekmatej added the DNM This PR should not be merged (Do Not Merge) label Apr 8, 2025
jukkar
jukkar previously approved these changes Apr 8, 2025
FacepalmMute added a commit to epneo/zephyr that referenced this pull request Apr 16, 2025
@marekmatej marekmatej dismissed stale reviews from jukkar, raffarost, sylvioalves, and wmrsouza via db4a464 April 24, 2025 16:48
@marekmatej marekmatej force-pushed the feature/psram_code_run branch from 1434ad9 to db4a464 Compare April 24, 2025 16:48
@marekmatej marekmatej removed the DNM This PR should not be merged (Do Not Merge) label Apr 28, 2025
Move SPIRAM heap under the noinit, where it get mapped.
Fixes SPIRAM startup by moving necessary object to SRAM.

Signed-off-by: Marek Matej <marek.matej@espressif.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Wi-Fi Wi-Fi platform: ESP32 Espressif ESP32
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants