| 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
| ||
|
Date: Wed, 22 May 2024 15:04:30 +0200 From: Bartosz Golaszewski <brgl@...ev.pl> To: neil.armstrong@...aro.org Cc: Bjorn Andersson <andersson@...nel.org>, Mathieu Poirier <mathieu.poirier@...aro.org>, Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, Konrad Dybcio <konrad.dybcio@...aro.org>, Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>, Jassi Brar <jassisinghbrar@...il.com>, Bartosz Golaszewski <bartosz.golaszewski@...aro.org>, linux-arm-msm@...r.kernel.org, linux-remoteproc@...r.kernel.org, devicetree@...r.kernel.org, linux-kernel@...r.kernel.org, Tengfei Fan <quic_tengfan@...cinc.com>, Srini Kandagatla <srinivas.kandagatla@...aro.org>, Alex Elder <elder@...nel.org> Subject: Re: [PATCH 1/5] dt-bindings: remoteproc: qcom,sm8550-pas: Document the SA8775p ADSP, CDSP and GPDSP On Wed, May 22, 2024 at 2:42 PM <neil.armstrong@...aro.org> wrote: > > On 22/05/2024 14:08, Bartosz Golaszewski wrote: > > From: Tengfei Fan <quic_tengfan@...cinc.com> > > > > Document the compatibles for the components used to boot the ADSP, CDSP0, > > CDSP1, GPDSP0 and GPDSP1 on the SA8775p SoC. > > > > Signed-off-by: Tengfei Fan <quic_tengfan@...cinc.com> > > Co-developed-by: Bartosz Golaszewski <bartosz.golaszewski@...aro.org> > > Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@...aro.org> > > --- > > .../bindings/remoteproc/qcom,sm8550-pas.yaml | 76 +++++++++++++++++++++- > > 1 file changed, 75 insertions(+), 1 deletion(-) > > > > diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,sm8550-pas.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,sm8550-pas.yaml > > index 73fda7565cd1..9d3a862c39e1 100644 > > --- a/Documentation/devicetree/bindings/remoteproc/qcom,sm8550-pas.yaml > > +++ b/Documentation/devicetree/bindings/remoteproc/qcom,sm8550-pas.yaml > > @@ -16,6 +16,11 @@ description: > > properties: > > compatible: > > enum: > > + - qcom,sa8775p-adsp-pas > > + - qcom,sa8775p-cdsp0-pas > > + - qcom,sa8775p-cdsp1-pas > > + - qcom,sa8775p-gpdsp0-pas > > + - qcom,sa8775p-gpdsp1-pas > > - qcom,sm8550-adsp-pas > > - qcom,sm8550-cdsp-pas > > - qcom,sm8550-mpss-pas > > @@ -44,12 +49,13 @@ properties: > > > > firmware-name: > > $ref: /schemas/types.yaml#/definitions/string-array > > + minItems: 1 > > This will allow a single firmware name for all compatible, > which is wrong > So increasing the limit from the default under allOf doesn't seem to work, should I instead keep this and make the lower limit stricter for all other models? Bart > > items: > > - description: Firmware name of the Hexagon core > > - description: Firmware name of the Hexagon Devicetree > > > > memory-region: > > - minItems: 2 > > + minItems: 1 > > Same here > > > items: > > - description: Memory region for main Firmware authentication > > - description: Memory region for Devicetree Firmware authentication > > @@ -81,6 +87,21 @@ allOf: > > maxItems: 5 > > memory-region: > > maxItems: 2 > > + - if: > > + properties: > > + compatible: > > + enum: > > + - qcom,sa8775p-adsp-pas > > + - qcom,sa8775p-cdsp0-pas > > + - qcom,sa8775p-cdsp1-pas > > + - qcom,sa8775p-gpdsp0-pas > > + - qcom,sa8775p-gpdsp1-pas > > + then: > > + properties: > > + interrupts: > > + maxItems: 5 > > + interrupt-names: > > + maxItems: 5 > > - if: > > properties: > > compatible: > > @@ -128,6 +149,7 @@ allOf: > > properties: > > compatible: > > enum: > > + - qcom,sa8775p-adsp-pas > > - qcom,sm8550-adsp-pas > > - qcom,sm8650-adsp-pas > > - qcom,x1e80100-adsp-pas > > @@ -177,6 +199,58 @@ allOf: > > - const: cx > > - const: mxc > > - const: nsp > > + - if: > > + properties: > > + compatible: > > + enum: > > + - qcom,sa8775p-cdsp-pas > > + then: > > + properties: > > + power-domains: > > + items: > > + - description: CX power domain > > + - description: MXC power domain > > + - description: NSP0 power domain > > + power-domain-names: > > + items: > > + - const: cx > > + - const: mxc > > + - const: nsp0 > > + > > + - if: > > + properties: > > + compatible: > > + enum: > > + - qcom,sa8775p-cdsp1-pas > > + then: > > + properties: > > + power-domains: > > + items: > > + - description: CX power domain > > + - description: MXC power domain > > + - description: NSP1 power domain > > + power-domain-names: > > + items: > > + - const: cx > > + - const: mxc > > + - const: nsp1 > > + > > + - if: > > + properties: > > + compatible: > > + enum: > > + - qcom,sa8775p-gpdsp0-pas > > + - qcom,sa8775p-gpdsp1-pas > > + then: > > + properties: > > + power-domains: > > + items: > > + - description: CX power domain > > + - description: MXC power domain > > + power-domain-names: > > + items: > > + - const: cx > > + - const: mxc > > > > unevaluatedProperties: false > > > > >
Powered by blists - more mailing lists