From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-10.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_2 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 542DBC33CA2 for ; Fri, 10 Jan 2020 10:01:54 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 2E77520721 for ; Fri, 10 Jan 2020 10:01:54 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727429AbgAJKBx convert rfc822-to-8bit (ORCPT ); Fri, 10 Jan 2020 05:01:53 -0500 Received: from lhrrgout.huawei.com ([185.176.76.210]:2248 "EHLO huawei.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1727336AbgAJKBx (ORCPT ); Fri, 10 Jan 2020 05:01:53 -0500 Received: from lhreml706-cah.china.huawei.com (unknown [172.18.7.108]) by Forcepoint Email with ESMTP id D047B70748EF312315BA; Fri, 10 Jan 2020 10:01:51 +0000 (GMT) Received: from lhreml710-chm.china.huawei.com (10.201.108.61) by lhreml706-cah.china.huawei.com (10.201.108.47) with Microsoft SMTP Server (TLS) id 14.3.408.0; Fri, 10 Jan 2020 10:01:51 +0000 Received: from localhost (10.202.226.57) by lhreml710-chm.china.huawei.com (10.201.108.61) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1713.5; Fri, 10 Jan 2020 10:01:51 +0000 Date: Fri, 10 Jan 2020 10:01:48 +0000 From: Jonathan Cameron To: Uwe =?ISO-8859-1?Q?Kleine-K=F6nig?= CC: Jonathan Cameron , Mark Rutland , Lars-Peter Clausen , Stefan Popa , Michael Hennerich , , Rob Herring , "Peter Meerwald-Stadler" , Hartmut Knaack , Alexandru Ardelean , Subject: Re: [PATCH v4 1/3] iio: adc: ltc2496: provide device tree binding document Message-ID: <20200110100148.00001033@Huawei.com> In-Reply-To: <20200109205954.hyjvtwpddvytzgpr@pengutronix.de> References: <20191209203248.21555-1-u.kleine-koenig@pengutronix.de> <20191209203248.21555-2-u.kleine-koenig@pengutronix.de> <20191215114958.6ad37d19@archlinux> <20191216083736.lzmborv33w6odlaj@pengutronix.de> <20191223175854.600c7254@archlinux> <20200109205954.hyjvtwpddvytzgpr@pengutronix.de> Organization: Huawei Technologies Research and Development (UK) Ltd. X-Mailer: Claws Mail 3.17.4 (GTK+ 2.24.32; i686-w64-mingw32) MIME-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 8BIT X-Originating-IP: [10.202.226.57] X-ClientProxiedBy: lhreml725-chm.china.huawei.com (10.201.108.76) To lhreml710-chm.china.huawei.com (10.201.108.61) X-CFilter-Loop: Reflected Sender: linux-iio-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-iio@vger.kernel.org On Thu, 9 Jan 2020 21:59:54 +0100 Uwe Kleine-König wrote: > Hello Jonathan, > > On Mon, Dec 23, 2019 at 05:58:54PM +0000, Jonathan Cameron wrote: > > On Mon, 16 Dec 2019 09:37:36 +0100 > > Uwe Kleine-König wrote: > > > > > Hello Jonathan, > > > > > > On Sun, Dec 15, 2019 at 11:49:58AM +0000, Jonathan Cameron wrote: > > > > On Mon, 9 Dec 2019 21:32:46 +0100 > > > > Uwe Kleine-König wrote: > > > > > > > > > The ADC only requires the standard stuff for spi devices and a reference > > > > > voltage. > > > > > > > > > > Signed-off-by: Uwe Kleine-König > > > > Thanks for figuring out what was wrong and fixing it up. > > > > > > It wasn't done primarily to do you a favour :-) > > > > > > > > + spi-max-frequency: > > > > > + description: maximal spi bus frequency supported by the chip > > > > > > > > dropped the "by the chip" as this is also about the wiring on the board. If it > > > > were just the chip, it could be put in the driver. The unknown bit is > > > > if there is some other reason why it might need to be set lower than the maximum. > > > > > > fine for me. > > > > > > > > +required: > > > > > + - compatible > > > > > + - vref-supply > > > > > + - reg > > > > > + > > > > > +examples: > > > > > + - | > > > > > + spi { > > > > > + #address-cells = <1>; > > > > > + #size-cells = <0>; > > > > > + > > > > > + adc@0 { > > > > > + compatible = "lltc,ltc2496"; > > > > You can't easily see it here, but this is a mixture of spaces > > > > and tabs. Should be all spaces. I've tidied that up. > > > > > > I did that on purpose, spaces to have the needed indention for the yaml > > > syntax and then tabs for dts indention (as done in the dts itself, too). > > > I thought this to be the right mix, but this was my first yaml binding, > > > so you're probably right. > > > > I'll be honest, I'm far from sure on this. > > > > I usually rely on Rob moaning about it if it's wrong :) > > > > Oh well. What's there builds. If we get a 'fix' later then so be it. > > I found my commits in your tree (at > https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git/log/?h=testing) > and wonder why they don't appear in next. IMHO it would be good to have > some exposure there to give more people the opportunity to moan. Is this > a mistake that your tree is not included in next? > > I assume being in your testing branch means I can reasonably expect they > will go into 5.6-rc1? Given timing yes. I tend to do last pull request 1-2 weeks before the merge window opens. Greg KH takes IIO through his staging tree. One day that might change but for now that's how we do it. Note that I'll carry on taking things into the togreg branch (which first becomes visible as testing) after that, but they'll not go anywhere until after the merge window. There was a slight delay this time around, but Greg took the pull request this morning so they should be in Linux-next on Monday. They are indeed heading for 5.6-rc1. Thanks, Jonathan > > Best regards > Uwe >