Messages in this thread | | | Date | Sat, 1 Jul 2023 12:55:55 +0200 | Subject | Re: [PATCH V2 10/13] remoteproc: qcom: Add Hexagon based multipd rproc driver | From | Krzysztof Kozlowski <> |
| |
On 30/06/2023 12:29, Manikanta Mylavarapu wrote: > > > On 6/27/2023 6:09 PM, Manikanta Mylavarapu wrote: >> >>
Do you see the lines above? You quote my reply and claim it is you who wrote it.
Please fix your mailer, it's making discussions unnecessary difficult.
>>>> + Say y here to support the Qualcomm Secure Peripheral Image Loader >>>> + for the Hexagon based MultiPD model remote processors on e.g. >>>> IPQ5018. >>>> + This is trustZone wireless subsystem. >>>> + >>> >>> ... >>> > I didn't understand. Can you please elaborate your comment?
https://en.wikipedia.org/wiki/Ellipsis
> >>>> + int (*powerup_scm)(u32 peripheral); >>>> + int (*powerdown_scm)(u32 peripheral); >>>> +}; >>>> + >>>> +/** >>>> + * qcom_get_pd_asid() - get the pd asid number from DT node >>> >>> From node name? NAK. It does not work like that. Node names can change >>> and you did not define this number as part of ABI. >>> >>> Probably you wanted unit address. >>> > > Yeah i got your point. Each of the WCSS PD's are internally > represented in Q6 with their corresponding "spawn" bit numbers. > I will use same and remove the "PD-" hardcodings. > > Is this fine ?
I don't get what you are going to use, so post a patch. Talk is cheap.
Best regards, Krzysztof
| |