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@xxxxxxxxxxxx> >> Reviewed-by: Ray Jui <ray.jui@xxxxxxxxxxxx> >> Reviewed-by: Scott Branden <scott.branden@xxxxxxxxxxxx> >> --- >> .../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@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html