[1/2] dt-bindings: clock: qcom,gcc-ipq8074: use common GCC schema

Message ID 20221028140326.43470-1-krzysztof.kozlowski@linaro.org
State New
Headers
Series [1/2] dt-bindings: clock: qcom,gcc-ipq8074: use common GCC schema |

Commit Message

Krzysztof Kozlowski Oct. 28, 2022, 2:03 p.m. UTC
  Reference common Qualcomm GCC schema to remove common pieces.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 .../bindings/clock/qcom,gcc-ipq8074.yaml      | 25 +++----------------
 1 file changed, 4 insertions(+), 21 deletions(-)
  

Comments

Rob Herring Oct. 31, 2022, 6:51 p.m. UTC | #1
On Fri, 28 Oct 2022 10:03:24 -0400, Krzysztof Kozlowski wrote:
> Reference common Qualcomm GCC schema to remove common pieces.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  .../bindings/clock/qcom,gcc-ipq8074.yaml      | 25 +++----------------
>  1 file changed, 4 insertions(+), 21 deletions(-)
> 

Acked-by: Rob Herring <robh@kernel.org>
  
Krzysztof Kozlowski Nov. 2, 2022, 4:28 p.m. UTC | #2
On 01/11/2022 15:48, Stephen Boyd wrote:
> Quoting Krzysztof Kozlowski (2022-10-28 07:03:24)
>> Reference common Qualcomm GCC schema to remove common pieces.
>>
>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>> ---
> 
> Should I pick this up into clk tree? Or are you going to apply it to
> binding tree?

I'll send v2 with subject fix pointed out by Rob and then please pick
them up via clk.

Best regards,
Krzysztof
  

Patch

diff --git a/Documentation/devicetree/bindings/clock/qcom,gcc-ipq8074.yaml b/Documentation/devicetree/bindings/clock/qcom,gcc-ipq8074.yaml
index 21470f52ce36..ac6711ed01ba 100644
--- a/Documentation/devicetree/bindings/clock/qcom,gcc-ipq8074.yaml
+++ b/Documentation/devicetree/bindings/clock/qcom,gcc-ipq8074.yaml
@@ -17,34 +17,17 @@  description: |
   See also:
   - dt-bindings/clock/qcom,gcc-ipq8074.h
 
+allOf:
+  - $ref: qcom,gcc.yaml#
+
 properties:
   compatible:
     const: qcom,gcc-ipq8074
 
-  '#clock-cells':
-    const: 1
-
-  '#power-domain-cells':
-    const: 1
-
-  '#reset-cells':
-    const: 1
-
-  reg:
-    maxItems: 1
-
-  protected-clocks:
-    description:
-      Protected clock specifier list as per common clock binding.
-
 required:
   - compatible
-  - reg
-  - '#clock-cells'
-  - '#power-domain-cells'
-  - '#reset-cells'
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |