Re: [PATCH v8 0/9] arm64: dts: ti: Split wkup R5F memory region for IO+DDR resume

[Bryan Brattlof]

On September  3, 2026 thus sayeth Markus Schneider-Pargmann (TI):
> Hi,
> 
> Split the firmware memory region in more specific parts so it is better
> described where which information is stored. Specifically the LPM metadata
> region is important as bootloader software like U-Boot has to know where
> that data is to be able to read that data and resume from RAM.
> 
> The bindings are already applied. The remaining patches use the new
> layout for the platforms that are capable to support IO+DDR. For IO+DDR
> the new layout is necessary as it defines the location of the LPM
> metadata.
> 
> Additionally the two important devicetree nodes for resuming from IO+DDR
> have the bootph-pre-ram flag added as this data needs to be read before
> the RAM is in use.
> 
> The changes in this series were suggested as part of the IO+DDR u-boot series:
>   https://lore.kernel.org/r/814c211f-a9eb-4311-bb84-165b1a69755f@xxxxxx
> 
> Note that concerns about uboot fixup of memory region fixups were
> discussed here:
>   https://lore.kernel.org/r/DJD1Y3G9S1SP.2GHOZ5X4RYJFA@xxxxxxxxxxxx
> 
> Best
> Markus
> 
> Signed-off-by: Markus Schneider-Pargmann (TI) <msp@xxxxxxxxxxxx>
> ---
> Changes in v8:
> - Rebase to v7.3-rc1
> - Increase the AM62A wkup R5F memory region to end at 0x9e700000 to
>   also fit the edgeai DM firmware as requested by Paresh
> - Move the memory region definitions back into the base board dts files
>   of all AM62A and AM62P boards
> - Added bootph-pre-ram to the memory regions are merged into the
>   bootph-pre-ram at the end of the series
> - Link to v7: https://lore.kernel.org/r/20260701-topic-am62a-ioddr-dt-v6-19-v7-0-e9db8b16821a@xxxxxxxxxxxx
> 
> Changes in v7:
> - Rebase to v7.1-rc1
> - Fix commit messages
> - Link to v6: https://lore.kernel.org/r/20260609-topic-am62a-ioddr-dt-v6-19-v6-0-16afba97fbe0@xxxxxxxxxxxx
> 
> Changes in v6:
> - Added fixes for the length of the wkup_r5fss0_core0_memory_region.
>   Thanks Francesco for pointing that out. am62a firmware is shorter than
>   am62p firmware. I calculated both and fixed all devicetrees using
>   them. All patches have different Fixes tags so I kept them separate.
> - Fixed the length of the split memory layout as well.
> - Removed the double definition of memory regions for the var-som board.
> - Link to v5: https://lore.kernel.org/r/20260601-topic-am62a-ioddr-dt-v6-19-v5-0-3856a023aff2@xxxxxxxxxxxx
> 
> Changes in v5:
> - Move all changes into k3-am62a/p-ti-ipc-firmware.dtsi
> - Dropped the patch that adds bootph-pre-ram to k3-am62a stuff as it is
>   already present in k3-am62d2-evm.dts and got moved into the
>   firmware.dtsi file which covers am62a as well then.
> - Link to v4: https://lore.kernel.org/r/20260429-topic-am62a-ioddr-dt-v6-19-v4-0-fc27d6ac753c@xxxxxxxxxxxx
> 
> Changes in v4:
> - Rebased to v7.1-rc1
> - Dropped all already applied patches that are the bindings and the
>   initial introduction of memory-region-names
> - Link to v3: https://lore.kernel.org/r/20260318-topic-am62a-ioddr-dt-v6-19-v3-0-c41473cb23c3@xxxxxxxxxxxx
> 
> Changes in v3:
> - Squash the enforcement of the memory-region-names requirement in the
>   patch adding the memory-region-names, as suggested.
> - Link to v2: https://lore.kernel.org/r/20260312-topic-am62a-ioddr-dt-v6-19-v2-0-37cb7ceec658@xxxxxxxxxxxx
> 
> Changes in v2:
> - Make memory-region-names required if memory-region is present
> - Fixup memory-region and memory-region-names conditions. Require either
>   2 or 6 regions for memory-region and memory-region-names
> - Reword and restructure the binding documentation for memory-region and
>   memory-region-names
> - Add memory-region-names to all uses of memory-region
> - Link to v1: https://lore.kernel.org/r/20260303-topic-am62a-ioddr-dt-v6-19-v1-0-12fe72bb40d2@xxxxxxxxxxxx
> 
> To: Nishanth Menon <nm@xxxxxx>
> To: Vignesh Raghavendra <vigneshr@xxxxxx>
> To: Tero Kristo <kristo@xxxxxxxxxx>
> To: Rob Herring <robh@xxxxxxxxxx>
> To: Krzysztof Kozlowski <krzk+dt@xxxxxxxxxx>
> To: Conor Dooley <conor+dt@xxxxxxxxxx>
> To: Nathan Chancellor <nathan@xxxxxxxxxx>
> To: Nick Desaulniers <ndesaulniers@xxxxxxxxxx>
> To: Bill Wendling <morbo@xxxxxxxxxx>
> To: Justin Stitt <justinstitt@xxxxxxxxxx>
> To: Andrew Davis <afd@xxxxxx>
> To: Judith Mendez <jm@xxxxxx>
> To: Daniel Schultz <d.schultz@xxxxxxxxx>
> To: Siddharth Vadapalli <s-vadapalli@xxxxxx>
> To: Bryan Brattlof <bb@xxxxxx>
> To: Paresh Bhagat <p-bhagat@xxxxxx>
> To: Devarsh Thakkar <devarsht@xxxxxx>
> To: Jai Luthra <jai.luthra@xxxxxxxxxxxxxxxx>
> To: Beleswar Padhi <b-padhi@xxxxxx>
> To: Francesco Dolcini <francesco.dolcini@xxxxxxxxxxx>
> To: Stefano Radaelli <stefano.radaelli21@xxxxxxxxx>
> Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> Cc: devicetree@xxxxxxxxxxxxxxx
> Cc: linux-kernel@xxxxxxxxxxxxxxx
> Cc: llvm@xxxxxxxxxxxxxxx
> Cc: Hari Nagalla <hnagalla@xxxxxx>
> 
> ---
> Markus Schneider-Pargmann (TI) (9):
>       arm64: dts: ti: k3-am62a-phycore-som: Fix wkup R5F memory region size
>       arm64: dts: ti: k3-am62d2-evm: Fix wkup R5F memory region size
>       arm64: dts: ti: k3-am62a7-sk: Fix wkup R5F memory region size
>       arm64: dts: ti: k3-am62p-verdin: Fix wkup R5F memory region size
>       arm64: dts: ti: k3-am62p5-sk: Fix wkup R5F memory region size
>       arm64: dts: ti: var-som-am62p: Fix wkup R5F memory region size
>       arm64: dts: ti: k3-am62a: Split r5f memory region
>       arm64: dts: ti: k3-am62p: Split r5f memory region
>       arm64: dts: ti: Add wkup R5F nodes to pre-ram bootphase
> 
>  arch/arm64/boot/dts/ti/k3-am62a-phycore-som.dtsi   | 29 ++++++++++++++++++--
>  .../boot/dts/ti/k3-am62a-ti-ipc-firmware.dtsi      | 11 ++++++--
>  arch/arm64/boot/dts/ti/k3-am62a7-sk.dts            | 29 ++++++++++++++++++--
>  arch/arm64/boot/dts/ti/k3-am62d2-evm.dts           | 32 ++++++++++++++++++----
>  .../boot/dts/ti/k3-am62p-ti-ipc-firmware.dtsi      | 11 ++++++--
>  arch/arm64/boot/dts/ti/k3-am62p-verdin.dtsi        | 29 ++++++++++++++++++--
>  arch/arm64/boot/dts/ti/k3-am62p5-sk.dts            | 29 ++++++++++++++++++--
>  arch/arm64/boot/dts/ti/k3-am62p5-var-som.dtsi      | 29 ++++++++++++++++++--
>  8 files changed, 179 insertions(+), 20 deletions(-)

Thanks Markus for dealing with all the back and forth on the Linux and 
bootloader sides. 

Reviewed-by: Bryan Brattlof <bb@xxxxxx>

~Bryan