From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Herring Subject: Re: [PATCH v2 1/4] iio: hx711: add data-ready-delay-ns property in DT Date: Tue, 3 Jul 2018 15:48:43 -0600 Message-ID: <20180703214843.GA484@rob-hp-laptop> References: <20180625151802.GA6369@arbeit> <20180630183232.540ce031@archlinux> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20180630183232.540ce031@archlinux> Sender: linux-kernel-owner@vger.kernel.org To: Jonathan Cameron Cc: Andreas Klinger , knaack.h@gmx.de, lars@metafoo.de, pmeerw@pmeerw.net, mark.rutland@arm.com, mchehab@kernel.org, davem@davemloft.net, gregkh@linuxfoundation.org, akpm@linux-foundation.org, linus.walleij@linaro.org, rdunlap@infradead.org, devicetree@vger.kernel.org, linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org List-Id: devicetree@vger.kernel.org On Sat, Jun 30, 2018 at 06:32:32PM +0100, Jonathan Cameron wrote: > On Mon, 25 Jun 2018 17:18:02 +0200 > Andreas Klinger wrote: > > > add data-ready-delay-ns property for hx711 ADC > > this is the delay after rising PD_SCK until DOUT is read and assumed to be > > ready > > > > Signed-off-by: Andreas Klinger > > Hmm. It's not particularly generic... I think this should have > have vendor prefix until we have some visibility of whether it > is a common thing we want to standardise around or not. Agreed. Doesn't look like any interface I've seen before. Since it's bit-banging, seems to me it would be better to specify the frequency for PD_SCK and sample DOUT just before or after setting PD_SCK low. Rob > I'll also be waiting for a devicetree ack on this one as it's a > tiny bit unusual. > > Thanks, > > Jonathan