| Message ID | 20221028034155.5580-10-quic_bjorande@quicinc.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | interconnect: osm-l3: SC8280XP L3 and DDR scaling | expand |
On 27/10/2022 23:41, 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 <quic_bjorande@quicinc.com> > --- > .../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..223cd6ecf279 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-bwmon qcom,sc8280xp-cpu-bwmon To match sc7280. I think it's better than my initial choice for qcom,sdm845-bwmon without the cpu part. Best regards, Krzysztof
On Fri, Oct 28, 2022 at 06:15:50PM -0400, Krzysztof Kozlowski wrote: > On 27/10/2022 23:41, 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 <quic_bjorande@quicinc.com> > > --- > > .../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..223cd6ecf279 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-bwmon > > qcom,sc8280xp-cpu-bwmon > To match sc7280. I think it's better than my initial choice for > qcom,sdm845-bwmon without the cpu part. > As discussed back then, we omitted "cpu" because there where multiple instances of the bwmon block. Would you prefer we give it the "cpu" compatible and potentially us it for non-cpu measurements? Regards, Bjorn
diff --git a/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml b/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml index be29e0b80995..223cd6ecf279 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-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
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 <quic_bjorande@quicinc.com> --- .../devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml | 5 +++++ 1 file changed, 5 insertions(+)