Re: [PATCH v6 4/9] dt-bindings: iio: iio-mux: document iio-mux bindings
From: Rob Herring <robh@kernel.org>
Date: 2016-12-05 23:26:32
Also in:
linux-i2c, linux-iio, lkml
On Wed, Nov 30, 2016 at 09:16:58AM +0100, Peter Rosin wrote:
Signed-off-by: Peter Rosin <redacted> --- .../bindings/iio/multiplexer/iio-mux.txt | 40 ++++++++++++++++++++++ MAINTAINERS | 6 ++++ 2 files changed, 46 insertions(+) create mode 100644 Documentation/devicetree/bindings/iio/multiplexer/iio-mux.txt
I'm still not convinced about this binding, but don't really have more comments ATM. Sending 6 versions in 2 weeks or so doesn't really help either.
quoted hunk ↗ jump to hunk
diff --git a/Documentation/devicetree/bindings/iio/multiplexer/iio-mux.txt b/Documentation/devicetree/bindings/iio/multiplexer/iio-mux.txt new file mode 100644 index 000000000000..8080cf790d82 --- /dev/null +++ b/Documentation/devicetree/bindings/iio/multiplexer/iio-mux.txt@@ -0,0 +1,40 @@ +IIO multiplexer bindings + +If a multiplexer is used to select which hardware signal is fed to +e.g. an ADC channel, these bindings describe that situation. + +Required properties: +- compatible : "iio-mux"
This is a Linuxism. perhaps "adc-mux".
quoted hunk ↗ jump to hunk
+- io-channels : Channel node of the parent channel that has multiplexed + input. +- io-channel-names : Should be "parent". +- #address-cells = <1>; +- #size-cells = <0>; +- mux-controls : Mux controller node to use for operating the mux +- channels : List of strings, labeling the mux controller states. + +The multiplexer state as described in ../misc/mux-controller.txt + +For each non-empty string in the channels property, an iio channel will +be created. The number of this iio channel is the same as the index into +the list of strings in the channels property, and also matches the mux +controller state. + +Example: + mux: mux-controller { + compatible = "mux-gpio"; + #mux-control-cells = <0>; + + mux-gpios = <&pioA 0 GPIO_ACTIVE_HIGH>, + <&pioA 1 GPIO_ACTIVE_HIGH>; + }; + + adc-mux { + compatible = "iio-mux"; + io-channels = <&adc 0>; + io-channel-names = "parent"; + + mux-controls = <&mux>; + + channels = "sync", "in", system-regulator"; + };