Path: csiph.com!news.mixmin.net!news.unit0.net!news.panservice.it!bofh.it!news.nic.it!robomod From: Rob Herring Newsgroups: linux.kernel Subject: Re: [PATCH v2 1/3] Documentation: DT: Add iproc-static-adc binding Date: Mon, 20 Jun 2016 20:10:02 +0200 Message-ID: References: X-Original-To: Raveendra Padasalagi X-Google-Dkim-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=77xZ3Syyq+Q9bfWng9hm7jZDfgu7HU0avNZgupRrzcs=; b=BQ263hp7l49pfqZTUTESHUvvWNiKYorGpBGpOqU/XKU8jzZ9AIB6X+2XCely4NvGKD 0WCXNW2cNj3Kjqc15fwX372a3LNm5v2S7RwX/ZAQa1AsjRLUvPJye2aQS7IPejq4BS+k CC5EyFGZib1XXtBO9PC+IZlBwZHGBmhtgJ6Bg2QoM3SBUzpAapxJIQ0fX5z0VIYjLrD1 tBrKHhZXI0lWeDCY4rKKudWDxC70DCpjj4h6imxaawVVIDnroUVqA4vGHS4uv0th2QFQ zNOU3HHH/2ez/bwyNOFGq1eB/yrHXtTbNDSBgROty0tjzWMD6pHZK6EaR6dwCwPc93PI wB4A== X-Gm-Message-State: ALyK8tIioaxATBo27HsQoou56wWmXjonzeNDbhnyGyqlGyeyzsOGK0M183OJEMMArZ4/dA== X-Received: by 10.157.33.67 with SMTP id l3mr10209015otd.61.1466445899521; Mon, 20 Jun 2016 11:04:59 -0700 (PDT) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.5.24 (2015-08-30) Sender: robomod@news.nic.it List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Approved: robomod@news.nic.it Lines: 68 Organization: linux.* mail to news gateway X-Original-Cc: Jonathan Cameron , Peter Meerwald-Stadler , Russell King , Arnd Bergmann , linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Jonathan Richardson , Jon Mason , Florian Fainelli , Anup Patel , Ray Jui , Scott Branden , Pramod Kumar , linux-kernel@vger.kernel.org, bcm-kernel-feedback-list@broadcom.com X-Original-Date: Mon, 20 Jun 2016 13:04:58 -0500 X-Original-Message-ID: <20160620180458.GA10171@rob-hp-laptop> X-Original-References: <1466330790-2820-1-git-send-email-raveendra.padasalagi@broadcom.com> <1466330790-2820-2-git-send-email-raveendra.padasalagi@broadcom.com> X-Original-Sender: linux-kernel-owner@vger.kernel.org Xref: csiph.com linux.kernel:1426893 On Sun, Jun 19, 2016 at 03:36:28PM +0530, Raveendra Padasalagi wrote: > The patch adds devicetree binding document for broadcom's > iproc-static-adc controller driver. > > Signed-off-by: Raveendra Padasalagi > Reviewed-by: Ray Jui > Reviewed-by: Scott Branden > --- > .../bindings/iio/adc/brcm,iproc-static-adc.txt | 43 ++++++++++++++++++++++ > 1 file changed, 43 insertions(+) > create mode 100644 Documentation/devicetree/bindings/iio/adc/brcm,iproc-static-adc.txt > > diff --git a/Documentation/devicetree/bindings/iio/adc/brcm,iproc-static-adc.txt b/Documentation/devicetree/bindings/iio/adc/brcm,iproc-static-adc.txt > new file mode 100644 > index 0000000..1de0dfa > --- /dev/null > +++ b/Documentation/devicetree/bindings/iio/adc/brcm,iproc-static-adc.txt > @@ -0,0 +1,43 @@ > +* Broadcom's IPROC Static ADC controller > + > +Required properties: > + > +- compatible: Must be "brcm,iproc-static-adc" > + > +- #address-cells: Specify the number of u32 entries needed in child nodes. > + Should set to 1. > + > +- #size-cells: Specify number of u32 entries needed to specify child nodes size > + in reg property. Should set to 1. Why are these needed? What child nodes can you have? > + > +- adc-syscon: Handler of syscon node defining physical base address of the > + controller and length of memory mapped region. > + > +- #io-channel-cells = <1>; As ADC has multiple outputs > + refer to Documentation/devicetree/bindings/iio/iio-bindings.txt for details. > + > +- clocks: Clock used for this block. > + > +- clock-names: Clock name should be given as tsc_clk. > + > +- interrupts: interrupt line number. > + > +For example: > + > + ts_adc_syscon: ts_adc_syscon@180a6000 { > + compatible = "brcm,iproc-ts-adc-syscon","syscon"; > + reg = <0x180a6000 0xc30>; > + }; > + > + adc: adc@180a6000 { > + compatible = "brcm,iproc-static-adc"; > + #address-cells = <1>; > + #size-cells = <1>; > + adc-syscon = <&ts_adc_syscon>; > + #io-channel-cells = <1>; > + io-channel-ranges; > + clocks = <&asiu_clks BCM_CYGNUS_ASIU_ADC_CLK>; > + clock-names = "tsc_clk"; > + interrupts = ; > + status = "disabled"; > + }; > -- > 1.9.1 >