[v2] arm64: dts: qcom: Add support for SDM660 CDSP and ADSP FastRPC

[PATCH v2 3/3] arm64: dts: qcom: sdm630: Add FastRPC nodes to ADSP

Posted by Nickolay Goppen 3 months, 2 weeks ago

Add FastRPC subnode with compute-cb subnodes to ADSP node.

Signed-off-by: Nickolay Goppen <setotau@mainlining.org>
---
 arch/arm64/boot/dts/qcom/sdm630.dtsi | 33 +++++++++++++++++++++++++++++++++
 1 file changed, 33 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/sdm630.dtsi b/arch/arm64/boot/dts/qcom/sdm630.dtsi
index f4906ee3f0c3..2764666714e6 100644
--- a/arch/arm64/boot/dts/qcom/sdm630.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm630.dtsi
@@ -2342,6 +2342,39 @@ q6routing: routing {
 						};
 					};
 				};
+
+				fastrpc {
+					compatible = "qcom,fastrpc";
+					qcom,glink-channels = "fastrpcglink-apps-dsp";
+					label = "adsp";
+					qcom,non-secure-domain;
+					#address-cells = <1>;
+					#size-cells = <0>;
+
+					compute-cb@1 {
+						compatible = "qcom,fastrpc-compute-cb";
+						reg = <1>;
+						iommus = <&lpass_smmu 3>;
+					};
+
+					compute-cb@2 {
+						compatible = "qcom,fastrpc-compute-cb";
+						reg = <2>;
+						iommus = <&lpass_smmu 7>;
+					};
+
+					compute-cb@3 {
+						compatible = "qcom,fastrpc-compute-cb";
+						reg = <3>;
+						iommus = <&lpass_smmu 8>;
+					};
+
+					compute-cb@4 {
+						compatible = "qcom,fastrpc-compute-cb";
+						reg = <4>;
+						iommus = <&lpass_smmu 9>;
+					};
+				};
 			};
 		};
 

-- 
2.51.1

Re: [PATCH v2 3/3] arm64: dts: qcom: sdm630: Add FastRPC nodes to ADSP

Posted by Konrad Dybcio 3 months, 2 weeks ago

On 10/23/25 9:52 PM, Nickolay Goppen wrote:
> Add FastRPC subnode with compute-cb subnodes to ADSP node.
> 
> Signed-off-by: Nickolay Goppen <setotau@mainlining.org>
> ---
>  arch/arm64/boot/dts/qcom/sdm630.dtsi | 33 +++++++++++++++++++++++++++++++++
>  1 file changed, 33 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/qcom/sdm630.dtsi b/arch/arm64/boot/dts/qcom/sdm630.dtsi
> index f4906ee3f0c3..2764666714e6 100644
> --- a/arch/arm64/boot/dts/qcom/sdm630.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm630.dtsi
> @@ -2342,6 +2342,39 @@ q6routing: routing {
>  						};
>  					};
>  				};
> +
> +				fastrpc {
> +					compatible = "qcom,fastrpc";
> +					qcom,glink-channels = "fastrpcglink-apps-dsp";
> +					label = "adsp";
> +					qcom,non-secure-domain;

I'm not sure this property is valid

Konrad

Re: [PATCH v2 3/3] arm64: dts: qcom: sdm630: Add FastRPC nodes to ADSP

Posted by Nickolay Goppen 3 months ago

24.10.2025 11:26, Konrad Dybcio пишет:
> On 10/23/25 9:52 PM, Nickolay Goppen wrote:
>> Add FastRPC subnode with compute-cb subnodes to ADSP node.
>>
>> Signed-off-by: Nickolay Goppen <setotau@mainlining.org>
>> ---
>>   arch/arm64/boot/dts/qcom/sdm630.dtsi | 33 +++++++++++++++++++++++++++++++++
>>   1 file changed, 33 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/sdm630.dtsi b/arch/arm64/boot/dts/qcom/sdm630.dtsi
>> index f4906ee3f0c3..2764666714e6 100644
>> --- a/arch/arm64/boot/dts/qcom/sdm630.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/sdm630.dtsi
>> @@ -2342,6 +2342,39 @@ q6routing: routing {
>>   						};
>>   					};
>>   				};
>> +
>> +				fastrpc {
>> +					compatible = "qcom,fastrpc";
>> +					qcom,glink-channels = "fastrpcglink-apps-dsp";
>> +					label = "adsp";
>> +					qcom,non-secure-domain;
> I'm not sure this property is valid

I've looked into FastRPC driver and found out, that the 
"qcom,non-secure-domain" property isn't valid at least for adsp

>
> Konrad

-- 
Best regards,
Nickolay

Re: [PATCH v2 3/3] arm64: dts: qcom: sdm630: Add FastRPC nodes to ADSP

Posted by Nickolay Goppen 3 months ago

09.11.2025 01:22, Nickolay Goppen пишет:
>
> 24.10.2025 11:26, Konrad Dybcio пишет:
>> On 10/23/25 9:52 PM, Nickolay Goppen wrote:
>>> Add FastRPC subnode with compute-cb subnodes to ADSP node.
>>>
>>> Signed-off-by: Nickolay Goppen <setotau@mainlining.org>
>>> ---
>>>   arch/arm64/boot/dts/qcom/sdm630.dtsi | 33 
>>> +++++++++++++++++++++++++++++++++
>>>   1 file changed, 33 insertions(+)
>>>
>>> diff --git a/arch/arm64/boot/dts/qcom/sdm630.dtsi 
>>> b/arch/arm64/boot/dts/qcom/sdm630.dtsi
>>> index f4906ee3f0c3..2764666714e6 100644
>>> --- a/arch/arm64/boot/dts/qcom/sdm630.dtsi
>>> +++ b/arch/arm64/boot/dts/qcom/sdm630.dtsi
>>> @@ -2342,6 +2342,39 @@ q6routing: routing {
>>>                           };
>>>                       };
>>>                   };
>>> +
>>> +                fastrpc {
>>> +                    compatible = "qcom,fastrpc";
>>> +                    qcom,glink-channels = "fastrpcglink-apps-dsp";
>>> +                    label = "adsp";
>>> +                    qcom,non-secure-domain;
>> I'm not sure this property is valid
>
> I've looked into FastRPC driver and found out, that the 
> "qcom,non-secure-domain" property isn't valid at least for adsp
>
nvm, it's used when registering the FastRPC device.
>>
>> Konrad
>
-- 
Best regards,
Nickolay