linux-kernel - Re: [PATCH 1/2] dt-bindings: remoteproc: qcom,sa8775p-pas: Document QCS8300 remoteproc

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <c0a85aa3-9829-4ac9-ab79-0ceb6fff7075@quicinc.com>
Date: Wed, 28 Aug 2024 17:38:44 +0800
From: Jingyi Wang <quic_jingyw@...cinc.com>
To: Krzysztof Kozlowski <krzk@...nel.org>
CC: <linux-arm-msm@...r.kernel.org>, <devicetree@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>, <linux-remoteproc@...r.kernel.org>,
        <andersson@...nel.org>, <robh@...nel.org>, <krzk+dt@...nel.org>,
        <conor+dt@...nel.org>, <mathieu.poirier@...aro.org>,
        <bartosz.golaszewski@...aro.org>, <quic_tingweiz@...cinc.com>,
        <quic_aiquny@...cinc.com>, <quic_tengfan@...cinc.com>,
        Xin Liu
	<quic_liuxin@...cinc.com>
Subject: Re: [PATCH 1/2] dt-bindings: remoteproc: qcom,sa8775p-pas: Document
 QCS8300 remoteproc



On 8/28/2024 5:07 PM, Krzysztof Kozlowski wrote:
> On 28/08/2024 09:28, Jingyi Wang wrote:
>>
>>
>> On 8/28/2024 3:22 PM, Krzysztof Kozlowski wrote:
>>> On Wed, Aug 28, 2024 at 11:05:10AM +0800, Jingyi Wang wrote:
>>>> Document the components used to boot the ADSP, CDSP and GPDSP on the
>>>> QCS8300 SoC.
>>>>
>>>> Co-developed-by: Xin Liu <quic_liuxin@...cinc.com>
>>>> Signed-off-by: Xin Liu <quic_liuxin@...cinc.com>
>>>> Signed-off-by: Jingyi Wang <quic_jingyw@...cinc.com>
>>>> ---
>>>>  .../bindings/remoteproc/qcom,sa8775p-pas.yaml | 22 +++++++++++++++++++
>>>>  1 file changed, 22 insertions(+)
>>>>
>>>> diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,sa8775p-pas.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,sa8775p-pas.yaml
>>>> index 7fe401a06805..44b070a17ca0 100644
>>>> --- a/Documentation/devicetree/bindings/remoteproc/qcom,sa8775p-pas.yaml
>>>> +++ b/Documentation/devicetree/bindings/remoteproc/qcom,sa8775p-pas.yaml
>>>> @@ -16,6 +16,9 @@ description:
>>>>  properties:
>>>>    compatible:
>>>>      enum:
>>>> +      - qcom,qcs8300-adsp-pas
>>>> +      - qcom,qcs8300-cdsp-pas
>>>> +      - qcom,qcs8300-gpdsp-pas
>>>>        - qcom,sa8775p-adsp-pas
>>>>        - qcom,sa8775p-cdsp0-pas
>>>>        - qcom,sa8775p-cdsp1-pas
>>>> @@ -64,6 +67,7 @@ allOf:
>>>>        properties:
>>>>          compatible:
>>>>            enum:
>>>> +            - qcom,qcs8300-adsp-pas
>>>>              - qcom,sa8775p-adsp-pas
>>>>      then:
>>>>        properties:
>>>> @@ -75,6 +79,23 @@ allOf:
>>>>            items:
>>>>              - const: lcx
>>>>              - const: lmx
>>>> +  - if:
>>>> +      properties:
>>>> +        compatible:
>>>> +          enum:
>>>> +            - qcom,qcs8300-cdsp-pas
>>>
>>> This looks the same as sa8775p cdsp. Why new entry?
>>>
>> There is difference in power domain, sa8775p use nsp while qcs8300 use nsp0
> 
> Please paste code where do you see the difference.
> 
> Best regards,
> Krzysztof
> 
sorry my mistake, I have checked that nsp0 is form the view of power domain, it
should be fine to use "nsp" to find the right power domain, will cleanup both
binding and driver patch.

Thanks,
Jingyi