* [PATCH v2 11/14] staging: iio: ad7746: Add devicetree bindings documentation [not found] <1523637411-8531-1-git-send-email-hernan@vanguardiasur.com.ar> @ 2018-04-13 16:36 ` Hernán Gonzalez 2018-04-15 15:37 ` Jonathan Cameron 2018-04-16 20:08 ` Rob Herring 0 siblings, 2 replies; 4+ messages in thread From: Hernán Gonzalez @ 2018-04-13 16:36 UTC (permalink / raw) To: jic23, knaack.h, lars, pmeerw, gregkh, Michael.Hennerich, linux-iio, linux-kernel Cc: Hernán Gonzalez, Rob Herring, Mark Rutland, devicetree Cc: Rob Herring <robh+dt@kernel.org> Cc: Mark Rutland <mark.rutland@arm.com> Cc: devicetree@vger.kernel.org Signed-off-by: Hernán Gonzalez <hernan@vanguardiasur.com.ar> --- .../devicetree/bindings/staging/iio/cdc/ad7746.txt | 34 ++++++++++++++++++++++ 1 file changed, 34 insertions(+) create mode 100644 Documentation/devicetree/bindings/staging/iio/cdc/ad7746.txt diff --git a/Documentation/devicetree/bindings/staging/iio/cdc/ad7746.txt b/Documentation/devicetree/bindings/staging/iio/cdc/ad7746.txt new file mode 100644 index 0000000..7740f05 --- /dev/null +++ b/Documentation/devicetree/bindings/staging/iio/cdc/ad7746.txt @@ -0,0 +1,34 @@ +Analog Devices AD7746/5/7 capacitive sensor driver + +Required properties: + - compatible: Should be one of + * "adi,ad7745" + * "adi,ad7746" + * "adi,ad7747" + - reg: The 7-bits long I2c address of the device + +Optional properties: + - adi,exclvl: This property defines the excitation voltage level for the + capacitance to be measured. Possible values are: + * 0 = +-VDD/8 + * 1 = +-VDD/4 + * 2 = +-VDD * 3/8 + * 3 = +-VDD/2 (Default) + - adi,nexca_en: Invert excitation output A. + - adi,nexcb_en: Invert excitation output B. + +Example: +Here exclvl would be 1 (VDD/4), Excitation pin A would be inverted and +Excitation pin B would NOT be inverted. + +i2c2 { + + < . . . > + + ad7746: ad7746@60 { + compatible = "ad7746"; + reg = <0x60>; + adi,exclvl = <1>; + adi,nexca_en; + }; +}; -- 2.7.4 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2 11/14] staging: iio: ad7746: Add devicetree bindings documentation 2018-04-13 16:36 ` [PATCH v2 11/14] staging: iio: ad7746: Add devicetree bindings documentation Hernán Gonzalez @ 2018-04-15 15:37 ` Jonathan Cameron 2018-04-16 14:55 ` Hernán Gonzalez 2018-04-16 20:08 ` Rob Herring 1 sibling, 1 reply; 4+ messages in thread From: Jonathan Cameron @ 2018-04-15 15:37 UTC (permalink / raw) To: Hernán Gonzalez Cc: knaack.h, lars, pmeerw, gregkh, Michael.Hennerich, linux-iio, linux-kernel, Rob Herring, Mark Rutland, devicetree On Fri, 13 Apr 2018 13:36:48 -0300 Hernán Gonzalez <hernan@vanguardiasur.com.ar> wrote: > Cc: Rob Herring <robh+dt@kernel.org> > Cc: Mark Rutland <mark.rutland@arm.com> > Cc: devicetree@vger.kernel.org > Signed-off-by: Hernán Gonzalez <hernan@vanguardiasur.com.ar> A few comments inline. Thanks, Jonathan > --- > .../devicetree/bindings/staging/iio/cdc/ad7746.txt | 34 ++++++++++++++++++++++ > 1 file changed, 34 insertions(+) > create mode 100644 Documentation/devicetree/bindings/staging/iio/cdc/ad7746.txt > > diff --git a/Documentation/devicetree/bindings/staging/iio/cdc/ad7746.txt b/Documentation/devicetree/bindings/staging/iio/cdc/ad7746.txt > new file mode 100644 > index 0000000..7740f05 > --- /dev/null > +++ b/Documentation/devicetree/bindings/staging/iio/cdc/ad7746.txt > @@ -0,0 +1,34 @@ > +Analog Devices AD7746/5/7 capacitive sensor driver > + > +Required properties: > + - compatible: Should be one of > + * "adi,ad7745" > + * "adi,ad7746" > + * "adi,ad7747" > + - reg: The 7-bits long I2c address of the device > + > +Optional properties: > + - adi,exclvl: This property defines the excitation voltage level for the > + capacitance to be measured. Possible values are: > + * 0 = +-VDD/8 > + * 1 = +-VDD/4 > + * 2 = +-VDD * 3/8 > + * 3 = +-VDD/2 (Default) > + - adi,nexca_en: Invert excitation output A. The naming is rather odd. Why _en? > + - adi,nexcb_en: Invert excitation output B. > + > +Example: > +Here exclvl would be 1 (VDD/4), Excitation pin A would be inverted and > +Excitation pin B would NOT be inverted. > + > +i2c2 { > + > + < . . . > What does this spacer add to the example? Sure, there can be other devices, but no need to illustrate that here. > + > + ad7746: ad7746@60 { > + compatible = "ad7746"; > + reg = <0x60>; > + adi,exclvl = <1>; > + adi,nexca_en; > + }; > +}; ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2 11/14] staging: iio: ad7746: Add devicetree bindings documentation 2018-04-15 15:37 ` Jonathan Cameron @ 2018-04-16 14:55 ` Hernán Gonzalez 0 siblings, 0 replies; 4+ messages in thread From: Hernán Gonzalez @ 2018-04-16 14:55 UTC (permalink / raw) To: Jonathan Cameron Cc: knaack.h, lars, Peter Meerwald-Stadler, gregkh, Michael.Hennerich, linux-iio, linux-kernel, Rob Herring, Mark Rutland, devicetree On Sun, Apr 15, 2018 at 12:37 PM, Jonathan Cameron <jic23@kernel.org> wrote: > On Fri, 13 Apr 2018 13:36:48 -0300 > Hernán Gonzalez <hernan@vanguardiasur.com.ar> wrote: > >> Cc: Rob Herring <robh+dt@kernel.org> >> Cc: Mark Rutland <mark.rutland@arm.com> >> Cc: devicetree@vger.kernel.org >> Signed-off-by: Hernán Gonzalez <hernan@vanguardiasur.com.ar> > A few comments inline. > > Thanks, > > Jonathan > >> --- >> .../devicetree/bindings/staging/iio/cdc/ad7746.txt | 34 ++++++++++++++++++++++ >> 1 file changed, 34 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/staging/iio/cdc/ad7746.txt >> >> diff --git a/Documentation/devicetree/bindings/staging/iio/cdc/ad7746.txt b/Documentation/devicetree/bindings/staging/iio/cdc/ad7746.txt >> new file mode 100644 >> index 0000000..7740f05 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/staging/iio/cdc/ad7746.txt >> @@ -0,0 +1,34 @@ >> +Analog Devices AD7746/5/7 capacitive sensor driver >> + >> +Required properties: >> + - compatible: Should be one of >> + * "adi,ad7745" >> + * "adi,ad7746" >> + * "adi,ad7747" >> + - reg: The 7-bits long I2c address of the device >> + >> +Optional properties: >> + - adi,exclvl: This property defines the excitation voltage level for the >> + capacitance to be measured. Possible values are: >> + * 0 = +-VDD/8 >> + * 1 = +-VDD/4 >> + * 2 = +-VDD * 3/8 >> + * 3 = +-VDD/2 (Default) >> + - adi,nexca_en: Invert excitation output A. > > The naming is rather odd. Why _en? > Just sticking to the original pdata naming. I'll rename it on next series. >> + - adi,nexcb_en: Invert excitation output B. >> + >> +Example: >> +Here exclvl would be 1 (VDD/4), Excitation pin A would be inverted and >> +Excitation pin B would NOT be inverted. >> + >> +i2c2 { >> + >> + < . . . > > What does this spacer add to the example? > Sure, there can be other devices, but no need to illustrate that here. > >> + >> + ad7746: ad7746@60 { >> + compatible = "ad7746"; >> + reg = <0x60>; >> + adi,exclvl = <1>; >> + adi,nexca_en; >> + }; >> +}; > ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2 11/14] staging: iio: ad7746: Add devicetree bindings documentation 2018-04-13 16:36 ` [PATCH v2 11/14] staging: iio: ad7746: Add devicetree bindings documentation Hernán Gonzalez 2018-04-15 15:37 ` Jonathan Cameron @ 2018-04-16 20:08 ` Rob Herring 1 sibling, 0 replies; 4+ messages in thread From: Rob Herring @ 2018-04-16 20:08 UTC (permalink / raw) To: Hernán Gonzalez Cc: jic23, knaack.h, lars, pmeerw, gregkh, Michael.Hennerich, linux-iio, linux-kernel, Mark Rutland, devicetree On Fri, Apr 13, 2018 at 01:36:48PM -0300, Hernán Gonzalez wrote: Commit msg? > Cc: Rob Herring <robh+dt@kernel.org> > Cc: Mark Rutland <mark.rutland@arm.com> > Cc: devicetree@vger.kernel.org > Signed-off-by: Hernán Gonzalez <hernan@vanguardiasur.com.ar> > --- > .../devicetree/bindings/staging/iio/cdc/ad7746.txt | 34 ++++++++++++++++++++++ > 1 file changed, 34 insertions(+) > create mode 100644 Documentation/devicetree/bindings/staging/iio/cdc/ad7746.txt > > diff --git a/Documentation/devicetree/bindings/staging/iio/cdc/ad7746.txt b/Documentation/devicetree/bindings/staging/iio/cdc/ad7746.txt > new file mode 100644 > index 0000000..7740f05 > --- /dev/null > +++ b/Documentation/devicetree/bindings/staging/iio/cdc/ad7746.txt > @@ -0,0 +1,34 @@ > +Analog Devices AD7746/5/7 capacitive sensor driver > + > +Required properties: > + - compatible: Should be one of > + * "adi,ad7745" > + * "adi,ad7746" > + * "adi,ad7747" Alignment. > + - reg: The 7-bits long I2c address of the device > + > +Optional properties: > + - adi,exclvl: This property defines the excitation voltage level for the > + capacitance to be measured. Possible values are: > + * 0 = +-VDD/8 > + * 1 = +-VDD/4 > + * 2 = +-VDD * 3/8 > + * 3 = +-VDD/2 (Default) > + - adi,nexca_en: Invert excitation output A. > + - adi,nexcb_en: Invert excitation output B. s/_/-/ > + > +Example: > +Here exclvl would be 1 (VDD/4), Excitation pin A would be inverted and > +Excitation pin B would NOT be inverted. > + > +i2c2 { > + > + < . . . > > + > + ad7746: ad7746@60 { > + compatible = "ad7746"; > + reg = <0x60>; > + adi,exclvl = <1>; > + adi,nexca_en; > + }; > +}; > -- > 2.7.4 > ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-04-16 20:08 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1523637411-8531-1-git-send-email-hernan@vanguardiasur.com.ar>
2018-04-13 16:36 ` [PATCH v2 11/14] staging: iio: ad7746: Add devicetree bindings documentation Hernán Gonzalez
2018-04-15 15:37 ` Jonathan Cameron
2018-04-16 14:55 ` Hernán Gonzalez
2018-04-16 20:08 ` Rob Herring
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).