[PATCH v3] arm64: dts: qcom: talos: Add memory-region for audio PD

[PATCH v3] arm64: dts: qcom: talos: Add memory-region for audio PD

[Ekansh Gupta via B4 Relay]

From: Ekansh Gupta <ekansh.gupta@oss.qualcomm.com>

Reserve memory region for audio PD dynamic loading and remote heap
requirements. Add the required VMID list for memory ownership
transfers.

Signed-off-by: Ekansh Gupta <ekansh.gupta@oss.qualcomm.com>
---
Changes in v3:
- Use shared-dma-pool representation for adsp_rpc_remote_heap_mem.
- Link to v2: https://lore.kernel.org/r/20260419-talosaudio-v2-1-3a3b5dd67dd4@oss.qualcomm.com

Changes in v2:
- Fixed VMID list alignment.
- Link to v1: https://lore.kernel.org/r/20260418-talosaudio-v1-1-585ab22faaf4@oss.qualcomm.com
---
 arch/arm64/boot/dts/qcom/talos.dtsi | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/talos.dtsi b/arch/arm64/boot/dts/qcom/talos.dtsi
index ff5afbfce2a4..c230d46616ea 100644
--- a/arch/arm64/boot/dts/qcom/talos.dtsi
+++ b/arch/arm64/boot/dts/qcom/talos.dtsi
@@ -11,6 +11,7 @@
 #include <dt-bindings/clock/qcom,qcs615-videocc.h>
 #include <dt-bindings/clock/qcom,rpmh.h>
 #include <dt-bindings/dma/qcom-gpi.h>
+#include <dt-bindings/firmware/qcom,scm.h>
 #include <dt-bindings/interconnect/qcom,icc.h>
 #include <dt-bindings/interconnect/qcom,osm-l3.h>
 #include <dt-bindings/interconnect/qcom,qcs615-rpmh.h>
@@ -657,6 +658,14 @@ pil_gpu_mem: pil-gpu@97715000 {
 			reg = <0x0 0x97715000 0x0 0x2000>;
 			no-map;
 		};
+
+		adsp_rpc_remote_heap_mem: adsp-rpc-remote-heap {
+			compatible = "shared-dma-pool";
+			alloc-ranges = <0x0 0x80000000 0x0 0x80000000>;
+			reusable;
+			alignment = <0x0 0x400000>;
+			size = <0x0 0x800000>;
+		};
 	};
 
 	soc: soc@0 {
@@ -5100,6 +5109,9 @@ fastrpc {
 					compatible = "qcom,fastrpc";
 					qcom,glink-channels = "fastrpcglink-apps-dsp";
 					label = "adsp";
+					memory-region = <&adsp_rpc_remote_heap_mem>;
+					qcom,vmids = <QCOM_SCM_VMID_LPASS
+						      QCOM_SCM_VMID_ADSP_HEAP>;
 					#address-cells = <1>;
 					#size-cells = <0>;
 

---
base-commit: c7275b05bc428c7373d97aa2da02d3a7fa6b9f66
change-id: 20260418-talosaudio-b8ecf8b9a1b3

Best regards,
-- 
Ekansh Gupta <ekansh.gupta@oss.qualcomm.com>

Re: [PATCH v3] arm64: dts: qcom: talos: Add memory-region for audio PD

[Dmitry Baryshkov]

On Fri, Apr 24, 2026 at 03:40:22PM +0530, Ekansh Gupta via B4 Relay wrote:
> From: Ekansh Gupta <ekansh.gupta@oss.qualcomm.com>
> 
> Reserve memory region for audio PD dynamic loading and remote heap
> requirements. Add the required VMID list for memory ownership
> transfers.

And what happens if there is no such region allocated / assigned?

Also, can we please get the remote heap set up for all the platforms
where it makes sense? I see that it is only currently set for a very few
devices (hamoa, kodiak, lemans, monaco and SM8750). Checking the
downstream sources, I see the remoteheap being set starting from the
era of MSM8996 / SDM660.

> Signed-off-by: Ekansh Gupta <ekansh.gupta@oss.qualcomm.com>

-- 
With best wishes
Dmitry

Re: [PATCH v3] arm64: dts: qcom: talos: Add memory-region for audio PD

[Ekansh Gupta]

On 25-04-2026 20:43, Dmitry Baryshkov wrote:
> On Fri, Apr 24, 2026 at 03:40:22PM +0530, Ekansh Gupta via B4 Relay wrote:
>> From: Ekansh Gupta <ekansh.gupta@oss.qualcomm.com>
>>
>> Reserve memory region for audio PD dynamic loading and remote heap
>> requirements. Add the required VMID list for memory ownership
>> transfers.
> 
> And what happens if there is no such region allocated / assigned?
If there is no region assigned and audio PD dynamic loading is requested
for any offload use-case, it would result in audio PD dynamic loading
failure which is the major purpose of audio fastrpc daemon.
> 
> Also, can we please get the remote heap set up for all the platforms
> where it makes sense? I see that it is only currently set for a very few
> devices (hamoa, kodiak, lemans, monaco and SM8750). Checking the
> downstream sources, I see the remoteheap being set starting from the
> era of MSM8996 / SDM660.
Noted. I'll bring this region for other platforms also.
> 
>> Signed-off-by: Ekansh Gupta <ekansh.gupta@oss.qualcomm.com>
> 

Re: [PATCH v3] arm64: dts: qcom: talos: Add memory-region for audio PD

[Konrad Dybcio]

On 4/24/26 12:10 PM, Ekansh Gupta via B4 Relay wrote:
> From: Ekansh Gupta <ekansh.gupta@oss.qualcomm.com>
> 
> Reserve memory region for audio PD dynamic loading and remote heap
> requirements. Add the required VMID list for memory ownership
> transfers.
> 
> Signed-off-by: Ekansh Gupta <ekansh.gupta@oss.qualcomm.com>
> ---

Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>

Konrad