| Message ID | 20200810033806.15503-1-chris.ruehl@gtsys.com.hk (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | iio/dac: convert ltc2632.txt to lltc,ltc2632.yaml | expand |
On Mon, 10 Aug 2020 11:37:52 +0800 Chris Ruehl <chris.ruehl@gtsys.com.hk> wrote: Hi Chris, This will need a dt-binding maintainer review, but in the meantime... Look at the title for dt-bindings patches and match that format. One question for Rob inline. A quick grep suggests we are very inconsistent on whether we use the multiline block thing for description fields or not. Is it needed? > Conversion of the ltc2632 to yaml format and name the file to > 'lltc,ltc2632.yaml'. > > Signed-off-by: Chris Ruehl <chris.ruehl@gtsys.com.hk> > --- > v6: > Remove tab and drop unused label > Related patches already in linux-next. > [PATCH v5 1/3] iio: documentation ltc2632_chip_info add num_channels > linux-next commit: 6f1c9e0da9aae51177457731357ae8a2c8af27cd > PATCH v5 2/3] iio: DAC extension for ltc2634-12/10/8 > linux-next commit: aefa5bc87c808dd08db2fc79ebdbf19ed4af7be2 > > .../bindings/iio/dac/lltc,ltc2632.yaml | 77 +++++++++++++++++++ > .../devicetree/bindings/iio/dac/ltc2632.txt | 49 ------------ > 2 files changed, 77 insertions(+), 49 deletions(-) > create mode 100644 Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml > delete mode 100644 Documentation/devicetree/bindings/iio/dac/ltc2632.txt > > diff --git a/Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml b/Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml > new file mode 100644 > index 000000000000..edf804d0aca2 > --- /dev/null > +++ b/Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml > @@ -0,0 +1,77 @@ > +# SPDX-License-Identifier: GPL-2.0 OR BSD-2-Clause > +%YAML 1.2 > +--- > +$id: "http://devicetree.org/schemas/iio/dac/lltc,ltc2632.yaml#" > +$schema: "http://devicetree.org/meta-schemas/core.yaml#" > + > +title: Linear Technology LTC263x 12-/10-/8-Bit Rail-to-Rail DAC > + > +maintainers: > + - Michael Hennerich <michael.hennerich@analog.com> > + > +description: | > + Bindings for the Linear Technology LTC2632/2634/2636 DAC > + Datasheet can be found here: https://www.analog.com/media/en/technical-documentation/data-sheets/LTC263[246].pdf > + > +properties: > + compatible: > + enum: > + - lltc,ltc2632-l12 > + - lltc,ltc2632-l10 > + - lltc,ltc2632-l8 > + - lltc,ltc2632-h12 > + - lltc,ltc2632-h10 > + - lltc,ltc2632-h8 > + - lltc,ltc2634-l12 > + - lltc,ltc2634-l10 > + - lltc,ltc2634-l8 > + - lltc,ltc2634-h12 > + - lltc,ltc2634-h10 > + - lltc,ltc2634-h8 > + - lltc,ltc2636-l12 > + - lltc,ltc2636-l10 > + - lltc,ltc2636-l8 > + - lltc,ltc2636-h12 > + - lltc,ltc2636-h10 > + - lltc,ltc2636-h8 > + > + reg: > + maxItems: 1 > + > + spi-max-frequency: > + maximum: 2000000 > + > + vref-supply: > + description: Does this need a | ? > + Phandle to the external reference voltage supply. This should > + only be set if there is an external reference voltage connected to the VREF > + pin. If the property is not set the internal reference is used. > + > +required: > + - compatible > + - reg > + > +additionalProperties: false > + > +examples: > + - | > + vref: regulator-vref { > + compatible = "regulator-fixed"; > + regulator-name = "vref-ltc2632"; > + regulator-min-microvolt = <1250000>; > + regulator-max-microvolt = <1250000>; > + regulator-always-on; > + }; > + > + spi { > + #address-cells = <1>; > + #size-cells = <0>; > + > + dac@0 { > + compatible = "lltc,ltc2632"; > + reg = <0>; /* CS0 */ > + spi-max-frequency = <1000000>; > + vref-supply = <&vref>; > + }; > + }; > +... > diff --git a/Documentation/devicetree/bindings/iio/dac/ltc2632.txt b/Documentation/devicetree/bindings/iio/dac/ltc2632.txt > deleted file mode 100644 > index 1ab9570cf219..000000000000 > --- a/Documentation/devicetree/bindings/iio/dac/ltc2632.txt > +++ /dev/null > @@ -1,49 +0,0 @@ > -Linear Technology LTC2632/2634/2636 DAC > - > -Required properties: > - - compatible: Has to contain one of the following: > - lltc,ltc2632-l12 > - lltc,ltc2632-l10 > - lltc,ltc2632-l8 > - lltc,ltc2632-h12 > - lltc,ltc2632-h10 > - lltc,ltc2632-h8 > - lltc,ltc2634-l12 > - lltc,ltc2634-l10 > - lltc,ltc2634-l8 > - lltc,ltc2634-h12 > - lltc,ltc2634-h10 > - lltc,ltc2634-h8 > - lltc,ltc2636-l12 > - lltc,ltc2636-l10 > - lltc,ltc2636-l8 > - lltc,ltc2636-h12 > - lltc,ltc2636-h10 > - lltc,ltc2636-h8 > - > -Property rules described in Documentation/devicetree/bindings/spi/spi-bus.txt > -apply. In particular, "reg" and "spi-max-frequency" properties must be given. > - > -Optional properties: > - - vref-supply: Phandle to the external reference voltage supply. This should > - only be set if there is an external reference voltage connected to the VREF > - pin. If the property is not set the internal reference is used. > - > -Example: > - > - vref: regulator-vref { > - compatible = "regulator-fixed"; > - regulator-name = "vref-ltc2632"; > - regulator-min-microvolt = <1250000>; > - regulator-max-microvolt = <1250000>; > - regulator-always-on; > - }; > - > - spi_master { > - dac: ltc2632@0 { > - compatible = "lltc,ltc2632-l12"; > - reg = <0>; /* CS0 */ > - spi-max-frequency = <1000000>; > - vref-supply = <&vref>; /* optional */ > - }; > - };
Hi Jonathan On 16/8/2020 5:27 pm, Jonathan Cameron wrote: > On Mon, 10 Aug 2020 11:37:52 +0800 > Chris Ruehl <chris.ruehl@gtsys.com.hk> wrote: > > Hi Chris, > > This will need a dt-binding maintainer review, but in the meantime... > > Look at the title for dt-bindings patches and match that format. > > One question for Rob inline. A quick grep suggests we are > very inconsistent on whether we use the multiline block > thing for description fields or not. > > Is it needed? ok, next submit [PATCH] dt-bindings: iio/dac: ... just wait for Rob and his advice on your question regarding the | in the block. Regards Chris
On Sun, Aug 16, 2020 at 10:27:33AM +0100, Jonathan Cameron wrote: > On Mon, 10 Aug 2020 11:37:52 +0800 > Chris Ruehl <chris.ruehl@gtsys.com.hk> wrote: > > Hi Chris, > > This will need a dt-binding maintainer review, but in the meantime... > > Look at the title for dt-bindings patches and match that format. > > One question for Rob inline. A quick grep suggests we are > very inconsistent on whether we use the multiline block > thing for description fields or not. > > Is it needed? > > > Conversion of the ltc2632 to yaml format and name the file to > > 'lltc,ltc2632.yaml'. > > > > Signed-off-by: Chris Ruehl <chris.ruehl@gtsys.com.hk> > > --- > > v6: > > Remove tab and drop unused label > > Related patches already in linux-next. > > [PATCH v5 1/3] iio: documentation ltc2632_chip_info add num_channels > > linux-next commit: 6f1c9e0da9aae51177457731357ae8a2c8af27cd > > PATCH v5 2/3] iio: DAC extension for ltc2634-12/10/8 > > linux-next commit: aefa5bc87c808dd08db2fc79ebdbf19ed4af7be2 > > > > .../bindings/iio/dac/lltc,ltc2632.yaml | 77 +++++++++++++++++++ > > .../devicetree/bindings/iio/dac/ltc2632.txt | 49 ------------ > > 2 files changed, 77 insertions(+), 49 deletions(-) > > create mode 100644 Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml > > delete mode 100644 Documentation/devicetree/bindings/iio/dac/ltc2632.txt > > > > diff --git a/Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml b/Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml > > new file mode 100644 > > index 000000000000..edf804d0aca2 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml > > @@ -0,0 +1,77 @@ > > +# SPDX-License-Identifier: GPL-2.0 OR BSD-2-Clause > > +%YAML 1.2 > > +--- > > +$id: "http://devicetree.org/schemas/iio/dac/lltc,ltc2632.yaml#" > > +$schema: "http://devicetree.org/meta-schemas/core.yaml#" > > + > > +title: Linear Technology LTC263x 12-/10-/8-Bit Rail-to-Rail DAC > > + > > +maintainers: > > + - Michael Hennerich <michael.hennerich@analog.com> > > + > > +description: | > > + Bindings for the Linear Technology LTC2632/2634/2636 DAC > > + Datasheet can be found here: https://www.analog.com/media/en/technical-documentation/data-sheets/LTC263[246].pdf > > + > > +properties: > > + compatible: > > + enum: > > + - lltc,ltc2632-l12 > > + - lltc,ltc2632-l10 > > + - lltc,ltc2632-l8 > > + - lltc,ltc2632-h12 > > + - lltc,ltc2632-h10 > > + - lltc,ltc2632-h8 > > + - lltc,ltc2634-l12 > > + - lltc,ltc2634-l10 > > + - lltc,ltc2634-l8 > > + - lltc,ltc2634-h12 > > + - lltc,ltc2634-h10 > > + - lltc,ltc2634-h8 > > + - lltc,ltc2636-l12 > > + - lltc,ltc2636-l10 > > + - lltc,ltc2636-l8 > > + - lltc,ltc2636-h12 > > + - lltc,ltc2636-h10 > > + - lltc,ltc2636-h8 > > + > > + reg: > > + maxItems: 1 > > + > > + spi-max-frequency: > > + maximum: 2000000 > > + > > + vref-supply: > > + description: > Does this need a | ? No. Only needed if you want to preserve line breaks and other formatting. > > + Phandle to the external reference voltage supply. This should > > + only be set if there is an external reference voltage connected to the VREF > > + pin. If the property is not set the internal reference is used. > > + > > +required: > > + - compatible > > + - reg > > + > > +additionalProperties: false > > + > > +examples: > > + - | > > + vref: regulator-vref { > > + compatible = "regulator-fixed"; > > + regulator-name = "vref-ltc2632"; > > + regulator-min-microvolt = <1250000>; > > + regulator-max-microvolt = <1250000>; > > + regulator-always-on; > > + }; > > + > > + spi { > > + #address-cells = <1>; > > + #size-cells = <0>; > > + > > + dac@0 { > > + compatible = "lltc,ltc2632"; > > + reg = <0>; /* CS0 */ > > + spi-max-frequency = <1000000>; > > + vref-supply = <&vref>; > > + }; > > + }; > > +... > > diff --git a/Documentation/devicetree/bindings/iio/dac/ltc2632.txt b/Documentation/devicetree/bindings/iio/dac/ltc2632.txt > > deleted file mode 100644 > > index 1ab9570cf219..000000000000 > > --- a/Documentation/devicetree/bindings/iio/dac/ltc2632.txt > > +++ /dev/null > > @@ -1,49 +0,0 @@ > > -Linear Technology LTC2632/2634/2636 DAC > > - > > -Required properties: > > - - compatible: Has to contain one of the following: > > - lltc,ltc2632-l12 > > - lltc,ltc2632-l10 > > - lltc,ltc2632-l8 > > - lltc,ltc2632-h12 > > - lltc,ltc2632-h10 > > - lltc,ltc2632-h8 > > - lltc,ltc2634-l12 > > - lltc,ltc2634-l10 > > - lltc,ltc2634-l8 > > - lltc,ltc2634-h12 > > - lltc,ltc2634-h10 > > - lltc,ltc2634-h8 > > - lltc,ltc2636-l12 > > - lltc,ltc2636-l10 > > - lltc,ltc2636-l8 > > - lltc,ltc2636-h12 > > - lltc,ltc2636-h10 > > - lltc,ltc2636-h8 > > - > > -Property rules described in Documentation/devicetree/bindings/spi/spi-bus.txt > > -apply. In particular, "reg" and "spi-max-frequency" properties must be given. > > - > > -Optional properties: > > - - vref-supply: Phandle to the external reference voltage supply. This should > > - only be set if there is an external reference voltage connected to the VREF > > - pin. If the property is not set the internal reference is used. > > - > > -Example: > > - > > - vref: regulator-vref { > > - compatible = "regulator-fixed"; > > - regulator-name = "vref-ltc2632"; > > - regulator-min-microvolt = <1250000>; > > - regulator-max-microvolt = <1250000>; > > - regulator-always-on; > > - }; > > - > > - spi_master { > > - dac: ltc2632@0 { > > - compatible = "lltc,ltc2632-l12"; > > - reg = <0>; /* CS0 */ > > - spi-max-frequency = <1000000>; > > - vref-supply = <&vref>; /* optional */ > > - }; > > - }; >
On Mon, 10 Aug 2020 11:37:52 +0800, Chris Ruehl wrote: > Conversion of the ltc2632 to yaml format and name the file to > 'lltc,ltc2632.yaml'. > > Signed-off-by: Chris Ruehl <chris.ruehl@gtsys.com.hk> > --- > v6: > Remove tab and drop unused label > Related patches already in linux-next. > [PATCH v5 1/3] iio: documentation ltc2632_chip_info add num_channels > linux-next commit: 6f1c9e0da9aae51177457731357ae8a2c8af27cd > PATCH v5 2/3] iio: DAC extension for ltc2634-12/10/8 > linux-next commit: aefa5bc87c808dd08db2fc79ebdbf19ed4af7be2 > > .../bindings/iio/dac/lltc,ltc2632.yaml | 77 +++++++++++++++++++ > .../devicetree/bindings/iio/dac/ltc2632.txt | 49 ------------ > 2 files changed, 77 insertions(+), 49 deletions(-) > create mode 100644 Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml > delete mode 100644 Documentation/devicetree/bindings/iio/dac/ltc2632.txt > Reviewed-by: Rob Herring <robh@kernel.org>
diff --git a/Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml b/Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml new file mode 100644 index 000000000000..edf804d0aca2 --- /dev/null +++ b/Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml @@ -0,0 +1,77 @@ +# SPDX-License-Identifier: GPL-2.0 OR BSD-2-Clause +%YAML 1.2 +--- +$id: "http://devicetree.org/schemas/iio/dac/lltc,ltc2632.yaml#" +$schema: "http://devicetree.org/meta-schemas/core.yaml#" + +title: Linear Technology LTC263x 12-/10-/8-Bit Rail-to-Rail DAC + +maintainers: + - Michael Hennerich <michael.hennerich@analog.com> + +description: | + Bindings for the Linear Technology LTC2632/2634/2636 DAC + Datasheet can be found here: https://www.analog.com/media/en/technical-documentation/data-sheets/LTC263[246].pdf + +properties: + compatible: + enum: + - lltc,ltc2632-l12 + - lltc,ltc2632-l10 + - lltc,ltc2632-l8 + - lltc,ltc2632-h12 + - lltc,ltc2632-h10 + - lltc,ltc2632-h8 + - lltc,ltc2634-l12 + - lltc,ltc2634-l10 + - lltc,ltc2634-l8 + - lltc,ltc2634-h12 + - lltc,ltc2634-h10 + - lltc,ltc2634-h8 + - lltc,ltc2636-l12 + - lltc,ltc2636-l10 + - lltc,ltc2636-l8 + - lltc,ltc2636-h12 + - lltc,ltc2636-h10 + - lltc,ltc2636-h8 + + reg: + maxItems: 1 + + spi-max-frequency: + maximum: 2000000 + + vref-supply: + description: + Phandle to the external reference voltage supply. This should + only be set if there is an external reference voltage connected to the VREF + pin. If the property is not set the internal reference is used. + +required: + - compatible + - reg + +additionalProperties: false + +examples: + - | + vref: regulator-vref { + compatible = "regulator-fixed"; + regulator-name = "vref-ltc2632"; + regulator-min-microvolt = <1250000>; + regulator-max-microvolt = <1250000>; + regulator-always-on; + }; + + spi { + #address-cells = <1>; + #size-cells = <0>; + + dac@0 { + compatible = "lltc,ltc2632"; + reg = <0>; /* CS0 */ + spi-max-frequency = <1000000>; + vref-supply = <&vref>; + }; + }; +... diff --git a/Documentation/devicetree/bindings/iio/dac/ltc2632.txt b/Documentation/devicetree/bindings/iio/dac/ltc2632.txt deleted file mode 100644 index 1ab9570cf219..000000000000 --- a/Documentation/devicetree/bindings/iio/dac/ltc2632.txt +++ /dev/null @@ -1,49 +0,0 @@ -Linear Technology LTC2632/2634/2636 DAC - -Required properties: - - compatible: Has to contain one of the following: - lltc,ltc2632-l12 - lltc,ltc2632-l10 - lltc,ltc2632-l8 - lltc,ltc2632-h12 - lltc,ltc2632-h10 - lltc,ltc2632-h8 - lltc,ltc2634-l12 - lltc,ltc2634-l10 - lltc,ltc2634-l8 - lltc,ltc2634-h12 - lltc,ltc2634-h10 - lltc,ltc2634-h8 - lltc,ltc2636-l12 - lltc,ltc2636-l10 - lltc,ltc2636-l8 - lltc,ltc2636-h12 - lltc,ltc2636-h10 - lltc,ltc2636-h8 - -Property rules described in Documentation/devicetree/bindings/spi/spi-bus.txt -apply. In particular, "reg" and "spi-max-frequency" properties must be given. - -Optional properties: - - vref-supply: Phandle to the external reference voltage supply. This should - only be set if there is an external reference voltage connected to the VREF - pin. If the property is not set the internal reference is used. - -Example: - - vref: regulator-vref { - compatible = "regulator-fixed"; - regulator-name = "vref-ltc2632"; - regulator-min-microvolt = <1250000>; - regulator-max-microvolt = <1250000>; - regulator-always-on; - }; - - spi_master { - dac: ltc2632@0 { - compatible = "lltc,ltc2632-l12"; - reg = <0>; /* CS0 */ - spi-max-frequency = <1000000>; - vref-supply = <&vref>; /* optional */ - }; - };
Conversion of the ltc2632 to yaml format and name the file to 'lltc,ltc2632.yaml'. Signed-off-by: Chris Ruehl <chris.ruehl@gtsys.com.hk> --- v6: Remove tab and drop unused label Related patches already in linux-next. [PATCH v5 1/3] iio: documentation ltc2632_chip_info add num_channels linux-next commit: 6f1c9e0da9aae51177457731357ae8a2c8af27cd PATCH v5 2/3] iio: DAC extension for ltc2634-12/10/8 linux-next commit: aefa5bc87c808dd08db2fc79ebdbf19ed4af7be2 .../bindings/iio/dac/lltc,ltc2632.yaml | 77 +++++++++++++++++++ .../devicetree/bindings/iio/dac/ltc2632.txt | 49 ------------ 2 files changed, 77 insertions(+), 49 deletions(-) create mode 100644 Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml delete mode 100644 Documentation/devicetree/bindings/iio/dac/ltc2632.txt