| Message ID | 1392206518-1457-3-git-send-email-paul.cercueil@analog.com |
|---|---|
| State | Superseded, archived |
| Headers | show |
On Wed, Feb 12, 2014 at 12:01:58PM +0000, Paul Cercueil wrote: > Signed-off-by: Paul Cercueil <paul.cercueil@analog.com> > Cc: Rob Herring <rob.herring@calxeda.com> > Cc: Pawel Moll <pawel.moll@arm.com> > Cc: Mark Rutland <mark.rutland@arm.com> > Cc: Ian Campbell <ijc+devicetree@hellion.org.uk> > Cc: Kumar Gala <galak@codeaurora.org> > Cc: devicetree@vger.kernel.org > --- > .../devicetree/bindings/iio/dac/ad5064.txt | 49 ++++++++++++++++++++ > 1 file changed, 49 insertions(+) > create mode 100644 Documentation/devicetree/bindings/iio/dac/ad5064.txt > > diff --git a/Documentation/devicetree/bindings/iio/dac/ad5064.txt b/Documentation/devicetree/bindings/iio/dac/ad5064.txt > new file mode 100644 > index 0000000..e5997b0 > --- /dev/null > +++ b/Documentation/devicetree/bindings/iio/dac/ad5064.txt > @@ -0,0 +1,49 @@ > +Analog Devices AD5064 DAC device driver It would be nice to have a short description here, with useful facts (e.g. this is an SPI device). Is there any public documentation? It might not need to go in the document, but having a link really helps with review. > + > +Required properties: > + - compatible: Must be one of: > + * "adi,ad5024" > + * "adi,ad5025" > + * "adi,ad5044" > + * "adi,ad5045" > + * "adi,ad5064" > + * "adi,ad5064-1" > + * "adi,ad5065" > + * "adi,ad5628-1" > + * "adi,ad5628-2" > + * "adi,ad5648-1" > + * "adi,ad5648-2" > + * "adi,ad5666-1" > + * "adi,ad5666-2" > + * "adi,ad5668-1" > + * "adi,ad5668-2" > + * "adi,ad5668-3" How do these differ? Are these just different revisions of the same chip? Are particular strings expected to be used as fallbacks in the compatible list? > + - reg: SPI chip select number for the device > + - spi-max-frequency: Max SPI frequency to use (< 30000000) > + - vrefA-supply, vrefB-supply: phandles to external reference voltage > + supplies for channels 0 and 1 respectively. > + This property must be present for ad5024, ad5025, ad5044, ad5045, > + ad5064, ad5065. Does ad5064 also imply ad5604-1 here? > + - vrefC-supply, vrefD-supply: phandles to external reference voltage > + supplies for channels 2 and 3 respectively. > + This property must be present for ad5024, ad5044, ad5064. Likewise. These seem oddly named given the description. Are these named A B C D in the documentation? If not, vref-channel-X-supply would seem to be clearer. Why are these grouped in pairs rather than listed individually? > + > +Optional properties: > + - adi,use-external-reference: If set, the external reference voltage > + supply is used. This should only be set if there is an external > + reference voltage connected to the vref or vref[A-D] pins. > + If the property is not set, the internal reference voltage supply > + is used if present, or an error is issued by the driver. When would you want to do this, and when would you want to use the internal reference? Could the driver not choose to do this based on whether a supply is listed in the dt? Strip the part about the error, that's not a description of the device and not a matter for the binding. > + - vref-supply: phandle to the external reference voltage supply. > + This property can be used with ad5064-1, ad5628-1, ad5628-2, ad5648-1, > + ad5648-2, ad5666-1, ad5666-2, ad5668-1, ad5668-2, ad5668-3. Just to check, on some versions (e.g. ad5668-3), this can work without any supply listed at all, yes? Cheers, Mark. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 02/12/14 13:18, Mark Rutland wrote: > On Wed, Feb 12, 2014 at 12:01:58PM +0000, Paul Cercueil wrote: >> Signed-off-by: Paul Cercueil <paul.cercueil@analog.com> >> Cc: Rob Herring <rob.herring@calxeda.com> >> Cc: Pawel Moll <pawel.moll@arm.com> >> Cc: Mark Rutland <mark.rutland@arm.com> >> Cc: Ian Campbell <ijc+devicetree@hellion.org.uk> >> Cc: Kumar Gala <galak@codeaurora.org> >> Cc: devicetree@vger.kernel.org >> --- >> .../devicetree/bindings/iio/dac/ad5064.txt | 49 ++++++++++++++++++++ >> 1 file changed, 49 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/iio/dac/ad5064.txt >> >> diff --git a/Documentation/devicetree/bindings/iio/dac/ad5064.txt b/Documentation/devicetree/bindings/iio/dac/ad5064.txt >> new file mode 100644 >> index 0000000..e5997b0 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/iio/dac/ad5064.txt >> @@ -0,0 +1,49 @@ >> +Analog Devices AD5064 DAC device driver > > It would be nice to have a short description here, with useful facts > (e.g. this is an SPI device). > > Is there any public documentation? It might not need to go in the > document, but having a link really helps with review. Allright, I will add a short description. I didn't add a link to the documentation as it moves every once in a while... >> + >> +Required properties: >> + - compatible: Must be one of: >> + * "adi,ad5024" >> + * "adi,ad5025" >> + * "adi,ad5044" >> + * "adi,ad5045" >> + * "adi,ad5064" >> + * "adi,ad5064-1" >> + * "adi,ad5065" >> + * "adi,ad5628-1" >> + * "adi,ad5628-2" >> + * "adi,ad5648-1" >> + * "adi,ad5648-2" >> + * "adi,ad5666-1" >> + * "adi,ad5666-2" >> + * "adi,ad5668-1" >> + * "adi,ad5668-2" >> + * "adi,ad5668-3" > > How do these differ? Are these just different revisions of the same > chip? > > Are particular strings expected to be used as fallbacks in the > compatible list? All of those chips are different, but are handled by the ad5064 driver. They differ in the number of channels available, in the presence of an internal vref and/or its voltage. Some of those use a shared vref, some others have a vref per channel. The only similar chips are the ad5668-2 and ad5668-3. >> + - reg: SPI chip select number for the device >> + - spi-max-frequency: Max SPI frequency to use (< 30000000) >> + - vrefA-supply, vrefB-supply: phandles to external reference voltage >> + supplies for channels 0 and 1 respectively. >> + This property must be present for ad5024, ad5025, ad5044, ad5045, >> + ad5064, ad5065. > > Does ad5064 also imply ad5604-1 here? > >> + - vrefC-supply, vrefD-supply: phandles to external reference voltage >> + supplies for channels 2 and 3 respectively. >> + This property must be present for ad5024, ad5044, ad5064. > > Likewise. The ad5064 and ad5064-1 chips are actually different; the first one has one vref per channel, the second one use a shared vref. > These seem oddly named given the description. Are these named A B C D in > the documentation? If not, vref-channel-X-supply would seem to be > clearer. They are named after the pin names on the datasheets: vrefA, vrefB, vrefC, vrefD. > Why are these grouped in pairs rather than listed individually? All the chips featuring separate vref per channel, have either 2 or 4 channels, which means that some chips have only the vrefA/vrefB pins, while others have them all. These properties were grouped in pairs because the description and the list of supported chips are the same, but I can list them individually if it's preferred. >> + >> +Optional properties: >> + - adi,use-external-reference: If set, the external reference voltage >> + supply is used. This should only be set if there is an external >> + reference voltage connected to the vref or vref[A-D] pins. >> + If the property is not set, the internal reference voltage supply >> + is used if present, or an error is issued by the driver. > > When would you want to do this, and when would you want to use the > internal reference? Could the driver not choose to do this based on > whether a supply is listed in the dt? Using or not the internal reference is up to the platform. But you make a valid point, it would be better for the driver to use the external reference if specified in the dt and revert to the internal one otherwise. I will fix that in the driver. > Strip the part about the error, that's not a description of the device > and not a matter for the binding. Allright. >> + - vref-supply: phandle to the external reference voltage supply. >> + This property can be used with ad5064-1, ad5628-1, ad5628-2, ad5648-1, >> + ad5648-2, ad5666-1, ad5666-2, ad5668-1, ad5668-2, ad5668-3. > > Just to check, on some versions (e.g. ad5668-3), this can work without > any supply listed at all, yes? All of those chips except for ad5064-1 have an internal vref, which will be used if no external regulator is specified, so yes. > Cheers, > Mark. > Thanks for your comments! Regards, Paul -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/Documentation/devicetree/bindings/iio/dac/ad5064.txt b/Documentation/devicetree/bindings/iio/dac/ad5064.txt new file mode 100644 index 0000000..e5997b0 --- /dev/null +++ b/Documentation/devicetree/bindings/iio/dac/ad5064.txt @@ -0,0 +1,49 @@ +Analog Devices AD5064 DAC device driver + +Required properties: + - compatible: Must be one of: + * "adi,ad5024" + * "adi,ad5025" + * "adi,ad5044" + * "adi,ad5045" + * "adi,ad5064" + * "adi,ad5064-1" + * "adi,ad5065" + * "adi,ad5628-1" + * "adi,ad5628-2" + * "adi,ad5648-1" + * "adi,ad5648-2" + * "adi,ad5666-1" + * "adi,ad5666-2" + * "adi,ad5668-1" + * "adi,ad5668-2" + * "adi,ad5668-3" + - reg: SPI chip select number for the device + - spi-max-frequency: Max SPI frequency to use (< 30000000) + - vrefA-supply, vrefB-supply: phandles to external reference voltage + supplies for channels 0 and 1 respectively. + This property must be present for ad5024, ad5025, ad5044, ad5045, + ad5064, ad5065. + - vrefC-supply, vrefD-supply: phandles to external reference voltage + supplies for channels 2 and 3 respectively. + This property must be present for ad5024, ad5044, ad5064. + +Optional properties: + - adi,use-external-reference: If set, the external reference voltage + supply is used. This should only be set if there is an external + reference voltage connected to the vref or vref[A-D] pins. + If the property is not set, the internal reference voltage supply + is used if present, or an error is issued by the driver. + - vref-supply: phandle to the external reference voltage supply. + This property can be used with ad5064-1, ad5628-1, ad5628-2, ad5648-1, + ad5648-2, ad5666-1, ad5666-2, ad5668-1, ad5668-2, ad5668-3. + +Example: + + ad5668-2@4 { + compatible = "adi,ad5668-2"; + reg = <4>; + spi-max-frequency = <10000000>; + adi,use-external-reference; + vref-supply = <&vref_supply>; + };
Signed-off-by: Paul Cercueil <paul.cercueil@analog.com> Cc: Rob Herring <rob.herring@calxeda.com> Cc: Pawel Moll <pawel.moll@arm.com> Cc: Mark Rutland <mark.rutland@arm.com> Cc: Ian Campbell <ijc+devicetree@hellion.org.uk> Cc: Kumar Gala <galak@codeaurora.org> Cc: devicetree@vger.kernel.org --- .../devicetree/bindings/iio/dac/ad5064.txt | 49 ++++++++++++++++++++ 1 file changed, 49 insertions(+) create mode 100644 Documentation/devicetree/bindings/iio/dac/ad5064.txt