Re: [PATCH v6 04/11] arm64: dts: ti: k3-am62p-verdin: Fix wkup R5F memory region size

[Paresh Bhagat]

Hi Markus,

Thanks for the patch

On 10/06/26 12:41, Markus Schneider-Pargmann wrote:
> Hi,
>
> On Tue Jun 9, 2026 at 8:56 PM CEST, Markus Schneider-Pargmann (TI) wrote:
> > The wkup_r5fss0_core0_memory_region was reserved with only
> > 0x0f00000 but the MCU SDK linker for the wkup R5F firmware on
> This and the var-som patch incorrectly mention the reserved size to be
> 0x0f00000 in the commit message while it should say 0x01e00000. I will
> fix the commit message with the next version.
>
> Best
> Markus
>
> > AM62P defines the DM code/data DDR footprint differently:
> >
> >      /* DDR for DM R5F code/data [ size 27 MiB + 396 KB ] */
> >      DDR                         : ORIGIN = 0x9CAA5000 LENGTH = 0x1B63000
> >
> > which results in an end at 0x9e608000. For this memory region which
> > starts at 0x9c900000 this means a length of:
> >
> >      0x9e608000 - 0x9c900000 = 0x1d08000
> >
> > Link: https://github.com/TexasInstruments/mcupsdk-core-k3/blob/k3_main/examples/drivers/ipc/ipc_rpmsg_echo_linux/am62px-sk/wkup-r5fss0-0_freertos/ti-arm-clang/linker.cmd
> > Fixes: 87f95ea316ac ("arm64: dts: ti: Add Toradex Verdin AM62P")
> > Signed-off-by: Markus Schneider-Pargmann (TI) <msp@xxxxxxxxxxxx>


I was investigating a similar issue on AM62D and implemented a slightly 
different solution based on internal discussions (although still under 
review)
https://lore.kernel.org/all/20260609181006.460401-1-p-bhagat@xxxxxx/

Would be great to get your feedback on this.


Thanks
Paresh


> > ---
> >   arch/arm64/boot/dts/ti/k3-am62p-verdin.dtsi | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/arch/arm64/boot/dts/ti/k3-am62p-verdin.dtsi b/arch/arm64/boot/dts/ti/k3-am62p-verdin.dtsi
> > index 7ee894d59113aa727d41b7ecd6b2bc7e12760823..8a5ff5c457579c7b1be7157d235fd4b4e5c6af11 100644
> > --- a/arch/arm64/boot/dts/ti/k3-am62p-verdin.dtsi
> > +++ b/arch/arm64/boot/dts/ti/k3-am62p-verdin.dtsi
> > @@ -170,7 +170,7 @@ wkup_r5fss0_core0_dma_memory_region: memory@9c800000 {
> >   
> >   		wkup_r5fss0_core0_memory_region: memory@9c900000 {
> >   			compatible = "shared-dma-pool";
> > -			reg = <0x00 0x9c900000 0x00 0x01e00000>;
> > +			reg = <0x00 0x9c900000 0x00 0x01d08000>;
> >   			no-map;
> >   		};
> >   	};