Skip to content

Conversation

@djiatsaf-st
Copy link
Contributor

This commit 23ebd62 adds changes to update the buffer size to the full PSRAM size when the PSRAM node is available.
Now, on a board like stm32h7s78_dk, where both internal and external flash nodes are enabled, the test runs into SRAM overflow with the following errors: ...

...zephyr/zephyr_pre0.elf section `SRAM1' will not fit in region `SRAM1' 
...zephyr/zephyr_pre0.elf section `SRAM2' will not fit in region `SRAM2' 
...
...region `SRAM1' overflowed by 2080768 bytes 
...region `SRAM2' overflowed by 2080768 bytes 

To run tests successfully, mutual exclusion between SRAM and PSRAM needs to be implemented, so the correct buffer size is chosen for the correct peripheral.

To reproduce :

west build -p -b stm32h7s78_dk/stm32h7s7xx tests/drivers/memc/ram -T drivers.memc

disable psram and memc nodes.

Signed-off-by: Fabrice DJIATSA <fabrice.djiatsa-ext@st.com>
disable sram1 and sram2 nodes.

Signed-off-by: Fabrice DJIATSA <fabrice.djiatsa-ext@st.com>
add a scenario to run test in CI for external flash.

Signed-off-by: Fabrice DJIATSA <fabrice.djiatsa-ext@st.com>
@zephyrbot zephyrbot added area: Tests Issues related to a particular existing or missing test area: MEMC labels Sep 29, 2025
@zephyrbot zephyrbot requested a review from nashif September 29, 2025 15:58
@sonarqubecloud
Copy link

Copy link
Contributor

@etienne-lms etienne-lms left a comment

Choose a reason for hiding this comment

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

I think P-R #96635 has fixed the issue you are addressing here. It's been merged very recently.

@djiatsaf-st
Copy link
Contributor Author

I think P-R #96635 has fixed the issue you are addressing here. It's been merged very recently.

Indeed, Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: MEMC area: Tests Issues related to a particular existing or missing test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants