There are several ADC drivers that depend on the same device tree
bindings. Rather than continue to duplicate the properties, this patch
adds a common adc binding document that can be referenced. For beginning,
only two properties are documented.
Signed-off-by: Stefan Popa <[email protected]>
---
Changes in v2, v3:
- N/A.
Changes in v4:
- Added this commit.
Changes in v5:
- Nothing changed.
Documentation/devicetree/bindings/iio/adc/adc.txt | 23 +++++++++++++++++++++++
1 file changed, 23 insertions(+)
create mode 100644 Documentation/devicetree/bindings/iio/adc/adc.txt
diff --git a/Documentation/devicetree/bindings/iio/adc/adc.txt b/Documentation/devicetree/bindings/iio/adc/adc.txt
new file mode 100644
index 0000000..5bbaa33
--- /dev/null
+++ b/Documentation/devicetree/bindings/iio/adc/adc.txt
@@ -0,0 +1,23 @@
+Common ADCs properties
+
+Optional properties for child nodes:
+- bipolar : Boolean, if set the channel is used in bipolar mode.
+- diff-channels : Differential channels muxed for this ADC. The first value
+ specifies the positive input pin, the second value the negative
+ input pin.
+
+Example:
+ adc@0 {
+ compatible = "some,adc";
+ ...
+ channel@0 {
+ bipolar;
+ diff-channels = <0 1>;
+ ...
+ };
+
+ channel@1 {
+ diff-channels = <2 3>;
+ ...
+ };
+ };
--
2.7.4
On Tue, 13 Nov 2018 13:21:01 +0200
Stefan Popa <[email protected]> wrote:
> There are several ADC drivers that depend on the same device tree
> bindings. Rather than continue to duplicate the properties, this patch
> adds a common adc binding document that can be referenced. For beginning,
> only two properties are documented.
>
> Signed-off-by: Stefan Popa <[email protected]>
Looks very sensible to me, but as we are looking at a some generalization
here, I'd like an Ack from Rob if possible (as he suggested it I think :)
Thanks,
Jonathan
> ---
> Changes in v2, v3:
> - N/A.
> Changes in v4:
> - Added this commit.
> Changes in v5:
> - Nothing changed.
>
> Documentation/devicetree/bindings/iio/adc/adc.txt | 23 +++++++++++++++++++++++
> 1 file changed, 23 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/iio/adc/adc.txt
>
> diff --git a/Documentation/devicetree/bindings/iio/adc/adc.txt b/Documentation/devicetree/bindings/iio/adc/adc.txt
> new file mode 100644
> index 0000000..5bbaa33
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/iio/adc/adc.txt
> @@ -0,0 +1,23 @@
> +Common ADCs properties
> +
> +Optional properties for child nodes:
> +- bipolar : Boolean, if set the channel is used in bipolar mode.
> +- diff-channels : Differential channels muxed for this ADC. The first value
> + specifies the positive input pin, the second value the negative
> + input pin.
> +
> +Example:
> + adc@0 {
> + compatible = "some,adc";
> + ...
> + channel@0 {
> + bipolar;
> + diff-channels = <0 1>;
> + ...
> + };
> +
> + channel@1 {
> + diff-channels = <2 3>;
> + ...
> + };
> + };
On Fri, Nov 16, 2018 at 06:38:38PM +0000, Jonathan Cameron wrote:
> On Tue, 13 Nov 2018 13:21:01 +0200
> Stefan Popa <[email protected]> wrote:
>
> > There are several ADC drivers that depend on the same device tree
> > bindings. Rather than continue to duplicate the properties, this patch
> > adds a common adc binding document that can be referenced. For beginning,
> > only two properties are documented.
> >
> > Signed-off-by: Stefan Popa <[email protected]>
> Looks very sensible to me, but as we are looking at a some generalization
> here, I'd like an Ack from Rob if possible (as he suggested it I think :)
Looks fine to me, but I don't have any clue if this will be flexible
enough for various h/w.
Rob
On Tue, 13 Nov 2018 13:21:01 +0200, Stefan Popa wrote:
> There are several ADC drivers that depend on the same device tree
> bindings. Rather than continue to duplicate the properties, this patch
> adds a common adc binding document that can be referenced. For beginning,
> only two properties are documented.
>
> Signed-off-by: Stefan Popa <[email protected]>
> ---
> Changes in v2, v3:
> - N/A.
> Changes in v4:
> - Added this commit.
> Changes in v5:
> - Nothing changed.
>
> Documentation/devicetree/bindings/iio/adc/adc.txt | 23 +++++++++++++++++++++++
> 1 file changed, 23 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/iio/adc/adc.txt
>
Reviewed-by: Rob Herring <[email protected]>
On Fri, 16 Nov 2018 16:53:02 -0600
Rob Herring <[email protected]> wrote:
> On Fri, Nov 16, 2018 at 06:38:38PM +0000, Jonathan Cameron wrote:
> > On Tue, 13 Nov 2018 13:21:01 +0200
> > Stefan Popa <[email protected]> wrote:
> >
> > > There are several ADC drivers that depend on the same device tree
> > > bindings. Rather than continue to duplicate the properties, this patch
> > > adds a common adc binding document that can be referenced. For beginning,
> > > only two properties are documented.
> > >
> > > Signed-off-by: Stefan Popa <[email protected]>
> > Looks very sensible to me, but as we are looking at a some generalization
> > here, I'd like an Ack from Rob if possible (as he suggested it I think :)
>
> Looks fine to me, but I don't have any clue if this will be flexible
> enough for various h/w.
I think we'll need more to cover all cases, but hopefully things that
make sense to be optional in general. Rather hard to predict. This is
the obvious stuff so far.
This is better than the current mess of different options!
Jonathan
>
> Rob
>
On Fri, 16 Nov 2018 16:53:19 -0600
Rob Herring <[email protected]> wrote:
> On Tue, 13 Nov 2018 13:21:01 +0200, Stefan Popa wrote:
> > There are several ADC drivers that depend on the same device tree
> > bindings. Rather than continue to duplicate the properties, this patch
> > adds a common adc binding document that can be referenced. For beginning,
> > only two properties are documented.
> >
> > Signed-off-by: Stefan Popa <[email protected]>
> > ---
> > Changes in v2, v3:
> > - N/A.
> > Changes in v4:
> > - Added this commit.
> > Changes in v5:
> > - Nothing changed.
> >
> > Documentation/devicetree/bindings/iio/adc/adc.txt | 23 +++++++++++++++++++++++
> > 1 file changed, 23 insertions(+)
> > create mode 100644 Documentation/devicetree/bindings/iio/adc/adc.txt
> >
>
> Reviewed-by: Rob Herring <[email protected]>
Applied to the togreg branch of iio.git and pushed out as testing for
the autobuilders to play with it.
Thanks,
Jonathan