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 for Android: free password hash cracker in your pocket

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

Message-ID: <fcf59ffe-69f7-48d3-bf20-fc223a2c2d10@kernel.org>
Date: Wed, 28 Aug 2024 11:07:00 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Jingyi Wang <quic_jingyw@...cinc.com>
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.or, 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 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