dts: arm: st: h7: add ITCM and DTCM memory regions for STM32H7 #100255
Conversation
zephyrbot
requested review from
FRASTM,
GeorgeCGV,
djiatsaf-st,
erwango,
etienne-lms,
gautierg-st,
juickar and
mathieuchopstm
erwango
left a comment
erwango
left a comment
There was a problem hiding this comment.
From the search I've just made, all STM32H7 based soc use 64/128K ITCM/DTCM.
Unless I'm wrong, it would be better to put it directly in h7.dtsi (and delete node from m4 cores).
We'll also need to delete any itcm/dtcm defined by _m7 files. This will be a much bigger change, 10+ files, is that okay? |
Sure indeed, a complete clean up is required. Thanks !
To be clear, I meant explicitly use /delete-node/ directive in _m4.dtsi files to preserve existing configuration avoid resource sharing conflicts. |
iabdalkader
force-pushed
the
add_h747_dtcm_itcm
branch
from
a4f83e0 to
e05d274
Compare
iabdalkader
changed the title
Add ITCM and DTCM memory region definitions to the common STM32H7 devicetree. ITCM: 64 KB @ 0x00000000 DTCM: 128 KB @ 0x20000000 Signed-off-by: Ibrahim Abdalkader <i.abdalkader@gmail.com>
Delete the ITCM and DTCM nodes from all M4-specific devicetree. Signed-off-by: Ibrahim Abdalkader <i.abdalkader@gmail.com>
iabdalkader
force-pushed
the
add_h747_dtcm_itcm
branch
from
e05d274 to
7d6fe43
Compare
|
erwango
left a comment
erwango
left a comment
There was a problem hiding this comment.
I could not find a H7 that is not compatible with this setting from datasheet.
mathieuchopstm
left a comment
mathieuchopstm
left a comment
There was a problem hiding this comment.
Seems correct per ST Product Selector (which seems to have a bug making no TCM appear despite some products having it... glanced at a few datasheets too)
6 participants
Add ITCM and DTCM memory region definitions for the STM32H7.
ITCM: 64 KB @ 0x00000000
DTCM: 128 KB @ 0x20000000