[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <469f2249-5f7b-4136-9f65-b58c9baf537c@linaro.org>
Date: Mon, 30 Oct 2023 09:29:53 +0100
From: Neil Armstrong <neil.armstrong@...aro.org>
To: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
Andy Gross <agross@...nel.org>,
Bjorn Andersson <andersson@...nel.org>,
Konrad Dybcio <konrad.dybcio@...aro.org>,
Mathieu Poirier <mathieu.poirier@...aro.org>,
Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Conor Dooley <conor+dt@...nel.org>,
Manivannan Sadhasivam <mani@...nel.org>
Cc: linux-arm-msm@...r.kernel.org, linux-remoteproc@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/3] dt-bindings: remoteproc: qcom,sm8550-pas: document
the SM8650 PAS
On 27/10/2023 09:36, Krzysztof Kozlowski wrote:
> On 25/10/2023 09:35, Neil Armstrong wrote:
>> Document the DSP Peripheral Authentication Service on the SM8650 Platform.
>>
>> Signed-off-by: Neil Armstrong <neil.armstrong@...aro.org>
>> ---
>> .../bindings/remoteproc/qcom,sm8550-pas.yaml | 41 +++++++++++++++++++++-
>> 1 file changed, 40 insertions(+), 1 deletion(-)
>>
>> diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,sm8550-pas.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,sm8550-pas.yaml
>> index 58120829fb06..316371c8ee6e 100644
>> --- a/Documentation/devicetree/bindings/remoteproc/qcom,sm8550-pas.yaml
>> +++ b/Documentation/devicetree/bindings/remoteproc/qcom,sm8550-pas.yaml
>> @@ -19,6 +19,9 @@ properties:
>> - qcom,sm8550-adsp-pas
>> - qcom,sm8550-cdsp-pas
>> - qcom,sm8550-mpss-pas
>> + - qcom,sm8650-adsp-pas
>> + - qcom,sm8650-cdsp-pas
>> + - qcom,sm8650-mpss-pas
>>
>> reg:
>> maxItems: 1
>> @@ -49,6 +52,7 @@ properties:
>> - description: Memory region for main Firmware authentication
>> - description: Memory region for Devicetree Firmware authentication
>> - description: DSM Memory region
>> + - description: DSM Memory region 2
>>
>> required:
>> - compatible
>> @@ -63,6 +67,7 @@ allOf:
>> enum:
>> - qcom,sm8550-adsp-pas
>> - qcom,sm8550-cdsp-pas
>> + - qcom,sm8650-adsp-pas
>> then:
>> properties:
>> interrupts:
>> @@ -71,7 +76,25 @@ allOf:
>> maxItems: 5
>> memory-region:
>> maxItems: 2
>> - else:
>> + - if:
>> + properties:
>> + compatible:
>> + enum:
>> + - qcom,sm8650-cdsp-pas
>> + then:
>> + properties:
>> + interrupts:
>> + minItems: 5
>
> maxItems
>
>
>> + interrupt-names:
>> + minItems: 5
>
> maxItems
>
>> + memory-region:
>> + minItems: 3
>
> maxItems: 3
>
>> + - if:
>> + properties:
>> + compatible:
>> + enum:
>> + - qcom,sm8550-mpss-pas
>> + then:
>> properties:
>> interrupts:
>> minItems: 6
>> @@ -79,12 +102,26 @@ allOf:
>> minItems: 6
>> memory-region:
>> minItems: 3
>
> You need to add here maxItems.
Ok, I fixed all that
>
>> + - if:
>> + properties:
>> + compatible:
>> + enum:
>> + - qcom,sm8650-mpss-pas
>> + then:
>
> I am not sure if keeping it in the same binding as sm8550 avoids that
> much duplication.
Yes it does, 70% is the bindings would be the same, still if it's still preferable I can duplicate.
Thanks,
Neil
>
> Best regards,
> Krzysztof
>
Powered by blists - more mailing lists