| Message ID | 20230309082446.37362-2-krzysztof.kozlowski@linaro.org (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [1/3] dt-bindings: remoteproc: qcom,pas: correct memory-region constraints | expand |
On 09/03/2023 09:24, Krzysztof Kozlowski wrote: > Qualcomm PAS devices expect exactly one memory region, not many. Also, > the memory-region is now defined in device specific binding, not in > qcom,pas-common.yaml, thus also require it in the same place. > > Fixes: 838c558bb8bc ("dt-bindings: remoteproc: qcom: Add sm6115 pas yaml file") > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> I think the sm6115 was not yet merged, thus maybe this patch is not needed. If it wasn't merged, please squash it with original submission and send new viersion. Best regards, Krzysztof
On Thu, 09 Mar 2023 09:24:45 +0100, Krzysztof Kozlowski wrote: > Qualcomm PAS devices expect exactly one memory region, not many. Also, > the memory-region is now defined in device specific binding, not in > qcom,pas-common.yaml, thus also require it in the same place. > > Fixes: 838c558bb8bc ("dt-bindings: remoteproc: qcom: Add sm6115 pas yaml file") > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > .../devicetree/bindings/remoteproc/qcom,sm6115-pas.yaml | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > Reviewed-by: Rob Herring <robh@kernel.org>
diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,sm6115-pas.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,sm6115-pas.yaml index f5d1fa9f45f1..1ac5b99b2fad 100644 --- a/Documentation/devicetree/bindings/remoteproc/qcom,sm6115-pas.yaml +++ b/Documentation/devicetree/bindings/remoteproc/qcom,sm6115-pas.yaml @@ -32,7 +32,7 @@ properties: - const: xo memory-region: - minItems: 1 + maxItems: 1 description: Reference to the reserved-memory for the Hexagon core smd-edge: false @@ -44,6 +44,7 @@ properties: required: - compatible - reg + - memory-region allOf: - $ref: /schemas/remoteproc/qcom,pas-common.yaml#
Qualcomm PAS devices expect exactly one memory region, not many. Also, the memory-region is now defined in device specific binding, not in qcom,pas-common.yaml, thus also require it in the same place. Fixes: 838c558bb8bc ("dt-bindings: remoteproc: qcom: Add sm6115 pas yaml file") Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- .../devicetree/bindings/remoteproc/qcom,sm6115-pas.yaml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)