diff mbox series

[v9,02/11] dt-bindings: rtc: sun6i: Add H616 compatible string

Message ID 20210802003952.19942-3-andre.przywara@arm.com
State New
Headers show
Series None | expand

Commit Message

Andre Przywara Aug. 2, 2021, 12:39 a.m. UTC
Add the obvious compatible name to the existing RTC binding.
The actual RTC part of the device uses a different day/month/year
storage scheme, so it's not compatible with the previous devices.
Also the clock part is quite different, as there is no external 32K LOSC
oscillator input.

Signed-off-by: Andre Przywara <andre.przywara@arm.com>
Reviewed-by: Rob Herring <robh@kernel.org>
---
 .../bindings/rtc/allwinner,sun6i-a31-rtc.yaml      | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

Comments

Maxime Ripard Aug. 18, 2021, 8:15 a.m. UTC | #1
On Mon, Aug 02, 2021 at 01:39:43AM +0100, Andre Przywara wrote:
> Add the obvious compatible name to the existing RTC binding.

> The actual RTC part of the device uses a different day/month/year

> storage scheme, so it's not compatible with the previous devices.

> Also the clock part is quite different, as there is no external 32K LOSC

> oscillator input.

> 

> Signed-off-by: Andre Przywara <andre.przywara@arm.com>

> Reviewed-by: Rob Herring <robh@kernel.org>

> ---

>  .../bindings/rtc/allwinner,sun6i-a31-rtc.yaml      | 14 ++++++++++++++

>  1 file changed, 14 insertions(+)

> 

> diff --git a/Documentation/devicetree/bindings/rtc/allwinner,sun6i-a31-rtc.yaml b/Documentation/devicetree/bindings/rtc/allwinner,sun6i-a31-rtc.yaml

> index beeb90e55727..d8a6500e5840 100644

> --- a/Documentation/devicetree/bindings/rtc/allwinner,sun6i-a31-rtc.yaml

> +++ b/Documentation/devicetree/bindings/rtc/allwinner,sun6i-a31-rtc.yaml

> @@ -26,6 +26,7 @@ properties:

>            - const: allwinner,sun50i-a64-rtc

>            - const: allwinner,sun8i-h3-rtc

>        - const: allwinner,sun50i-h6-rtc

> +      - const: allwinner,sun50i-h616-rtc

>  

>    reg:

>      maxItems: 1

> @@ -104,6 +105,19 @@ allOf:

>            minItems: 3

>            maxItems: 3

>  

> +  - if:

> +      properties:

> +        compatible:

> +          contains:

> +            const: allwinner,sun50i-h616-rtc

> +

> +    then:

> +      properties:

> +        clock-output-names:

> +          minItems: 3

> +          maxItems: 3


The comments I made here on the v7 are still relevant: you only need one
of these two, and the list of clocks should be documented.

Maxime
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/rtc/allwinner,sun6i-a31-rtc.yaml b/Documentation/devicetree/bindings/rtc/allwinner,sun6i-a31-rtc.yaml
index beeb90e55727..d8a6500e5840 100644
--- a/Documentation/devicetree/bindings/rtc/allwinner,sun6i-a31-rtc.yaml
+++ b/Documentation/devicetree/bindings/rtc/allwinner,sun6i-a31-rtc.yaml
@@ -26,6 +26,7 @@  properties:
           - const: allwinner,sun50i-a64-rtc
           - const: allwinner,sun8i-h3-rtc
       - const: allwinner,sun50i-h6-rtc
+      - const: allwinner,sun50i-h616-rtc
 
   reg:
     maxItems: 1
@@ -104,6 +105,19 @@  allOf:
           minItems: 3
           maxItems: 3
 
+  - if:
+      properties:
+        compatible:
+          contains:
+            const: allwinner,sun50i-h616-rtc
+
+    then:
+      properties:
+        clock-output-names:
+          minItems: 3
+          maxItems: 3
+        clocks: false
+
   - if:
       properties:
         compatible: