Linux-IIO Archive on lore.kernel.org
 help / Atom feed
* [PATCH 2/2] dt-bindings: iio: adc: Add docs for AD7768-1
@ 2019-01-07 13:26 Stefan Popa
  2019-01-12 18:13 ` Jonathan Cameron
  0 siblings, 1 reply; 2+ messages in thread
From: Stefan Popa @ 2019-01-07 13:26 UTC (permalink / raw)
  To: jic23, robh+dt
  Cc: mark.rutland, knaack.h, lars, pmeerw, Michael.Hennerich, gregkh,
	linux-iio, devicetree, linux-kernel, stefan.popa

Document support for AD7768-1 Analog to Digital Converter.

Signed-off-by: Stefan Popa <stefan.popa@analog.com>
---
 .../devicetree/bindings/iio/adc/adi,ad7768-1.txt   | 34 ++++++++++++++++++++++
 MAINTAINERS                                        |  1 +
 2 files changed, 35 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.txt

diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.txt b/Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.txt
new file mode 100644
index 0000000..bf619fa8
--- /dev/null
+++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.txt
@@ -0,0 +1,34 @@
+Analog Devices AD7768-1 ADC device driver
+
+Required properties for the AD7768-1:
+
+- compatible: Must be "adi,ad7768-1"
+- 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
+- vref-supply: vref supply can be used as reference for conversion
+- adi,sync-in-gpios: must be the device tree identifier of the SYNC-IN pin. A pulse
+	is always required if the configuration is changed in any way, for example
+	if the filter decimation rate changes. As the line is active low, it should
+	be marked GPIO_ACTIVE_LOW.
+
+Example:
+
+	adc@0 {
+		compatible = "adi,ad7768-1";
+		reg = <0>;
+		spi-max-frequency = <2000000>;
+		spi-cpol;
+		spi-cpha;
+		vref-supply = <&adc_vref>;
+		interrupts = <25 IRQ_TYPE_EDGE_RISING>;
+		interrupt-parent = <&gpio>;
+		adi,sync-in-gpios = <&gpio 22 GPIO_ACTIVE_LOW>;
+		clocks = <&ad7768_mclk>;
+		clock-names = "mclk";
+	};
diff --git a/MAINTAINERS b/MAINTAINERS
index 3ba3811..e5613b6 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -868,6 +868,7 @@ L:	linux-iio@vger.kernel.org
 W:	http://ez.analog.com/community/linux-device-drivers
 S:	Supported
 F:	drivers/iio/adc/ad7768-1.c
+F:	Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.txt
 
 ANALOG DEVICES INC AD9389B DRIVER
 M:	Hans Verkuil <hans.verkuil@cisco.com>
-- 
2.7.4


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH 2/2] dt-bindings: iio: adc: Add docs for AD7768-1
  2019-01-07 13:26 [PATCH 2/2] dt-bindings: iio: adc: Add docs for AD7768-1 Stefan Popa
@ 2019-01-12 18:13 ` Jonathan Cameron
  0 siblings, 0 replies; 2+ messages in thread
From: Jonathan Cameron @ 2019-01-12 18:13 UTC (permalink / raw)
  To: Stefan Popa
  Cc: robh+dt, mark.rutland, knaack.h, lars, pmeerw, Michael.Hennerich,
	gregkh, linux-iio, devicetree, linux-kernel

On Mon, 7 Jan 2019 15:26:49 +0200
Stefan Popa <stefan.popa@analog.com> wrote:

> Document support for AD7768-1 Analog to Digital Converter.
> 
> Signed-off-by: Stefan Popa <stefan.popa@analog.com>

A few comments inline.

J
> ---
>  .../devicetree/bindings/iio/adc/adi,ad7768-1.txt   | 34 ++++++++++++++++++++++
>  MAINTAINERS                                        |  1 +
>  2 files changed, 35 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.txt
> 
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.txt b/Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.txt
> new file mode 100644
> index 0000000..bf619fa8
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.txt
> @@ -0,0 +1,34 @@
> +Analog Devices AD7768-1 ADC device driver
> +
> +Required properties for the AD7768-1:
> +
> +- compatible: Must be "adi,ad7768-1"
> +- 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
> +- vref-supply: vref supply can be used as reference for conversion
> +- adi,sync-in-gpios: must be the device tree identifier of the SYNC-IN pin. A pulse
> +	is always required if the configuration is changed in any way, for example
> +	if the filter decimation rate changes. As the line is active low, it should
> +	be marked GPIO_ACTIVE_LOW.
Should mention it's also used for simultaneous sampling.  The name makes that
kind of obvious, but then the comment doesn't mention it.

There are some other potential gpios it seems.  Good to get them in the binding
now even if we don't do anything much with them.  Reset definitely.  The others
maybe not as we are only supporting the spi control mode and I can't see why
we would want to do anything different. I suppose there might be boards wired
with it set to pin mode but hopefully not.

> +
> +Example:
> +
> +	adc@0 {
> +		compatible = "adi,ad7768-1";
> +		reg = <0>;
> +		spi-max-frequency = <2000000>;
> +		spi-cpol;
> +		spi-cpha;
> +		vref-supply = <&adc_vref>;
> +		interrupts = <25 IRQ_TYPE_EDGE_RISING>;
> +		interrupt-parent = <&gpio>;
> +		adi,sync-in-gpios = <&gpio 22 GPIO_ACTIVE_LOW>;
> +		clocks = <&ad7768_mclk>;
> +		clock-names = "mclk";
> +	};
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 3ba3811..e5613b6 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -868,6 +868,7 @@ L:	linux-iio@vger.kernel.org
>  W:	http://ez.analog.com/community/linux-device-drivers
>  S:	Supported
>  F:	drivers/iio/adc/ad7768-1.c
> +F:	Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.txt
>  
>  ANALOG DEVICES INC AD9389B DRIVER
>  M:	Hans Verkuil <hans.verkuil@cisco.com>


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, back to index

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-01-07 13:26 [PATCH 2/2] dt-bindings: iio: adc: Add docs for AD7768-1 Stefan Popa
2019-01-12 18:13 ` Jonathan Cameron

Linux-IIO Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-iio/0 linux-iio/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-iio linux-iio/ https://lore.kernel.org/linux-iio \
		linux-iio@vger.kernel.org linux-iio@archiver.kernel.org
	public-inbox-index linux-iio


Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-iio


AGPL code for this site: git clone https://public-inbox.org/ public-inbox