diff mbox series

[v2] dt-bindings: timer: add Ralink SoCs timer

Message ID 20230620100231.1412582-1-sergio.paracuellos@gmail.com
State Not Applicable, archived
Headers show
Series [v2] dt-bindings: timer: add Ralink SoCs timer | expand

Checks

Context Check Description
robh/checkpatch success
robh/patch-applied success
robh/dtbs-check warning build log
robh/dt-meta-schema success

Commit Message

Sergio Paracuellos June 20, 2023, 10:02 a.m. UTC
Add YAML documentation for the timer which is present on Ralink SoCs.

Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
---
Changes in v2:
- Remove redundant 'documentation' word from subject.
- Drop clock-names property.

 .../bindings/timer/ralink,rt2880-timer.yaml   | 44 +++++++++++++++++++
 1 file changed, 44 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/timer/ralink,rt2880-timer.yaml

Comments

Rob Herring (Arm) June 22, 2023, 1:51 a.m. UTC | #1
On Tue, 20 Jun 2023 12:02:31 +0200, Sergio Paracuellos wrote:
> Add YAML documentation for the timer which is present on Ralink SoCs.
> 
> Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> ---
> Changes in v2:
> - Remove redundant 'documentation' word from subject.
> - Drop clock-names property.
> 
>  .../bindings/timer/ralink,rt2880-timer.yaml   | 44 +++++++++++++++++++
>  1 file changed, 44 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/timer/ralink,rt2880-timer.yaml
> 

Reviewed-by: Rob Herring <robh@kernel.org>
Sergio Paracuellos June 22, 2023, 5:14 a.m. UTC | #2
On Thu, Jun 22, 2023 at 3:51 AM Rob Herring <robh@kernel.org> wrote:
>
>
> On Tue, 20 Jun 2023 12:02:31 +0200, Sergio Paracuellos wrote:
> > Add YAML documentation for the timer which is present on Ralink SoCs.
> >
> > Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> > ---
> > Changes in v2:
> > - Remove redundant 'documentation' word from subject.
> > - Drop clock-names property.
> >
> >  .../bindings/timer/ralink,rt2880-timer.yaml   | 44 +++++++++++++++++++
> >  1 file changed, 44 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/timer/ralink,rt2880-timer.yaml
> >
>
> Reviewed-by: Rob Herring <robh@kernel.org>
>

Thanks! Which tree is this supposed to go through?

Best regards,
    Sergio Paracuellos
Krzysztof Kozlowski June 22, 2023, 6:16 a.m. UTC | #3
On 22/06/2023 07:14, Sergio Paracuellos wrote:
> On Thu, Jun 22, 2023 at 3:51 AM Rob Herring <robh@kernel.org> wrote:
>>
>>
>> On Tue, 20 Jun 2023 12:02:31 +0200, Sergio Paracuellos wrote:
>>> Add YAML documentation for the timer which is present on Ralink SoCs.
>>>
>>> Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
>>> ---
>>> Changes in v2:
>>> - Remove redundant 'documentation' word from subject.
>>> - Drop clock-names property.
>>>
>>>  .../bindings/timer/ralink,rt2880-timer.yaml   | 44 +++++++++++++++++++
>>>  1 file changed, 44 insertions(+)
>>>  create mode 100644 Documentation/devicetree/bindings/timer/ralink,rt2880-timer.yaml
>>>
>>
>> Reviewed-by: Rob Herring <robh@kernel.org>
>>
> 
> Thanks! Which tree is this supposed to go through?

Preferably clocksource/clockevents/timer drivers tree/

Best regards,
Krzysztof
Sergio Paracuellos June 22, 2023, 6:47 a.m. UTC | #4
On Thu, Jun 22, 2023 at 8:16 AM Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:
>
> On 22/06/2023 07:14, Sergio Paracuellos wrote:
> > On Thu, Jun 22, 2023 at 3:51 AM Rob Herring <robh@kernel.org> wrote:
> >>
> >>
> >> On Tue, 20 Jun 2023 12:02:31 +0200, Sergio Paracuellos wrote:
> >>> Add YAML documentation for the timer which is present on Ralink SoCs.
> >>>
> >>> Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> >>> ---
> >>> Changes in v2:
> >>> - Remove redundant 'documentation' word from subject.
> >>> - Drop clock-names property.
> >>>
> >>>  .../bindings/timer/ralink,rt2880-timer.yaml   | 44 +++++++++++++++++++
> >>>  1 file changed, 44 insertions(+)
> >>>  create mode 100644 Documentation/devicetree/bindings/timer/ralink,rt2880-timer.yaml
> >>>
> >>
> >> Reviewed-by: Rob Herring <robh@kernel.org>
> >>
> >
> > Thanks! Which tree is this supposed to go through?
>
> Preferably clocksource/clockevents/timer drivers tree/

Thanks for letting me know :)

Best regards,
    Sergio Paracuellos

>
> Best regards,
> Krzysztof
>
Daniel Lezcano June 22, 2023, 9:59 a.m. UTC | #5
On 20/06/2023 12:02, Sergio Paracuellos wrote:
> Add YAML documentation for the timer which is present on Ralink SoCs.
> 
> Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> ---
> Changes in v2:
> - Remove redundant 'documentation' word from subject.
> - Drop clock-names property.
> 
>   .../bindings/timer/ralink,rt2880-timer.yaml   | 44 +++++++++++++++++++
>   1 file changed, 44 insertions(+)
>   create mode 100644 Documentation/devicetree/bindings/timer/ralink,rt2880-timer.yaml
> 
> diff --git a/Documentation/devicetree/bindings/timer/ralink,rt2880-timer.yaml b/Documentation/devicetree/bindings/timer/ralink,rt2880-timer.yaml
> new file mode 100644
> index 000000000000..daa7832babe3
> --- /dev/null

Applied, thanks
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/timer/ralink,rt2880-timer.yaml b/Documentation/devicetree/bindings/timer/ralink,rt2880-timer.yaml
new file mode 100644
index 000000000000..daa7832babe3
--- /dev/null
+++ b/Documentation/devicetree/bindings/timer/ralink,rt2880-timer.yaml
@@ -0,0 +1,44 @@ 
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/timer/ralink,rt2880-timer.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Timer present in Ralink family SoCs
+
+maintainers:
+  - Sergio Paracuellos <sergio.paracuellos@gmail.com>
+
+properties:
+  compatible:
+    const: ralink,rt2880-timer
+
+  reg:
+    maxItems: 1
+
+  clocks:
+    maxItems: 1
+
+  interrupts:
+    maxItems: 1
+
+required:
+  - compatible
+  - reg
+  - clocks
+  - interrupts
+
+additionalProperties: false
+
+examples:
+  - |
+    timer@100 {
+        compatible = "ralink,rt2880-timer";
+        reg = <0x100 0x20>;
+
+        clocks = <&sysc 3>;
+
+        interrupt-parent = <&intc>;
+        interrupts = <1>;
+    };
+...