From: Stefano Radaelli <stefano.radaelli21@gmail.com>
To: "Markus Schneider-Pargmann (TI)" <msp@baylibre.com>
Cc: Nishanth Menon <nm@ti.com>, Vignesh Raghavendra <vigneshr@ti.com>,
Tero Kristo <kristo@kernel.org>, Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Nathan Chancellor <nathan@kernel.org>,
Nick Desaulniers <nick.desaulniers+lkml@gmail.com>,
Bill Wendling <morbo@google.com>,
Justin Stitt <justinstitt@google.com>, Judith Mendez <jm@ti.com>,
Daniel Schultz <d.schultz@phytec.de>, Andrew Davis <afd@ti.com>,
Siddharth Vadapalli <s-vadapalli@ti.com>,
Paresh Bhagat <p-bhagat@ti.com>, Bryan Brattlof <bb@ti.com>,
Jai Luthra <jai.luthra@ideasonboard.com>,
Devarsh Thakkar <devarsht@ti.com>,
Beleswar Padhi <b-padhi@ti.com>,
Francesco Dolcini <francesco.dolcini@toradex.com>,
Nick Desaulniers <ndesaulniers@google.com>,
Vishal Mahaveer <vishalm@ti.com>,
Kevin Hilman <khilman@baylibre.com>,
Sebin Francis <sebin.francis@ti.com>,
Kendall Willis <k-willis@ti.com>, Akashdeep Kaur <a-kaur@ti.com>,
linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, llvm@lists.linux.dev,
Hari Nagalla <hnagalla@ti.com>
Subject: Re: [PATCH v9 9/9] arm64: dts: ti: Add wkup R5F nodes to pre-ram bootphase
Date: Fri, 18 Sep 2026 13:48:12 +0200 [thread overview]
Message-ID: <aq0k_NyB05KTBs9f@Lord-Beerus.station> (raw)
In-Reply-To: <20260918-topic-am62a-ioddr-dt-v6-19-v9-9-40972ef1f663@baylibre.com>
On Fri, Sep 18, 2026 at 11:17:53AM +0200, Markus Schneider-Pargmann (TI) wrote:
> For IO+DDR the wkup_r5fss0_core0 and the
> wkup_r5fss0_core0_lpm_metadata_region need to be accessed before RAM
> setup is done. These are used to read the lpm metadata region in
> which data is stored to resume. This needs to be done before RAM is
> in use to avoid overwriting data.
>
> Add the bootph-pre-ram flag to the wkup_r5fss0_core0 node in both
> ipc-firmware dtsi files and to the lpm metadata region nodes of the
> AM62A and AM62P boards.
>
> The board level bootph-pre-ram flag of k3-am62d2-evm.dts becomes
> redundant with the dtsi-wide flag and is dropped.
>
> Signed-off-by: Markus Schneider-Pargmann (TI) <msp@baylibre.com>
> ---
> arch/arm64/boot/dts/ti/k3-am62a-phycore-som.dtsi | 1 +
> arch/arm64/boot/dts/ti/k3-am62a-ti-ipc-firmware.dtsi | 1 +
> arch/arm64/boot/dts/ti/k3-am62a7-sk.dts | 1 +
> arch/arm64/boot/dts/ti/k3-am62d2-evm.dts | 4 ----
> arch/arm64/boot/dts/ti/k3-am62p-ti-ipc-firmware.dtsi | 1 +
> arch/arm64/boot/dts/ti/k3-am62p-verdin.dtsi | 1 +
> arch/arm64/boot/dts/ti/k3-am62p5-sk.dts | 1 +
> arch/arm64/boot/dts/ti/k3-am62p5-var-som.dtsi | 1 +
> 8 files changed, 7 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am62a-phycore-som.dtsi b/arch/arm64/boot/dts/ti/k3-am62a-phycore-som.dtsi
> index 7fa19ced1a23..12be777a35ca 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62a-phycore-som.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am62a-phycore-som.dtsi
> @@ -80,6 +80,7 @@ wkup_r5fss0_core0_lpm_fs_stub_region: memory@9ca00000 {
> wkup_r5fss0_core0_lpm_metadata_region: memory@9ca08000 {
> compatible = "shared-dma-pool";
> reg = <0x00 0x9ca08000 0x00 0x1000>;
> + bootph-pre-ram;
> no-map;
> };
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am62a-ti-ipc-firmware.dtsi b/arch/arm64/boot/dts/ti/k3-am62a-ti-ipc-firmware.dtsi
> index 94c1d60d8d7a..7beca54e75ee 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62a-ti-ipc-firmware.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am62a-ti-ipc-firmware.dtsi
> @@ -63,6 +63,7 @@ &wkup_r5fss0 {
> };
>
> &wkup_r5fss0_core0 {
> + bootph-pre-ram;
> mboxes = <&mailbox0_cluster0>, <&mbox_r5_0>;
> memory-region = <&wkup_r5fss0_core0_dma_memory_region>,
> <&wkup_r5fss0_core0_ipc_region>,
> diff --git a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
> index 871ec227def9..fe75f6b6b50b 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
> +++ b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
> @@ -74,6 +74,7 @@ wkup_r5fss0_core0_lpm_fs_stub_region: memory@9ca00000 {
> wkup_r5fss0_core0_lpm_metadata_region: memory@9ca08000 {
> compatible = "shared-dma-pool";
> reg = <0x00 0x9ca08000 0x00 0x1000>;
> + bootph-pre-ram;
> no-map;
> };
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am62d2-evm.dts b/arch/arm64/boot/dts/ti/k3-am62d2-evm.dts
> index 7cba3ff99e6a..f87d6d515e2b 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62d2-evm.dts
> +++ b/arch/arm64/boot/dts/ti/k3-am62d2-evm.dts
> @@ -800,10 +800,6 @@ partition@3fc0000 {
> };
> };
>
> -&wkup_r5fss0_core0 {
> - bootph-pre-ram;
> -};
> -
> &mcu_r5fss0_core0 {
> firmware-name = "am62d-mcu-r5f0_0-fw";
> };
> diff --git a/arch/arm64/boot/dts/ti/k3-am62p-ti-ipc-firmware.dtsi b/arch/arm64/boot/dts/ti/k3-am62p-ti-ipc-firmware.dtsi
> index 84dfbfd1986c..277d7e268cfe 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62p-ti-ipc-firmware.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am62p-ti-ipc-firmware.dtsi
> @@ -42,6 +42,7 @@ &wkup_r5fss0 {
> };
>
> &wkup_r5fss0_core0 {
> + bootph-pre-ram;
> mboxes = <&mailbox0_cluster0 &mbox_r5_0>;
> memory-region = <&wkup_r5fss0_core0_dma_memory_region>,
> <&wkup_r5fss0_core0_ipc_region>,
> diff --git a/arch/arm64/boot/dts/ti/k3-am62p-verdin.dtsi b/arch/arm64/boot/dts/ti/k3-am62p-verdin.dtsi
> index 45ab9d947b3a..62a518dbbdbe 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62p-verdin.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am62p-verdin.dtsi
> @@ -183,6 +183,7 @@ wkup_r5fss0_core0_lpm_fs_stub_region: memory@9ca00000 {
> wkup_r5fss0_core0_lpm_metadata_region: memory@9ca08000 {
> compatible = "shared-dma-pool";
> reg = <0x00 0x9ca08000 0x00 0x1000>;
> + bootph-pre-ram;
> no-map;
> };
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am62p5-sk.dts b/arch/arm64/boot/dts/ti/k3-am62p5-sk.dts
> index bb027c65eddd..39af098e4db2 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62p5-sk.dts
> +++ b/arch/arm64/boot/dts/ti/k3-am62p5-sk.dts
> @@ -70,6 +70,7 @@ wkup_r5fss0_core0_lpm_fs_stub_region: memory@9ca00000 {
> wkup_r5fss0_core0_lpm_metadata_region: memory@9ca08000 {
> compatible = "shared-dma-pool";
> reg = <0x00 0x9ca08000 0x00 0x1000>;
> + bootph-pre-ram;
> no-map;
> };
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am62p5-var-som.dtsi b/arch/arm64/boot/dts/ti/k3-am62p5-var-som.dtsi
> index 31f670fdd91d..2eefbd8d47a4 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62p5-var-som.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am62p5-var-som.dtsi
> @@ -84,6 +84,7 @@ wkup_r5fss0_core0_lpm_fs_stub_region: r5f-memory@9ca00000 {
> wkup_r5fss0_core0_lpm_metadata_region: r5f-memory@9ca08000 {
> compatible = "shared-dma-pool";
> reg = <0x00 0x9ca08000 0x00 0x1000>;
> + bootph-pre-ram;
> no-map;
> };
>
>
> --
> 2.55.0
>
Reviewed-by: Stefano Radaelli <stefano.r@variscite.com>
Best Regards,
Stefano
next prev parent reply other threads:[~2026-09-18 11:48 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-18 9:17 [PATCH v9 0/9] arm64: dts: ti: Split wkup R5F memory region for IO+DDR resume Markus Schneider-Pargmann (TI)
2026-09-18 9:17 ` [PATCH v9 1/9] arm64: dts: ti: k3-am62a-phycore-som: Fix wkup R5F memory region size Markus Schneider-Pargmann (TI)
2026-09-18 11:31 ` Wadim Egorov
2026-09-18 9:17 ` [PATCH v9 2/9] arm64: dts: ti: k3-am62d2-evm: " Markus Schneider-Pargmann (TI)
2026-09-18 9:17 ` [PATCH v9 3/9] arm64: dts: ti: k3-am62a7-sk: " Markus Schneider-Pargmann (TI)
2026-09-18 9:17 ` [PATCH v9 4/9] arm64: dts: ti: k3-am62p-verdin: " Markus Schneider-Pargmann (TI)
2026-09-18 9:17 ` [PATCH v9 5/9] arm64: dts: ti: k3-am62p5-sk: " Markus Schneider-Pargmann (TI)
2026-09-18 9:17 ` [PATCH v9 6/9] arm64: dts: ti: var-som-am62p: " Markus Schneider-Pargmann (TI)
2026-09-18 9:17 ` [PATCH v9 7/9] arm64: dts: ti: k3-am62a: Split r5f memory region Markus Schneider-Pargmann (TI)
2026-09-18 9:17 ` [PATCH v9 8/9] arm64: dts: ti: k3-am62p: " Markus Schneider-Pargmann (TI)
2026-09-18 11:47 ` Stefano Radaelli
2026-09-18 9:17 ` [PATCH v9 9/9] arm64: dts: ti: Add wkup R5F nodes to pre-ram bootphase Markus Schneider-Pargmann (TI)
2026-09-18 11:48 ` Stefano Radaelli [this message]
2026-10-02 1:00 ` [PATCH v9 0/9] arm64: dts: ti: Split wkup R5F memory region for IO+DDR resume Nishanth Menon
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=aq0k_NyB05KTBs9f@Lord-Beerus.station \
--to=stefano.radaelli21@gmail.com \
--cc=a-kaur@ti.com \
--cc=afd@ti.com \
--cc=b-padhi@ti.com \
--cc=bb@ti.com \
--cc=conor+dt@kernel.org \
--cc=d.schultz@phytec.de \
--cc=devarsht@ti.com \
--cc=devicetree@vger.kernel.org \
--cc=francesco.dolcini@toradex.com \
--cc=hnagalla@ti.com \
--cc=jai.luthra@ideasonboard.com \
--cc=jm@ti.com \
--cc=justinstitt@google.com \
--cc=k-willis@ti.com \
--cc=khilman@baylibre.com \
--cc=kristo@kernel.org \
--cc=krzk+dt@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=llvm@lists.linux.dev \
--cc=morbo@google.com \
--cc=msp@baylibre.com \
--cc=nathan@kernel.org \
--cc=ndesaulniers@google.com \
--cc=nick.desaulniers+lkml@gmail.com \
--cc=nm@ti.com \
--cc=p-bhagat@ti.com \
--cc=robh@kernel.org \
--cc=s-vadapalli@ti.com \
--cc=sebin.francis@ti.com \
--cc=vigneshr@ti.com \
--cc=vishalm@ti.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox