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

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: Windows password security audit tool. GUI, reports in PDF.

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

Date:   Thu, 16 Mar 2023 18:01:48 -0500
From:   Rob Herring <robh@...nel.org>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
Cc:     Neil Armstrong <neil.armstrong@...aro.org>,
        Konrad Dybcio <konrad.dybcio@...aro.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        linux-arm-msm@...r.kernel.org,
        Mathieu Poirier <mathieu.poirier@...aro.org>,
        linux-remoteproc@...r.kernel.org,
        Bjorn Andersson <andersson@...nel.org>,
        devicetree@...r.kernel.org, Andy Gross <agross@...nel.org>,
        Manivannan Sadhasivam <mani@...nel.org>,
        Bhupesh Sharma <bhupesh.sharma@...aro.org>,
        linux-kernel@...r.kernel.org, Rob Herring <robh+dt@...nel.org>
Subject: Re: [PATCH 1/3] dt-bindings: remoteproc: qcom,pas: correct
 memory-region constraints


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@...aro.org>
> 
> ---
> 
> 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@...nel.org>