From: Jonathan Cameron <jic23@kernel.org>
To: Stefan Popa <stefan.popa@analog.com>
Cc: <robh+dt@kernel.org>, <mark.rutland@arm.com>, <knaack.h@gmx.de>,
<lars@metafoo.de>, <pmeerw@pmeerw.net>,
<Michael.Hennerich@analog.com>, <gregkh@linuxfoundation.org>,
<linux-iio@vger.kernel.org>, <devicetree@vger.kernel.org>,
<linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v4 4/4] dt-bindings: iio: adc: Add docs for ad7124
Date: Sun, 11 Nov 2018 12:19:54 +0000 [thread overview]
Message-ID: <20181111121954.145cbd2f@archlinux> (raw)
In-Reply-To: <1541778180-7862-1-git-send-email-stefan.popa@analog.com>
On Fri, 9 Nov 2018 17:43:00 +0200
Stefan Popa <stefan.popa@analog.com> wrote:
> Add support for Analog Devices AD7124 4-channels and 8-channels ADC.
>
> Signed-off-by: Stefan Popa <stefan.popa@analog.com>
Your example still includes the other things that I think you have now dropped.
gain and odr.
> ---
> Changes in v2:
> - Nothing changed.
> Changes in v3:
> - Removed the "adi,channels" property.
> - Used the "reg" property to get the channel number and "adi,diff-channels"
> for the differential pins. The "adi,channel-number" property was removed.
> - adi,bipolar is of boolean type.
> Changes in v4:
> - Used the bipolar and diff-channels properties defined in the new adc.txt doc.
>
> .../devicetree/bindings/iio/adc/adi,ad7124.txt | 81 ++++++++++++++++++++++
> MAINTAINERS | 1 +
> 2 files changed, 82 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7124.txt
>
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7124.txt b/Documentation/devicetree/bindings/iio/adc/adi,ad7124.txt
> new file mode 100644
> index 0000000..fa0c43b
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7124.txt
> @@ -0,0 +1,81 @@
> +Analog Devices AD7124 ADC device driver
> +
> +Required properties for the AD7124:
> + - compatible: Must be one of "adi,ad7124-4" or "adi,ad7124-8"
> + - reg: SPI chip select number for the device
> + - spi-max-frequency: Max SPI frequency to use
> + see: Documentation/devicetree/bindings/spi/spi-bus.txt
> + - clocks: phandle to the master clock (mclk)
> + see: Documentation/devicetree/bindings/clock/clock-bindings.txt
> + - clock-names: Must be "mclk".
> + - interrupts: IRQ line for the ADC
> + see: Documentation/devicetree/bindings/interrupt-controller/interrupts.txt
Given the driver doesn't currently use it, should perhaps be optional?
> +
> + Required properties:
> + * #address-cells: Must be 1.
> + * #size-cells: Must be 0.
> +
> + Subnode(s) represent the external channels which are connected to the ADC.
> + Each subnode represents one channel and has the following properties:
> + Required properties:
> + * reg: The channel number. It can have up to 4 channels on ad7124-4
> + and 8 channels on ad7124-8, numbered from 0 to 15.
> + * diff-channels: see: Documentation/devicetree/bindings/iio/adc/adc.txt
> +
> + Optional properties:
> + * bipolar: see: Documentation/devicetree/bindings/iio/adc/adc.txt
> + * adi,reference-select: Select the reference source to use when
> + converting on the the specific channel. Valid values are:
> + 0: REFIN1(+)/REFIN1(−).
> + 1: REFIN2(+)/REFIN2(−).
> + 3: AVDD
> + If this field is left empty, internal reference is selected.
> +
> +Optional properties:
> + - refin1-supply: refin1 supply can be used as reference for conversion.
> + - refin2-supply: refin2 supply can be used as reference for conversion.
> + - avdd-supply: avdd supply can be used as reference for conversion.
> +
> +Example:
> + adc@0 {
> + compatible = "adi,ad7124-4";
> + reg = <0>;
> + spi-max-frequency = <5000000>;
> + interrupts = <25 2>;
> + interrupt-parent = <&gpio>;
> + refin1-supply = <&adc_vref>;
> + clocks = <&ad7124_mclk>;
> + clock-names = "mclk";
> +
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + channel@0 {
> + reg = <0>;
> + adi,diff-channels = <0 1>;
> + adi,reference-select = <0>;
> + adi,gain = <2>;
> + adi,odr-hz = <10>;
I think you have dropped these two..
> + };
> +
> + channel@1 {
> + reg = <1>;
> + adi,bipolar;
> + adi,diff-channels = <2 3>;
> + adi,reference-select = <0>;
> + adi,gain = <4>;
> + adi,odr-hz = <50>;
> + };
> +
> + channel@2 {
> + reg = <2>;
> + adi,diff-channels = <4 5>;
> + adi,gain = <128>;
> + adi,odr-hz = <19200>;
> + };
> +
> + channel@3 {
> + reg = <3>;
> + adi,diff-channels = <6 7>;
> + };
> + };
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 3a1bfcb..f2fa508 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -845,6 +845,7 @@ L: linux-iio@vger.kernel.org
> W: http://ez.analog.com/community/linux-device-drivers
> S: Supported
> F: drivers/iio/adc/ad7124.c
> +F: Documentation/devicetree/bindings/iio/adc/adi,ad7124.txt
>
> ANALOG DEVICES INC AD9389B DRIVER
> M: Hans Verkuil <hans.verkuil@cisco.com>
next prev parent reply other threads:[~2018-11-11 12:20 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-11-09 15:43 [PATCH v4 4/4] dt-bindings: iio: adc: Add docs for ad7124 Stefan Popa
2018-11-11 12:19 ` Jonathan Cameron [this message]
2018-11-12 9:30 ` Popa, Stefan Serban
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20181111121954.145cbd2f@archlinux \
--to=jic23@kernel.org \
--cc=Michael.Hennerich@analog.com \
--cc=devicetree@vger.kernel.org \
--cc=gregkh@linuxfoundation.org \
--cc=knaack.h@gmx.de \
--cc=lars@metafoo.de \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=pmeerw@pmeerw.net \
--cc=robh+dt@kernel.org \
--cc=stefan.popa@analog.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).