Re: [PATCH] misc: fastrpc: add ioctl for attaching to sensors pd

[Srinivas Kandagatla]

On 07/09/2020 15:02, Jonathan Marek wrote:
> > > Srini do you have any suggestions for how to name these values?
> >
> > These are domain id corresponding to each core.
> > you can use SDSP_DOMAIN_ID in here!
> > these are already defined in the file as:
> >
> > #define ADSP_DOMAIN_ID (0)
> > #define MDSP_DOMAIN_ID (1)
> > #define SDSP_DOMAIN_ID (2)
> > #define CDSP_DOMAIN_ID (3)
>
> I don't think this is right:
>
> FASTRPC_IOCTL_INIT_ATTACH uses pd = 0
> FASTRPC_IOCTL_INIT_CREATE uses pd = 1
>
> And these two ioctl are used with all DSP cores. So it wouldn't make 
> sense for the pd value to correspond to the domain id.
You are right, values are pretty much similar to domain ids but not 
exactly the same as Protection Domain(PD) ids.

I spoke to qcom guys about this, and this is what I have.

0 is Audio Process PD
1 is Dynamic User PD, cases like SNPE or CV
2 is Sensor Process PD.


Hope this helps!

--srini