Re: [PATCH v4 3/3] remoteproc: qcom: pas: Add Nord ADSP and CDSP support

[Shawn Guo]

On Tue, Jul 28, 2026 at 02:57:49PM +0530, Mukesh Ojha wrote:
> On Tue, Jul 28, 2026 at 04:40:12PM +0800, Shawn Guo wrote:
> > Add support for ADSP (HPASS DSP) and 4 CDSPs found on Nord SoC. The ADSP
> > is pre-booted by XBL before Linux starts, so set early_boot flag for
> > attach path rather than a cold boot sequence.
> > 
> > Signed-off-by: Shawn Guo <shengchao.guo@xxxxxxxxxxxxxxxx>
> > ---
> >  drivers/remoteproc/qcom_q6v5_pas.c | 110 +++++++++++++++++++++++++++++
> >  1 file changed, 110 insertions(+)
...
> > +static const struct qcom_pas_data nord_cdsp0_resource = {
> > +	.crash_reason_smem = 601,
> > +	.firmware_name = "cdsp.mdt",
> > +	.dtb_firmware_name = "cdsp_dtb.mbn",
> > +	.pas_id = 18,
> > +	.dtb_pas_id = 37,
> > +	.minidump_id = 7,
> > +	.auto_boot = true,
> > +	.proxy_pd_names = (char*[]){
> > +		"cx",
> > +		"mx",
> > +		"nsp",
> > +		NULL
> > +	},
> > +	.load_state = "cdsp",
> 
> is this a miss or a known thing to not use cdsp0 here but use X it at other
> cdspX instances ?

This is what I found from downstream pil-rm code.

Shawn

> 
> > +	.ssr_name = "cdsp0",
> > +	.sysmon_name = "cdsp0",
> > +	.ssctl_id = 0x17,
> > +	.smem_host_id = 5,
> > +};