On 06/26/2017 10:38 PM, Jonathan Cameron wrote:

On 26 June 2017 19:14:16 BST, Rob Herring [off-list ref] wrote:

quoted

On Wed, Jun 21, 2017 at 04:30:12PM +0200, Fabrice Gasnier wrote:

quoted

Add documentation for STMicroelectronics STM32 Low-Power Timer

Trigger

quoted

binding.

Signed-off-by: Fabrice Gasnier <redacted>
---
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

++++++++++++++++++++++

quoted

 1 file changed, 23 insertions(+)
 create mode 100644

Documentation/devicetree/bindings/iio/timer/stm32-lptimer-trigger.txt

quoted

diff --git

a/Documentation/devicetree/bindings/iio/timer/stm32-lptimer-trigger.txt
b/Documentation/devicetree/bindings/iio/timer/stm32-lptimer-trigger.txt

quoted

new file mode 100644
index 0000000..466d99f

--- /dev/null
+++

b/Documentation/devicetree/bindings/iio/timer/stm32-lptimer-trigger.txt

quoted

@@ -0,0 +1,23 @@
+STMicroelectronics STM32 Low-Power Timer Trigger
+
+STM32 Low-Power Timer provides trigger source (LPTIM output) that

can be used

quoted

+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.

quoted

Rob
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html