Re: [PATCH v5 2/4] dt-bindings: iio: adc: Add common ADCs properties to a separate file
On Fri, 16 Nov 2018 16:53:19 -0600 Rob Herring 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 > > --- > > 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 Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to play with it. Thanks, Jonathan
Re: [PATCH v5 2/4] dt-bindings: iio: adc: Add common ADCs properties to a separate file
On Fri, 16 Nov 2018 16:53:02 -0600 Rob Herring wrote: > On Fri, Nov 16, 2018 at 06:38:38PM +, Jonathan Cameron 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 > > 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 >
Re: [PATCH v5 2/4] dt-bindings: iio: adc: Add common ADCs properties to a separate file
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 > --- > 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
Re: [PATCH v5 2/4] dt-bindings: iio: adc: Add common ADCs properties to a separate file
On Fri, Nov 16, 2018 at 06:38:38PM +, Jonathan Cameron 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 > 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
Re: [PATCH v5 2/4] dt-bindings: iio: adc: Add common ADCs properties to a separate file
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 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 000..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>; > + ... > + }; > + };
[PATCH v5 2/4] dt-bindings: iio: adc: Add common ADCs properties to a separate file
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 --- 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 000..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