Re: [PATCH 2/5] Documentation: add the binding file for Freescale imx7d ADC driver
On 11/10/15 14:31, Jonathan Cameron wrote: > On 08/10/15 11:59, Haibo Chen wrote: >> The patch adds the binding file for Freescale imx7d ADC driver. >> >> Signed-off-by: Haibo Chen > Couple of trivial typos.. otherwise fine. >> --- >> .../devicetree/bindings/iio/adc/imx7d-adc.txt | 26 >> ++ >> 1 file changed, 26 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/iio/adc/imx7d-adc.txt >> >> diff --git a/Documentation/devicetree/bindings/iio/adc/imx7d-adc.txt >> b/Documentation/devicetree/bindings/iio/adc/imx7d-adc.txt >> new file mode 100644 >> index 000..6b21fd27 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/iio/adc/imx7d-adc.txt >> @@ -0,0 +1,26 @@ >> +Freescale imx7d ADC bindings >> + >> +The devicetree bingdings are for the nwe ADC driver written for >> +imx7d SoC. > bindings and new Actually I'd drop the new. Makes sense in a patch comment, but in the docs it'll rapidly stop being new. >> + >> +Required properties: >> +- compatible: Should be "fsl,imx7d-adc" >> +- reg: Offset and length of the register set for the ADC device >> +- interrupts: The interrupt number for the ADC device >> +- clocks: The root clock of the ADC controller >> +- clock-names: Must contain "adc", matching entry in the clocks property >> +- vref-supply: The regulator supply ADC reference voltage >> + >> +Optional properties: >> +- num-channels: the number of channels used >> + >> +Example: >> +adc1: adc@3061 { >> +compatible = "fsl,imx7d-adc"; >> +reg = <0x3061 0x1>; >> +interrupts = ; >> +clocks = <&clks IMX7D_ADC_ROOT_CLK>; >> +clock-names = "adc"; >> +num-channels = <4>; >> +vref-supply = <®_vcc_3v3_mcu>; >> +}; >> > > -- > To unsubscribe from this list: send the line "unsubscribe linux-iio" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-clk" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 2/5] Documentation: add the binding file for Freescale imx7d ADC driver
On 08/10/15 11:59, Haibo Chen wrote: > The patch adds the binding file for Freescale imx7d ADC driver. > > Signed-off-by: Haibo Chen Couple of trivial typos.. otherwise fine. > --- > .../devicetree/bindings/iio/adc/imx7d-adc.txt | 26 > ++ > 1 file changed, 26 insertions(+) > create mode 100644 Documentation/devicetree/bindings/iio/adc/imx7d-adc.txt > > diff --git a/Documentation/devicetree/bindings/iio/adc/imx7d-adc.txt > b/Documentation/devicetree/bindings/iio/adc/imx7d-adc.txt > new file mode 100644 > index 000..6b21fd27 > --- /dev/null > +++ b/Documentation/devicetree/bindings/iio/adc/imx7d-adc.txt > @@ -0,0 +1,26 @@ > +Freescale imx7d ADC bindings > + > +The devicetree bingdings are for the nwe ADC driver written for > +imx7d SoC. bindings and new > + > +Required properties: > +- compatible: Should be "fsl,imx7d-adc" > +- reg: Offset and length of the register set for the ADC device > +- interrupts: The interrupt number for the ADC device > +- clocks: The root clock of the ADC controller > +- clock-names: Must contain "adc", matching entry in the clocks property > +- vref-supply: The regulator supply ADC reference voltage > + > +Optional properties: > +- num-channels: the number of channels used > + > +Example: > +adc1: adc@3061 { > + compatible = "fsl,imx7d-adc"; > + reg = <0x3061 0x1>; > + interrupts = ; > + clocks = <&clks IMX7D_ADC_ROOT_CLK>; > + clock-names = "adc"; > + num-channels = <4>; > + vref-supply = <®_vcc_3v3_mcu>; > +}; > -- To unsubscribe from this list: send the line "unsubscribe linux-clk" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 2/5] Documentation: add the binding file for Freescale imx7d ADC driver
On Thu, Oct 8, 2015 at 5:59 AM, Haibo Chen wrote: > The patch adds the binding file for Freescale imx7d ADC driver. > > Signed-off-by: Haibo Chen Acked-by: Rob Herring > --- > .../devicetree/bindings/iio/adc/imx7d-adc.txt | 26 > ++ > 1 file changed, 26 insertions(+) > create mode 100644 Documentation/devicetree/bindings/iio/adc/imx7d-adc.txt > > diff --git a/Documentation/devicetree/bindings/iio/adc/imx7d-adc.txt > b/Documentation/devicetree/bindings/iio/adc/imx7d-adc.txt > new file mode 100644 > index 000..6b21fd27 > --- /dev/null > +++ b/Documentation/devicetree/bindings/iio/adc/imx7d-adc.txt > @@ -0,0 +1,26 @@ > +Freescale imx7d ADC bindings > + > +The devicetree bingdings are for the nwe ADC driver written for > +imx7d SoC. > + > +Required properties: > +- compatible: Should be "fsl,imx7d-adc" > +- reg: Offset and length of the register set for the ADC device > +- interrupts: The interrupt number for the ADC device > +- clocks: The root clock of the ADC controller > +- clock-names: Must contain "adc", matching entry in the clocks property > +- vref-supply: The regulator supply ADC reference voltage > + > +Optional properties: > +- num-channels: the number of channels used > + > +Example: > +adc1: adc@3061 { > + compatible = "fsl,imx7d-adc"; > + reg = <0x3061 0x1>; > + interrupts = ; > + clocks = <&clks IMX7D_ADC_ROOT_CLK>; > + clock-names = "adc"; > + num-channels = <4>; > + vref-supply = <®_vcc_3v3_mcu>; > +}; > -- > 1.9.1 > > -- > To unsubscribe from this list: send the line "unsubscribe devicetree" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-clk" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html