diff for duplicates of <20181201170046.13eef1fc@archlinux> diff --git a/a/1.txt b/N1/1.txt index e8f5da7..03508be 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -2,7 +2,7 @@ On Thu, 29 Nov 2018 18:57:43 +0200 Stefan Popa <stefan.popa@analog.com> wrote: > Document support for AD7606 Analog to Digital Converter. ->=20 +> > Signed-off-by: Stefan Popa <stefan.popa@analog.com> Looks good to me, but as it's a non trivial binding in complexity I'd definitely like a DT maintainers input before applying this. @@ -19,16 +19,13 @@ Jonathan > and range-gpios. > Changes in v3: > - Added adi,oversampling-ratio-gpios. ->=20 -> .../devicetree/bindings/iio/adc/adi,ad7606.txt | 54 ++++++++++++++++= -++++++ +> +> .../devicetree/bindings/iio/adc/adi,ad7606.txt | 54 ++++++++++++++++++++++ > MAINTAINERS | 1 + > 2 files changed, 55 insertions(+) -> create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7606.= -txt ->=20 -> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7606.txt b/D= -ocumentation/devicetree/bindings/iio/adc/adi,ad7606.txt +> create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7606.txt +> +> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7606.txt b/Documentation/devicetree/bindings/iio/adc/adi,ad7606.txt > new file mode 100644 > index 0000000..02fae23 > --- /dev/null @@ -49,53 +46,44 @@ ocumentation/devicetree/bindings/iio/adc/adi,ad7606.txt > +- spi-cpha: See Documentation/devicetree/bindings/spi/spi-bus.txt > +- avcc-supply: phandle to the Avcc power supply > +- interrupts: IRQ line for the ADC -> + see: Documentation/devicetree/bindings/interrupt-controller/interrupts.= -txt -> +- adi,conversion-start-gpios: must be the device tree identifier of the = -CONVST pin. +> + see: Documentation/devicetree/bindings/interrupt-controller/interrupts.txt +> +- adi,conversion-start-gpios: must be the device tree identifier of the CONVST pin. > + This logic input is used to initiate conversions on the analog > + input channels. > + > +Optional properties: > + -> +- reset-gpios: must be the device tree identifier of the RESET pin. If s= -pecified, +> +- reset-gpios: must be the device tree identifier of the RESET pin. If specified, > + it will be asserted during driver probe. -> +- standby-gpios: must be the device tree identifier of the STBY pin. Thi= -s pin is used +> +- standby-gpios: must be the device tree identifier of the STBY pin. This pin is used > + to place the AD7606 into one of two power-down modes, Standby mode or > + Shutdown mode. -> +- adi,first-data-gpios: must be the device tree identifier of the FRSTDA= -TA pin. +> +- adi,first-data-gpios: must be the device tree identifier of the FRSTDATA pin. > + The FRSTDATA output indicates when the first channel, V1, is > + being read back on either the parallel, byte or serial interface. -> +- adi,range-gpios: must be the device tree identifier of the RANGE pin. = -The polarity on -> + this pin determines the input range of the analog input channels.= - If -> + this pin is tied to a logic high, the analog input range is =C2= -=B110V for -> + all channels. If this pin is tied to a logic low, the analog inpu= -t range -> + is =C2=B15V for all channels. +> +- adi,range-gpios: must be the device tree identifier of the RANGE pin. The polarity on +> + this pin determines the input range of the analog input channels. If +> + this pin is tied to a logic high, the analog input range is ±10V for +> + all channels. If this pin is tied to a logic low, the analog input range +> + is ±5V for all channels. > + > +Example: > + > + adc@0 { -> + compatible =3D "adi,ad7606-8"; -> + reg =3D <0>; -> + spi-max-frequency =3D <1000000>; +> + compatible = "adi,ad7606-8"; +> + reg = <0>; +> + spi-max-frequency = <1000000>; > + spi-cpol; > + -> + avcc-supply =3D <&adc_vref>; +> + avcc-supply = <&adc_vref>; > + -> + interrupts =3D <25 IRQ_TYPE_EDGE_FALLING>; -> + interrupt-parent =3D <&gpio>; +> + interrupts = <25 IRQ_TYPE_EDGE_FALLING>; +> + interrupt-parent = <&gpio>; > + -> + adi,conversion-start-gpios =3D <&gpio 17 0>; -> + reset-gpios =3D <&gpio 27 0>; -> + adi,first-data-gpios =3D <&gpio 22 0>; -> + standby-gpios =3D <&gpio 24 0>; +> + adi,conversion-start-gpios = <&gpio 17 0>; +> + reset-gpios = <&gpio 27 0>; +> + adi,first-data-gpios = <&gpio 22 0>; +> + standby-gpios = <&gpio 24 0>; > + }; > diff --git a/MAINTAINERS b/MAINTAINERS > index 843545d..6d63db4 100644 @@ -106,6 +94,6 @@ t range > S: Supported > F: drivers/iio/adc/ad7606.c > +F: Documentation/devicetree/bindings/iio/adc/ad7606.txt -> =20 +> > ANALOG DEVICES INC AD9389B DRIVER > M: Hans Verkuil <hans.verkuil@cisco.com> diff --git a/a/content_digest b/N1/content_digest index 17ff496..1c3c43b 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -19,7 +19,7 @@ "Stefan Popa <stefan.popa@analog.com> wrote:\n" "\n" "> Document support for AD7606 Analog to Digital Converter.\n" - ">=20\n" + "> \n" "> Signed-off-by: Stefan Popa <stefan.popa@analog.com>\n" "Looks good to me, but as it's a non trivial binding in complexity I'd\n" "definitely like a DT maintainers input before applying this.\n" @@ -36,16 +36,13 @@ "> \tand range-gpios.\n" "> Changes in v3:\n" "> \t- Added adi,oversampling-ratio-gpios.\n" - ">=20\n" - "> .../devicetree/bindings/iio/adc/adi,ad7606.txt | 54 ++++++++++++++++=\n" - "++++++\n" + "> \n" + "> .../devicetree/bindings/iio/adc/adi,ad7606.txt | 54 ++++++++++++++++++++++\n" "> MAINTAINERS | 1 +\n" "> 2 files changed, 55 insertions(+)\n" - "> create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7606.=\n" - "txt\n" - ">=20\n" - "> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7606.txt b/D=\n" - "ocumentation/devicetree/bindings/iio/adc/adi,ad7606.txt\n" + "> create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7606.txt\n" + "> \n" + "> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7606.txt b/Documentation/devicetree/bindings/iio/adc/adi,ad7606.txt\n" "> new file mode 100644\n" "> index 0000000..02fae23\n" "> --- /dev/null\n" @@ -66,53 +63,44 @@ "> +- spi-cpha: See Documentation/devicetree/bindings/spi/spi-bus.txt\n" "> +- avcc-supply: phandle to the Avcc power supply\n" "> +- interrupts: IRQ line for the ADC\n" - "> +\tsee: Documentation/devicetree/bindings/interrupt-controller/interrupts.=\n" - "txt\n" - "> +- adi,conversion-start-gpios: must be the device tree identifier of the =\n" - "CONVST pin.\n" + "> +\tsee: Documentation/devicetree/bindings/interrupt-controller/interrupts.txt\n" + "> +- adi,conversion-start-gpios: must be the device tree identifier of the CONVST pin.\n" "> +\t\t This logic input is used to initiate conversions on the analog\n" "> +\t\t input channels.\n" "> +\n" "> +Optional properties:\n" "> +\n" - "> +- reset-gpios: must be the device tree identifier of the RESET pin. If s=\n" - "pecified,\n" + "> +- reset-gpios: must be the device tree identifier of the RESET pin. If specified,\n" "> +\t it will be asserted during driver probe.\n" - "> +- standby-gpios: must be the device tree identifier of the STBY pin. Thi=\n" - "s pin is used\n" + "> +- standby-gpios: must be the device tree identifier of the STBY pin. This pin is used\n" "> +\t\tto place the AD7606 into one of two power-down modes, Standby mode or\n" "> +\t\tShutdown mode.\n" - "> +- adi,first-data-gpios: must be the device tree identifier of the FRSTDA=\n" - "TA pin.\n" + "> +- adi,first-data-gpios: must be the device tree identifier of the FRSTDATA pin.\n" "> +\t\t The FRSTDATA output indicates when the first channel, V1, is\n" "> +\t\t being read back on either the parallel, byte or serial interface.\n" - "> +- adi,range-gpios: must be the device tree identifier of the RANGE pin. =\n" - "The polarity on\n" - "> +\t this pin determines the input range of the analog input channels.=\n" - " If\n" - "> +\t this pin is tied to a logic high, the analog input range is =C2=\n" - "=B110V for\n" - "> +\t all channels. If this pin is tied to a logic low, the analog inpu=\n" - "t range\n" - "> +\t is =C2=B15V for all channels.\n" + "> +- adi,range-gpios: must be the device tree identifier of the RANGE pin. The polarity on\n" + "> +\t this pin determines the input range of the analog input channels. If\n" + "> +\t this pin is tied to a logic high, the analog input range is \302\26110V for\n" + "> +\t all channels. If this pin is tied to a logic low, the analog input range\n" + "> +\t is \302\2615V for all channels.\n" "> +\n" "> +Example:\n" "> +\n" "> +\tadc@0 {\n" - "> +\t\tcompatible =3D \"adi,ad7606-8\";\n" - "> +\t\treg =3D <0>;\n" - "> +\t\tspi-max-frequency =3D <1000000>;\n" + "> +\t\tcompatible = \"adi,ad7606-8\";\n" + "> +\t\treg = <0>;\n" + "> +\t\tspi-max-frequency = <1000000>;\n" "> +\t\tspi-cpol;\n" "> +\n" - "> +\t\tavcc-supply =3D <&adc_vref>;\n" + "> +\t\tavcc-supply = <&adc_vref>;\n" "> +\n" - "> +\t\tinterrupts =3D <25 IRQ_TYPE_EDGE_FALLING>;\n" - "> +\t\tinterrupt-parent =3D <&gpio>;\n" + "> +\t\tinterrupts = <25 IRQ_TYPE_EDGE_FALLING>;\n" + "> +\t\tinterrupt-parent = <&gpio>;\n" "> +\n" - "> +\t\tadi,conversion-start-gpios =3D <&gpio 17 0>;\n" - "> +\t\treset-gpios =3D <&gpio 27 0>;\n" - "> +\t\tadi,first-data-gpios =3D <&gpio 22 0>;\n" - "> +\t\tstandby-gpios =3D <&gpio 24 0>;\n" + "> +\t\tadi,conversion-start-gpios = <&gpio 17 0>;\n" + "> +\t\treset-gpios = <&gpio 27 0>;\n" + "> +\t\tadi,first-data-gpios = <&gpio 22 0>;\n" + "> +\t\tstandby-gpios = <&gpio 24 0>;\n" "> +\t};\n" "> diff --git a/MAINTAINERS b/MAINTAINERS\n" "> index 843545d..6d63db4 100644\n" @@ -123,8 +111,8 @@ "> S:\tSupported\n" "> F:\tdrivers/iio/adc/ad7606.c\n" "> +F:\tDocumentation/devicetree/bindings/iio/adc/ad7606.txt\n" - "> =20\n" + "> \n" "> ANALOG DEVICES INC AD9389B DRIVER\n" "> M:\tHans Verkuil <hans.verkuil@cisco.com>" -f067a4b8933ce540a1a0c679ea719828e3383379bf003164a5e1f8241b25729f +eccae762235bcb34618dfad122176470e6fbd0364226e62da22db944cd9f8c5d
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox