| Message ID | 20220521150835.1488683-1-dmitry.baryshkov@linaro.org (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | dt-bindings: clock: gcc-apq8084: merge to gcc-other.yaml | expand |
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
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>
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
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