Received: by 2002:a25:ab43:0:0:0:0:0 with SMTP id u61csp2882393ybi; Sun, 26 May 2019 09:41:48 -0700 (PDT) X-Google-Smtp-Source: APXvYqzkl0HyqHJbw+uqdot6V8DpAJT33wV38VmMbcIsg9pomw9keT0gYpHu4BAxNAyc5ijn6dd0 X-Received: by 2002:a62:386:: with SMTP id 128mr105561157pfd.10.1558888908459; Sun, 26 May 2019 09:41:48 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1558888908; cv=none; d=google.com; s=arc-20160816; b=rkJ1MowNoukKsIa6SUAAoA35jmAc1lwnDyFSHFt76PF6rGoHnsJ5Btcmw/op4l59PH KCHs+sgKOJaCWeDDVwP9tixENl0VHRkWXBRlKIZE8cVBoGuAUIdHcaP0uC93875CEUEA XxtoavK496hvQ1pz9p9AdMiAszPBVSsTGgAsduDr35Imwr/MeXNAa7W5itHWo8MoQu9E zSM1A/Mna0PXWQQruw+VSoEN/be5+4LAMdNhBKiZZOyz2dqBEISjkUbVZkKmamh0DeWb MaucrRJVxK0B1DqNyQACVKeFJbh/qsBfhdIAJRb298ODvcqKQ/hYyajObGUGJ/LVGUBi 5hag== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :references:in-reply-to:message-id:subject:cc:to:from:date :dkim-signature; bh=14n6dshfjWPAxXlzkBFLuBtRPxKZM0UiIwfOqvz35L4=; b=Ach15KAjXzxyusN/AbgjmD5V8JEJsw/7sXVq33lYXV1uRvy0vZL6dFKuPsiHBJEVoK mJnl5aPdxnuDQIqt5ohjlXIjhjIdS83H9NxRRyTAJ0i+vju/0EC6B+BQTL2zuq5krYvl u+7ss795bs06BU1ZJBYey8/6EbIo7zU7sCGBVXve3QOO0KEFtJ5VUTr/aQKqFuIrwsfr U2Xw4HcJbY6kcdIhnqSX8qmlsdv/Vg///f7EFbruZH5NnVbkGtiVTXj4vCL9VMrO5uRG ODkEQNfqLoph2VBFxD70PUcVUV2wkjvtF0tqXMgMsqvVgTmPKGN3Usxc+lA45CGmH4zi r8KQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=bTydKIUT; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id b12si13794755pgl.77.2019.05.26.09.41.31; Sun, 26 May 2019 09:41:48 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=bTydKIUT; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727940AbfEZQjT (ORCPT + 99 others); Sun, 26 May 2019 12:39:19 -0400 Received: from mail.kernel.org ([198.145.29.99]:40436 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726622AbfEZQjT (ORCPT ); Sun, 26 May 2019 12:39:19 -0400 Received: from archlinux (cpc149474-cmbg20-2-0-cust94.5-4.cable.virginm.net [82.4.196.95]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id D3A542075B; Sun, 26 May 2019 16:39:14 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1558888757; bh=ZRGTIVYaHYRt51w0uSjhKU/386SEDJ1jWuPg7ckiXDM=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=bTydKIUT2J0nOZF9/pmjOFRvdPIeF8Zg5xRFX+NrRtEW4tVnO3Np8pk3ljl3h66AN U2AiOtugeZE+7vTYGJWNnVWDSOKSgdV77/VKBrYkp/H/UNPPnLuQWNAzKFs6QqcRJX BTb3zpSgztXgw2G1sECkK1EWvtArIhMD4zgpXdAo= Date: Sun, 26 May 2019 17:39:11 +0100 From: Jonathan Cameron To: Renato Lui Geh Cc: 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 Message-ID: <20190526173911.57ae3d11@archlinux> In-Reply-To: <2426649b2d8224ae72e7706bcb8c4f2c44c581d2.1558746978.git.renatogeh@gmail.com> References: <2426649b2d8224ae72e7706bcb8c4f2c44c581d2.1558746978.git.renatogeh@gmail.com> X-Mailer: Claws Mail 3.17.3 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 24 May 2019 22:26:30 -0300 Renato Lui Geh 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 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! 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 > + > +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 > + 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>; > + }; > + };