* [PATCH] dt-bindings: interconnect: qcom,msm8998-bwmon: Correct SC7280 CPU compatible
@ 2022-10-11 14:07 Krzysztof Kozlowski
2022-10-12 6:08 ` Rajendra Nayak
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Krzysztof Kozlowski @ 2022-10-11 14:07 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Konrad Dybcio, Krzysztof Kozlowski,
Georgi Djakov, Rob Herring, Rajendra Nayak, linux-arm-msm,
linux-pm, devicetree, linux-kernel
Two different compatibles for SC7280 CPU BWMON instance were used
in DTS and bindings. Correct the bindings to use the same one as in
DTS, because it is more specific.
Fixes: b7c84ae757c2 ("dt-bindings: interconnect: qcom,msm8998-bwmon: Add support for sc7280 BWMONs")
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
.../devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml b/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml
index 2684562df4d9..be29e0b80995 100644
--- a/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml
+++ b/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml
@@ -24,7 +24,7 @@ properties:
oneOf:
- items:
- enum:
- - qcom,sc7280-bwmon
+ - qcom,sc7280-cpu-bwmon
- qcom,sdm845-bwmon
- const: qcom,msm8998-bwmon
- const: qcom,msm8998-bwmon # BWMON v4
--
2.34.1
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] dt-bindings: interconnect: qcom,msm8998-bwmon: Correct SC7280 CPU compatible
2022-10-11 14:07 [PATCH] dt-bindings: interconnect: qcom,msm8998-bwmon: Correct SC7280 CPU compatible Krzysztof Kozlowski
@ 2022-10-12 6:08 ` Rajendra Nayak
2022-10-12 16:11 ` Rob Herring
2022-10-28 7:14 ` Georgi Djakov
2 siblings, 0 replies; 5+ messages in thread
From: Rajendra Nayak @ 2022-10-12 6:08 UTC (permalink / raw)
To: Krzysztof Kozlowski, Andy Gross, Bjorn Andersson, Konrad Dybcio,
Georgi Djakov, Rob Herring, linux-arm-msm, linux-pm, devicetree,
linux-kernel
On 10/11/2022 7:37 PM, Krzysztof Kozlowski wrote:
> Two different compatibles for SC7280 CPU BWMON instance were used
> in DTS and bindings. Correct the bindings to use the same one as in
> DTS, because it is more specific.
>
> Fixes: b7c84ae757c2 ("dt-bindings: interconnect: qcom,msm8998-bwmon: Add support for sc7280 BWMONs")
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
ah, thanks for the fixing this,
Reviewed-by: Rajendra Nayak <quic_rjendra@quicinc.com>
> ---
> .../devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml b/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml
> index 2684562df4d9..be29e0b80995 100644
> --- a/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml
> +++ b/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml
> @@ -24,7 +24,7 @@ properties:
> oneOf:
> - items:
> - enum:
> - - qcom,sc7280-bwmon
> + - qcom,sc7280-cpu-bwmon
> - qcom,sdm845-bwmon
> - const: qcom,msm8998-bwmon
> - const: qcom,msm8998-bwmon # BWMON v4
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] dt-bindings: interconnect: qcom,msm8998-bwmon: Correct SC7280 CPU compatible
2022-10-11 14:07 [PATCH] dt-bindings: interconnect: qcom,msm8998-bwmon: Correct SC7280 CPU compatible Krzysztof Kozlowski
2022-10-12 6:08 ` Rajendra Nayak
@ 2022-10-12 16:11 ` Rob Herring
2022-10-28 7:14 ` Georgi Djakov
2 siblings, 0 replies; 5+ messages in thread
From: Rob Herring @ 2022-10-12 16:11 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: devicetree, Bjorn Andersson, Georgi Djakov, linux-kernel,
Konrad Dybcio, Rajendra Nayak, Rob Herring, linux-pm, Andy Gross,
linux-arm-msm
On Tue, 11 Oct 2022 10:07:44 -0400, Krzysztof Kozlowski wrote:
> Two different compatibles for SC7280 CPU BWMON instance were used
> in DTS and bindings. Correct the bindings to use the same one as in
> DTS, because it is more specific.
>
> Fixes: b7c84ae757c2 ("dt-bindings: interconnect: qcom,msm8998-bwmon: Add support for sc7280 BWMONs")
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
> .../devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] dt-bindings: interconnect: qcom,msm8998-bwmon: Correct SC7280 CPU compatible
2022-10-11 14:07 [PATCH] dt-bindings: interconnect: qcom,msm8998-bwmon: Correct SC7280 CPU compatible Krzysztof Kozlowski
2022-10-12 6:08 ` Rajendra Nayak
2022-10-12 16:11 ` Rob Herring
@ 2022-10-28 7:14 ` Georgi Djakov
2022-10-28 11:24 ` Krzysztof Kozlowski
2 siblings, 1 reply; 5+ messages in thread
From: Georgi Djakov @ 2022-10-28 7:14 UTC (permalink / raw)
To: Krzysztof Kozlowski, Andy Gross, Bjorn Andersson, Konrad Dybcio,
Rob Herring, Rajendra Nayak, linux-arm-msm, linux-pm, devicetree,
linux-kernel
On 11.10.22 17:07, Krzysztof Kozlowski wrote:
> Two different compatibles for SC7280 CPU BWMON instance were used
> in DTS and bindings. Correct the bindings to use the same one as in
> DTS, because it is more specific.
>
> Fixes: b7c84ae757c2 ("dt-bindings: interconnect: qcom,msm8998-bwmon: Add support for sc7280 BWMONs")
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
I assume that this will be picked by Bjorn. Please let me know otherwise.
Acked-by: Georgi Djakov <djakov@kernel.org>
> ---
> .../devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml b/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml
> index 2684562df4d9..be29e0b80995 100644
> --- a/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml
> +++ b/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml
> @@ -24,7 +24,7 @@ properties:
> oneOf:
> - items:
> - enum:
> - - qcom,sc7280-bwmon
> + - qcom,sc7280-cpu-bwmon
> - qcom,sdm845-bwmon
> - const: qcom,msm8998-bwmon
> - const: qcom,msm8998-bwmon # BWMON v4
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] dt-bindings: interconnect: qcom,msm8998-bwmon: Correct SC7280 CPU compatible
2022-10-28 7:14 ` Georgi Djakov
@ 2022-10-28 11:24 ` Krzysztof Kozlowski
0 siblings, 0 replies; 5+ messages in thread
From: Krzysztof Kozlowski @ 2022-10-28 11:24 UTC (permalink / raw)
To: Georgi Djakov, Andy Gross, Bjorn Andersson, Konrad Dybcio,
Rob Herring, Rajendra Nayak, linux-arm-msm, linux-pm, devicetree,
linux-kernel
On 28/10/2022 03:14, Georgi Djakov wrote:
> On 11.10.22 17:07, Krzysztof Kozlowski wrote:
>> Two different compatibles for SC7280 CPU BWMON instance were used
>> in DTS and bindings. Correct the bindings to use the same one as in
>> DTS, because it is more specific.
>>
>> Fixes: b7c84ae757c2 ("dt-bindings: interconnect: qcom,msm8998-bwmon: Add support for sc7280 BWMONs")
>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>
> I assume that this will be picked by Bjorn. Please let me know otherwise.
>
Bjorn marked it as not applicable, so please pick it up for interconnect.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2022-10-28 11:24 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-10-11 14:07 [PATCH] dt-bindings: interconnect: qcom,msm8998-bwmon: Correct SC7280 CPU compatible Krzysztof Kozlowski
2022-10-12 6:08 ` Rajendra Nayak
2022-10-12 16:11 ` Rob Herring
2022-10-28 7:14 ` Georgi Djakov
2022-10-28 11:24 ` Krzysztof Kozlowski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®