Re: [PATCH V2 10/13] remoteproc: qcom: Add Hexagon based multipd rproc driver
From: Krzysztof Kozlowski
Date: Sat Jul 01 2023 - 06:56:04 EST
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