On 11/08/2014 09:57 AM, Jonathan Cameron wrote: > On 05/11/14 21:56, Ezequiel Garcia wrote: >> From: Phani Movva <[email protected]> >> >> Add the devicetree binding document for Cosmic Circuits 10001 ADC device. >> >> Signed-off-by: Phani Movva <[email protected]> >> Signed-off-by: Naidu Tellapati <[email protected]> >> [Ezequiel: minor style cleaning] >> Signed-off-by: Ezequiel Garcia <[email protected]> > Please resend including the device tree list and maintainers. > All bindings must go to them whatever part they are for. >
Right, sorry about that. > >> --- >> .../devicetree/bindings/iio/adc/cc_10001_adc.txt | 19 >> +++++++++++++++++++ >> 1 file changed, 19 insertions(+) >> create mode 100644 >> Documentation/devicetree/bindings/iio/adc/cc_10001_adc.txt >> >> diff --git a/Documentation/devicetree/bindings/iio/adc/cc_10001_adc.txt >> b/Documentation/devicetree/bindings/iio/adc/cc_10001_adc.txt >> new file mode 100644 >> index 0000000..6491839 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/iio/adc/cc_10001_adc.txt >> @@ -0,0 +1,19 @@ >> +* Cosmic Circuits - Analog to Digital Converter (CC-10001-ADC) >> + >> +Required properties: >> + - compatible: Should be "cosmic,10001-adc" >> + - reg: Should contain adc registers location and length. >> + - cosmic,adc-available-channels: Bitmask of the channels currently >> enabled. > I suspect there may be other preferred ways of doing this (child nodes perhaps > for each enabled channel?) Hm, I'm not sure about this. As far I can recall, if a node has children then it *must* be a bus, which is not the case. Of course, you'll find examples doing this wrong. Maybe some of the devicetree maintainers can clarify this point? -- Ezequiel -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
