From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755533AbcLNLTt (ORCPT ); Wed, 14 Dec 2016 06:19:49 -0500 Received: from www381.your-server.de ([78.46.137.84]:53000 "EHLO www381.your-server.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755393AbcLNLTd (ORCPT ); Wed, 14 Dec 2016 06:19:33 -0500 Subject: Re: [PATCH v2 1/2] iio: adc: hx711: Add DT binding for avia,hx711 To: Andreas Klinger References: <20161214095945.GA26511@andreas> <470f0017-2607-d5bd-bcd3-69f892cdc88b@metafoo.de> <20161214103455.GA27362@andreas> 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 From: Lars-Peter Clausen Message-ID: <10cef06e-a3d0-ef9e-2a6a-b044f02d8df3@metafoo.de> Date: Wed, 14 Dec 2016 12:18:11 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Icedove/45.5.1 MIME-Version: 1.0 In-Reply-To: <20161214103455.GA27362@andreas> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-Authenticated-Sender: lars@metafoo.de Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 12/14/2016 11:34 AM, Andreas Klinger wrote: > 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? I believe so, especially considering that there are two channels and your driver currently only exposes one of them, which is elected by the gain configuration. I'd expect that there are use-cases where both inputs are connected and the application wants to switch between them dynamically at runtime. The only tricky part here seems to be that the configuration for the next conversion is selected on the current conversion. This means you probably need to do a dummy conversion whenever the settings change.