Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1671715 > unrolled thread
| Started by | Fabrice Gasnier <fabrice.gasnier@st.com> |
|---|---|
| First post | 2017-06-21 16:40 +0200 |
| Last post | 2017-06-27 11:10 +0200 |
| Articles | 5 — 4 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
[PATCH v2 5/8] dt-bindings: iio: Add STM32 LPTimer trigger binding Fabrice Gasnier <fabrice.gasnier@st.com> - 2017-06-21 16:40 +0200
Re: [PATCH v2 5/8] dt-bindings: iio: Add STM32 LPTimer trigger binding Jonathan Cameron <jic23@kernel.org> - 2017-06-24 22:20 +0200
Re: [PATCH v2 5/8] dt-bindings: iio: Add STM32 LPTimer trigger binding Rob Herring <robh@kernel.org> - 2017-06-26 20:20 +0200
Re: [PATCH v2 5/8] dt-bindings: iio: Add STM32 LPTimer trigger binding Jonathan Cameron <jic23@jic23.retrosnub.co.uk> - 2017-06-26 22:40 +0200
Re: [PATCH v2 5/8] dt-bindings: iio: Add STM32 LPTimer trigger binding Fabrice Gasnier <fabrice.gasnier@st.com> - 2017-06-27 11:10 +0200
| From | Fabrice Gasnier <fabrice.gasnier@st.com> |
|---|---|
| Date | 2017-06-21 16:40 +0200 |
| Subject | [PATCH v2 5/8] dt-bindings: iio: Add STM32 LPTimer trigger binding |
| Message-ID | <tUOXw-7i3-23@gated-at.bofh.it> |
Add documentation for STMicroelectronics STM32 Low-Power Timer Trigger
binding.
Signed-off-by: Fabrice Gasnier <fabrice.gasnier@st.com>
---
Changes in v2:
- s/Low Power/Low-Power
- remove leading 0x in example (parent node)
- improve reg property description
---
.../bindings/iio/timer/stm32-lptimer-trigger.txt | 23 ++++++++++++++++++++++
1 file changed, 23 insertions(+)
create mode 100644 Documentation/devicetree/bindings/iio/timer/stm32-lptimer-trigger.txt
diff --git a/Documentation/devicetree/bindings/iio/timer/stm32-lptimer-trigger.txt b/Documentation/devicetree/bindings/iio/timer/stm32-lptimer-trigger.txt
new file mode 100644
index 0000000..466d99f
--- /dev/null
+++ b/Documentation/devicetree/bindings/iio/timer/stm32-lptimer-trigger.txt
@@ -0,0 +1,23 @@
+STMicroelectronics STM32 Low-Power Timer Trigger
+
+STM32 Low-Power Timer provides trigger source (LPTIM output) that can be used
+by STM32 internal ADC and/or DAC.
+
+Must be a sub-node of an STM32 Low-Power Timer device tree node.
+See ../mfd/stm32-lptimer.txt for details about the parent node.
+
+Required properties:
+- compatible: Must be "st,stm32-lptimer-trigger".
+- reg: Selects trigger hardware block. Must be 0, 1 or 2
+ respectively for lptimer1, lptimer2 or lptimer3
+ trigger output.
+
+Example:
+ lptimer1: lptimer@40002400 {
+ compatible = "st,stm32-lptimer";
+ ...
+ trigger@0 {
+ compatible = "st,stm32-lptimer-trigger";
+ reg = <0>;
+ };
+ };
--
1.9.1
[toc] | [next] | [standalone]
| From | Jonathan Cameron <jic23@kernel.org> |
|---|---|
| Date | 2017-06-24 22:20 +0200 |
| Subject | Re: [PATCH v2 5/8] dt-bindings: iio: Add STM32 LPTimer trigger binding |
| Message-ID | <tVZHc-3U2-11@gated-at.bofh.it> |
| In reply to | #1671715 |
On Wed, 21 Jun 2017 16:30:12 +0200
Fabrice Gasnier <fabrice.gasnier@st.com> wrote:
> Add documentation for STMicroelectronics STM32 Low-Power Timer Trigger
> binding.
>
> Signed-off-by: Fabrice Gasnier <fabrice.gasnier@st.com>
looks sensible to me.
I'm kind of assuming this whole series will end up going through Lee and MFD.
Acked-by: Jonathan Cameron <jic23@kernel.org>
> ---
> Changes in v2:
> - s/Low Power/Low-Power
> - remove leading 0x in example (parent node)
> - improve reg property description
> ---
> .../bindings/iio/timer/stm32-lptimer-trigger.txt | 23 ++++++++++++++++++++++
> 1 file changed, 23 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/iio/timer/stm32-lptimer-trigger.txt
>
> diff --git a/Documentation/devicetree/bindings/iio/timer/stm32-lptimer-trigger.txt b/Documentation/devicetree/bindings/iio/timer/stm32-lptimer-trigger.txt
> new file mode 100644
> index 0000000..466d99f
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/iio/timer/stm32-lptimer-trigger.txt
> @@ -0,0 +1,23 @@
> +STMicroelectronics STM32 Low-Power Timer Trigger
> +
> +STM32 Low-Power Timer provides trigger source (LPTIM output) that can be used
> +by STM32 internal ADC and/or DAC.
> +
> +Must be a sub-node of an STM32 Low-Power Timer device tree node.
> +See ../mfd/stm32-lptimer.txt for details about the parent node.
> +
> +Required properties:
> +- compatible: Must be "st,stm32-lptimer-trigger".
> +- reg: Selects trigger hardware block. Must be 0, 1 or 2
> + respectively for lptimer1, lptimer2 or lptimer3
> + trigger output.
> +
> +Example:
> + lptimer1: lptimer@40002400 {
> + compatible = "st,stm32-lptimer";
> + ...
> + trigger@0 {
> + compatible = "st,stm32-lptimer-trigger";
> + reg = <0>;
> + };
> + };
[toc] | [prev] | [next] | [standalone]
| From | Rob Herring <robh@kernel.org> |
|---|---|
| Date | 2017-06-26 20:20 +0200 |
| Subject | Re: [PATCH v2 5/8] dt-bindings: iio: Add STM32 LPTimer trigger binding |
| Message-ID | <tWGM9-5KY-1@gated-at.bofh.it> |
| In reply to | #1671715 |
On Wed, Jun 21, 2017 at 04:30:12PM +0200, Fabrice Gasnier wrote: > Add documentation for STMicroelectronics STM32 Low-Power Timer Trigger > binding. > > Signed-off-by: Fabrice Gasnier <fabrice.gasnier@st.com> > --- > Changes in v2: > - s/Low Power/Low-Power > - remove leading 0x in example (parent node) > - improve reg property description > --- > .../bindings/iio/timer/stm32-lptimer-trigger.txt | 23 ++++++++++++++++++++++ > 1 file changed, 23 insertions(+) > create mode 100644 Documentation/devicetree/bindings/iio/timer/stm32-lptimer-trigger.txt > > diff --git a/Documentation/devicetree/bindings/iio/timer/stm32-lptimer-trigger.txt b/Documentation/devicetree/bindings/iio/timer/stm32-lptimer-trigger.txt > new file mode 100644 > index 0000000..466d99f > --- /dev/null > +++ b/Documentation/devicetree/bindings/iio/timer/stm32-lptimer-trigger.txt > @@ -0,0 +1,23 @@ > +STMicroelectronics STM32 Low-Power Timer Trigger > + > +STM32 Low-Power Timer provides trigger source (LPTIM output) that can be used > +by STM32 internal ADC and/or DAC. > + > +Must be a sub-node of an STM32 Low-Power Timer device tree node. > +See ../mfd/stm32-lptimer.txt for details about the parent node. > + > +Required properties: > +- compatible: Must be "st,stm32-lptimer-trigger". > +- reg: Selects trigger hardware block. Must be 0, 1 or 2 > + respectively for lptimer1, lptimer2 or lptimer3 > + trigger output. I guess this answers my question. However, this seems like abuse of the reg prop. This should not be how you select a trigger. The DT should describe all the h/w blocks and then the adc connection is a separate property. Rob
[toc] | [prev] | [next] | [standalone]
| From | Jonathan Cameron <jic23@jic23.retrosnub.co.uk> |
|---|---|
| Date | 2017-06-26 22:40 +0200 |
| Message-ID | <tWIXE-72s-17@gated-at.bofh.it> |
| In reply to | #1674986 |
On 26 June 2017 19:14:16 BST, Rob Herring <robh@kernel.org> wrote: >On Wed, Jun 21, 2017 at 04:30:12PM +0200, Fabrice Gasnier wrote: >> Add documentation for STMicroelectronics STM32 Low-Power Timer >Trigger >> binding. >> >> Signed-off-by: Fabrice Gasnier <fabrice.gasnier@st.com> >> --- >> Changes in v2: >> - s/Low Power/Low-Power >> - remove leading 0x in example (parent node) >> - improve reg property description >> --- >> .../bindings/iio/timer/stm32-lptimer-trigger.txt | 23 >++++++++++++++++++++++ >> 1 file changed, 23 insertions(+) >> create mode 100644 >Documentation/devicetree/bindings/iio/timer/stm32-lptimer-trigger.txt >> >> diff --git >a/Documentation/devicetree/bindings/iio/timer/stm32-lptimer-trigger.txt >b/Documentation/devicetree/bindings/iio/timer/stm32-lptimer-trigger.txt >> new file mode 100644 >> index 0000000..466d99f >> --- /dev/null >> +++ >b/Documentation/devicetree/bindings/iio/timer/stm32-lptimer-trigger.txt >> @@ -0,0 +1,23 @@ >> +STMicroelectronics STM32 Low-Power Timer Trigger >> + >> +STM32 Low-Power Timer provides trigger source (LPTIM output) that >can be used >> +by STM32 internal ADC and/or DAC. >> + >> +Must be a sub-node of an STM32 Low-Power Timer device tree node. >> +See ../mfd/stm32-lptimer.txt for details about the parent node. >> + >> +Required properties: >> +- compatible: Must be "st,stm32-lptimer-trigger". >> +- reg: Selects trigger hardware block. Must be 0, 1 or 2 >> + respectively for lptimer1, lptimer2 or lptimer3 >> + trigger output. > >I guess this answers my question. However, this seems like abuse of the > >reg prop. This should not be how you select a trigger. The DT should >describe all the h/w blocks and then the adc connection is a separate >property. I think this is miss described. Reg is labelling instances of the hardware block which is providing the trigger. The connection to an ADC etc is done from userspace. > >Rob >-- >To unsubscribe from this list: send the line "unsubscribe linux-iio" in >the body of a message to majordomo@vger.kernel.org >More majordomo info at http://vger.kernel.org/majordomo-info.html -- Sent from my Android device with K-9 Mail. Please excuse my brevity.
[toc] | [prev] | [next] | [standalone]
| From | Fabrice Gasnier <fabrice.gasnier@st.com> |
|---|---|
| Date | 2017-06-27 11:10 +0200 |
| Subject | Re: [PATCH v2 5/8] dt-bindings: iio: Add STM32 LPTimer trigger binding |
| Message-ID | <tWUFs-6Nx-19@gated-at.bofh.it> |
| In reply to | #1675091 |
On 06/26/2017 10:38 PM, Jonathan Cameron wrote: > > > On 26 June 2017 19:14:16 BST, Rob Herring <robh@kernel.org> wrote: >> On Wed, Jun 21, 2017 at 04:30:12PM +0200, Fabrice Gasnier wrote: >>> Add documentation for STMicroelectronics STM32 Low-Power Timer >> Trigger >>> binding. >>> >>> Signed-off-by: Fabrice Gasnier <fabrice.gasnier@st.com> >>> --- >>> Changes in v2: >>> - s/Low Power/Low-Power >>> - remove leading 0x in example (parent node) >>> - improve reg property description >>> --- >>> .../bindings/iio/timer/stm32-lptimer-trigger.txt | 23 >> ++++++++++++++++++++++ >>> 1 file changed, 23 insertions(+) >>> create mode 100644 >> Documentation/devicetree/bindings/iio/timer/stm32-lptimer-trigger.txt >>> >>> diff --git >> a/Documentation/devicetree/bindings/iio/timer/stm32-lptimer-trigger.txt >> b/Documentation/devicetree/bindings/iio/timer/stm32-lptimer-trigger.txt >>> new file mode 100644 >>> index 0000000..466d99f >>> --- /dev/null >>> +++ >> b/Documentation/devicetree/bindings/iio/timer/stm32-lptimer-trigger.txt >>> @@ -0,0 +1,23 @@ >>> +STMicroelectronics STM32 Low-Power Timer Trigger >>> + >>> +STM32 Low-Power Timer provides trigger source (LPTIM output) that >> can be used >>> +by STM32 internal ADC and/or DAC. >>> + >>> +Must be a sub-node of an STM32 Low-Power Timer device tree node. >>> +See ../mfd/stm32-lptimer.txt for details about the parent node. >>> + >>> +Required properties: >>> +- compatible: Must be "st,stm32-lptimer-trigger". >>> +- reg: Selects trigger hardware block. Must be 0, 1 or 2 >>> + respectively for lptimer1, lptimer2 or lptimer3 >>> + trigger output. >> >> I guess this answers my question. However, this seems like abuse of the >> >> reg prop. This should not be how you select a trigger. The DT should >> describe all the h/w blocks and then the adc connection is a separate >> property. > I think this is miss described. Reg is labelling instances of the hardware block which is providing the trigger. Hi Rob, Jonathan, yes, reg identifies trigger hardware block. I'll better describe in v3: - reg: Identify trigger hardware block. Best regards, Fabrice > > The connection to an ADC etc is done from userspace. > > >> >> Rob >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-iio" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html >
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web