[v3,1/2] dt-bindings: mfd: Document STM32 low power clkevent bindings
diff mbox series

Message ID 20200128120702.26166-2-benjamin.gaignard@st.com
State Superseded
Headers show
Series
  • clockevent: add low power STM32 timer
Related show

Checks

Context Check Description
robh/dt-meta-schema success
robh/checkpatch success

Commit Message

Benjamin Gaignard Jan. 28, 2020, 12:07 p.m. UTC
Add a subnode to STM low power timer bindings to support clkevent driver

Signed-off-by: Benjamin Gaignard <benjamin.gaignard@st.com>
---
 .../devicetree/bindings/mfd/st,stm32-lptimer.yaml          | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

Comments

Daniel Lezcano Feb. 12, 2020, 4:51 p.m. UTC | #1
On Tue, Jan 28, 2020 at 01:07:01PM +0100, Benjamin Gaignard wrote:
> Add a subnode to STM low power timer bindings to support clkevent driver

clkevent is Linux specific.

> Signed-off-by: Benjamin Gaignard <benjamin.gaignard@st.com>
> ---
>  .../devicetree/bindings/mfd/st,stm32-lptimer.yaml          | 14 ++++++++++++++
>  1 file changed, 14 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml b/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml
> index 1a4cc5f3fb33..535e0b40cdf9 100644
> --- a/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml
> +++ b/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml
> @@ -81,6 +81,16 @@ patternProperties:
>      required:
>        - compatible
>  
> +  clkevent:
> +    type: object
> +
> +    properties:
> +      compatible:
> +        const: st,stm32-clkevent-lp
> +
> +    required:
> +      - compatible
> +
>  required:
>    - "#address-cells"
>    - "#size-cells"
> @@ -115,6 +125,10 @@ examples:
>        counter {
>          compatible = "st,stm32-lptimer-counter";
>        };
> +
> +      clkevent {
> +        compatible = "st,stm32-clkevent-lp";
> +      };
>      };
>  
>  ...
> -- 
> 2.15.0
>
Benjamin Gaignard Feb. 12, 2020, 5:07 p.m. UTC | #2
On 2/12/20 5:51 PM, Daniel Lezcano wrote:
> On Tue, Jan 28, 2020 at 01:07:01PM +0100, Benjamin Gaignard wrote:
>> Add a subnode to STM low power timer bindings to support clkevent driver
> clkevent is Linux specific.
Does 'event' or 'broadcast' would sound better ?
>
>> Signed-off-by: Benjamin Gaignard <benjamin.gaignard@st.com>
>> ---
>>   .../devicetree/bindings/mfd/st,stm32-lptimer.yaml          | 14 ++++++++++++++
>>   1 file changed, 14 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml b/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml
>> index 1a4cc5f3fb33..535e0b40cdf9 100644
>> --- a/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml
>> +++ b/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml
>> @@ -81,6 +81,16 @@ patternProperties:
>>       required:
>>         - compatible
>>   
>> +  clkevent:
>> +    type: object
>> +
>> +    properties:
>> +      compatible:
>> +        const: st,stm32-clkevent-lp
>> +
>> +    required:
>> +      - compatible
>> +
>>   required:
>>     - "#address-cells"
>>     - "#size-cells"
>> @@ -115,6 +125,10 @@ examples:
>>         counter {
>>           compatible = "st,stm32-lptimer-counter";
>>         };
>> +
>> +      clkevent {
>> +        compatible = "st,stm32-clkevent-lp";
>> +      };
>>       };
>>   
>>   ...
>> -- 
>> 2.15.0
>>
Daniel Lezcano Feb. 12, 2020, 7:23 p.m. UTC | #3
Hi Benjamin,

On Wed, Feb 12, 2020 at 05:07:49PM +0000, Benjamin GAIGNARD wrote:
> On 2/12/20 5:51 PM, Daniel Lezcano wrote:
> > On Tue, Jan 28, 2020 at 01:07:01PM +0100, Benjamin Gaignard wrote:
> >> Add a subnode to STM low power timer bindings to support clkevent driver
> > clkevent is Linux specific.
> Does 'event' or 'broadcast' would sound better ?

As this describes the timer hardware block of the mfd, I would say 'timer' is
more appropriate.

Patch
diff mbox series

diff --git a/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml b/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml
index 1a4cc5f3fb33..535e0b40cdf9 100644
--- a/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml
+++ b/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml
@@ -81,6 +81,16 @@  patternProperties:
     required:
       - compatible
 
+  clkevent:
+    type: object
+
+    properties:
+      compatible:
+        const: st,stm32-clkevent-lp
+
+    required:
+      - compatible
+
 required:
   - "#address-cells"
   - "#size-cells"
@@ -115,6 +125,10 @@  examples:
       counter {
         compatible = "st,stm32-lptimer-counter";
       };
+
+      clkevent {
+        compatible = "st,stm32-clkevent-lp";
+      };
     };
 
 ...