RTEMS | bsp/stm32h7: fix SDRAM init loops and clean up constants and style (!1152)

Mohamed Ayman (@mohamedayman23) gitlab at rtems.org
Sat Mar 21 19:42:00 UTC 2026



Mohamed Ayman created a merge request: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1152

Project:Branches: mohamedayman23/rtems:fix-potential-bugs-stm32h7-ext-mem-ctl to rtems/rtos/rtems:main
Author:   Mohamed Ayman



bsp/stm32h7: fix SDRAM init loops and clean up constants and style

Fix potential issues in SDRAM initialization loops by ensuring the status register value is refreshed before each polling loop.

Replace busy-wait delay loop with \__NOP() to prevent compiler optimization and improve reliability.

Remove use of deprecated 'register' keyword and use standard uint32_t types.

Add 'U' suffix to constants to ensure correct unsigned behavior and avoid compiler warnings.

Add clarifying comments for FMC commands and improve overall code readability and consistency.

AI usage: yes

Prompt used: explain this file for me in details, which part in the data sheet for this, why the developer neglect the unsigned symbol what is can led to and tell me the flow it written by

AI model: GPT-5 mini

How AI was used: for explaining the code to understand its tricks and flow and resolve my questions

-- 
View it on GitLab: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1152
You're receiving this email because of your account on gitlab.rtems.org.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/bugs/attachments/20260321/e54d89c7/attachment.htm>


More information about the bugs mailing list