From: Jonathan Cameron <jic23@kernel.org>
To: Rob Herring <robh@kernel.org>,
Raveendra Padasalagi <raveendra.padasalagi@broadcom.com>
Cc: Peter Meerwald-Stadler <pmeerw@pmeerw.net>,
Russell King <linux@arm.linux.org.uk>,
Arnd Bergmann <arnd@arndb.de>,
linux-iio@vger.kernel.org, devicetree@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
Pawel Moll <pawel.moll@arm.com>,
Mark Rutland <mark.rutland@arm.com>,
Ian Campbell <ijc+devicetree@hellion.org.uk>,
Kumar Gala <galak@codeaurora.org>,
Jonathan Richardson <jonathar@broadcom.com>,
Jon Mason <jonmason@broadcom.com>,
Florian Fainelli <f.fainelli@gmail.com>,
Anup Patel <anup.patel@broadcom.com>, Ray Jui <rjui@broadcom.com>,
Scott Branden <sbranden@broadcom.com>,
Pramod Kumar <pramod.kumar@broadcom.com>,
linux-kernel@vger.kernel.org,
bcm-kernel-feedback-list@broadcom.com
Subject: Re: [PATCH v3 1/3] Documentation: DT: Add iproc-static-adc binding
Date: Sun, 26 Jun 2016 11:24:36 +0100 [thread overview]
Message-ID: <4c8870ef-8777-7016-465b-9c03463d01fa@kernel.org> (raw)
In-Reply-To: <20160624154323.GA6046@rob-hp-laptop>
On 24/06/16 16:43, Rob Herring wrote:
> On Wed, Jun 22, 2016 at 11:41:51AM +0530, Raveendra Padasalagi wrote:
>> The patch adds devicetree binding document for broadcom's
>> iproc-static-adc controller driver.
>>
>> Signed-off-by: Raveendra Padasalagi <raveendra.padasalagi@broadcom.com>
>> Reviewed-by: Ray Jui <ray.jui@broadcom.com>
>> Reviewed-by: Scott Branden <scott.branden@broadcom.com>
>> ---
>> .../bindings/iio/adc/brcm,iproc-static-adc.txt | 38 ++++++++++++++++++++++
>> 1 file changed, 38 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..a82076e
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/iio/adc/brcm,iproc-static-adc.txt
>> @@ -0,0 +1,38 @@
>> +* Broadcom's IPROC Static ADC controller
>> +
>> +Broadcom iProc ADC controller has 8 channels 10bit ADC.
>> +Allows user to convert analog input voltage values to digital.
>> +
>> +Required properties:
>> +
>> +- compatible: Must be "brcm,iproc-static-adc"
>> +
>> +- 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";
>> + adc-syscon = <&ts_adc_syscon>;
>> + #io-channel-cells = <1>;
>> + io-channel-ranges;
>
> Not documented.
Core iio docs, but obviously a cross reference would be good.
>
>> + clocks = <&asiu_clks BCM_CYGNUS_ASIU_ADC_CLK>;
>> + clock-names = "tsc_clk";
>> + interrupts = <GIC_SPI 164 IRQ_TYPE_LEVEL_HIGH>;
>> + status = "disabled";
>> + };
>> --
>> 1.9.1
>>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
next prev parent reply other threads:[~2016-06-26 10:24 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-06-22 6:11 [PATCH v3 0/3] Add Broadcom iproc-static-adc controller driver Raveendra Padasalagi
2016-06-22 6:11 ` [PATCH v3 1/3] Documentation: DT: Add iproc-static-adc binding Raveendra Padasalagi
2016-06-24 15:43 ` Rob Herring
2016-06-26 10:24 ` Jonathan Cameron [this message]
2016-06-27 6:27 ` Raveendra Padasalagi
2016-06-22 6:11 ` [PATCH v3 2/3] iio: Add driver for Broadcom iproc-static-adc Raveendra Padasalagi
2016-06-26 10:38 ` Jonathan Cameron
2016-06-27 6:25 ` Raveendra Padasalagi
2016-06-27 19:11 ` Jonathan Cameron
2016-06-28 5:13 ` Raveendra Padasalagi
2016-06-22 6:11 ` [PATCH v3 3/3] ARM:dts-Add dt node " Raveendra Padasalagi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4c8870ef-8777-7016-465b-9c03463d01fa@kernel.org \
--to=jic23@kernel.org \
--cc=anup.patel@broadcom.com \
--cc=arnd@arndb.de \
--cc=bcm-kernel-feedback-list@broadcom.com \
--cc=devicetree@vger.kernel.org \
--cc=f.fainelli@gmail.com \
--cc=galak@codeaurora.org \
--cc=ijc+devicetree@hellion.org.uk \
--cc=jonathar@broadcom.com \
--cc=jonmason@broadcom.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=mark.rutland@arm.com \
--cc=pawel.moll@arm.com \
--cc=pmeerw@pmeerw.net \
--cc=pramod.kumar@broadcom.com \
--cc=raveendra.padasalagi@broadcom.com \
--cc=rjui@broadcom.com \
--cc=robh@kernel.org \
--cc=sbranden@broadcom.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).