* [PATCH] dt-bindings: clock: gcc-apq8084: merge to gcc-other.yaml
@ 2022-05-21 15:08 Dmitry Baryshkov
2022-05-21 15:26 ` Krzysztof Kozlowski
0 siblings, 1 reply; 3+ messages in thread
From: Dmitry Baryshkov @ 2022-05-21 15:08 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Rob Herring, Krzysztof Kozlowski
Cc: linux-arm-msm, devicetree
There is no need to have separate bindings file for the
qcom,gcc-apq8084, merge it to qcom,gcc-other.yaml.
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
---
.../bindings/clock/qcom,gcc-apq8084.yaml | 42 -------------------
.../bindings/clock/qcom,gcc-other.yaml | 3 ++
2 files changed, 3 insertions(+), 42 deletions(-)
delete mode 100644 Documentation/devicetree/bindings/clock/qcom,gcc-apq8084.yaml
diff --git a/Documentation/devicetree/bindings/clock/qcom,gcc-apq8084.yaml b/Documentation/devicetree/bindings/clock/qcom,gcc-apq8084.yaml
deleted file mode 100644
index 397fb918e032..000000000000
--- a/Documentation/devicetree/bindings/clock/qcom,gcc-apq8084.yaml
+++ /dev/null
@@ -1,42 +0,0 @@
-# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
-%YAML 1.2
----
-$id: http://devicetree.org/schemas/clock/qcom,gcc-apq8084.yaml#
-$schema: http://devicetree.org/meta-schemas/core.yaml#
-
-title: Qualcomm Global Clock & Reset Controller Binding for APQ8084
-
-maintainers:
- - Stephen Boyd <sboyd@kernel.org>
- - Taniya Das <quic_tdas@quicinc.com>
-
-description: |
- Qualcomm global clock control module which supports the clocks, resets and
- power domains on APQ8084.
-
- See also::
- - dt-bindings/clock/qcom,gcc-apq8084.h
- - dt-bindings/reset/qcom,gcc-apq8084.h
-
-allOf:
- - $ref: qcom,gcc.yaml#
-
-properties:
- compatible:
- const: qcom,gcc-apq8084
-
-required:
- - compatible
-
-unevaluatedProperties: false
-
-examples:
- - |
- clock-controller@fc400000 {
- compatible = "qcom,gcc-apq8084";
- reg = <0xfc400000 0x4000>;
- #clock-cells = <1>;
- #reset-cells = <1>;
- #power-domain-cells = <1>;
- };
-...
diff --git a/Documentation/devicetree/bindings/clock/qcom,gcc-other.yaml b/Documentation/devicetree/bindings/clock/qcom,gcc-other.yaml
index 6c45e0f85494..142402f29d98 100644
--- a/Documentation/devicetree/bindings/clock/qcom,gcc-other.yaml
+++ b/Documentation/devicetree/bindings/clock/qcom,gcc-other.yaml
@@ -15,6 +15,8 @@ description: |
power domains.
See also:
+ - dt-bindings/clock/qcom,gcc-apq8084.h
+ - dt-bindings/reset/qcom,gcc-apq8084.h
- dt-bindings/clock/qcom,gcc-ipq4019.h
- dt-bindings/clock/qcom,gcc-ipq6018.h
- dt-bindings/reset/qcom,gcc-ipq6018.h
@@ -36,6 +38,7 @@ allOf:
properties:
compatible:
enum:
+ - qcom,gcc-apq8084
- qcom,gcc-ipq4019
- qcom,gcc-ipq6018
- qcom,gcc-mdm9607
--
2.35.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] dt-bindings: clock: gcc-apq8084: merge to gcc-other.yaml
2022-05-21 15:08 [PATCH] dt-bindings: clock: gcc-apq8084: merge to gcc-other.yaml Dmitry Baryshkov
@ 2022-05-21 15:26 ` Krzysztof Kozlowski
2022-05-21 19:44 ` Dmitry Baryshkov
0 siblings, 1 reply; 3+ messages in thread
From: Krzysztof Kozlowski @ 2022-05-21 15:26 UTC (permalink / raw)
To: Dmitry Baryshkov, Andy Gross, Bjorn Andersson, Rob Herring
Cc: linux-arm-msm, devicetree
On 21/05/2022 17:08, Dmitry Baryshkov wrote:
> There is no need to have separate bindings file for the
> qcom,gcc-apq8084, merge it to qcom,gcc-other.yaml.
>
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
This makes sense and we already discussed it was preferred over my patch.
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] dt-bindings: clock: gcc-apq8084: merge to gcc-other.yaml
2022-05-21 15:26 ` Krzysztof Kozlowski
@ 2022-05-21 19:44 ` Dmitry Baryshkov
0 siblings, 0 replies; 3+ messages in thread
From: Dmitry Baryshkov @ 2022-05-21 19:44 UTC (permalink / raw)
To: Krzysztof Kozlowski, Andy Gross, Bjorn Andersson, Rob Herring
Cc: linux-arm-msm, devicetree
On 21/05/2022 18:26, Krzysztof Kozlowski wrote:
> On 21/05/2022 17:08, Dmitry Baryshkov wrote:
>> There is no need to have separate bindings file for the
>> qcom,gcc-apq8084, merge it to qcom,gcc-other.yaml.
>>
>> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
>
> This makes sense and we already discussed it was preferred over my patch.
Yes, but for some reason Bjorn has merged your patch, so I had to rebase
this one.
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-05-21 19:44 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-05-21 15:08 [PATCH] dt-bindings: clock: gcc-apq8084: merge to gcc-other.yaml Dmitry Baryshkov
2022-05-21 15:26 ` Krzysztof Kozlowski
2022-05-21 19:44 ` Dmitry Baryshkov
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.