2022-11-11 03:29:33

by Bjorn Andersson

[permalink] [raw]
Subject: [PATCH v2 09/10] dt-bindings: interconnect: qcom,msm8998-bwmon: Add sc8280xp bwmon instances

The sc8280xp platform has two BWMON instances, one v4 and one v5. Extend
the existing qcom,msm8998-bwmon and qcom,sc7280-llcc-bwmon to describe
these.

Signed-off-by: Bjorn Andersson <[email protected]>
Tested-by: Steev Klimaszewski <[email protected]>
---

Changes since v1:
- Added "cpu" to compatible

.../devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml | 5 +++++
1 file changed, 5 insertions(+)

diff --git a/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml b/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml
index be29e0b80995..0c720dbde36e 100644
--- a/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml
+++ b/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml
@@ -25,9 +25,14 @@ properties:
- items:
- enum:
- qcom,sc7280-cpu-bwmon
+ - qcom,sc8280xp-cpu-bwmon
- qcom,sdm845-bwmon
- const: qcom,msm8998-bwmon
- const: qcom,msm8998-bwmon # BWMON v4
+ - items:
+ - enum:
+ - qcom,sc8280xp-llcc-bwmon
+ - const: qcom,sc7280-llcc-bwmon
- const: qcom,sc7280-llcc-bwmon # BWMON v5
- const: qcom,sdm845-llcc-bwmon # BWMON v5

--
2.17.1



2022-11-11 08:58:59

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH v2 09/10] dt-bindings: interconnect: qcom,msm8998-bwmon: Add sc8280xp bwmon instances

On 11/11/2022 04:25, Bjorn Andersson wrote:
> The sc8280xp platform has two BWMON instances, one v4 and one v5. Extend
> the existing qcom,msm8998-bwmon and qcom,sc7280-llcc-bwmon to describe
> these.
>
> Signed-off-by: Bjorn Andersson <[email protected]>
> Tested-by: Steev Klimaszewski <[email protected]>


Reviewed-by: Krzysztof Kozlowski <[email protected]>

Best regards,
Krzysztof


2022-11-11 11:37:25

by Sibi Sankar

[permalink] [raw]
Subject: Re: [PATCH v2 09/10] dt-bindings: interconnect: qcom,msm8998-bwmon: Add sc8280xp bwmon instances



On 11/11/22 08:55, Bjorn Andersson wrote:
> The sc8280xp platform has two BWMON instances, one v4 and one v5. Extend
> the existing qcom,msm8998-bwmon and qcom,sc7280-llcc-bwmon to describe
> these.
>
> Signed-off-by: Bjorn Andersson <[email protected]>
> Tested-by: Steev Klimaszewski <[email protected]>

Reviewed-by: Sibi Sankar <[email protected]>

> ---
>
> Changes since v1:
> - Added "cpu" to compatible
>
> .../devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml b/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml
> index be29e0b80995..0c720dbde36e 100644
> --- a/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml
> +++ b/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml
> @@ -25,9 +25,14 @@ properties:
> - items:
> - enum:
> - qcom,sc7280-cpu-bwmon
> + - qcom,sc8280xp-cpu-bwmon
> - qcom,sdm845-bwmon
> - const: qcom,msm8998-bwmon
> - const: qcom,msm8998-bwmon # BWMON v4
> + - items:
> + - enum:
> + - qcom,sc8280xp-llcc-bwmon
> + - const: qcom,sc7280-llcc-bwmon
> - const: qcom,sc7280-llcc-bwmon # BWMON v5
> - const: qcom,sdm845-llcc-bwmon # BWMON v5
>