Re: [PATCH v3 1/2] soc: qcom: pd-mapper: Add support for SA8775P

[Mohammad Rafi Shaik]

On 2/9/2026 6:10 PM, Dmitry Baryshkov wrote:
> On Mon, Feb 09, 2026 at 04:59:46PM +0530, Mohammad Rafi Shaik wrote:
> > Add support for the Qualcomm SA8775P SoC to the protection
> > domain mapper. SA8775P share the same protection domain
> > configuration as SC8280XP with an additional gpdsp domain,
> > except for charger_pd.
> >
> > Add an entry to the kernel, to avoid the need for userspace
> > to provide this service.
> >
> > Signed-off-by: Mohammad Rafi Shaik <mohammad.rafi.shaik@xxxxxxxxxxxxxxxx>
> > ---
> >   drivers/soc/qcom/qcom_pd_mapper.c | 22 ++++++++++++++++++++++
> >   1 file changed, 22 insertions(+)
> >
> > diff --git a/drivers/soc/qcom/qcom_pd_mapper.c b/drivers/soc/qcom/qcom_pd_mapper.c
> > index 1bcbe6968..8b899ec7f 100644
> > --- a/drivers/soc/qcom/qcom_pd_mapper.c
> > +++ b/drivers/soc/qcom/qcom_pd_mapper.c
> > @@ -305,6 +305,18 @@ static const struct qcom_pdm_domain_data cdsp_root_pd = {
> >   	.services = { NULL },
> >   };
> >   
> > +static const struct qcom_pdm_domain_data gpdsp_root_pd = {
> > +	.domain = "msm/gpdsp/root_pd",
> > +	.instance_id = 192,
> > +	.services = { NULL },
> > +};
> > +
> > +static const struct qcom_pdm_domain_data gpdsp1_root_pd = {
> > +	.domain = "msm/gpdsp1/root_pd",
> > +	.instance_id = 241,
> > +	.services = { NULL },
> > +};
> > +
> >   static const struct qcom_pdm_domain_data slpi_root_pd = {
> >   	.domain = "msm/slpi/root_pd",
> >   	.instance_id = 90,
> > @@ -401,6 +413,15 @@ static const struct qcom_pdm_domain_data *qcs404_domains[] = {
> >   	NULL,
> >   };
> >   
> > +static const struct qcom_pdm_domain_data *sa8775p_domains[] = {
> > +	&adsp_audio_pd,
> > +	&adsp_root_pd,
> > +	&cdsp_root_pd,
>
> Is there no separate PD for CDSP1? What is the difference between CDSP
> and CDSP1?

Yes, there is no dedicated PD entry for CDSP1 in the linux‑firmware,

In the device tree, there is no functional change in CDSP, only the base 
address is different.

Thanks & Regards,
Rafi.

> > +	&gpdsp_root_pd,
> > +	&gpdsp1_root_pd,
> > +	NULL,
> > +};
> > +
> >   static const struct qcom_pdm_domain_data *sc7180_domains[] = {
> >   	&adsp_audio_pd,
> >   	&adsp_root_pd_pdr,
> > @@ -572,6 +593,7 @@ static const struct of_device_id qcom_pdm_domains[] __maybe_unused = {
> >   	{ .compatible = "qcom,qcm2290", .data = qcm2290_domains, },
> >   	{ .compatible = "qcom,qcm6490", .data = sc7280_domains, },
> >   	{ .compatible = "qcom,qcs404", .data = qcs404_domains, },
> > +	{ .compatible = "qcom,sa8775p", .data = sa8775p_domains, },
> >   	{ .compatible = "qcom,sc7180", .data = sc7180_domains, },
> >   	{ .compatible = "qcom,sc7280", .data = sc7280_domains, },
> >   	{ .compatible = "qcom,sc8180x", .data = sc8180x_domains, },
> > --
> > 2.34.1