iio:dac:ti-dac7612: Add driver for Texas Instruments DAC7612
Subsystems (3)
PATCH v4 1/2 iio:dac:ti-dac7612: Add driver for Texas Instruments DAC7612
It is a driver for Texas Instruments Dual, 12-Bit Serial Input Digital-to-Analog Converter.
Datasheet of this chip:
http://www.ti.com/lit/ds/sbas106/sbas106.pdf
---
Changelog v4 by: Jonathan Cameron <[email protected]>,
dt related:
-Fix pin name
-Fix part name
-improve comments
MAINTAINERS | 6 ++ drivers/iio/dac/Kconfig | 10 ++ drivers/iio/dac/Makefile | 1 + drivers/iio/dac/ti-dac7612.c | 184 +++++++++++++++++++++++++++++++++++ 4 files changed, 201 insertions(+) create mode 100644 drivers/iio/dac/ti-dac7612.c
MAINTAINERS +6 -0
@@ -14877,6 +14877,12 @@ F: Documentation/devicetree/bindings/clock/ti,sci-clk.txt F: drivers/clk/keystone/sci-clk.c F: drivers/reset/reset-ti-sci.c +Texas Instruments' DAC7612 DAC Driver +M: Ricardo Ribalda <[email protected]> +L: [email protected] +S: Supported +F: drivers/iio/dac/ti-dac7612.c + THANKO'S RAREMONO AM/FM/SW RADIO RECEIVER USB DRIVER M: Hans Verkuil <[email protected]> L: [email protected]
drivers/iio/dac/Kconfig +10 -0
@@ -375,6 +375,16 @@ config TI_DAC7311 If compiled as a module, it will be called ti-dac7311. +config TI_DAC7612 + tristate "Texas Instruments 12-bit 2-channel DAC driver" + depends on SPI_MASTER && GPIOLIB + help + Driver for the Texas Instruments DAC7612, DAC7612U, DAC7612UB + The driver hand drive the load pin automatically, otherwise + it needs to be toggled manually. + + If compiled as a module, it will be called ti-dac7612. + config VF610_DAC tristate "Vybrid vf610 DAC driver" depends on OF
drivers/iio/dac/Makefile +1 -0
@@ -41,4 +41,5 @@ obj-$(CONFIG_STM32_DAC) += stm32-dac.o obj-$(CONFIG_TI_DAC082S085) += ti-dac082s085.o obj-$(CONFIG_TI_DAC5571) += ti-dac5571.o obj-$(CONFIG_TI_DAC7311) += ti-dac7311.o +obj-$(CONFIG_TI_DAC7612) += ti-dac7612.o obj-$(CONFIG_VF610_DAC) += vf610_dac.o
drivers/iio/dac/ti-dac7612.c new file +184 -0
184 lines not on this page · raw message
Signature · 1 line
2.20.1
PATCH v4 2/2 iio:dac:dac7612: device tree bindings
Bindings for dac7612.
---
v4 Changelog by Rob Herring <[email protected]>
-Add ti, prefix to gpio
-Fix example da@1
-Fix partname
.../bindings/iio/dac/ti,dac7612.txt | 29 +++++++++++++++++++ MAINTAINERS | 1 + 2 files changed, 30 insertions(+) create mode 100644 Documentation/devicetree/bindings/iio/dac/ti,dac7612.txt
Documentation/devicetree/bindings/iio/dac/ti,dac7612.txt new file +29 -0
@@ -0,0 +1,29 @@ +* Texas Instruments Dual, 12-Bit Serial Input Digital-to-Analog Converter + +The DAC7612 is a dual, 12-bit digital-to-analog converter (DAC) with guaranteed +12-bit monotonicity performance over the industrial temperature range. +Is is programmable through an SPI interface. + +The internal DACs are loaded when the LOADDACS pin is pulled down. + +http://www.ti.com/lit/ds/sbas106/sbas106.pdf + +Required Properties: +- compatible: Should be one of: + "ti,dac7612" + "ti,dac7612u" + "ti,dac7612ub" +- reg: Definition as per Documentation/devicetree/bindings/spi/spi-bus.txt + +Optional Properties: +- ti,loaddacs-gpios: GPIO descriptor for the LOADDACS pin. +- spi-*: Definition as per Documentation/devicetree/bindings/spi/spi-bus.txt + +Example: + + dac@1 { + compatible = "ti,dac7612"; + reg = <0x1>; + ti,loaddacs-gpios = <&msmgpio 25 GPIO_ACTIVE_LOW>; + }; +
MAINTAINERS +1 -0
@@ -14882,6 +14882,7 @@ M: Ricardo Ribalda <[email protected]> L: [email protected] S: Supported F: drivers/iio/dac/ti-dac7612.c +F: Documentation/devicetree/bindings/iio/dac/ti,dac7612.txt THANKO'S RAREMONO AM/FM/SW RADIO RECEIVER USB DRIVER M: Hans Verkuil <[email protected]>
Signature · 1 line
2.20.1
Re: [PATCH v4 2/2] iio:dac:dac7612: device tree bindings
Quoted 4 lines · On Mon, 4 Feb 2019 13:48:33 +0100 Ricardo Ribalda Delgado…
Bindings for dac7612.
Signed-off-by Ricardo Ribalda Delgado <[email protected]>
Simple binding so I'm happy to take with out a DT review unless someone feels like giving one. Seems to have answered Rob's previous comments.
Applied to the togreg branch of iio.git with the tiny tweak of dropping the extra blank line at the end of the file.
Thanks,
Jonathan
Quoted 59 lines
---
v4 Changelog by Rob Herring <[email protected]>(quoted diff) +0
-3-Add ti, prefix to gpio -Fix example da@1 -Fix partname .../bindings/iio/dac/ti,dac7612.txt | 29 +++++++++++++++++++ MAINTAINERS | 1 + 2 files changed, 30 insertions(+) create mode 100644 Documentation/devicetree/bindings/iio/dac/ti,dac7612.txtDocumentation/devicetree/bindings/iio/dac/ti,dac7612.txt new file +29
-0@@ -0,0 +1,29 @@ +* Texas Instruments Dual, 12-Bit Serial Input Digital-to-Analog Converter + +The DAC7612 is a dual, 12-bit digital-to-analog converter (DAC) with guaranteed +12-bit monotonicity performance over the industrial temperature range. +Is is programmable through an SPI interface. + +The internal DACs are loaded when the LOADDACS pin is pulled down. + +http://www.ti.com/lit/ds/sbas106/sbas106.pdf + +Required Properties: +- compatible: Should be one of: + "ti,dac7612" + "ti,dac7612u" + "ti,dac7612ub" +- reg: Definition as per Documentation/devicetree/bindings/spi/spi-bus.txt + +Optional Properties: +- ti,loaddacs-gpios: GPIO descriptor for the LOADDACS pin. +- spi-*: Definition as per Documentation/devicetree/bindings/spi/spi-bus.txt + +Example: + + dac@1 { + compatible = "ti,dac7612"; + reg = <0x1>; + ti,loaddacs-gpios = <&msmgpio 25 GPIO_ACTIVE_LOW>; + }; +MAINTAINERS +1
-0@@ -14882,6 +14882,7 @@ M: Ricardo Ribalda <[email protected]> L: [email protected] S: Supported F: drivers/iio/dac/ti-dac7612.c +F: Documentation/devicetree/bindings/iio/dac/ti,dac7612.txt THANKO'S RAREMONO AM/FM/SW RADIO RECEIVER USB DRIVER M: Hans Verkuil <[email protected]>
Jonathan Cameron
maintainerQuoted 7 lines · On Mon, 4 Feb 2019 13:48:32 +0100 Ricardo Ribalda Delgado…
It is a driver for Texas Instruments Dual, 12-Bit Serial Input Digital-to-Analog Converter.
Datasheet of this chip:
http://www.ti.com/lit/ds/sbas106/sbas106.pdfSigned-off-by Ricardo Ribalda Delgado <[email protected]>
Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to play with it.
Thanks,
Jonathan
Quoted 254 lines
---
Changelog v4 by: Jonathan Cameron <[email protected]>,dt related:
(quoted diff) +0
-3-Fix pin name -Fix part name -improve comments MAINTAINERS | 6 ++ drivers/iio/dac/Kconfig | 10 ++ drivers/iio/dac/Makefile | 1 + drivers/iio/dac/ti-dac7612.c | 184 +++++++++++++++++++++++++++++++++++ 4 files changed, 201 insertions(+) create mode 100644 drivers/iio/dac/ti-dac7612.cMAINTAINERS +6
-0@@ -14877,6 +14877,12 @@ F: Documentation/devicetree/bindings/clock/ti,sci-clk.txt F: drivers/clk/keystone/sci-clk.c F: drivers/reset/reset-ti-sci.c +Texas Instruments' DAC7612 DAC Driver +M: Ricardo Ribalda <[email protected]> +L: [email protected] +S: Supported +F: drivers/iio/dac/ti-dac7612.c + THANKO'S RAREMONO AM/FM/SW RADIO RECEIVER USB DRIVER M: Hans Verkuil <[email protected]> L: [email protected]drivers/iio/dac/Kconfig +10
-0@@ -375,6 +375,16 @@ config TI_DAC7311 If compiled as a module, it will be called ti-dac7311. +config TI_DAC7612 + tristate "Texas Instruments 12-bit 2-channel DAC driver" + depends on SPI_MASTER && GPIOLIB + help + Driver for the Texas Instruments DAC7612, DAC7612U, DAC7612UB + The driver hand drive the load pin automatically, otherwise + it needs to be toggled manually. + + If compiled as a module, it will be called ti-dac7612. + config VF610_DAC tristate "Vybrid vf610 DAC driver" depends on OFdrivers/iio/dac/Makefile +1
-0@@ -41,4 +41,5 @@ obj-$(CONFIG_STM32_DAC) += stm32-dac.o obj-$(CONFIG_TI_DAC082S085) += ti-dac082s085.o obj-$(CONFIG_TI_DAC5571) += ti-dac5571.o obj-$(CONFIG_TI_DAC7311) += ti-dac7311.o +obj-$(CONFIG_TI_DAC7612) += ti-dac7612.o obj-$(CONFIG_VF610_DAC) += vf610_dac.odrivers/iio/dac/ti-dac7612.c new file +184
-0@@ -0,0 +1,184 @@ +// SPDX-License-Identifier: GPL-2.0 +/* +* DAC7612 Dual, 12-Bit Serial input Digital-to-Analog Converter +* +* Copyright 2019 Qtechnology A/S +* 2019 Ricardo Ribalda <[email protected]> +* +* Licensed under the GPL-2. +*/ +#include <linux/kernel.h> +#include <linux/module.h> +#include <linux/spi/spi.h> +#include <linux/gpio/consumer.h> +#include <linux/iio/iio.h> + +#define DAC7612_RESOLUTION 12 +#define DAC7612_ADDRESS 4 +#define DAC7612_START 5 + +struct dac7612 { +struct spi_device *spi; +struct gpio_desc *loaddacs; +uint16_t cache[2 ]; + +/* +* DMA (thus cache coherency maintenance) requires the +* transfer buffers to live in their own cache lines. +*/ +uint8_t data[2 ]____cacheline_aligned ; +}; + +static int dac7612_cmd_single (struct dac7612 *priv,int channel,u16 val) +{ +int ret; + + priv->data[0 ] =BIT (DAC7612_START ) | (channel <<DAC7612_ADDRESS ); + priv->data[0 ] |= val >>8 ; + priv->data[1 ] = val &0xff ; + + priv->cache[channel] = val; + + ret =spi_write (priv->spi, priv->data,sizeof (priv->data)); +if (ret) +return ret; + +gpiod_set_value (priv->loaddacs,1 ); +gpiod_set_value (priv->loaddacs,0 ); + +return 0 ; +} + +#define dac7612_CHANNEL (chan, name) { \ + .type = IIO_VOLTAGE, \ + .channel = (chan), \ + .indexed = 1, \ + .output = 1, \ + .datasheet_name = name, \ + .info_mask_separate = BIT(IIO_CHAN_INFO_RAW), \ + .info_mask_shared_by_type = BIT(IIO_CHAN_INFO_SCALE), \ +} + +static const struct iio_chan_spec dac7612_channels[] = { +dac7612_CHANNEL (0 ,"OUTA" ), +dac7612_CHANNEL (1 ,"OUTB" ), +}; + +static int dac7612_read_raw (struct iio_dev *iio_dev, +const struct iio_chan_spec *chan, +int *val,int *val2,long mask) +{ +struct dac7612 *priv; + +switch (mask) { +case IIO_CHAN_INFO_RAW : + priv =iio_priv (iio_dev); + *val = priv->cache[chan->channel]; +return IIO_VAL_INT ; + +case IIO_CHAN_INFO_SCALE : + *val =1 ; +return IIO_VAL_INT ; + +default : +return -EINVAL ; + } +} + +static int dac7612_write_raw (struct iio_dev *iio_dev, +const struct iio_chan_spec *chan, +int val,int val2,long mask) +{ +struct dac7612 *priv =iio_priv (iio_dev); +int ret; + +if (mask !=IIO_CHAN_INFO_RAW ) +return -EINVAL ; + +if ((val >=BIT (DAC7612_RESOLUTION )) || val <0 || val2) +return -EINVAL ; + +if (val == priv->cache[chan->channel]) +return 0 ; + +mutex_lock (&iio_dev->mlock); + ret =dac7612_cmd_single (priv, chan->channel, val); +mutex_unlock (&iio_dev->mlock); + +return ret; +} + +static const struct iio_info dac7612_info = { + .read_raw = dac7612_read_raw, + .write_raw = dac7612_write_raw, +}; + +static int dac7612_probe (struct spi_device *spi) +{ +struct iio_dev *iio_dev; +struct dac7612 *priv; +int i; +int ret; + + iio_dev =devm_iio_device_alloc (&spi->dev,sizeof (*priv)); +if (!iio_dev) +return -ENOMEM ; + + priv =iio_priv (iio_dev); +/* +* LOADDACS pin can be controlled by the driver or externally. +* When controlled by the driver, the DAC value is updated after +* every write. +* When the driver does not control the PIN, the user or an external +* event can change the value of all DACs by pulsing down the LOADDACs +* pin. +*/ + priv->loaddacs =devm_gpiod_get_optional (&spi->dev,"ti,loaddacs" , +GPIOD_OUT_LOW ); +if (IS_ERR (priv->loaddacs)) +return PTR_ERR (priv->loaddacs); + priv->spi = spi; +spi_set_drvdata (spi, iio_dev); + iio_dev->dev.parent = &spi->dev; + iio_dev->info = &dac7612_info; + iio_dev->modes =INDIO_DIRECT_MODE ; + iio_dev->channels = dac7612_channels; + iio_dev->num_channels =ARRAY_SIZE (priv->cache); + iio_dev->name =spi_get_device_id (spi)->name; + +for (i =0 ; i <ARRAY_SIZE (priv->cache); i++) { + ret =dac7612_cmd_single (priv, i,0 ); +if (ret) +return ret; + } + +return devm_iio_device_register (&spi->dev, iio_dev); +} + +static const struct spi_device_id dac7612_id[] = { + {"ti-dac7612" }, + {} +}; +MODULE_DEVICE_TABLE (spi, dac7612_id); + +static const struct of_device_id dac7612_of_match[] = { + { .compatible ="ti,dac7612" }, + { .compatible ="ti,dac7612u" }, + { .compatible ="ti,dac7612ub" }, + { }, +}; +MODULE_DEVICE_TABLE (of, dac7612_of_match); + +static struct spi_driver dac7612_driver = { + .driver = { + .name ="ti-dac7612" , + .of_match_table = dac7612_of_match, + }, + .probe = dac7612_probe, + .id_table = dac7612_id, +}; +module_spi_driver (dac7612_driver); + +MODULE_AUTHOR ("Ricardo Ribalda <[email protected]>" ); +MODULE_DESCRIPTION ("Texas Instruments DAC7612 DAC driver" ); +MODULE_LICENSE ("GPL v2" );