Skip to content

Conversation

@avolmat-st
Copy link

This serie enable the display / ltdc device on the STM32H7S78 discovery board.

@avolmat-st
Copy link
Author

Tested with display sample. With PR #93805 applied, the colors are correct.

JarmouniA
JarmouniA previously approved these changes Aug 2, 2025
Copy link
Contributor

@JarmouniA JarmouniA left a comment

Choose a reason for hiding this comment

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

Some nits

kartben
kartben previously approved these changes Aug 2, 2025
@kartben kartben assigned erwango and unassigned kartben Aug 2, 2025
@avolmat-st avolmat-st marked this pull request as draft August 3, 2025 19:38
@avolmat-st
Copy link
Author

Converted to draft in order to fix @JarmouniA comments and as well check a potential stability issue.

@avolmat-st avolmat-st dismissed stale reviews from kartben and JarmouniA via 0aa487d August 4, 2025 14:18
@sonarqubecloud
Copy link

sonarqubecloud bot commented Aug 4, 2025

@github-actions
Copy link

github-actions bot commented Oct 4, 2025

This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time.

@avolmat-st
Copy link
Author

Simply rebased on top of HEAD

Alain Volmat added 5 commits October 4, 2025 20:29
Add a first region in the MPU to disable all access to the whole
memory range.  With that ensure that the MPU will block all
access to regions that aren't defined in further regions.
Ensure as well that the peripheral area is accessible.

This handles the errata 2.1.1 PLD might perform linefill to address
that would generate a MemManage Fault of the STM32H7Rxx / STM32H7Sxx
device errate ES0596 - Rev 6.

Signed-off-by: Alain Volmat <alain.volmat@foss.st.com>
The stm32h7rs embeds a LTDC display controller. Add the node
describing it in stm32h7rs.dtsi.

Signed-off-by: Alain Volmat <alain.volmat@foss.st.com>
In case of DISPLAY is enabled, ensure that MEMC as well as
Shared-Multi-Heap is enabled in order to use for framebuffer.
Moreover, since PSRAM is 32MB large, HEAP size should be set
AUTO instead of SMALL which is the default since SRAM is rather
small.

Signed-off-by: Alain Volmat <alain.volmat@foss.st.com>
Enable the display panel of the stm32h7s78_dk with framebuffer
located within the PSRAM.

Signed-off-by: Alain Volmat <alain.volmat@foss.st.com>
Due to the large display of the STM32H7S78 DK, it is necessary
to have larger POOL_SIZE for display buffers.

Signed-off-by: Alain Volmat <alain.volmat@foss.st.com>
@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 4, 2025

@avolmat-st avolmat-st marked this pull request as ready for review October 4, 2025 19:23
@github-actions github-actions bot removed the Stale label Oct 5, 2025
@JarmouniA JarmouniA added this to the v4.3.0 milestone Oct 7, 2025
@cfriedt cfriedt merged commit 7c47371 into zephyrproject-rtos:main Oct 8, 2025
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants