On 11/03/16 14:12, Sean Nyekjaer wrote:
> Signed-off-by: Sean Nyekjaer <sean.nyekjaer@xxxxxxxxx>
Looks much better to me.
Looking for Acks from Lars and a device tree binding maintainer
on this one though.
(added CCs)
As an aside Device Tree bindings maintainers, do you prefer a personal cc
or just getting them directly from the devicetree list?
Jonthan
> ---
> Changes since v4:
> - Appended -hz to dc-dc-freq
> - Added adi,dc-dc-max-microvolt
> - Added reg property
>
> Changes since v3:
> - replaced '_' with '-'
> - Now used actual values instead of register values.
>
> Changes since v2:
> - Removed defines, alot easier to read
>
> .../devicetree/bindings/iio/dac/ad5755.txt | 124 +++++++++++++++++++++
> 1 file changed, 124 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/iio/dac/ad5755.txt
>
> diff --git a/Documentation/devicetree/bindings/iio/dac/ad5755.txt b/Documentation/devicetree/bindings/iio/dac/ad5755.txt
> new file mode 100644
> index 0000000..f0bbd7e
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/iio/dac/ad5755.txt
> @@ -0,0 +1,124 @@
> +* Analog Device AD5755 IIO Multi-Channel DAC Linux Driver
> +
> +Required properties:
> + - compatible: Has to contain one of the following:
> + adi,ad5755
> + adi,ad5755-1
> + adi,ad5757
> + adi,ad5735
> + adi,ad5737
> +
> + - reg: spi chip select number for the device
> + - spi-cpha or spi-cpol: is the only modes that is supported
> +
> +Recommended properties:
> + - spi-max-frequency: Definition as per
> + Documentation/devicetree/bindings/spi/spi-bus.txt
> +
> +Optional properties:
> +See include/dt-bindings/iio/ad5755.h
> + - adi,ext-dc-dc-compenstation-resistor: boolean set if the hardware have an
> + external resistor and thereby bypasses
> + the internal compensation resistor.
> + - adi,dc-dc-phase:
> + Valid values for DC DC Phase control is:
> + 0: All dc-to-dc converters clock on the same edge.
> + 1: Channel A and Channel B clock on the same edge,
> + Channel C and Channel D clock on opposite edges.
> + 2: Channel A and Channel C clock on the same edge,
> + Channel B and Channel D clock on opposite edges.
> + 3: Channel A, Channel B, Channel C, and Channel D
> + clock 90 degrees out of phase from each other.
> + - adi,dc-dc-freq-hz:
> + Valid values for DC DC frequency is [Hz]:
> + 250000
> + 410000
> + 650000
> + - adi,dc-dc-max-microvolt:
> + Valid values for the maximum allowed Vboost voltage supplied by
> + the dc-to-dc converter is:
> + 23000000
> + 24500000
> + 27000000
> + 29500000
> +
> +Optional for every channel:
> + - adi,mode:
> + Valid values for DAC modes is:
> + 0: 0 V to 5 V voltage range.
> + 1: 0 V to 10 V voltage range.
> + 2: Plus minus 5 V voltage range.
> + 3: Plus minus 10 V voltage range.
> + 4: 4 mA to 20 mA current range.
> + 5: 0 mA to 20 mA current range.
> + 6: 0 mA to 24 mA current range.
> + - adi,ext-current-sense-resistor: boolean set if the hardware a external
> + current sense resistor.
> + - adi,enable-voltage-overrange: boolean enable voltage overrange
> + - adi,slew: Array of slewrate settings should contain 3 fields:
> + 1: Should be either 0 or 1 in order to enable or disable slewrate.
> + 2: Slew rate settings:
> + Valid values for the slew rate update frequency:
> + 64000
> + 32000
> + 16000
> + 8000
> + 4000
> + 2000
> + 1000
> + 500
> + 250
> + 125
> + 64
> + 32
> + 16
> + 8
> + 4
> + 0
> + 3: Slew step size:
> + Valid values for the step size LSBs:
> + 1
> + 2
> + 4
> + 16
> + 32
> + 64
> + 128
> + 256
> +
> +Example:
> +dac@0 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> + compatible = "adi,ad5755";
> + reg = <0>;
> + spi-max-frequency = <1000000>;
> + spi-cpha;
> + adi,dc-dc-phase = <0>;
> + adi,dc-dc-freq-hz = <410000>;
> + adi,dc-dc-max-microvolt = <23000000>;
> + channel@0 {
> + reg = <0>;
> + adi,mode = <4>;
> + adi,ext-current-sense-resistor;
> + adi,slew = <0 64000 1>;
> + };
> + channel@1 {
> + reg = <1>;
> + adi,mode = <4>;
> + adi,ext-current-sense-resistor;
> + adi,slew = <0 64000 1>;
> + };
> + channel@2 {
> + reg = <2>;
> + adi,mode = <4>;
> + adi,ext-current-sense-resistor;
> + adi,slew = <0 64000 1>;
> + };
> + channel@3 {
> + reg = <3>;
> + adi,mode = <4>;
> + adi,ext-current-sense-resistor;
> + adi,slew = <0 64000 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
