From: Renato Lui Geh <renatogeh@gmail.com>
To: Jonathan Cameron <jic23@kernel.org>
Cc: Renato Lui Geh <renatogeh@gmail.com>,
lars@metafoo.de, Michael.Hennerich@analog.com, knaack.h@gmx.de,
pmeerw@pmeerw.net, gregkh@linuxfoundation.org,
stefan.popa@analog.com, alexandru.Ardelean@analog.com,
robh+dt@kernel.org, mark.rutland@arm.com,
linux-iio@vger.kernel.org, devel@driverdev.osuosl.org,
linux-kernel@vger.kernel.org, kernel-usp@googlegroups.com,
devicetree@vger.kernel.org
Subject: Re: [PATCH v2 1/2] dt-bindings: iio: adc: add adi,ad7780.yaml binding
Date: Wed, 5 Jun 2019 17:35:56 -0300 [thread overview]
Message-ID: <20190605203554.podktlonhp527iqq@renatolg> (raw)
In-Reply-To: <20190526173911.57ae3d11@archlinux>
On 05/26, Jonathan Cameron wrote:
>On Fri, 24 May 2019 22:26:30 -0300
>Renato Lui Geh <renatogeh@gmail.com> wrote:
>
>> This patch adds a YAML binding for the Analog Devices AD7780/1 and
>> AD7170/1 analog-to-digital converters.
>>
>> Signed-off-by: Renato Lui Geh <renatogeh@gmail.com>
>Looks good to me, but I'm still finding my feet with these so will
>leave it for a few days for others to have time to comment.
>
>Michael, looking for a quick reply from you to say if you are happy
>being explicitly listed as maintainer for this one, or if you'd
>rather land it on someone else. Same applies for patch 2.
>
>Renato, if I seem to have forgotten this in a week or so, feel
>free to give me a poke. I've been known to loose patches entirely!
Hi Jonathan,
Just here to give you a poke. :)
By the way, in these cases, which would be easier for you? To send you
an email like I'm doing right now on last week's thread; or to resend
the entire patch(set)?
Thanks,
Renato
>
>Thanks,
>
>Jonathan
>> ---
>> Changes in v2:
>> - vref-supply to avdd-supply
>> - remove avdd-supply from required list
>> - include adc block in an spi block
>>
>> .../bindings/iio/adc/adi,ad7780.txt | 48 ----------
>> .../bindings/iio/adc/adi,ad7780.yaml | 87 +++++++++++++++++++
>> 2 files changed, 87 insertions(+), 48 deletions(-)
>> delete mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
>> create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
>>
>> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
>> deleted file mode 100644
>> index 440e52555349..000000000000
>> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
>> +++ /dev/null
>> @@ -1,48 +0,0 @@
>> -* Analog Devices AD7170/AD7171/AD7780/AD7781
>> -
>> -Data sheets:
>> -
>> -- AD7170:
>> - * https://www.analog.com/media/en/technical-documentation/data-sheets/AD7170.pdf
>> -- AD7171:
>> - * https://www.analog.com/media/en/technical-documentation/data-sheets/AD7171.pdf
>> -- AD7780:
>> - * https://www.analog.com/media/en/technical-documentation/data-sheets/ad7780.pdf
>> -- AD7781:
>> - * https://www.analog.com/media/en/technical-documentation/data-sheets/AD7781.pdf
>> -
>> -Required properties:
>> -
>> -- compatible: should be one of
>> - * "adi,ad7170"
>> - * "adi,ad7171"
>> - * "adi,ad7780"
>> - * "adi,ad7781"
>> -- reg: spi chip select number for the device
>> -- vref-supply: the regulator supply for the ADC reference voltage
>> -
>> -Optional properties:
>> -
>> -- powerdown-gpios: must be the device tree identifier of the PDRST pin. If
>> - specified, it will be asserted during driver probe. As the
>> - line is active high, it should be marked GPIO_ACTIVE_HIGH.
>> -- adi,gain-gpios: must be the device tree identifier of the GAIN pin. Only for
>> - the ad778x chips. If specified, it will be asserted during
>> - driver probe. As the line is active low, it should be marked
>> - GPIO_ACTIVE_LOW.
>> -- adi,filter-gpios: must be the device tree identifier of the FILTER pin. Only
>> - for the ad778x chips. If specified, it will be asserted
>> - during driver probe. As the line is active low, it should be
>> - marked GPIO_ACTIVE_LOW.
>> -
>> -Example:
>> -
>> -adc@0 {
>> - compatible = "adi,ad7780";
>> - reg = <0>;
>> - vref-supply = <&vdd_supply>
>> -
>> - powerdown-gpios = <&gpio 12 GPIO_ACTIVE_HIGH>;
>> - adi,gain-gpios = <&gpio 5 GPIO_ACTIVE_LOW>;
>> - adi,filter-gpios = <&gpio 15 GPIO_ACTIVE_LOW>;
>> -};
>> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
>> new file mode 100644
>> index 000000000000..d1109416963c
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
>> @@ -0,0 +1,87 @@
>> +# SPDX-License-Identifier: GPL-2.0
>> +%YAML 1.2
>> +---
>> +$id: http://devicetree.org/schemas/iio/adc/adi,ad7780.yaml#
>> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>> +
>> +title: Analog Devices AD7170/AD7171/AD7780/AD7781 analog to digital converters
>> +
>> +maintainers:
>> + - Michael Hennerich <michael.hennerich@analog.com>
>> +
>> +description: |
>> + The ad7780 is a sigma-delta analog to digital converter. This driver provides
>> + reading voltage values and status bits from both the ad778x and ad717x series.
>> + Its interface also allows writing on the FILTER and GAIN GPIO pins on the
>> + ad778x.
>> +
>> + Specifications on the converters can be found at:
>> + AD7170:
>> + https://www.analog.com/media/en/technical-documentation/data-sheets/AD7170.pdf
>> + AD7171:
>> + https://www.analog.com/media/en/technical-documentation/data-sheets/AD7171.pdf
>> + AD7780:
>> + https://www.analog.com/media/en/technical-documentation/data-sheets/ad7780.pdf
>> + AD7781:
>> + https://www.analog.com/media/en/technical-documentation/data-sheets/AD7781.pdf
>> +
>> +properties:
>> + compatible:
>> + enum:
>> + - adi,ad7170
>> + - adi,ad7171
>> + - adi,ad7780
>> + - adi,ad7781
>> +
>> + reg:
>> + maxItems: 1
>> +
>> + avdd-supply:
>> + description:
>> + The regulator supply for the ADC reference voltage.
>> + maxItems: 1
>> +
>> + powerdown-gpios:
>> + description:
>> + Must be the device tree identifier of the PDRST pin. If
>> + specified, it will be asserted during driver probe. As the
>> + line is active high, it should be marked GPIO_ACTIVE_HIGH.
>> + maxItems: 1
>> +
>> + adi,gain-gpios:
>> + description:
>> + Must be the device tree identifier of the GAIN pin. Only for
>> + the ad778x chips. If specified, it will be asserted during
>> + driver probe. As the line is active low, it should be marked
>> + GPIO_ACTIVE_LOW.
>> + maxItems: 1
>> +
>> + adi,filter-gpios:
>> + description:
>> + Must be the device tree identifier of the FILTER pin. Only
>> + for the ad778x chips. If specified, it will be asserted
>> + during driver probe. As the line is active low, it should be
>> + marked GPIO_ACTIVE_LOW.
>> + maxItems: 1
>> +
>> +required:
>> + - compatible
>> + - reg
>> +
>> +examples:
>> + - |
>> + #include <dt-bindings/gpio/gpio.h>
>> + spi0 {
>> + #address-cells = <1>;
>> + #size-cells = <0>;
>> +
>> + adc@0 {
>> + compatible = "adi,ad7780";
>> + reg = <0>;
>> +
>> + avdd-supply = <&vdd_supply>;
>> + powerdown-gpios = <&gpio0 12 GPIO_ACTIVE_HIGH>;
>> + adi,gain-gpios = <&gpio1 5 GPIO_ACTIVE_LOW>;
>> + adi,filter-gpios = <&gpio2 15 GPIO_ACTIVE_LOW>;
>> + };
>> + };
>
next prev parent reply other threads:[~2019-06-05 20:36 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-05-25 1:26 [PATCH v2 0/2] dt-bindings: iio: adc: add ad7780 yaml and MAINTAINERS entry Renato Lui Geh
2019-05-25 1:26 ` [PATCH v2 1/2] dt-bindings: iio: adc: add adi,ad7780.yaml binding Renato Lui Geh
2019-05-26 16:39 ` Jonathan Cameron
2019-05-27 6:29 ` Hennerich, Michael
2019-06-05 20:35 ` Renato Lui Geh [this message]
2019-06-06 11:13 ` Ardelean, Alexandru
2019-06-06 15:52 ` Jonathan Cameron
2019-06-08 11:18 ` Jonathan Cameron
2019-06-14 19:43 ` Rob Herring
2019-06-16 15:45 ` Jonathan Cameron
2019-05-25 1:26 ` [PATCH v2 2/2] MAINTAINERS: add entry for ad7780 adc driver Renato Lui Geh
2019-06-08 11:20 ` Jonathan Cameron
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=20190605203554.podktlonhp527iqq@renatolg \
--to=renatogeh@gmail.com \
--cc=Michael.Hennerich@analog.com \
--cc=alexandru.Ardelean@analog.com \
--cc=devel@driverdev.osuosl.org \
--cc=devicetree@vger.kernel.org \
--cc=gregkh@linuxfoundation.org \
--cc=jic23@kernel.org \
--cc=kernel-usp@googlegroups.com \
--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).