[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <bc56e3be-7d0b-5420-978e-125d66ae8f72@linaro.org>
Date: Sat, 1 Jul 2023 12:55:55 +0200
From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To: Manikanta Mylavarapu <quic_mmanikan@...cinc.com>,
agross@...nel.org, andersson@...nel.org, konrad.dybcio@...aro.org,
robh+dt@...nel.org, krzysztof.kozlowski+dt@...aro.org,
conor+dt@...nel.org, jassisinghbrar@...il.com,
mathieu.poirier@...aro.org, mturquette@...libre.com,
sboyd@...nel.org, quic_eberman@...cinc.com, quic_mojha@...cinc.com,
kvalo@...nel.org, loic.poulain@...aro.org,
linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-remoteproc@...r.kernel.org,
linux-clk@...r.kernel.org
Cc: quic_srichara@...cinc.com, quic_sjaganat@...cinc.com,
quic_kathirav@...cinc.com, quic_anusha@...cinc.com,
quic_poovendh@...cinc.com, quic_varada@...cinc.com,
quic_devipriy@...cinc.com
Subject: Re: [PATCH V2 10/13] remoteproc: qcom: Add Hexagon based multipd
rproc driver
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
Powered by blists - more mailing lists