-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
soc: xtensa: esp32s3: add support for SPIRAM #59770
soc: xtensa: esp32s3: add support for SPIRAM #59770
Conversation
The following west manifest projects have been modified in this Pull Request:
Note: This message is automatically posted and updated by the Manifest GitHub Action. |
@@ -18,3 +18,6 @@ config HEAP_MEM_POOL_SIZE | |||
choice BT_HCI_BUS_TYPE | |||
default BT_ESP32 if BT | |||
endchoice | |||
|
|||
config ESP_SPIRAM | |||
default y |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think SPIRAM should be enabled in the application code rather on board level.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would be great if if would be selected automatically based on devicetree node corresponding to SPIRAM.
Could we please wait until #58454 gets merged? |
d007997
to
d54676b
Compare
|
||
.. code-block:: console | ||
|
||
west build -b esp32_devkitc_wrover tests/boards/espressif_esp32/cache_coex | ||
west flash --esp-device /dev/ttyUSB0 | ||
west build -b <board> tests/boards/espressif_esp32/cache_coex |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good idea :)
d54676b
to
c7ca100
Compare
. = ALIGN(0x10000); | ||
. = ALIGN(4); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This alignment was previously wrong / just arbitrary?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It was arbitrary, for testing purposes. Ended up forgetting to put it on the first commit. Thanks for pointing it out
Add support for external PSRAM for esp32s3. Signed-off-by: Lucas Tamborrino <[email protected]>
Test esp32s3 for cache coexistence. Update test documentation. Signed-off-by: Lucas Tamborrino <[email protected]>
Add section to allocate memory of WiFi and NET stack in SPIRAM Signed-off-by: Lucas Tamborrino <[email protected]>
c7ca100
to
2cd6bd6
Compare
Add support for external PSRAM for esp32s3.
Test esp32s3 for cache coexistence. Update test documentation.
Add section to allocate memory of WiFi and NET stack in SPIRAM