From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755485AbcLNKfZ (ORCPT ); Wed, 14 Dec 2016 05:35:25 -0500 Received: from mout.kundenserver.de ([212.227.126.131]:61475 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754789AbcLNKfY (ORCPT ); Wed, 14 Dec 2016 05:35:24 -0500 Date: Wed, 14 Dec 2016 11:34:55 +0100 From: Andreas Klinger To: Lars-Peter Clausen Cc: devicetree@vger.kernel.org, linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org, robh+dt@kernel.org, pawel.moll@arm.com, mark.rutland@arm.com, ijc+devicetree@hellion.org.uk, galak@codeaurora.org, jic23@kernel.org, knaack.h@gmx.de, pmeerw@pmeerw.net Subject: Re: [PATCH v2 1/2] iio: adc: hx711: Add DT binding for avia,hx711 Message-ID: <20161214103455.GA27362@andreas> References: <20161214095945.GA26511@andreas> <470f0017-2607-d5bd-bcd3-69f892cdc88b@metafoo.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <470f0017-2607-d5bd-bcd3-69f892cdc88b@metafoo.de> User-Agent: Mutt/1.5.23 (2014-03-12) X-Provags-ID: V03:K0:6U8Ily/PreK3pKzOgGeg92bR4kVgseININleH9urPDcmpFfVheE 80/2XB44wIQv3dLTDp2vbrGCHwhHHWl5EVUHl1utByHzNm8PmkniHN0S8VLYirJdoDZKDKN lyeVQyAq7iceeuMZoSZ5GFUoqIPtPXMEZ0hiGRGRAqI/MqyFYMHpDmbEPTxROZ25sdbPTlP kXVMc1dkN6KPW5atKiiQw== X-UI-Out-Filterresults: notjunk:1;V01:K0:EwQYODxW/ig=:8YSplgycKKoPMTtyBjdnXg CaK43ThqfI49+N65UoDz2aK/w4QHGwE6SCz2OZmGut2nDfFOd+wphI2dRc6vnwMZiI8nNpMb8 WCN/5z9HJTHVEuFK+1pNvCd3fJNOjNmbAn0jjzpQu+7pzY/YZY3A3+XKM2UTo9D4SLPhnHnwd uPJgioSeVCgeQsPwXrflWRriXMAAHJWN7A/JYD1YiO4+rmE1+LM8xrs4jshkVeC8hGDHlmiuB VZUjdP4VlkXEytmQa6JuV2m4hvcTs0Xj4APatiDLWB3gYTM0AEozrrULGgQwk0G78oOmGMLlM YUwV0EjSxLVB6UI16lWFBR2yzzK+zwzvUCsBjwUshf4aqCephqcRcLt0tc4+Us0fqCX3T+SmF gNMw+oB/cjWyFjiDm4ZYtODStrYcbY9XrFnaUF3UouvE10HCHmF3pDyFjaprLVkg26Ai79dtP 2rX7BB1Lg76xfIj3f6RC8ZnAc8cn30brKiWylwF2tLDEHyogCKD2dVvIoMtcTbMewA8VctAMB cgh21Avk8UyuggZaexo4fqp5c81hj5nLM+A6jSuc8uNvCfh0HGZzLPwn91jLx2VWF22G85CQl ucdQq7vCi3eS6wx4pyHCTrF+ysixkWMZVqZwWppdrT2opEAQ3tmXI3+4cqpbiflP8TIqksnAY aya7wzzcqpra3eHHLw56bTnfuicuum7S7OCgyzgSp5oevSFrYu+/Eg3CWW99kAIYbZ4k= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Lars-Peter, Lars-Peter Clausen schrieb am Wed, 14. Dec 11:11: > On 12/14/2016 10:59 AM, Andreas Klinger wrote: > > Add DT bindings for avia,hx711 > > Add vendor avia to vendor list > > > > Signed-off-by: Andreas Klinger > > --- > > .../devicetree/bindings/iio/adc/avia-hx711.txt | 21 +++++++++++++++++++++ > > .../devicetree/bindings/vendor-prefixes.txt | 1 + > > 2 files changed, 22 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/iio/adc/avia-hx711.txt > > > > diff --git a/Documentation/devicetree/bindings/iio/adc/avia-hx711.txt b/Documentation/devicetree/bindings/iio/adc/avia-hx711.txt > > new file mode 100644 > > index 000000000000..6a4659fc7a4f > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/iio/adc/avia-hx711.txt > > @@ -0,0 +1,21 @@ > > +* AVIA HX711 ADC chip for weight cells > > + Bit-banging driver > > + > > +Required properties: > > + - compatible: Should be "avia,hx711" > > + - sck-gpios: Definition of the GPIO for the clock > > + - dout-gpios: Definition of the GPIO for data-out > > + See Documentation/devicetree/bindings/gpio/gpio.txt > > + > > +Recommended properties: > > + - gain: Gain select, can be 32, 64 or 128 > > + default is 128 > > If the gain is software programmable it should be exposed by the driver > allowing the application to change it rather than putting it in the devicetree. > There is also a hardware dependency in terms of where to connect the input channel to: - gain 64 and gain 128 are available only on input channel A - gain 32 is only available on input channel B Do you still think, it should be removed from the devicetree? Andreas > -- > To unsubscribe from this list: send the line "unsubscribe linux-iio" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html