* [PATCH v5] iio: patch set ltc2632 @ 2020-04-20 4:26 Chris Ruehl 2020-04-20 4:26 ` [PATCH v5 1/3] iio: documentation ltc2632_chip_info add num_channels Chris Ruehl ` (2 more replies) 0 siblings, 3 replies; 10+ messages in thread From: Chris Ruehl @ 2020-04-20 4:26 UTC (permalink / raw) To: Chris Ruehl Cc: devicetree, Michael Hennerich, Jonathan Cameron, Hartmut Knaack, Lars-Peter Clausen, Peter Meerwald-Stadler, Uwe Kleine-König, Richard Fontana, Alexios Zavras, Thomas Gleixner, linux-iio, linux-kernel Patchset to extend ltc2632 spi driver to support the similar chip set ltc2634. * Patch v2 1/3 update documentation struct ltc2632_chip_info * Patch v2 2/3 patch ltc2632.c,Kconfig,ltc2632.txt(devtree) * Patch v2 3/3 convert ltc2632.txt to yaml format named lltc,ltc2632.yaml * Patch v3 2/3 : correct help text * Patch v4 1/3 : correct spelling in commit */3 hide change history below --- * Patch v5 : 1/3,2/3 add Accept-by and fixed tag, 3/3 maintainer & correct required seciton Signed-off-by: Chris Ruehl <chris.ruehl@gtsys.com.hk> --- ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v5 1/3] iio: documentation ltc2632_chip_info add num_channels 2020-04-20 4:26 [PATCH v5] iio: patch set ltc2632 Chris Ruehl @ 2020-04-20 4:26 ` Chris Ruehl 2020-04-25 19:37 ` Jonathan Cameron 2020-04-20 4:26 ` [PATCH v5 2/3] iio: DAC extension for ltc2634-12/10/8 Chris Ruehl 2020-04-20 4:26 ` [PATCH v5 3/3] iio/dac: convert ltc2632.txt to lltc,ltc2632.yaml Chris Ruehl 2 siblings, 1 reply; 10+ messages in thread From: Chris Ruehl @ 2020-04-20 4:26 UTC (permalink / raw) To: Chris Ruehl Cc: devicetree, Michael Hennerich, Uwe Kleine-König, Jonathan Cameron, Hartmut Knaack, Lars-Peter Clausen, Peter Meerwald-Stadler, Alexios Zavras, Greg Kroah-Hartman, Thomas Gleixner, linux-iio, linux-kernel The documentation for ltc_2632_chip_info missed the desciption for the num_channels. This trivial patch adds it. Signed-off-by: Chris Ruehl <chris.ruehl@gtsys.com.hk> Fixes: 9f15a4a0adc9 ("iio: dac: ltc2632: add support for LTC2636 family") Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> --- v5: tags added v4: Fix commit text. drivers/iio/dac/ltc2632.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/iio/dac/ltc2632.c b/drivers/iio/dac/ltc2632.c index 7adc91056aa1..2a84ea654645 100644 --- a/drivers/iio/dac/ltc2632.c +++ b/drivers/iio/dac/ltc2632.c @@ -24,6 +24,7 @@ /** * struct ltc2632_chip_info - chip specific information * @channels: channel spec for the DAC + * @num_channels: DAC channel count of the chip * @vref_mv: internal reference voltage */ struct ltc2632_chip_info { -- 2.20.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v5 1/3] iio: documentation ltc2632_chip_info add num_channels 2020-04-20 4:26 ` [PATCH v5 1/3] iio: documentation ltc2632_chip_info add num_channels Chris Ruehl @ 2020-04-25 19:37 ` Jonathan Cameron 0 siblings, 0 replies; 10+ messages in thread From: Jonathan Cameron @ 2020-04-25 19:37 UTC (permalink / raw) To: Chris Ruehl Cc: devicetree, Michael Hennerich, Uwe Kleine-König, Hartmut Knaack, Lars-Peter Clausen, Peter Meerwald-Stadler, Alexios Zavras, Greg Kroah-Hartman, Thomas Gleixner, linux-iio, linux-kernel On Mon, 20 Apr 2020 12:26:06 +0800 Chris Ruehl <chris.ruehl@gtsys.com.hk> wrote: > The documentation for ltc_2632_chip_info missed the desciption for the > num_channels. This trivial patch adds it. > > Signed-off-by: Chris Ruehl <chris.ruehl@gtsys.com.hk> > > Fixes: 9f15a4a0adc9 ("iio: dac: ltc2632: add support for LTC2636 family") > Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> Might as well take this one as it stands on its own. Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to play with it. Jonathan > --- > v5: > tags added > > v4: > Fix commit text. > drivers/iio/dac/ltc2632.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/iio/dac/ltc2632.c b/drivers/iio/dac/ltc2632.c > index 7adc91056aa1..2a84ea654645 100644 > --- a/drivers/iio/dac/ltc2632.c > +++ b/drivers/iio/dac/ltc2632.c > @@ -24,6 +24,7 @@ > /** > * struct ltc2632_chip_info - chip specific information > * @channels: channel spec for the DAC > + * @num_channels: DAC channel count of the chip > * @vref_mv: internal reference voltage > */ > struct ltc2632_chip_info { ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v5 2/3] iio: DAC extension for ltc2634-12/10/8 2020-04-20 4:26 [PATCH v5] iio: patch set ltc2632 Chris Ruehl 2020-04-20 4:26 ` [PATCH v5 1/3] iio: documentation ltc2632_chip_info add num_channels Chris Ruehl @ 2020-04-20 4:26 ` Chris Ruehl 2020-04-30 15:17 ` Rob Herring 2020-04-20 4:26 ` [PATCH v5 3/3] iio/dac: convert ltc2632.txt to lltc,ltc2632.yaml Chris Ruehl 2 siblings, 1 reply; 10+ messages in thread From: Chris Ruehl @ 2020-04-20 4:26 UTC (permalink / raw) To: Chris Ruehl Cc: devicetree, Michael Hennerich, Uwe Kleine-König, Jonathan Cameron, Hartmut Knaack, Lars-Peter Clausen, Peter Meerwald-Stadler, Rob Herring, Alexandru Ardelean, Alexandru Tachici, Stefan Popa, Thomas Gleixner, Richard Fontana, linux-iio, linux-kernel This patch add support for Analog Devices (Linear Technology) LTC2634 Quad 12-/10-/8-Bit Rail-to-Rail DAC. The SPI functionality based on them from LTC2632 therefor add the definitions only and update the Kconfig. Signed-off-by: Chris Ruehl <chris.ruehl@gtsys.com.hk> Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> --- v5: -add Acked-by v4: -hide version from commit text v3: -Consistens decimal separator in help text v2: -Send the documentation fix in a extra patch -Kconfig simplify the driver description, details to the help text -ltc2632.txt add to this patch-set (prepare to convert to yaml) .../devicetree/bindings/iio/dac/ltc2632.txt | 8 ++- drivers/iio/dac/Kconfig | 6 +- drivers/iio/dac/ltc2632.c | 60 +++++++++++++++++++ 3 files changed, 70 insertions(+), 4 deletions(-) diff --git a/Documentation/devicetree/bindings/iio/dac/ltc2632.txt b/Documentation/devicetree/bindings/iio/dac/ltc2632.txt index 338c3220f01a..1ab9570cf219 100644 --- a/Documentation/devicetree/bindings/iio/dac/ltc2632.txt +++ b/Documentation/devicetree/bindings/iio/dac/ltc2632.txt @@ -1,4 +1,4 @@ -Linear Technology LTC2632/2636 DAC +Linear Technology LTC2632/2634/2636 DAC Required properties: - compatible: Has to contain one of the following: @@ -8,6 +8,12 @@ Required properties: 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 diff --git a/drivers/iio/dac/Kconfig b/drivers/iio/dac/Kconfig index 93744011b63f..3728f6325501 100644 --- a/drivers/iio/dac/Kconfig +++ b/drivers/iio/dac/Kconfig @@ -279,12 +279,12 @@ config LTC1660 module will be called ltc1660. config LTC2632 - tristate "Linear Technology LTC2632-12/10/8 and LTC2636-12/10/8 DAC spi driver" + tristate "Linear Technology LTC2632-12/10/8 and similar DAC spi driver" depends on SPI help Say yes here to build support for Linear Technology - LTC2632-12, LTC2632-10, LTC2632-8, LTC2636-12, LTC2636-10 and - LTC2636-8 converters (DAC). + LTC2632, LTC2634 and LTC2636 DAC resolution 12/10/8 bit + low 0-2.5V and high 0-4.096V range converters. To compile this driver as a module, choose M here: the module will be called ltc2632. diff --git a/drivers/iio/dac/ltc2632.c b/drivers/iio/dac/ltc2632.c index 2a84ea654645..e939d7f81014 100644 --- a/drivers/iio/dac/ltc2632.c +++ b/drivers/iio/dac/ltc2632.c @@ -54,6 +54,12 @@ enum ltc2632_supported_device_ids { ID_LTC2632H12, ID_LTC2632H10, ID_LTC2632H8, + ID_LTC2634L12, + ID_LTC2634L10, + ID_LTC2634L8, + ID_LTC2634H12, + ID_LTC2634H10, + ID_LTC2634H8, ID_LTC2636L12, ID_LTC2636L10, ID_LTC2636L8, @@ -236,6 +242,36 @@ static const struct ltc2632_chip_info ltc2632_chip_info_tbl[] = { .num_channels = 2, .vref_mv = 4096, }, + [ID_LTC2634L12] = { + .channels = ltc2632x12_channels, + .num_channels = 4, + .vref_mv = 2500, + }, + [ID_LTC2634L10] = { + .channels = ltc2632x10_channels, + .num_channels = 4, + .vref_mv = 2500, + }, + [ID_LTC2634L8] = { + .channels = ltc2632x8_channels, + .num_channels = 4, + .vref_mv = 2500, + }, + [ID_LTC2634H12] = { + .channels = ltc2632x12_channels, + .num_channels = 4, + .vref_mv = 4096, + }, + [ID_LTC2634H10] = { + .channels = ltc2632x10_channels, + .num_channels = 4, + .vref_mv = 4096, + }, + [ID_LTC2634H8] = { + .channels = ltc2632x8_channels, + .num_channels = 4, + .vref_mv = 4096, + }, [ID_LTC2636L12] = { .channels = ltc2632x12_channels, .num_channels = 8, @@ -357,6 +393,12 @@ static const struct spi_device_id ltc2632_id[] = { { "ltc2632-h12", (kernel_ulong_t)<c2632_chip_info_tbl[ID_LTC2632H12] }, { "ltc2632-h10", (kernel_ulong_t)<c2632_chip_info_tbl[ID_LTC2632H10] }, { "ltc2632-h8", (kernel_ulong_t)<c2632_chip_info_tbl[ID_LTC2632H8] }, + { "ltc2634-l12", (kernel_ulong_t)<c2632_chip_info_tbl[ID_LTC2634L12] }, + { "ltc2634-l10", (kernel_ulong_t)<c2632_chip_info_tbl[ID_LTC2634L10] }, + { "ltc2634-l8", (kernel_ulong_t)<c2632_chip_info_tbl[ID_LTC2634L8] }, + { "ltc2634-h12", (kernel_ulong_t)<c2632_chip_info_tbl[ID_LTC2634H12] }, + { "ltc2634-h10", (kernel_ulong_t)<c2632_chip_info_tbl[ID_LTC2634H10] }, + { "ltc2634-h8", (kernel_ulong_t)<c2632_chip_info_tbl[ID_LTC2634H8] }, { "ltc2636-l12", (kernel_ulong_t)<c2632_chip_info_tbl[ID_LTC2636L12] }, { "ltc2636-l10", (kernel_ulong_t)<c2632_chip_info_tbl[ID_LTC2636L10] }, { "ltc2636-l8", (kernel_ulong_t)<c2632_chip_info_tbl[ID_LTC2636L8] }, @@ -386,6 +428,24 @@ static const struct of_device_id ltc2632_of_match[] = { }, { .compatible = "lltc,ltc2632-h8", .data = <c2632_chip_info_tbl[ID_LTC2632H8] + }, { + .compatible = "lltc,ltc2634-l12", + .data = <c2632_chip_info_tbl[ID_LTC2634L12] + }, { + .compatible = "lltc,ltc2634-l10", + .data = <c2632_chip_info_tbl[ID_LTC2634L10] + }, { + .compatible = "lltc,ltc2634-l8", + .data = <c2632_chip_info_tbl[ID_LTC2634L8] + }, { + .compatible = "lltc,ltc2634-h12", + .data = <c2632_chip_info_tbl[ID_LTC2634H12] + }, { + .compatible = "lltc,ltc2634-h10", + .data = <c2632_chip_info_tbl[ID_LTC2634H10] + }, { + .compatible = "lltc,ltc2634-h8", + .data = <c2632_chip_info_tbl[ID_LTC2634H8] }, { .compatible = "lltc,ltc2636-l12", .data = <c2632_chip_info_tbl[ID_LTC2636L12] -- 2.20.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v5 2/3] iio: DAC extension for ltc2634-12/10/8 2020-04-20 4:26 ` [PATCH v5 2/3] iio: DAC extension for ltc2634-12/10/8 Chris Ruehl @ 2020-04-30 15:17 ` Rob Herring 0 siblings, 0 replies; 10+ messages in thread From: Rob Herring @ 2020-04-30 15:17 UTC (permalink / raw) To: Chris Ruehl Cc: Chris Ruehl, devicetree, Michael Hennerich, Uwe Kleine-König, Jonathan Cameron, Hartmut Knaack, Lars-Peter Clausen, Peter Meerwald-Stadler, Alexandru Ardelean, Alexandru Tachici, Stefan Popa, Thomas Gleixner, Richard Fontana, linux-iio, linux-kernel On Mon, 20 Apr 2020 12:26:07 +0800, Chris Ruehl wrote: > This patch add support for Analog Devices (Linear Technology) > LTC2634 Quad 12-/10-/8-Bit Rail-to-Rail DAC. > The SPI functionality based on them from LTC2632 therefor > add the definitions only and update the Kconfig. > > Signed-off-by: Chris Ruehl <chris.ruehl@gtsys.com.hk> > > Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> > --- > v5: > -add Acked-by > v4: > -hide version from commit text > v3: > -Consistens decimal separator in help text > v2: > -Send the documentation fix in a extra patch > -Kconfig simplify the driver description, details to the help text > -ltc2632.txt add to this patch-set (prepare to convert to yaml) > .../devicetree/bindings/iio/dac/ltc2632.txt | 8 ++- > drivers/iio/dac/Kconfig | 6 +- > drivers/iio/dac/ltc2632.c | 60 +++++++++++++++++++ > 3 files changed, 70 insertions(+), 4 deletions(-) > Acked-by: Rob Herring <robh@kernel.org> ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v5 3/3] iio/dac: convert ltc2632.txt to lltc,ltc2632.yaml 2020-04-20 4:26 [PATCH v5] iio: patch set ltc2632 Chris Ruehl 2020-04-20 4:26 ` [PATCH v5 1/3] iio: documentation ltc2632_chip_info add num_channels Chris Ruehl 2020-04-20 4:26 ` [PATCH v5 2/3] iio: DAC extension for ltc2634-12/10/8 Chris Ruehl @ 2020-04-20 4:26 ` Chris Ruehl 2020-04-25 19:35 ` Jonathan Cameron 2020-04-30 15:24 ` Rob Herring 2 siblings, 2 replies; 10+ messages in thread From: Chris Ruehl @ 2020-04-20 4:26 UTC (permalink / raw) To: Chris Ruehl Cc: devicetree, Michael Hennerich, Jonathan Cameron, Hartmut Knaack, Lars-Peter Clausen, Peter Meerwald-Stadler, Rob Herring, Uwe Kleine-König, Alexios Zavras, Steve Winslow, Thomas Gleixner, linux-iio, linux-kernel Conversion of the ltc2632 to yaml format and name the file 'lltc,ltc2632.yaml'. Signed-off-by: Chris Ruehl <chris.ruehl@gtsys.com.hk> --- v5: correct require section set maintainer of analog.com v4..v2: no change .../bindings/iio/dac/lltc,ltc2632.yaml | 76 +++++++++++++++++++ .../devicetree/bindings/iio/dac/ltc2632.txt | 49 ------------ 2 files changed, 76 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..b0043144fbc4 --- /dev/null +++ b/Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml @@ -0,0 +1,76 @@ +# 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_master { + #address-cells = <1>; + #size-cells = <0>; + + dac: ltc2632@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 */ - }; - }; -- 2.20.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v5 3/3] iio/dac: convert ltc2632.txt to lltc,ltc2632.yaml 2020-04-20 4:26 ` [PATCH v5 3/3] iio/dac: convert ltc2632.txt to lltc,ltc2632.yaml Chris Ruehl @ 2020-04-25 19:35 ` Jonathan Cameron 2020-04-27 1:09 ` Chris Ruehl 2020-04-30 15:24 ` Rob Herring 1 sibling, 1 reply; 10+ messages in thread From: Jonathan Cameron @ 2020-04-25 19:35 UTC (permalink / raw) To: Chris Ruehl Cc: devicetree, Michael Hennerich, Hartmut Knaack, Lars-Peter Clausen, Peter Meerwald-Stadler, Rob Herring, Uwe Kleine-König, Alexios Zavras, Steve Winslow, Thomas Gleixner, linux-iio, linux-kernel On Mon, 20 Apr 2020 12:26:08 +0800 Chris Ruehl <chris.ruehl@gtsys.com.hk> wrote: > Conversion of the ltc2632 to yaml format and name the file 'lltc,ltc2632.yaml'. > > Signed-off-by: Chris Ruehl <chris.ruehl@gtsys.com.hk> There is something odd going on with indentation but otherwise looks fine to me. However we need to leave time for a final DT review. Thanks, Jonathan > --- > v5: > correct require section > set maintainer of analog.com > v4..v2: no change > > .../bindings/iio/dac/lltc,ltc2632.yaml | 76 +++++++++++++++++++ > .../devicetree/bindings/iio/dac/ltc2632.txt | 49 ------------ > 2 files changed, 76 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..b0043144fbc4 > --- /dev/null > +++ b/Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml > @@ -0,0 +1,76 @@ > +# 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. Odd intent going on there. > + > +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_master { > + #address-cells = <1>; > + #size-cells = <0>; > + > + dac: ltc2632@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 */ > - }; > - }; ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v5 3/3] iio/dac: convert ltc2632.txt to lltc,ltc2632.yaml 2020-04-25 19:35 ` Jonathan Cameron @ 2020-04-27 1:09 ` Chris Ruehl 0 siblings, 0 replies; 10+ messages in thread From: Chris Ruehl @ 2020-04-27 1:09 UTC (permalink / raw) To: Jonathan Cameron Cc: devicetree, Michael Hennerich, Hartmut Knaack, Lars-Peter Clausen, Peter Meerwald-Stadler, Rob Herring, Uwe Kleine-König, Alexios Zavras, Steve Winslow, Thomas Gleixner, linux-iio, linux-kernel On 26/4/2020 3:35 am, Jonathan Cameron wrote: > On Mon, 20 Apr 2020 12:26:08 +0800 > Chris Ruehl <chris.ruehl@gtsys.com.hk> wrote: > >> Conversion of the ltc2632 to yaml format and name the file 'lltc,ltc2632.yaml'. >> >> Signed-off-by: Chris Ruehl <chris.ruehl@gtsys.com.hk> > There is something odd going on with indentation but otherwise looks fine to me. > However we need to leave time for a final DT review. > > Thanks, > > Jonathan I will wait for the review, and fix the indent together with / or without any changes lining up. I think a good Idea to get a yaml extension for my 'vim'. Cheers, Chris >> --- >> v5: >> correct require section >> set maintainer of analog.com >> v4..v2: no change >> >> .../bindings/iio/dac/lltc,ltc2632.yaml | 76 +++++++++++++++++++ >> .../devicetree/bindings/iio/dac/ltc2632.txt | 49 ------------ >> 2 files changed, 76 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..b0043144fbc4 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml >> @@ -0,0 +1,76 @@ >> +# 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. > Odd intent going on there. > >> + >> +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_master { >> + #address-cells = <1>; >> + #size-cells = <0>; >> + >> + dac: ltc2632@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 */ >> - }; >> - }; -- GTSYS Limited RFID Technology 9/F, Unit E, R07, Kwai Shing Industrial Building Phase 2, 42-46 Tai Lin Pai Road, Kwai Chung, N.T., Hong Kong Tel (852) 9079 9521 Disclaimer: https://www.gtsys.com.hk/email/classified.html ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v5 3/3] iio/dac: convert ltc2632.txt to lltc,ltc2632.yaml 2020-04-20 4:26 ` [PATCH v5 3/3] iio/dac: convert ltc2632.txt to lltc,ltc2632.yaml Chris Ruehl 2020-04-25 19:35 ` Jonathan Cameron @ 2020-04-30 15:24 ` Rob Herring 2020-08-10 3:01 ` Chris Ruehl 1 sibling, 1 reply; 10+ messages in thread From: Rob Herring @ 2020-04-30 15:24 UTC (permalink / raw) To: Chris Ruehl Cc: devicetree, Michael Hennerich, Jonathan Cameron, Hartmut Knaack, Lars-Peter Clausen, Peter Meerwald-Stadler, Uwe Kleine-König, Alexios Zavras, Steve Winslow, Thomas Gleixner, linux-iio, linux-kernel On Mon, Apr 20, 2020 at 12:26:08PM +0800, Chris Ruehl wrote: > Conversion of the ltc2632 to yaml format and name the file 'lltc,ltc2632.yaml'. > > Signed-off-by: Chris Ruehl <chris.ruehl@gtsys.com.hk> > --- > v5: > correct require section > set maintainer of analog.com > v4..v2: no change > > .../bindings/iio/dac/lltc,ltc2632.yaml | 76 +++++++++++++++++++ > .../devicetree/bindings/iio/dac/ltc2632.txt | 49 ------------ > 2 files changed, 76 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..b0043144fbc4 > --- /dev/null > +++ b/Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml > @@ -0,0 +1,76 @@ > +# 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 There's a tab here which the yaml parser isn't going to like. Not sure why my checker didn't run on this, but make sure 'make dt_binding_check' passes. > + 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_master { spi { > + #address-cells = <1>; > + #size-cells = <0>; > + > + dac: ltc2632@0 { dac@0 And drop the unused label. > + 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 */ > - }; > - }; > -- > 2.20.1 > ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v5 3/3] iio/dac: convert ltc2632.txt to lltc,ltc2632.yaml 2020-04-30 15:24 ` Rob Herring @ 2020-08-10 3:01 ` Chris Ruehl 0 siblings, 0 replies; 10+ messages in thread From: Chris Ruehl @ 2020-08-10 3:01 UTC (permalink / raw) To: Rob Herring Cc: devicetree, Michael Hennerich, Jonathan Cameron, Hartmut Knaack, Lars-Peter Clausen, Peter Meerwald-Stadler, Uwe Kleine-König, Alexios Zavras, Steve Winslow, Thomas Gleixner, linux-iio, linux-kernel Rob, On 30/4/2020 11:24 pm, Rob Herring wrote: > On Mon, Apr 20, 2020 at 12:26:08PM +0800, Chris Ruehl wrote: >> Conversion of the ltc2632 to yaml format and name the file 'lltc,ltc2632.yaml'. >> >> Signed-off-by: Chris Ruehl <chris.ruehl@gtsys.com.hk> >> --- >> v5: >> correct require section >> set maintainer of analog.com >> v4..v2: no change 1/3 and 2/3 make it already to linux-next. I will send 3/3 with the corrections you requested as a new stand alone patch and refer commit 6f1c9e0da9aae51177457731357ae8a2c8af27cd for reference. Regards Chris ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2020-08-10 3:01 UTC | newest] Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-04-20 4:26 [PATCH v5] iio: patch set ltc2632 Chris Ruehl 2020-04-20 4:26 ` [PATCH v5 1/3] iio: documentation ltc2632_chip_info add num_channels Chris Ruehl 2020-04-25 19:37 ` Jonathan Cameron 2020-04-20 4:26 ` [PATCH v5 2/3] iio: DAC extension for ltc2634-12/10/8 Chris Ruehl 2020-04-30 15:17 ` Rob Herring 2020-04-20 4:26 ` [PATCH v5 3/3] iio/dac: convert ltc2632.txt to lltc,ltc2632.yaml Chris Ruehl 2020-04-25 19:35 ` Jonathan Cameron 2020-04-27 1:09 ` Chris Ruehl 2020-04-30 15:24 ` Rob Herring 2020-08-10 3:01 ` Chris Ruehl
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).