Re: [PATCH v2 1/3] arm64: dts: qcom: sdm630/660: Add CDSP-related nodes

[Nickolay Goppen]

24.10.2025 11:28, Konrad Dybcio пишет:
> On 10/23/25 9:51 PM, Nickolay Goppen wrote:
> > In order to enable CDSP support for SDM660 SoC:
> >   * add shared memory p2p nodes for CDSP
> >   * add CDSP-specific smmu node
> >   * add CDSP peripheral image loader node
> >
> > Memory region for CDSP in SDM660 occupies the same spot as
> > TZ buffer mem defined in sdm630.dtsi (which does not have CDSP).
> > In sdm660.dtsi replace buffer_mem inherited from SDM630 with
> > cdsp_region, which is also larger in size.
> >
> > SDM636 also doesn't have CDSP, so remove inherited from sdm660.dtsi
> > related nodes and add buffer_mem back.
> >
> > Signed-off-by: Nickolay Goppen <setotau@xxxxxxxxxxxxxx>
> > ---
> [...]
>
> > +			label = "turing";
> "cdsp"
Ok, I'll change this in the next revision.
> > +			mboxes = <&apcs_glb 29>;
> > +			qcom,remote-pid = <5>;
> > +
> > +			fastrpc {
> > +				compatible = "qcom,fastrpc";
> > +				qcom,glink-channels = "fastrpcglink-apps-dsp";
> > +				label = "cdsp";
> > +				qcom,non-secure-domain;
> This shouldn't matter, both a secure and a non-secure device is
> created for CDSP
I've added this property, because it is used in other SoC's, such as 
SDM845 and SM6115 for both ADSP and CDSP
> Konrad

--
Best regards,
Nickolay