| Message ID | 20200128120702.26166-2-benjamin.gaignard@st.com |
|---|---|
| State | Superseded, archived |
| Headers | show |
| Series | clockevent: add low power STM32 timer | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success | |
| robh/dt-meta-schema | success |
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 >
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 >>
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.
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"; + }; }; ...
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(+)