Re: [PATCH 1/3] dt-bindings: remoteproc: qcom,pas: correct memory-region constraints

From: Rob Herring
Date: Thu Mar 16 2023 - 19:02:04 EST



On Thu, 09 Mar 2023 09:24:44 +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: cee616c68846 ("dt-bindings: remoteproc: qcom: adsp: move memory-region and firmware-name out of pas-common")
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
>
> ---
>
> Following dtbs_check errors are expected (not related):
> - qcom,halt-regs:0: [142] is too short
> - clocks: [[24, 222], [24, 223], [24, 157], [16], [24, 229], [24, 224], [24, 225], [27, 2], [27, 8]] is too short
> - 'px-supply' is a required property
> ---
> Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml | 1 +
> .../devicetree/bindings/remoteproc/qcom,pas-common.yaml | 1 -
> .../devicetree/bindings/remoteproc/qcom,qcs404-pas.yaml | 3 ++-
> .../devicetree/bindings/remoteproc/qcom,sc7180-pas.yaml | 3 ++-
> .../devicetree/bindings/remoteproc/qcom,sc8180x-pas.yaml | 3 ++-
> .../devicetree/bindings/remoteproc/qcom,sc8280xp-pas.yaml | 3 ++-
> .../devicetree/bindings/remoteproc/qcom,sdx55-pas.yaml | 3 ++-
> .../devicetree/bindings/remoteproc/qcom,sm6350-pas.yaml | 3 ++-
> .../devicetree/bindings/remoteproc/qcom,sm8150-pas.yaml | 3 ++-
> .../devicetree/bindings/remoteproc/qcom,sm8350-pas.yaml | 3 ++-
> 10 files changed, 17 insertions(+), 9 deletions(-)
>

Reviewed-by: Rob Herring <robh@xxxxxxxxxx>