From patchwork Fri Aug 16 06:39:35 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Oleksandr Kozaruk X-Patchwork-Id: 2845486 Return-Path: X-Original-To: patchwork-linux-arm@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork2.web.kernel.org (Postfix) with ESMTP id 6422BBF546 for ; Fri, 16 Aug 2013 06:40:46 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id AE892202B8 for ; Fri, 16 Aug 2013 06:40:41 +0000 (UTC) Received: from casper.infradead.org (casper.infradead.org [85.118.1.10]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 9EB4620214 for ; Fri, 16 Aug 2013 06:40:40 +0000 (UTC) Received: from merlin.infradead.org ([2001:4978:20e::2]) by casper.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1VADhy-0007Nt-ET; Fri, 16 Aug 2013 06:40:26 +0000 Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1VADht-0001kY-GQ; Fri, 16 Aug 2013 06:40:21 +0000 Received: from bear.ext.ti.com ([192.94.94.41]) by merlin.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1VADhn-0001iv-Ai for linux-arm-kernel@lists.infradead.org; Fri, 16 Aug 2013 06:40:16 +0000 Received: from dlelxv90.itg.ti.com ([172.17.2.17]) by bear.ext.ti.com (8.13.7/8.13.7) with ESMTP id r7G6dbGB004005; Fri, 16 Aug 2013 01:39:37 -0500 Received: from DNCE71.ent.ti.com (dnce71.ent.ti.com [137.167.131.20]) by dlelxv90.itg.ti.com (8.14.3/8.13.8) with ESMTP id r7G6dagS027711; Fri, 16 Aug 2013 01:39:36 -0500 Received: from DNCE04.ent.ti.com ([fe80::50a2:cda3:1471:a76]) by DNCE71.ent.ti.com ([fe80::7d60:3983:a688:ea7d%20]) with mapi id 14.02.0342.003; Fri, 16 Aug 2013 08:39:35 +0200 From: "Kozaruk, Oleksandr" To: Jonathan Cameron , Mark Rutland Subject: RE: [PATCH v8 0/2] TWL6030, TWL6032 GPADC driver Thread-Topic: [PATCH v8 0/2] TWL6030, TWL6032 GPADC driver Thread-Index: AQHOmZePlU54UJBg2Ui+gbUHbnKlYJmV+U4AgAAgbgCAAFPUAIAA9Q2L Date: Fri, 16 Aug 2013 06:39:35 +0000 Message-ID: <2A7ABDFCE21540479A5AEB0244A684D5E3EE9C@DNCE04.ent.ti.com> References: <1374758813-6539-1-git-send-email-oleksandr.kozaruk@ti.com> <20130815091216.GB28366@e106331-lin.cambridge.arm.com> <520CB566.2050201@kernel.org> <20130815125906.GA32421@e106331-lin.cambridge.arm.com>, <520D16EC.90604@kernel.org> In-Reply-To: <520D16EC.90604@kernel.org> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [128.247.5.41] x-exclaimer-md-config: f9c360f5-3d1e-4c3c-8703-f45bf52eff6b MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20130816_024015_526303_BECFCD3F X-CRM114-Status: GOOD ( 17.23 ) X-Spam-Score: -9.7 (---------) Cc: "Kim, Milo" , "lars@metafoo.de" , "sameo@linux.intel.com" , "devicetree@vger.kernel.org" , "tony@atomide.com" , "Nayak, Rajendra" , "poeschel@lemonage.de" , "linux-kernel@vger.kernel.org" , "rob.herring@calxeda.com" , "ABRAHAM, KISHON VIJAY" , "Ujfalusi, Peter" , "linux-iio@vger.kernel.org" , "linux-omap@vger.kernel.org" , "jic23@cam.ac.uk" , "grant.likely@linaro.org" , "benoit.cousson@linaro.org" , "gg@slimlogic.co.uk" , "ch.naveen@samsung.com" , "linux-arm-kernel@lists.infradead.org" , "Krishnamoorthy, Balaji T" X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Spam-Status: No, score=-6.9 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_MED, RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP >On 08/15/13 13:59, Mark Rutland wrote: >> On Thu, Aug 15, 2013 at 12:03:02PM +0100, Jonathan Cameron wrote: >>> >>>>> The changes to the original driver: >>>>> - device tree adaptation; >>>> >>>> I couldn't see a binding document in this series or in mainline. Have I >>>> looked in the wrong places? >>> >>> Nothing explicit supplied, but does it need one given it is doing only >>> iio bindings (bindings/iio/iio-bindings.txt) >>> plus twl child bindings >>> (bindings/mfd/twl-family.txt) >> >> Every binding needs to be documented. >> >>> >>> If it does, I guess absolutely everything does, then fair enough! >>> I guess that would make sense as there is no way for someone writing >>> a device tree to know that there is nothing else to be specified. >> >> Yup, that's why. Also, Linux isn't necessarily the only consumer, and >> other consumers shouldn't need to read Linux code to figure out how a >> particular binding is supposed to look. >> >Fair enough. Thanks for clearing that up. > >Oleksandr, could you send a follow up patch adding the required documentation? >(mostly a cut and paste job from similar elements by the look of it). > Hello, Is this good enough? From 211e81ff4a146d9ec27443696a429e795c58dc30 Mon Sep 17 00:00:00 2001 From: Oleksandr Kozaruk Date: Thu, 15 Aug 2013 16:14:11 +0300 Subject: [PATCH] iio: adc: Add bindigs documentation for twl6030 GPADC Add required documentation for twl6030 GPADC device tree bindings. Signed-off-by: Oleksandr Kozaruk --- .../devicetree/bindings/iio/adc/twl6030-gpadc.txt | 14 ++++++++++++++ 1 file changed, 14 insertions(+) create mode 100644 Documentation/devicetree/bindings/iio/adc/twl6030-gpadc.txt diff --git a/Documentation/devicetree/bindings/iio/adc/twl6030-gpadc.txt b/Documentation/devicetree/bindings/iio/adc/twl6030-gpadc.txt new file mode 100644 index 0000000..6829420 --- /dev/null +++ b/Documentation/devicetree/bindings/iio/adc/twl6030-gpadc.txt @@ -0,0 +1,14 @@ +Texas Instruments twl6030/twl6032 GPADC device driver + +Required properties: + - compatible: must be "ti,twl6030-gpadc" for TWL6030 or + "ti,twl6032-gpadc" for TWL6032 + - interrupts: interrupt number associated with it + - #io-channel-cells: must be <1> - multiple IIO outputs + +Example: + adc { + compatible = "ti,twl6030-gpadc"; + interrupts = <3>; + #io-channel-cells = <1>; + };