linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/2] iio: dac: AD5770R: Add support
@ 2020-02-06  9:23 Alexandru Tachici
  2020-02-06  9:23 ` [PATCH 1/2] iio: dac: ad5770r: Add AD5770R support Alexandru Tachici
                   ` (2 more replies)
  0 siblings, 3 replies; 9+ messages in thread
From: Alexandru Tachici @ 2020-02-06  9:23 UTC (permalink / raw)
  To: linux-iio, linux-kernel; +Cc: jic23, Alexandru Tachici

This series of patches adds support for the AD5570R
14-bit current DAC.

It contains changes discussed in:
https://patchwork.kernel.org/patch/10549129/

Alexandru Tachici (2):
  iio: dac: ad5770r: Add AD5770R support
  dt-bindings: iio: dac: Add docs for AD5770R DAC

 .../bindings/iio/dac/adi,ad5770r.yaml         | 176 +++++
 drivers/iio/dac/Kconfig                       |  10 +
 drivers/iio/dac/Makefile                      |   1 +
 drivers/iio/dac/ad5770r.c                     | 662 ++++++++++++++++++
 include/linux/iio/common/adi_spi_regs.h       |  44 ++
 5 files changed, 893 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml
 create mode 100644 drivers/iio/dac/ad5770r.c
 create mode 100644 include/linux/iio/common/adi_spi_regs.h

-- 
2.20.1


^ permalink raw reply	[flat|nested] 9+ messages in thread

* [PATCH 1/2] iio: dac: ad5770r: Add AD5770R support
  2020-02-06  9:23 [PATCH 0/2] iio: dac: AD5770R: Add support Alexandru Tachici
@ 2020-02-06  9:23 ` Alexandru Tachici
  2020-02-08  9:05   ` kbuild test robot
  2020-02-08 14:33   ` Jonathan Cameron
  2020-02-06  9:23 ` [PATCH 2/2] dt-bindings: iio: dac: Add docs for AD5770R DAC Alexandru Tachici
  2020-02-08 14:24 ` [PATCH 0/2] iio: dac: AD5770R: Add support Jonathan Cameron
  2 siblings, 2 replies; 9+ messages in thread
From: Alexandru Tachici @ 2020-02-06  9:23 UTC (permalink / raw)
  To: linux-iio, linux-kernel; +Cc: jic23, Alexandru Tachici, Mircea Caprioru

The AD5770R is a 6-channel, 14-bit resolution, low noise, programmable
current output digital-to-analog converter (DAC) for photonics control
applications.

It contains five 14-bit resolution current sourcing DAC channels and one
14-bit resolution current sourcing/sinking DAC channel.

Signed-off-by: Mircea Caprioru <mircea.caprioru@analog.com>
Signed-off-by: Alexandru Tachici <alexandru.tachici@analog.com>
---
 drivers/iio/dac/Kconfig                 |  10 +
 drivers/iio/dac/Makefile                |   1 +
 drivers/iio/dac/ad5770r.c               | 662 ++++++++++++++++++++++++
 include/linux/iio/common/adi_spi_regs.h |  44 ++
 4 files changed, 717 insertions(+)
 create mode 100644 drivers/iio/dac/ad5770r.c
 create mode 100644 include/linux/iio/common/adi_spi_regs.h

diff --git a/drivers/iio/dac/Kconfig b/drivers/iio/dac/Kconfig
index 979070196da9..c74f2151ee42 100644
--- a/drivers/iio/dac/Kconfig
+++ b/drivers/iio/dac/Kconfig
@@ -208,6 +208,16 @@ config AD5764
 	  To compile this driver as a module, choose M here: the
 	  module will be called ad5764.
 
+config AD5770R
+	tristate "Analog Devices AD5770R IDAC driver"
+	depends on SPI_MASTER
+	help
+	  Say yes here to build support for Analog Devices AD5770R Digital to
+	  Analog Converter.
+
+	  To compile this driver as a module, choose M here: the
+	  module will be called ad5770r.
+
 config AD5791
 	tristate "Analog Devices AD5760/AD5780/AD5781/AD5790/AD5791 DAC SPI driver"
 	depends on SPI
diff --git a/drivers/iio/dac/Makefile b/drivers/iio/dac/Makefile
index 1369fa1d2f0e..2fc481167724 100644
--- a/drivers/iio/dac/Makefile
+++ b/drivers/iio/dac/Makefile
@@ -19,6 +19,7 @@ obj-$(CONFIG_AD5755) += ad5755.o
 obj-$(CONFIG_AD5755) += ad5758.o
 obj-$(CONFIG_AD5761) += ad5761.o
 obj-$(CONFIG_AD5764) += ad5764.o
+obj-$(CONFIG_AD5770R) += ad5770r.o
 obj-$(CONFIG_AD5791) += ad5791.o
 obj-$(CONFIG_AD5686) += ad5686.o
 obj-$(CONFIG_AD5686_SPI) += ad5686-spi.o
diff --git a/drivers/iio/dac/ad5770r.c b/drivers/iio/dac/ad5770r.c
new file mode 100644
index 000000000000..a8b2c1850ffd
--- /dev/null
+++ b/drivers/iio/dac/ad5770r.c
@@ -0,0 +1,662 @@
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * AD5770R Digital to analog converters driver
+ *
+ * Copyright 2018 Analog Devices Inc.
+ */
+
+#include <linux/delay.h>
+#include <linux/device.h>
+#include <linux/gpio/consumer.h>
+#include <linux/iio/common/adi_spi_regs.h>
+#include <linux/iio/iio.h>
+#include <linux/iio/sysfs.h>
+#include <linux/kernel.h>
+#include <linux/module.h>
+#include <linux/property.h>
+#include <linux/regmap.h>
+#include <linux/regulator/consumer.h>
+#include <linux/spi/spi.h>
+
+/* AD5770R configuration registers */
+#define AD5770R_CHANNEL_CONFIG		0x14
+#define AD5770R_OUTPUT_RANGE(ch)	(0x15 + (ch))
+#define AD5770R_FILTER_RESISTOR(ch)	(0x1D + (ch))
+#define AD5770R_REFERENCE		0x1B
+#define AD5770R_DAC_LSB(ch)		(0x26 + 2 * (ch))
+#define AD5770R_DAC_MSB(ch)		(0x27 + 2 * (ch))
+#define AD5770R_CH_SELECT		0x34
+#define AD5770R_CH_ENABLE		0x44
+
+/* AD5770R_CHANNEL_CONFIG */
+#define AD5770R_CFG_CH0_SINK_EN(x)		(((x) & 0x1) << 7)
+#define AD5770R_CFG_SHUTDOWN_B(x, ch)		(((x) & 0x1) << (ch))
+
+/* AD5770R_OUTPUT_RANGE */
+#define AD5770R_RANGE_OUTPUT_SCALING(x)		(((x) & 0x3F) << 2)
+#define AD5770R_RANGE_MODE(x)			((x) & 0x03)
+
+/* AD5770R_REFERENCE */
+#define AD5770R_REF_RESISTOR_SEL(x)		(((x) & 0x1) << 2)
+#define AD5770R_REF_SEL(x)			(((x) & 0x3) << 0)
+
+/* AD5770R_CH_ENABLE */
+#define AD5770R_CH_SET(x, channel)		(((x) & 0x1) << (channel))
+
+#define AD5770R_MAX_CHANNELS	6
+#define AD5770R_MAX_CH_MODES	14
+#define AD5770R_LOW_VREF_mV	1250
+#define AD5770R_HIGH_VREF_mV	2500
+
+enum ad5770r_ch0_modes {
+	AD5770R_CH0_0_300 = 0,
+	AD5770R_CH0_NEG_60_0,
+	AD5770R_CH0_NEG_60_300
+};
+
+enum ad5770r_ch1_modes {
+	AD5770R_CH1_0_140_LOW_HEAD = 1,
+	AD5770R_CH1_0_140_LOW_NOISE,
+	AD5770R_CH1_0_250
+};
+
+enum ad5770r_ch2_5_modes {
+	AD5770R_CH_LOW_RANGE = 0,
+	AD5770R_CH_HIGH_RANGE
+};
+
+enum ad5770r_ref_v {
+	AD5770R_EXT_2_5_V = 0,
+	AD5770R_INT_1_25_V_OUT_ON,
+	AD5770R_EXT_1_25_V,
+	AD5770R_INT_1_25_V_OUT_OFF
+};
+
+enum ad5770r_output_filter_resistor {
+	AD5770R_FILTER_60_OHM = 0x0,
+	AD5770R_FILTER_5_6_KOHM = 0x5,
+	AD5770R_FILTER_11_2_KOHM,
+	AD5770R_FILTER_22_2_KOHM,
+	AD5770R_FILTER_44_4_KOHM,
+	AD5770R_FILTER_104_KOHM,
+};
+
+struct ad5770r_out_range {
+	u8	out_scale;
+	u8	out_range_mode;
+};
+
+/**
+ * struct ad5770R_state - driver instance specific data
+ * @spi:		spi_device
+ * @regmap:		regmap
+ * @vref_reg:		fixed regulator for reference configuration
+ * @gpio_reset:		gpio descriptor
+ * @output_mode:		array contains channels output ranges
+ * @vref:		reference value
+ * @ch_pwr_down:		powerdown flags
+ * @internal_ref:	internal reference flag
+ * @external_res:	external 2.5k resistor flag
+ */
+struct ad5770r_state {
+	struct spi_device		*spi;
+	struct regmap			*regmap;
+	struct regulator		*vref_reg;
+	struct gpio_desc		*gpio_reset;
+	struct ad5770r_out_range	output_mode[AD5770R_MAX_CHANNELS];
+	int				vref;
+	bool				ch_pwr_down[AD5770R_MAX_CHANNELS];
+	bool				internal_ref;
+	bool				external_res;
+};
+
+static const struct regmap_config ad5770r_spi_regmap_config = {
+	.reg_bits = 8,
+	.val_bits = 8,
+	.read_flag_mask = BIT(7),
+};
+
+struct ad5770r_output_modes {
+	unsigned int ch;
+	u8 mode;
+	int min;
+	int max;
+};
+
+static struct ad5770r_output_modes ad5770r_rng_tbl[] = {
+	{ 0, AD5770R_CH0_0_300, 0, 300 },
+	{ 0, AD5770R_CH0_NEG_60_0, -60, 0 },
+	{ 0, AD5770R_CH0_NEG_60_300, -60, 300 },
+	{ 1, AD5770R_CH1_0_140_LOW_HEAD, 0, 140 },
+	{ 1, AD5770R_CH1_0_140_LOW_NOISE, 0, 140 },
+	{ 1, AD5770R_CH1_0_250, 0, 250 },
+	{ 2, AD5770R_CH_LOW_RANGE, 0, 55 },
+	{ 2, AD5770R_CH_HIGH_RANGE, 0, 150 },
+	{ 3, AD5770R_CH_LOW_RANGE, 0, 45 },
+	{ 3, AD5770R_CH_HIGH_RANGE, 0, 100 },
+	{ 4, AD5770R_CH_LOW_RANGE, 0, 45 },
+	{ 4, AD5770R_CH_HIGH_RANGE, 0, 100 },
+	{ 5, AD5770R_CH_LOW_RANGE, 0, 45 },
+	{ 5, AD5770R_CH_HIGH_RANGE, 0, 100 },
+};
+
+static const unsigned int ad5770r_filter_freqs[] = {
+	153, 357, 715, 1400, 2800, 262000,
+};
+
+static const unsigned int ad5770r_filter_reg_vals[] = {
+	AD5770R_FILTER_104_KOHM,
+	AD5770R_FILTER_44_4_KOHM,
+	AD5770R_FILTER_22_2_KOHM,
+	AD5770R_FILTER_11_2_KOHM,
+	AD5770R_FILTER_5_6_KOHM,
+	AD5770R_FILTER_60_OHM
+};
+
+static int ad5770r_set_output_mode(struct ad5770r_state *st,
+				   const struct ad5770r_out_range *out_mode,
+				   int channel)
+{
+	unsigned int regval;
+
+	regval = AD5770R_RANGE_OUTPUT_SCALING(out_mode->out_scale) |
+		 AD5770R_RANGE_MODE(out_mode->out_range_mode);
+
+	return regmap_write(st->regmap,
+			    AD5770R_OUTPUT_RANGE(channel), regval);
+}
+
+static int ad5770r_set_reference(struct ad5770r_state *st)
+{
+	unsigned int regval = 0;
+
+	regval = AD5770R_REF_RESISTOR_SEL(st->external_res);
+
+	if (st->internal_ref) {
+		regval |= AD5770R_REF_SEL(AD5770R_INT_1_25_V_OUT_OFF);
+	} else {
+		switch (st->vref) {
+		case AD5770R_LOW_VREF_mV:
+			regval |= AD5770R_REF_SEL(AD5770R_EXT_1_25_V);
+			break;
+		case AD5770R_HIGH_VREF_mV:
+			regval |= AD5770R_REF_SEL(AD5770R_EXT_2_5_V);
+			break;
+		default:
+			regval = AD5770R_REF_SEL(AD5770R_INT_1_25_V_OUT_OFF);
+			break;
+		}
+	}
+
+	return regmap_write(st->regmap, AD5770R_REFERENCE, regval);
+}
+
+static int ad5770r_soft_reset(struct ad5770r_state *st)
+{
+	return regmap_write(st->regmap, ADI_SPI_IF_CONFIG_A,
+			    ADI_SPI_IF_SW_RESET_SEL(1));
+}
+
+static int ad5770r_reset(struct ad5770r_state *st)
+{
+	/* Perform software reset if no GPIO provided */
+	if (!st->gpio_reset)
+		return ad5770r_soft_reset(st);
+
+	gpiod_set_value_cansleep(st->gpio_reset, 0);
+	usleep_range(10, 20);
+	gpiod_set_value_cansleep(st->gpio_reset, 1);
+
+	/* data must not be written during reset timeframe */
+	usleep_range(100, 200);
+
+	return 0;
+}
+
+static int ad5770r_get_range(struct ad5770r_state *st,
+			     int ch, int *min, int *max)
+{
+	int i;
+	u8 tbl_ch, tbl_mode, out_range;
+
+	out_range = st->output_mode[ch].out_range_mode;
+
+	for (i = 0; i < AD5770R_MAX_CH_MODES; i++) {
+		tbl_ch = ad5770r_rng_tbl[i].ch;
+		tbl_mode = ad5770r_rng_tbl[i].mode;
+		if (tbl_ch == ch && tbl_mode == out_range) {
+			*min = ad5770r_rng_tbl[i].min;
+			*max = ad5770r_rng_tbl[i].max;
+			return 0;
+		}
+	}
+
+	return -EINVAL;
+}
+
+static int ad5770r_get_filter_freq(struct iio_dev *indio_dev,
+				   const struct iio_chan_spec *chan, int *freq)
+{
+	struct ad5770r_state *st = iio_priv(indio_dev);
+	int ret;
+	unsigned int regval, i;
+
+	ret = regmap_read(st->regmap,
+			  AD5770R_FILTER_RESISTOR(chan->channel), &regval);
+	if (ret < 0)
+		return ret;
+
+	for (i = 0; i < ARRAY_SIZE(ad5770r_filter_reg_vals); i++)
+		if (regval == ad5770r_filter_reg_vals[i])
+			break;
+	if (i == ARRAY_SIZE(ad5770r_filter_reg_vals))
+		return -EINVAL;
+
+	*freq = ad5770r_filter_freqs[i];
+
+	return IIO_VAL_INT;
+}
+
+static int ad5770r_set_filter_freq(struct iio_dev *indio_dev,
+				   const struct iio_chan_spec *chan,
+				   unsigned int freq)
+{
+	struct ad5770r_state *st = iio_priv(indio_dev);
+	unsigned int regval, i;
+
+	for (i = 0; i < ARRAY_SIZE(ad5770r_filter_freqs); i++)
+		if (ad5770r_filter_freqs[i] >= freq)
+			break;
+	if (i == ARRAY_SIZE(ad5770r_filter_freqs))
+		return -EINVAL;
+
+	regval = ad5770r_filter_reg_vals[i];
+
+	return regmap_write(st->regmap, AD5770R_FILTER_RESISTOR(chan->channel),
+			    regval);
+}
+
+static int ad5770r_read_raw(struct iio_dev *indio_dev,
+			    struct iio_chan_spec const *chan,
+			    int *val, int *val2, long info)
+{
+	struct ad5770r_state *st = iio_priv(indio_dev);
+	int max, min, ret;
+	__le16 buf16;
+
+	switch (info) {
+	case IIO_CHAN_INFO_RAW:
+		ret = regmap_bulk_read(st->regmap,
+				       chan->address,
+				       &buf16, 2);
+		if (ret)
+			return 0;
+		*val = le16_to_cpu(buf16) >> 2;
+		return IIO_VAL_INT;
+	case IIO_CHAN_INFO_SCALE:
+		ret = ad5770r_get_range(st, chan->channel, &min, &max);
+		if (ret < 0)
+			return ret;
+		*val = max - min;
+		/* There is no sign bit. (negative current is mapped from 0)
+		 * (sourced/sinked) current = raw * scale + offset
+		 * where offset in case of CH0 can be negative.
+		 */
+		*val2 = 14;
+		return IIO_VAL_FRACTIONAL_LOG2;
+	case IIO_CHAN_INFO_LOW_PASS_FILTER_3DB_FREQUENCY:
+		return ad5770r_get_filter_freq(indio_dev, chan, val);
+	case IIO_CHAN_INFO_OFFSET:
+		ret = ad5770r_get_range(st, chan->channel, &min, &max);
+		if (ret < 0)
+			return ret;
+		*val = min;
+		return IIO_VAL_INT;
+	default:
+		return -EINVAL;
+	}
+}
+
+static int ad5770r_write_raw(struct iio_dev *indio_dev,
+			     struct iio_chan_spec const *chan,
+			     int val, int val2, long info)
+{
+	struct ad5770r_state *st = iio_priv(indio_dev);
+	u8 buf[2];
+
+	switch (info) {
+	case IIO_CHAN_INFO_RAW:
+		buf[0] = ((u16)val >> 6);
+		buf[1] = (val & 0x3F) << 2;
+		return regmap_bulk_write(st->regmap, chan->address,
+					 buf, ARRAY_SIZE(buf));
+	case IIO_CHAN_INFO_LOW_PASS_FILTER_3DB_FREQUENCY:
+		return ad5770r_set_filter_freq(indio_dev, chan, val);
+	default:
+		return -EINVAL;
+	}
+}
+
+static int ad5770r_read_freq_avail(struct iio_dev *indio_dev,
+				   struct iio_chan_spec const *chan,
+				   const int **vals, int *type, int *length,
+				   long mask)
+{
+	switch (mask) {
+	case IIO_CHAN_INFO_LOW_PASS_FILTER_3DB_FREQUENCY:
+		*type = IIO_VAL_INT;
+		*vals = ad5770r_filter_freqs;
+		*length = ARRAY_SIZE(ad5770r_filter_freqs);
+		return IIO_AVAIL_LIST;
+	}
+
+	return -EINVAL;
+}
+
+static int ad5770r_reg_access(struct iio_dev *indio_dev,
+			      unsigned int reg,
+			      unsigned int writeval,
+			      unsigned int *readval)
+{
+	struct ad5770r_state *st = iio_priv(indio_dev);
+
+	if (readval)
+		return regmap_read(st->regmap, reg, readval);
+	else
+		return regmap_write(st->regmap, reg, writeval);
+}
+
+static const struct iio_info ad5770r_info = {
+	.read_raw = ad5770r_read_raw,
+	.write_raw = ad5770r_write_raw,
+	.read_avail = ad5770r_read_freq_avail,
+	.debugfs_reg_access = &ad5770r_reg_access,
+};
+
+static int ad5770r_store_output_range(struct ad5770r_state *st,
+				      int min, int max, int index)
+{
+	int i;
+
+	for (i = 0; i < AD5770R_MAX_CH_MODES; i++) {
+		if (ad5770r_rng_tbl[i].ch != index)
+			continue;
+		if (ad5770r_rng_tbl[i].min != min ||
+		    ad5770r_rng_tbl[i].max != max)
+			continue;
+		st->output_mode[index].out_range_mode = ad5770r_rng_tbl[i].mode;
+
+		return 0;
+	}
+
+	return -EINVAL;
+}
+
+static ssize_t ad5770r_read_dac_powerdown(struct iio_dev *indio_dev,
+					  uintptr_t private,
+					  const struct iio_chan_spec *chan,
+					  char *buf)
+{
+	struct ad5770r_state *st = iio_priv(indio_dev);
+
+	return sprintf(buf, "%d\n", st->ch_pwr_down[chan->channel]);
+}
+
+static ssize_t ad5770r_write_dac_powerdown(struct iio_dev *indio_dev,
+					   uintptr_t private,
+					   const struct iio_chan_spec *chan,
+					   const char *buf, size_t len)
+{
+	struct ad5770r_state *st = iio_priv(indio_dev);
+	unsigned int regval;
+	unsigned int mask;
+	bool readin;
+	int ret;
+
+	ret = kstrtobool(buf, &readin);
+	if (ret)
+		return ret;
+
+	readin = !readin;
+
+	regval = AD5770R_CFG_SHUTDOWN_B(readin, chan->channel);
+	if (chan->channel == 0 &&
+	    st->output_mode[0].out_range_mode > AD5770R_CH0_0_300) {
+		regval |= AD5770R_CFG_CH0_SINK_EN(readin);
+		mask = BIT(chan->channel) + BIT(7);
+	} else {
+		mask = BIT(chan->channel);
+	}
+	ret = regmap_update_bits(st->regmap, AD5770R_CHANNEL_CONFIG, mask,
+				 regval);
+	if (ret)
+		return ret;
+
+	regval = AD5770R_CH_SET(readin, chan->channel);
+	ret = regmap_update_bits(st->regmap, AD5770R_CH_ENABLE,
+				 BIT(chan->channel), regval);
+	if (ret)
+		return ret;
+
+	st->ch_pwr_down[chan->channel] = !readin;
+
+	return len;
+}
+
+static const struct iio_chan_spec_ext_info ad5770r_ext_info[] = {
+	{
+		.name = "powerdown",
+		.read = ad5770r_read_dac_powerdown,
+		.write = ad5770r_write_dac_powerdown,
+		.shared = IIO_SEPARATE,
+	},
+	{ }
+};
+
+#define AD5770R_IDAC_CHANNEL(index, reg) {				\
+	.type = IIO_CURRENT,						\
+	.address = reg,							\
+	.indexed = 1,							\
+	.channel = index,						\
+	.output = 1,							\
+	.info_mask_separate = BIT(IIO_CHAN_INFO_RAW) |			\
+		BIT(IIO_CHAN_INFO_SCALE) |				\
+		BIT(IIO_CHAN_INFO_OFFSET) |				\
+		BIT(IIO_CHAN_INFO_LOW_PASS_FILTER_3DB_FREQUENCY),	\
+	.info_mask_shared_by_type_available =				\
+		BIT(IIO_CHAN_INFO_LOW_PASS_FILTER_3DB_FREQUENCY),	\
+	.ext_info = ad5770r_ext_info,					\
+}
+
+static const struct iio_chan_spec ad5770r_channels[] = {
+	AD5770R_IDAC_CHANNEL(0, AD5770R_DAC_MSB(0)),
+	AD5770R_IDAC_CHANNEL(1, AD5770R_DAC_MSB(1)),
+	AD5770R_IDAC_CHANNEL(2, AD5770R_DAC_MSB(2)),
+	AD5770R_IDAC_CHANNEL(3, AD5770R_DAC_MSB(3)),
+	AD5770R_IDAC_CHANNEL(4, AD5770R_DAC_MSB(4)),
+	AD5770R_IDAC_CHANNEL(5, AD5770R_DAC_MSB(5)),
+};
+
+static int ad5770r_channel_config(struct ad5770r_state *st)
+{
+	int ret, tmp[2], min, max;
+	unsigned int num;
+	struct fwnode_handle *child;
+
+	num = device_get_child_node_count(&st->spi->dev);
+	if (num != AD5770R_MAX_CHANNELS)
+		return -EINVAL;
+
+	device_for_each_child_node(&st->spi->dev, child) {
+		ret = fwnode_property_read_u32(child, "num", &num);
+		if (ret)
+			return ret;
+		if (num > AD5770R_MAX_CHANNELS)
+			return -EINVAL;
+
+		ret = fwnode_property_read_u32_array(child,
+						     "adi,range-microamp",
+						     tmp, 2);
+		if (ret)
+			return ret;
+
+		min = tmp[0] / 1000;
+		max = tmp[1] / 1000;
+		ret = ad5770r_store_output_range(st, min, max, num);
+		if (ret)
+			return ret;
+	}
+
+	return ret;
+}
+
+static int ad5770r_init(struct ad5770r_state *st)
+{
+	int ret, i;
+
+	st->gpio_reset = devm_gpiod_get_optional(&st->spi->dev, "reset",
+						 GPIOD_OUT_HIGH);
+	if (IS_ERR(st->gpio_reset))
+		return PTR_ERR(st->gpio_reset);
+
+	/* Perform a reset */
+	ret = ad5770r_reset(st);
+	if (ret)
+		return ret;
+
+	/* Set output range */
+	ret = ad5770r_channel_config(st);
+	if (ret)
+		return ret;
+
+	for (i = 0; i < AD5770R_MAX_CHANNELS; i++) {
+		ret = ad5770r_set_output_mode(st,  &st->output_mode[i], i);
+		if (ret)
+			return ret;
+	}
+
+	st->external_res = fwnode_property_read_bool(st->spi->dev.fwnode,
+						     "adi,external-resistor");
+
+	ret = ad5770r_set_reference(st);
+	if (ret)
+		return ret;
+
+	/* Set outputs off */
+	ret = regmap_write(st->regmap, AD5770R_CHANNEL_CONFIG, 0x00);
+	if (ret)
+		return ret;
+
+	ret = regmap_write(st->regmap, AD5770R_CH_ENABLE, 0x00);
+	if (ret)
+		return ret;
+
+	for (i = 0; i < AD5770R_MAX_CHANNELS; i++)
+		st->ch_pwr_down[i] = true;
+
+	return ret;
+}
+
+static void ad5770r_disable_regulator(void *data)
+{
+	struct spi_device *spi = (struct spi_device *)data;
+	struct iio_dev *indio_dev = spi_get_drvdata(spi);
+	struct ad5770r_state *st = iio_priv(indio_dev);
+
+	if (st->vref_reg)
+		regulator_disable(st->vref_reg);
+}
+
+static int ad5770r_probe(struct spi_device *spi)
+{
+	struct ad5770r_state *st;
+	struct iio_dev *indio_dev;
+	struct regmap *regmap;
+	int ret;
+
+	indio_dev = devm_iio_device_alloc(&spi->dev, sizeof(*st));
+	if (!indio_dev)
+		return -ENOMEM;
+
+	st = iio_priv(indio_dev);
+	spi_set_drvdata(spi, indio_dev);
+
+	st->spi = spi;
+
+	regmap = devm_regmap_init_spi(spi, &ad5770r_spi_regmap_config);
+	if (IS_ERR(regmap)) {
+		dev_err(&spi->dev, "Error initializing spi regmap: %ld\n",
+			PTR_ERR(regmap));
+		return PTR_ERR(regmap);
+	}
+	st->regmap = regmap;
+
+	ret = devm_add_action_or_reset(&spi->dev, ad5770r_disable_regulator,
+				       st);
+	if (ret < 0)
+		return ret;
+
+	st->vref_reg = devm_regulator_get_optional(&spi->dev, "vref");
+	if (!IS_ERR(st->vref_reg)) {
+		ret = regulator_enable(st->vref_reg);
+		if (ret) {
+			dev_err(&spi->dev,
+				"Failed to enable vref regulators: %d\n", ret);
+			return ret;
+		}
+
+		ret = regulator_get_voltage(st->vref_reg);
+		if (ret < 0)
+			return ret;
+
+		st->vref = ret / 1000;
+	} else {
+		if (PTR_ERR(st->vref_reg) == -ENODEV) {
+			st->vref = AD5770R_LOW_VREF_mV;
+			st->internal_ref = true;
+		} else {
+			return PTR_ERR(st->vref_reg);
+		}
+	}
+
+	indio_dev->dev.parent = &spi->dev;
+	indio_dev->name = spi_get_device_id(spi)->name;
+	indio_dev->info = &ad5770r_info;
+	indio_dev->modes = INDIO_DIRECT_MODE;
+	indio_dev->channels = ad5770r_channels;
+	indio_dev->num_channels = ARRAY_SIZE(ad5770r_channels);
+
+	ret = ad5770r_init(st);
+	if (ret < 0) {
+		dev_err(&spi->dev, "AD5770R init failed\n");
+		return ret;
+	}
+
+	return devm_iio_device_register(&st->spi->dev, indio_dev);
+}
+
+static const struct of_device_id ad5770r_of_id[] = {
+	{ .compatible = "adi,ad5770r", },
+};
+MODULE_DEVICE_TABLE(of, ad5770r_of_id);
+
+static const struct spi_device_id ad5770r_id[] = {
+	{ "ad5770r", 0 },
+	{}
+};
+MODULE_DEVICE_TABLE(spi, ad5770r_id);
+
+static struct spi_driver ad5770r_driver = {
+	.driver = {
+		.name = KBUILD_MODNAME,
+		.of_match_table = of_match_ptr(ad5770r_of_id),
+	},
+	.probe = ad5770r_probe,
+	.id_table = ad5770r_id,
+};
+
+module_spi_driver(ad5770r_driver);
+
+MODULE_AUTHOR("Mircea Caprioru <mircea.caprioru@analog.com>");
+MODULE_DESCRIPTION("Analog Devices AD5770R IDAC");
+MODULE_LICENSE("GPL v2");
diff --git a/include/linux/iio/common/adi_spi_regs.h b/include/linux/iio/common/adi_spi_regs.h
new file mode 100644
index 000000000000..1fc3bfc7845d
--- /dev/null
+++ b/include/linux/iio/common/adi_spi_regs.h
@@ -0,0 +1,44 @@
+/* SPDX-License-Identifier: GPL-2.0 */
+/*
+ * Common Analog Devices SPI configuration registers
+ *
+ * Copyright 2019 Analog Devices Inc.
+ */
+
+#ifndef _ADI_SPI_REGS_H_
+#define _ADI_SPI_REGS_H_
+
+#include <linux/bits.h>
+
+#define ADI_SPI_IF_CONFIG_A		0x00
+#define ADI_SPI_IF_CONFIG_B		0x01
+#define ADI_SPI_IF_DEVICE_CONFIG	0x02
+#define ADI_SPI_IF_CHIP_TYPE		0x03
+#define ADI_SPI_IF_PRODUCT_ID_L		0x04
+#define ADI_SPI_IF_PRODUCT_ID_H		0x05
+#define ADI_SPI_IF_CHIP_GRADE		0x06
+#define ADI_SPI_IF_SCRACTH_PAD		0x0A
+#define ADI_SPI_IF_SPI_REVISION		0x0B
+#define ADI_SPI_IF_SPI_VENDOR_L		0x0C
+#define ADI_SPI_IF_SPI_VENDOR_H		0x0D
+#define ADI_SPI_IF_SPI_STREAM_MODE	0x0E
+#define ADI_SPI_IF_CONFIG_C		0x10
+#define ADI_SPI_IF_STATUS_A		0x11
+
+/* ADI_SPI_IF_CONFIG_A */
+#define ADI_SPI_IF_SW_RESET_MSK		(BIT(0) | BIT(7))
+#define ADI_SPI_IF_SW_RESET_SEL(x)	((x) & ADI_SPI_IF_SW_RESET_MSK)
+#define ADI_SPI_IF_ADDR_ASC_MSK		(BIT(2) | BIT(5))
+#define ADI_SPI_IF_ADDR_ASC_SEL(x)	(((x) << 2) & ADI_SPI_IF_ADDR_ASC_MSK)
+
+/* ADI_SPI_IF_CONFIG_B */
+#define ADI_SPI_IF_SINGLE_INS_MSK	BIT(7)
+#define ADI_SPI_IF_SINGLE_INS_SEL(x)	FIELD_PREP(ADI_SPI_IF_SINGLE_INS_MSK, x)
+#define ADI_SPI_IF_SHORT_INS_MSK	BIT(7)
+#define ADI_SPI_IF_SHORT_INS_SEL(x)	FIELD_PREP(ADI_SPI_IF_SINGLE_INS_MSK, x)
+
+/* ADI_SPI_IF_CONFIG_C */
+#define ADI_SPI_IF_STRICT_REG_MSK	BIT(5)
+#define ADI_SPI_IF_STRICT_REG_GET(x)	FIELD_GET(ADI_SPI_IF_STRICT_REG_MSK, x)
+
+#endif /* _ADI_SPI_REGS_H_ */
-- 
2.20.1


^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [PATCH 2/2] dt-bindings: iio: dac: Add docs for AD5770R DAC
  2020-02-06  9:23 [PATCH 0/2] iio: dac: AD5770R: Add support Alexandru Tachici
  2020-02-06  9:23 ` [PATCH 1/2] iio: dac: ad5770r: Add AD5770R support Alexandru Tachici
@ 2020-02-06  9:23 ` Alexandru Tachici
  2020-02-08 14:36   ` Jonathan Cameron
  2020-02-08 14:24 ` [PATCH 0/2] iio: dac: AD5770R: Add support Jonathan Cameron
  2 siblings, 1 reply; 9+ messages in thread
From: Alexandru Tachici @ 2020-02-06  9:23 UTC (permalink / raw)
  To: linux-iio, linux-kernel; +Cc: jic23, Alexandru Tachici, Mircea Caprioru

Adding dt-bindings documentation for AD5770R DAC.

Signed-off-by: Mircea Caprioru <mircea.caprioru@analog.com>
Signed-off-by: Alexandru Tachici <alexandru.tachici@analog.com>
---
 .../bindings/iio/dac/adi,ad5770r.yaml         | 176 ++++++++++++++++++
 1 file changed, 176 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml

diff --git a/Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml b/Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml
new file mode 100644
index 000000000000..02b2f6e67583
--- /dev/null
+++ b/Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml
@@ -0,0 +1,176 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+# Copyright 2020 Analog Devices Inc.
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/bindings/iio/adc/adi,ad5770r.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Analog Devices AD5770R DAC device driver
+
+maintainers:
+  - Mircea Caprioru <mircea.caprioru@analog.com>
+
+description: |
+  Bindings for the Analog Devices AD5770R current DAC device. Datasheet can be
+  found here:
+    https://www.analog.com/media/en/technical-documentation/data-sheets/AD5770R.pdf
+
+properties:
+  compatible:
+    enum:
+      - adi,ad5770r
+
+  reg:
+    maxItems: 1
+
+  vref-supply:
+    description: Specify the voltage of the external reference used.
+      Available reference options are 1.25 V or 2.5 V. If no
+      external reference declared then the device will use the
+      internal reference of 1.25 V.
+
+  adi,external-resistor:
+    description: Specify if an external 2.5k ohm resistor is used. If not
+      specified the device will use and internal 2.5k ohm resistor.
+      The precision resistor is used for reference current generation.
+    type: boolean
+
+  reset-gpios:
+    description: GPIO spec for the RESET pin. If specified, it will be
+      asserted during driver probe.
+    maxItems: 1
+
+  channel0:
+    description: Represents an external channel which are
+      connected to the DAC. Channel 0 can act both as a current
+      source and sink.
+    type: object
+
+    properties:
+      num:
+        description: This represents the channel number.
+        items:
+          const: 0
+
+      adi,range-microamp:
+          description: Output range of the channel.
+          oneOf:
+            - $ref: /schemas/types.yaml#/definitions/int32-array
+            - items:
+                - enum: [0 300000]
+                - enum: [-60000 0]
+                - enum: [-60000 300000]
+
+  channel1:
+    description: Represents an external channel which are
+      connected to the DAC.
+    type: object
+
+    properties:
+      num:
+        description: This represents the channel number.
+        items:
+          const: 1
+
+      adi,range-microamp:
+          description: Output range of the channel.
+          allOf:
+            - $ref: /schemas/types.yaml#/definitions/uint32-array
+            - items:
+                - enum: [0 140000]
+                - enum: [0 250000]
+
+  channel2:
+    description: Represents an external channel which are
+      connected to the DAC.
+    type: object
+
+    properties:
+      num:
+        description: This represents the channel number.
+        items:
+          const: 2
+
+      adi,range-microamp:
+          description: Output range of the channel.
+          allOf:
+            - $ref: /schemas/types.yaml#/definitions/uint32-array
+            - items:
+                - enum: [0 140000]
+                - enum: [0 250000]
+
+patternProperties:
+  "^channel@([3-5])$":
+    type: object
+    description: Represents the external channels which are connected to the DAC.
+    properties:
+      num:
+        description: This represents the channel number.
+        items:
+          minimum: 3
+          maximum: 5
+
+      adi,range-microamp:
+          description: Output range of the channel.
+          oneOf:
+            - $ref: /schemas/types.yaml#/definitions/uint32-array
+            - items:
+                - enum: [0 45000]
+                - enum: [0 100000]
+
+required:
+- reg
+- diff-channels
+- channel0
+- channel1
+- channel2
+- channel3
+- channel4
+- channel5
+
+examples:
+  - |
+        spi {
+                #address-cells = <1>;
+                #size-cells = <0>;
+
+                ad5770r@0 {
+                        compatible = "ad5770r";
+                        reg = <0>;
+                        spi-max-frequency = <1000000>;
+                        vref-supply = <&vref>;
+                        adi,external-resistor;
+                        reset-gpios = <&gpio 22 0>;
+
+                        channel@0 {
+                                num = <0>;
+                                adi,range-microamp = <(-60000) 300000>;
+                        };
+
+                        channel@1 {
+                                num = <1>;
+                                adi,range-microamp = <0 140000>;
+                        };
+
+                        channel@2 {
+                                num = <2>;
+                                adi,range-microamp = <0 55000>;
+                        };
+
+                        channel@3 {
+                                num = <3>;
+                                adi,range-microamp = <0 45000>;
+                        };
+
+                        channel@4 {
+                                num = <4>;
+                                adi,range-microamp = <0 45000>;
+                        };
+
+                        channel@5 {
+                                num = <5>;
+                                adi,range-microamp = <0 45000>;
+                        };
+                };
+        };
+...
\ No newline at end of file
-- 
2.20.1


^ permalink raw reply related	[flat|nested] 9+ messages in thread

* Re: [PATCH 1/2] iio: dac: ad5770r: Add AD5770R support
  2020-02-06  9:23 ` [PATCH 1/2] iio: dac: ad5770r: Add AD5770R support Alexandru Tachici
@ 2020-02-08  9:05   ` kbuild test robot
  2020-02-08 14:33   ` Jonathan Cameron
  1 sibling, 0 replies; 9+ messages in thread
From: kbuild test robot @ 2020-02-08  9:05 UTC (permalink / raw)
  To: Alexandru Tachici
  Cc: kbuild-all, linux-iio, linux-kernel, jic23, Alexandru Tachici,
	Mircea Caprioru

[-- Attachment #1: Type: text/plain, Size: 2345 bytes --]

Hi Alexandru,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on iio/togreg]
[also build test ERROR on linux/master linus/master v5.5 next-20200207]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]

url:    https://github.com/0day-ci/linux/commits/Alexandru-Tachici/iio-dac-AD5770R-Add-support/20200208-030558
base:   https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git togreg
config: sh-allmodconfig (attached as .config)
compiler: sh4-linux-gcc (GCC) 7.5.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        GCC_VERSION=7.5.0 make.cross ARCH=sh 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

   drivers/iio/dac/ad5770r: struct of_device_id is 196 bytes.  The last of 1 is:
   0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x61 0x64 0x69 0x2c 0x61 0x64 0x35 0x37 0x37 0x30 0x72 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 
>> FATAL: drivers/iio/dac/ad5770r: struct of_device_id is not terminated with a NULL entry!

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 53553 bytes --]

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 0/2] iio: dac: AD5770R: Add support
  2020-02-06  9:23 [PATCH 0/2] iio: dac: AD5770R: Add support Alexandru Tachici
  2020-02-06  9:23 ` [PATCH 1/2] iio: dac: ad5770r: Add AD5770R support Alexandru Tachici
  2020-02-06  9:23 ` [PATCH 2/2] dt-bindings: iio: dac: Add docs for AD5770R DAC Alexandru Tachici
@ 2020-02-08 14:24 ` Jonathan Cameron
  2 siblings, 0 replies; 9+ messages in thread
From: Jonathan Cameron @ 2020-02-08 14:24 UTC (permalink / raw)
  To: Alexandru Tachici; +Cc: linux-iio, linux-kernel

On Thu, 6 Feb 2020 11:23:11 +0200
Alexandru Tachici <alexandru.tachici@analog.com> wrote:

> This series of patches adds support for the AD5570R
> 14-bit current DAC.
> 
> It contains changes discussed in:
> https://patchwork.kernel.org/patch/10549129/

Should have been marked as v2 in the patch titles.

Jonathan

> 
> Alexandru Tachici (2):
>   iio: dac: ad5770r: Add AD5770R support
>   dt-bindings: iio: dac: Add docs for AD5770R DAC
> 
>  .../bindings/iio/dac/adi,ad5770r.yaml         | 176 +++++
>  drivers/iio/dac/Kconfig                       |  10 +
>  drivers/iio/dac/Makefile                      |   1 +
>  drivers/iio/dac/ad5770r.c                     | 662 ++++++++++++++++++
>  include/linux/iio/common/adi_spi_regs.h       |  44 ++
>  5 files changed, 893 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml
>  create mode 100644 drivers/iio/dac/ad5770r.c
>  create mode 100644 include/linux/iio/common/adi_spi_regs.h
> 


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 1/2] iio: dac: ad5770r: Add AD5770R support
  2020-02-06  9:23 ` [PATCH 1/2] iio: dac: ad5770r: Add AD5770R support Alexandru Tachici
  2020-02-08  9:05   ` kbuild test robot
@ 2020-02-08 14:33   ` Jonathan Cameron
  2020-02-11 13:51     ` Ardelean, Alexandru
  1 sibling, 1 reply; 9+ messages in thread
From: Jonathan Cameron @ 2020-02-08 14:33 UTC (permalink / raw)
  To: Alexandru Tachici; +Cc: linux-iio, linux-kernel, Mircea Caprioru

On Thu, 6 Feb 2020 11:23:12 +0200
Alexandru Tachici <alexandru.tachici@analog.com> wrote:

> The AD5770R is a 6-channel, 14-bit resolution, low noise, programmable
> current output digital-to-analog converter (DAC) for photonics control
> applications.
> 
> It contains five 14-bit resolution current sourcing DAC channels and one
> 14-bit resolution current sourcing/sinking DAC channel.

I'd like to see considerably more discussion of the common header.
What devices do these shared registers apply to?
Current header naming kind of implies all ADI devices with an
spi interface which I find very unlikely.

Otherwise one minor comment inline.

Thanks,

Jonathan


> 
> Signed-off-by: Mircea Caprioru <mircea.caprioru@analog.com>
> Signed-off-by: Alexandru Tachici <alexandru.tachici@analog.com>
> ---
>  drivers/iio/dac/Kconfig                 |  10 +
>  drivers/iio/dac/Makefile                |   1 +
>  drivers/iio/dac/ad5770r.c               | 662 ++++++++++++++++++++++++
>  include/linux/iio/common/adi_spi_regs.h |  44 ++
>  4 files changed, 717 insertions(+)
>  create mode 100644 drivers/iio/dac/ad5770r.c
>  create mode 100644 include/linux/iio/common/adi_spi_regs.h
> 
> diff --git a/drivers/iio/dac/Kconfig b/drivers/iio/dac/Kconfig
> index 979070196da9..c74f2151ee42 100644
> --- a/drivers/iio/dac/Kconfig
> +++ b/drivers/iio/dac/Kconfig
> @@ -208,6 +208,16 @@ config AD5764
>  	  To compile this driver as a module, choose M here: the
>  	  module will be called ad5764.
>  
> +config AD5770R
> +	tristate "Analog Devices AD5770R IDAC driver"
> +	depends on SPI_MASTER
> +	help
> +	  Say yes here to build support for Analog Devices AD5770R Digital to
> +	  Analog Converter.
> +
> +	  To compile this driver as a module, choose M here: the
> +	  module will be called ad5770r.
> +
>  config AD5791
>  	tristate "Analog Devices AD5760/AD5780/AD5781/AD5790/AD5791 DAC SPI driver"
>  	depends on SPI
> diff --git a/drivers/iio/dac/Makefile b/drivers/iio/dac/Makefile
> index 1369fa1d2f0e..2fc481167724 100644
> --- a/drivers/iio/dac/Makefile
> +++ b/drivers/iio/dac/Makefile
> @@ -19,6 +19,7 @@ obj-$(CONFIG_AD5755) += ad5755.o
>  obj-$(CONFIG_AD5755) += ad5758.o
>  obj-$(CONFIG_AD5761) += ad5761.o
>  obj-$(CONFIG_AD5764) += ad5764.o
> +obj-$(CONFIG_AD5770R) += ad5770r.o
>  obj-$(CONFIG_AD5791) += ad5791.o
>  obj-$(CONFIG_AD5686) += ad5686.o
>  obj-$(CONFIG_AD5686_SPI) += ad5686-spi.o
> diff --git a/drivers/iio/dac/ad5770r.c b/drivers/iio/dac/ad5770r.c
> new file mode 100644
> index 000000000000..a8b2c1850ffd
> --- /dev/null
> +++ b/drivers/iio/dac/ad5770r.c
> @@ -0,0 +1,662 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +/*
> + * AD5770R Digital to analog converters driver
> + *
> + * Copyright 2018 Analog Devices Inc.
> + */
> +
> +#include <linux/delay.h>
> +#include <linux/device.h>
> +#include <linux/gpio/consumer.h>
> +#include <linux/iio/common/adi_spi_regs.h>
> +#include <linux/iio/iio.h>
> +#include <linux/iio/sysfs.h>
> +#include <linux/kernel.h>
> +#include <linux/module.h>
> +#include <linux/property.h>
> +#include <linux/regmap.h>
> +#include <linux/regulator/consumer.h>
> +#include <linux/spi/spi.h>
> +
> +/* AD5770R configuration registers */
> +#define AD5770R_CHANNEL_CONFIG		0x14
> +#define AD5770R_OUTPUT_RANGE(ch)	(0x15 + (ch))
> +#define AD5770R_FILTER_RESISTOR(ch)	(0x1D + (ch))
> +#define AD5770R_REFERENCE		0x1B
> +#define AD5770R_DAC_LSB(ch)		(0x26 + 2 * (ch))
> +#define AD5770R_DAC_MSB(ch)		(0x27 + 2 * (ch))
> +#define AD5770R_CH_SELECT		0x34
> +#define AD5770R_CH_ENABLE		0x44
> +
> +/* AD5770R_CHANNEL_CONFIG */
> +#define AD5770R_CFG_CH0_SINK_EN(x)		(((x) & 0x1) << 7)
> +#define AD5770R_CFG_SHUTDOWN_B(x, ch)		(((x) & 0x1) << (ch))
> +
> +/* AD5770R_OUTPUT_RANGE */
> +#define AD5770R_RANGE_OUTPUT_SCALING(x)		(((x) & 0x3F) << 2)
> +#define AD5770R_RANGE_MODE(x)			((x) & 0x03)
> +
> +/* AD5770R_REFERENCE */
> +#define AD5770R_REF_RESISTOR_SEL(x)		(((x) & 0x1) << 2)
> +#define AD5770R_REF_SEL(x)			(((x) & 0x3) << 0)
> +
> +/* AD5770R_CH_ENABLE */
> +#define AD5770R_CH_SET(x, channel)		(((x) & 0x1) << (channel))
> +
> +#define AD5770R_MAX_CHANNELS	6
> +#define AD5770R_MAX_CH_MODES	14
> +#define AD5770R_LOW_VREF_mV	1250
> +#define AD5770R_HIGH_VREF_mV	2500
> +
> +enum ad5770r_ch0_modes {
> +	AD5770R_CH0_0_300 = 0,
> +	AD5770R_CH0_NEG_60_0,
> +	AD5770R_CH0_NEG_60_300
> +};
> +
> +enum ad5770r_ch1_modes {
> +	AD5770R_CH1_0_140_LOW_HEAD = 1,
> +	AD5770R_CH1_0_140_LOW_NOISE,
> +	AD5770R_CH1_0_250
> +};
> +
> +enum ad5770r_ch2_5_modes {
> +	AD5770R_CH_LOW_RANGE = 0,
> +	AD5770R_CH_HIGH_RANGE
> +};
> +
> +enum ad5770r_ref_v {
> +	AD5770R_EXT_2_5_V = 0,
> +	AD5770R_INT_1_25_V_OUT_ON,
> +	AD5770R_EXT_1_25_V,
> +	AD5770R_INT_1_25_V_OUT_OFF
> +};
> +
> +enum ad5770r_output_filter_resistor {
> +	AD5770R_FILTER_60_OHM = 0x0,
> +	AD5770R_FILTER_5_6_KOHM = 0x5,
> +	AD5770R_FILTER_11_2_KOHM,
> +	AD5770R_FILTER_22_2_KOHM,
> +	AD5770R_FILTER_44_4_KOHM,
> +	AD5770R_FILTER_104_KOHM,
> +};
> +
> +struct ad5770r_out_range {
> +	u8	out_scale;
> +	u8	out_range_mode;
> +};
> +
> +/**
> + * struct ad5770R_state - driver instance specific data
> + * @spi:		spi_device
> + * @regmap:		regmap
> + * @vref_reg:		fixed regulator for reference configuration
> + * @gpio_reset:		gpio descriptor
> + * @output_mode:		array contains channels output ranges
> + * @vref:		reference value
> + * @ch_pwr_down:		powerdown flags
> + * @internal_ref:	internal reference flag
> + * @external_res:	external 2.5k resistor flag
> + */
> +struct ad5770r_state {
> +	struct spi_device		*spi;
> +	struct regmap			*regmap;
> +	struct regulator		*vref_reg;
> +	struct gpio_desc		*gpio_reset;
> +	struct ad5770r_out_range	output_mode[AD5770R_MAX_CHANNELS];
> +	int				vref;
> +	bool				ch_pwr_down[AD5770R_MAX_CHANNELS];
> +	bool				internal_ref;
> +	bool				external_res;
> +};
> +
> +static const struct regmap_config ad5770r_spi_regmap_config = {
> +	.reg_bits = 8,
> +	.val_bits = 8,
> +	.read_flag_mask = BIT(7),
> +};
> +
> +struct ad5770r_output_modes {
> +	unsigned int ch;
> +	u8 mode;
> +	int min;
> +	int max;
> +};
> +
> +static struct ad5770r_output_modes ad5770r_rng_tbl[] = {
> +	{ 0, AD5770R_CH0_0_300, 0, 300 },
> +	{ 0, AD5770R_CH0_NEG_60_0, -60, 0 },
> +	{ 0, AD5770R_CH0_NEG_60_300, -60, 300 },
> +	{ 1, AD5770R_CH1_0_140_LOW_HEAD, 0, 140 },
> +	{ 1, AD5770R_CH1_0_140_LOW_NOISE, 0, 140 },
> +	{ 1, AD5770R_CH1_0_250, 0, 250 },
> +	{ 2, AD5770R_CH_LOW_RANGE, 0, 55 },
> +	{ 2, AD5770R_CH_HIGH_RANGE, 0, 150 },
> +	{ 3, AD5770R_CH_LOW_RANGE, 0, 45 },
> +	{ 3, AD5770R_CH_HIGH_RANGE, 0, 100 },
> +	{ 4, AD5770R_CH_LOW_RANGE, 0, 45 },
> +	{ 4, AD5770R_CH_HIGH_RANGE, 0, 100 },
> +	{ 5, AD5770R_CH_LOW_RANGE, 0, 45 },
> +	{ 5, AD5770R_CH_HIGH_RANGE, 0, 100 },
> +};
> +
> +static const unsigned int ad5770r_filter_freqs[] = {
> +	153, 357, 715, 1400, 2800, 262000,
> +};
> +
> +static const unsigned int ad5770r_filter_reg_vals[] = {
> +	AD5770R_FILTER_104_KOHM,
> +	AD5770R_FILTER_44_4_KOHM,
> +	AD5770R_FILTER_22_2_KOHM,
> +	AD5770R_FILTER_11_2_KOHM,
> +	AD5770R_FILTER_5_6_KOHM,
> +	AD5770R_FILTER_60_OHM
> +};
> +
> +static int ad5770r_set_output_mode(struct ad5770r_state *st,
> +				   const struct ad5770r_out_range *out_mode,
> +				   int channel)
> +{
> +	unsigned int regval;
> +
> +	regval = AD5770R_RANGE_OUTPUT_SCALING(out_mode->out_scale) |
> +		 AD5770R_RANGE_MODE(out_mode->out_range_mode);
> +
> +	return regmap_write(st->regmap,
> +			    AD5770R_OUTPUT_RANGE(channel), regval);
> +}
> +
> +static int ad5770r_set_reference(struct ad5770r_state *st)
> +{
> +	unsigned int regval = 0;

It's immediately set to something else, so don't initialize it here.

> +
> +	regval = AD5770R_REF_RESISTOR_SEL(st->external_res);
> +
> +	if (st->internal_ref) {
> +		regval |= AD5770R_REF_SEL(AD5770R_INT_1_25_V_OUT_OFF);
> +	} else {
> +		switch (st->vref) {
> +		case AD5770R_LOW_VREF_mV:
> +			regval |= AD5770R_REF_SEL(AD5770R_EXT_1_25_V);
> +			break;
> +		case AD5770R_HIGH_VREF_mV:
> +			regval |= AD5770R_REF_SEL(AD5770R_EXT_2_5_V);
> +			break;
> +		default:
> +			regval = AD5770R_REF_SEL(AD5770R_INT_1_25_V_OUT_OFF);
> +			break;
> +		}
> +	}
> +
> +	return regmap_write(st->regmap, AD5770R_REFERENCE, regval);
> +}
> +
> +static int ad5770r_soft_reset(struct ad5770r_state *st)
> +{
> +	return regmap_write(st->regmap, ADI_SPI_IF_CONFIG_A,
> +			    ADI_SPI_IF_SW_RESET_SEL(1));
> +}
> +
> +static int ad5770r_reset(struct ad5770r_state *st)
> +{
> +	/* Perform software reset if no GPIO provided */
> +	if (!st->gpio_reset)
> +		return ad5770r_soft_reset(st);
> +
> +	gpiod_set_value_cansleep(st->gpio_reset, 0);
> +	usleep_range(10, 20);
> +	gpiod_set_value_cansleep(st->gpio_reset, 1);
> +
> +	/* data must not be written during reset timeframe */
> +	usleep_range(100, 200);
> +
> +	return 0;
> +}
> +
> +static int ad5770r_get_range(struct ad5770r_state *st,
> +			     int ch, int *min, int *max)
> +{
> +	int i;
> +	u8 tbl_ch, tbl_mode, out_range;
> +
> +	out_range = st->output_mode[ch].out_range_mode;
> +
> +	for (i = 0; i < AD5770R_MAX_CH_MODES; i++) {
> +		tbl_ch = ad5770r_rng_tbl[i].ch;
> +		tbl_mode = ad5770r_rng_tbl[i].mode;
> +		if (tbl_ch == ch && tbl_mode == out_range) {
> +			*min = ad5770r_rng_tbl[i].min;
> +			*max = ad5770r_rng_tbl[i].max;
> +			return 0;
> +		}
> +	}
> +
> +	return -EINVAL;
> +}
> +
> +static int ad5770r_get_filter_freq(struct iio_dev *indio_dev,
> +				   const struct iio_chan_spec *chan, int *freq)
> +{
> +	struct ad5770r_state *st = iio_priv(indio_dev);
> +	int ret;
> +	unsigned int regval, i;
> +
> +	ret = regmap_read(st->regmap,
> +			  AD5770R_FILTER_RESISTOR(chan->channel), &regval);
> +	if (ret < 0)
> +		return ret;
> +
> +	for (i = 0; i < ARRAY_SIZE(ad5770r_filter_reg_vals); i++)
> +		if (regval == ad5770r_filter_reg_vals[i])
> +			break;
> +	if (i == ARRAY_SIZE(ad5770r_filter_reg_vals))
> +		return -EINVAL;
> +
> +	*freq = ad5770r_filter_freqs[i];
> +
> +	return IIO_VAL_INT;
> +}
> +
> +static int ad5770r_set_filter_freq(struct iio_dev *indio_dev,
> +				   const struct iio_chan_spec *chan,
> +				   unsigned int freq)
> +{
> +	struct ad5770r_state *st = iio_priv(indio_dev);
> +	unsigned int regval, i;
> +
> +	for (i = 0; i < ARRAY_SIZE(ad5770r_filter_freqs); i++)
> +		if (ad5770r_filter_freqs[i] >= freq)
> +			break;
> +	if (i == ARRAY_SIZE(ad5770r_filter_freqs))
> +		return -EINVAL;
> +
> +	regval = ad5770r_filter_reg_vals[i];
> +
> +	return regmap_write(st->regmap, AD5770R_FILTER_RESISTOR(chan->channel),
> +			    regval);
> +}
> +
> +static int ad5770r_read_raw(struct iio_dev *indio_dev,
> +			    struct iio_chan_spec const *chan,
> +			    int *val, int *val2, long info)
> +{
> +	struct ad5770r_state *st = iio_priv(indio_dev);
> +	int max, min, ret;
> +	__le16 buf16;
> +
> +	switch (info) {
> +	case IIO_CHAN_INFO_RAW:
> +		ret = regmap_bulk_read(st->regmap,
> +				       chan->address,
> +				       &buf16, 2);
> +		if (ret)
> +			return 0;
> +		*val = le16_to_cpu(buf16) >> 2;
> +		return IIO_VAL_INT;
> +	case IIO_CHAN_INFO_SCALE:
> +		ret = ad5770r_get_range(st, chan->channel, &min, &max);
> +		if (ret < 0)
> +			return ret;
> +		*val = max - min;
> +		/* There is no sign bit. (negative current is mapped from 0)
> +		 * (sourced/sinked) current = raw * scale + offset
> +		 * where offset in case of CH0 can be negative.
> +		 */
> +		*val2 = 14;
> +		return IIO_VAL_FRACTIONAL_LOG2;
> +	case IIO_CHAN_INFO_LOW_PASS_FILTER_3DB_FREQUENCY:
> +		return ad5770r_get_filter_freq(indio_dev, chan, val);
> +	case IIO_CHAN_INFO_OFFSET:
> +		ret = ad5770r_get_range(st, chan->channel, &min, &max);
> +		if (ret < 0)
> +			return ret;
> +		*val = min;
> +		return IIO_VAL_INT;
> +	default:
> +		return -EINVAL;
> +	}
> +}
> +
> +static int ad5770r_write_raw(struct iio_dev *indio_dev,
> +			     struct iio_chan_spec const *chan,
> +			     int val, int val2, long info)
> +{
> +	struct ad5770r_state *st = iio_priv(indio_dev);
> +	u8 buf[2];
> +
> +	switch (info) {
> +	case IIO_CHAN_INFO_RAW:
> +		buf[0] = ((u16)val >> 6);
> +		buf[1] = (val & 0x3F) << 2;
> +		return regmap_bulk_write(st->regmap, chan->address,
> +					 buf, ARRAY_SIZE(buf));
> +	case IIO_CHAN_INFO_LOW_PASS_FILTER_3DB_FREQUENCY:
> +		return ad5770r_set_filter_freq(indio_dev, chan, val);
> +	default:
> +		return -EINVAL;
> +	}
> +}
> +
> +static int ad5770r_read_freq_avail(struct iio_dev *indio_dev,
> +				   struct iio_chan_spec const *chan,
> +				   const int **vals, int *type, int *length,
> +				   long mask)
> +{
> +	switch (mask) {
> +	case IIO_CHAN_INFO_LOW_PASS_FILTER_3DB_FREQUENCY:
> +		*type = IIO_VAL_INT;
> +		*vals = ad5770r_filter_freqs;
> +		*length = ARRAY_SIZE(ad5770r_filter_freqs);
> +		return IIO_AVAIL_LIST;
> +	}
> +
> +	return -EINVAL;
> +}
> +
> +static int ad5770r_reg_access(struct iio_dev *indio_dev,
> +			      unsigned int reg,
> +			      unsigned int writeval,
> +			      unsigned int *readval)
> +{
> +	struct ad5770r_state *st = iio_priv(indio_dev);
> +
> +	if (readval)
> +		return regmap_read(st->regmap, reg, readval);
> +	else
> +		return regmap_write(st->regmap, reg, writeval);
> +}
> +
> +static const struct iio_info ad5770r_info = {
> +	.read_raw = ad5770r_read_raw,
> +	.write_raw = ad5770r_write_raw,
> +	.read_avail = ad5770r_read_freq_avail,
> +	.debugfs_reg_access = &ad5770r_reg_access,
> +};
> +
> +static int ad5770r_store_output_range(struct ad5770r_state *st,
> +				      int min, int max, int index)
> +{
> +	int i;
> +
> +	for (i = 0; i < AD5770R_MAX_CH_MODES; i++) {
> +		if (ad5770r_rng_tbl[i].ch != index)
> +			continue;
> +		if (ad5770r_rng_tbl[i].min != min ||
> +		    ad5770r_rng_tbl[i].max != max)
> +			continue;
> +		st->output_mode[index].out_range_mode = ad5770r_rng_tbl[i].mode;
> +
> +		return 0;
> +	}
> +
> +	return -EINVAL;
> +}
> +
> +static ssize_t ad5770r_read_dac_powerdown(struct iio_dev *indio_dev,
> +					  uintptr_t private,
> +					  const struct iio_chan_spec *chan,
> +					  char *buf)
> +{
> +	struct ad5770r_state *st = iio_priv(indio_dev);
> +
> +	return sprintf(buf, "%d\n", st->ch_pwr_down[chan->channel]);
> +}
> +
> +static ssize_t ad5770r_write_dac_powerdown(struct iio_dev *indio_dev,
> +					   uintptr_t private,
> +					   const struct iio_chan_spec *chan,
> +					   const char *buf, size_t len)
> +{
> +	struct ad5770r_state *st = iio_priv(indio_dev);
> +	unsigned int regval;
> +	unsigned int mask;
> +	bool readin;
> +	int ret;
> +
> +	ret = kstrtobool(buf, &readin);
> +	if (ret)
> +		return ret;
> +
> +	readin = !readin;
> +
> +	regval = AD5770R_CFG_SHUTDOWN_B(readin, chan->channel);
> +	if (chan->channel == 0 &&
> +	    st->output_mode[0].out_range_mode > AD5770R_CH0_0_300) {
> +		regval |= AD5770R_CFG_CH0_SINK_EN(readin);
> +		mask = BIT(chan->channel) + BIT(7);
> +	} else {
> +		mask = BIT(chan->channel);
> +	}
> +	ret = regmap_update_bits(st->regmap, AD5770R_CHANNEL_CONFIG, mask,
> +				 regval);
> +	if (ret)
> +		return ret;
> +
> +	regval = AD5770R_CH_SET(readin, chan->channel);
> +	ret = regmap_update_bits(st->regmap, AD5770R_CH_ENABLE,
> +				 BIT(chan->channel), regval);
> +	if (ret)
> +		return ret;
> +
> +	st->ch_pwr_down[chan->channel] = !readin;
> +
> +	return len;
> +}
> +
> +static const struct iio_chan_spec_ext_info ad5770r_ext_info[] = {
> +	{
> +		.name = "powerdown",
> +		.read = ad5770r_read_dac_powerdown,
> +		.write = ad5770r_write_dac_powerdown,
> +		.shared = IIO_SEPARATE,
> +	},
> +	{ }
> +};
> +
> +#define AD5770R_IDAC_CHANNEL(index, reg) {				\
> +	.type = IIO_CURRENT,						\
> +	.address = reg,							\
> +	.indexed = 1,							\
> +	.channel = index,						\
> +	.output = 1,							\
> +	.info_mask_separate = BIT(IIO_CHAN_INFO_RAW) |			\
> +		BIT(IIO_CHAN_INFO_SCALE) |				\
> +		BIT(IIO_CHAN_INFO_OFFSET) |				\
> +		BIT(IIO_CHAN_INFO_LOW_PASS_FILTER_3DB_FREQUENCY),	\
> +	.info_mask_shared_by_type_available =				\
> +		BIT(IIO_CHAN_INFO_LOW_PASS_FILTER_3DB_FREQUENCY),	\
> +	.ext_info = ad5770r_ext_info,					\
> +}
> +
> +static const struct iio_chan_spec ad5770r_channels[] = {
> +	AD5770R_IDAC_CHANNEL(0, AD5770R_DAC_MSB(0)),
> +	AD5770R_IDAC_CHANNEL(1, AD5770R_DAC_MSB(1)),
> +	AD5770R_IDAC_CHANNEL(2, AD5770R_DAC_MSB(2)),
> +	AD5770R_IDAC_CHANNEL(3, AD5770R_DAC_MSB(3)),
> +	AD5770R_IDAC_CHANNEL(4, AD5770R_DAC_MSB(4)),
> +	AD5770R_IDAC_CHANNEL(5, AD5770R_DAC_MSB(5)),
> +};
> +
> +static int ad5770r_channel_config(struct ad5770r_state *st)
> +{
> +	int ret, tmp[2], min, max;
> +	unsigned int num;
> +	struct fwnode_handle *child;
> +
> +	num = device_get_child_node_count(&st->spi->dev);
> +	if (num != AD5770R_MAX_CHANNELS)
> +		return -EINVAL;
> +
> +	device_for_each_child_node(&st->spi->dev, child) {
> +		ret = fwnode_property_read_u32(child, "num", &num);
> +		if (ret)
> +			return ret;
> +		if (num > AD5770R_MAX_CHANNELS)
> +			return -EINVAL;
> +
> +		ret = fwnode_property_read_u32_array(child,
> +						     "adi,range-microamp",
> +						     tmp, 2);
> +		if (ret)
> +			return ret;
> +
> +		min = tmp[0] / 1000;
> +		max = tmp[1] / 1000;
> +		ret = ad5770r_store_output_range(st, min, max, num);
> +		if (ret)
> +			return ret;
> +	}
> +
> +	return ret;
> +}
> +
> +static int ad5770r_init(struct ad5770r_state *st)
> +{
> +	int ret, i;
> +
> +	st->gpio_reset = devm_gpiod_get_optional(&st->spi->dev, "reset",
> +						 GPIOD_OUT_HIGH);
> +	if (IS_ERR(st->gpio_reset))
> +		return PTR_ERR(st->gpio_reset);
> +
> +	/* Perform a reset */
> +	ret = ad5770r_reset(st);
> +	if (ret)
> +		return ret;
> +
> +	/* Set output range */
> +	ret = ad5770r_channel_config(st);
> +	if (ret)
> +		return ret;
> +
> +	for (i = 0; i < AD5770R_MAX_CHANNELS; i++) {
> +		ret = ad5770r_set_output_mode(st,  &st->output_mode[i], i);
> +		if (ret)
> +			return ret;
> +	}
> +
> +	st->external_res = fwnode_property_read_bool(st->spi->dev.fwnode,
> +						     "adi,external-resistor");
> +
> +	ret = ad5770r_set_reference(st);
> +	if (ret)
> +		return ret;
> +
> +	/* Set outputs off */
> +	ret = regmap_write(st->regmap, AD5770R_CHANNEL_CONFIG, 0x00);
> +	if (ret)
> +		return ret;
> +
> +	ret = regmap_write(st->regmap, AD5770R_CH_ENABLE, 0x00);
> +	if (ret)
> +		return ret;
> +
> +	for (i = 0; i < AD5770R_MAX_CHANNELS; i++)
> +		st->ch_pwr_down[i] = true;
> +
> +	return ret;
> +}
> +
> +static void ad5770r_disable_regulator(void *data)
> +{
> +	struct spi_device *spi = (struct spi_device *)data;
> +	struct iio_dev *indio_dev = spi_get_drvdata(spi);
> +	struct ad5770r_state *st = iio_priv(indio_dev);
> +
> +	if (st->vref_reg)
> +		regulator_disable(st->vref_reg);
> +}
> +
> +static int ad5770r_probe(struct spi_device *spi)
> +{
> +	struct ad5770r_state *st;
> +	struct iio_dev *indio_dev;
> +	struct regmap *regmap;
> +	int ret;
> +
> +	indio_dev = devm_iio_device_alloc(&spi->dev, sizeof(*st));
> +	if (!indio_dev)
> +		return -ENOMEM;
> +
> +	st = iio_priv(indio_dev);
> +	spi_set_drvdata(spi, indio_dev);
> +
> +	st->spi = spi;
> +
> +	regmap = devm_regmap_init_spi(spi, &ad5770r_spi_regmap_config);
> +	if (IS_ERR(regmap)) {
> +		dev_err(&spi->dev, "Error initializing spi regmap: %ld\n",
> +			PTR_ERR(regmap));
> +		return PTR_ERR(regmap);
> +	}
> +	st->regmap = regmap;
> +
> +	ret = devm_add_action_or_reset(&spi->dev, ad5770r_disable_regulator,
> +				       st);
> +	if (ret < 0)
> +		return ret;
> +
> +	st->vref_reg = devm_regulator_get_optional(&spi->dev, "vref");
> +	if (!IS_ERR(st->vref_reg)) {
> +		ret = regulator_enable(st->vref_reg);
> +		if (ret) {
> +			dev_err(&spi->dev,
> +				"Failed to enable vref regulators: %d\n", ret);
> +			return ret;
> +		}
> +
> +		ret = regulator_get_voltage(st->vref_reg);
> +		if (ret < 0)
> +			return ret;
> +
> +		st->vref = ret / 1000;
> +	} else {
> +		if (PTR_ERR(st->vref_reg) == -ENODEV) {
> +			st->vref = AD5770R_LOW_VREF_mV;
> +			st->internal_ref = true;
> +		} else {
> +			return PTR_ERR(st->vref_reg);
> +		}
> +	}
> +
> +	indio_dev->dev.parent = &spi->dev;
> +	indio_dev->name = spi_get_device_id(spi)->name;
> +	indio_dev->info = &ad5770r_info;
> +	indio_dev->modes = INDIO_DIRECT_MODE;
> +	indio_dev->channels = ad5770r_channels;
> +	indio_dev->num_channels = ARRAY_SIZE(ad5770r_channels);
> +
> +	ret = ad5770r_init(st);
> +	if (ret < 0) {
> +		dev_err(&spi->dev, "AD5770R init failed\n");
> +		return ret;
> +	}
> +
> +	return devm_iio_device_register(&st->spi->dev, indio_dev);
> +}
> +
> +static const struct of_device_id ad5770r_of_id[] = {
> +	{ .compatible = "adi,ad5770r", },
> +};
> +MODULE_DEVICE_TABLE(of, ad5770r_of_id);
> +
> +static const struct spi_device_id ad5770r_id[] = {
> +	{ "ad5770r", 0 },
> +	{}
> +};
> +MODULE_DEVICE_TABLE(spi, ad5770r_id);
> +
> +static struct spi_driver ad5770r_driver = {
> +	.driver = {
> +		.name = KBUILD_MODNAME,
> +		.of_match_table = of_match_ptr(ad5770r_of_id),
> +	},
> +	.probe = ad5770r_probe,
> +	.id_table = ad5770r_id,
> +};
> +
> +module_spi_driver(ad5770r_driver);
> +
> +MODULE_AUTHOR("Mircea Caprioru <mircea.caprioru@analog.com>");
> +MODULE_DESCRIPTION("Analog Devices AD5770R IDAC");
> +MODULE_LICENSE("GPL v2");
> diff --git a/include/linux/iio/common/adi_spi_regs.h b/include/linux/iio/common/adi_spi_regs.h
> new file mode 100644
> index 000000000000..1fc3bfc7845d
> --- /dev/null
> +++ b/include/linux/iio/common/adi_spi_regs.h
> @@ -0,0 +1,44 @@
> +/* SPDX-License-Identifier: GPL-2.0 */
> +/*
> + * Common Analog Devices SPI configuration registers
> + *
> + * Copyright 2019 Analog Devices Inc.
> + */
> +
> +#ifndef _ADI_SPI_REGS_H_
> +#define _ADI_SPI_REGS_H_
> +
> +#include <linux/bits.h>
> +
> +#define ADI_SPI_IF_CONFIG_A		0x00
> +#define ADI_SPI_IF_CONFIG_B		0x01
> +#define ADI_SPI_IF_DEVICE_CONFIG	0x02
> +#define ADI_SPI_IF_CHIP_TYPE		0x03
> +#define ADI_SPI_IF_PRODUCT_ID_L		0x04
> +#define ADI_SPI_IF_PRODUCT_ID_H		0x05
> +#define ADI_SPI_IF_CHIP_GRADE		0x06
> +#define ADI_SPI_IF_SCRACTH_PAD		0x0A
> +#define ADI_SPI_IF_SPI_REVISION		0x0B
> +#define ADI_SPI_IF_SPI_VENDOR_L		0x0C
> +#define ADI_SPI_IF_SPI_VENDOR_H		0x0D
> +#define ADI_SPI_IF_SPI_STREAM_MODE	0x0E
> +#define ADI_SPI_IF_CONFIG_C		0x10
> +#define ADI_SPI_IF_STATUS_A		0x11
> +
> +/* ADI_SPI_IF_CONFIG_A */
> +#define ADI_SPI_IF_SW_RESET_MSK		(BIT(0) | BIT(7))
> +#define ADI_SPI_IF_SW_RESET_SEL(x)	((x) & ADI_SPI_IF_SW_RESET_MSK)
> +#define ADI_SPI_IF_ADDR_ASC_MSK		(BIT(2) | BIT(5))
> +#define ADI_SPI_IF_ADDR_ASC_SEL(x)	(((x) << 2) & ADI_SPI_IF_ADDR_ASC_MSK)
> +
> +/* ADI_SPI_IF_CONFIG_B */
> +#define ADI_SPI_IF_SINGLE_INS_MSK	BIT(7)
> +#define ADI_SPI_IF_SINGLE_INS_SEL(x)	FIELD_PREP(ADI_SPI_IF_SINGLE_INS_MSK, x)
> +#define ADI_SPI_IF_SHORT_INS_MSK	BIT(7)
> +#define ADI_SPI_IF_SHORT_INS_SEL(x)	FIELD_PREP(ADI_SPI_IF_SINGLE_INS_MSK, x)
> +
> +/* ADI_SPI_IF_CONFIG_C */
> +#define ADI_SPI_IF_STRICT_REG_MSK	BIT(5)
> +#define ADI_SPI_IF_STRICT_REG_GET(x)	FIELD_GET(ADI_SPI_IF_STRICT_REG_MSK, x)
> +
> +#endif /* _ADI_SPI_REGS_H_ */


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 2/2] dt-bindings: iio: dac: Add docs for AD5770R DAC
  2020-02-06  9:23 ` [PATCH 2/2] dt-bindings: iio: dac: Add docs for AD5770R DAC Alexandru Tachici
@ 2020-02-08 14:36   ` Jonathan Cameron
  0 siblings, 0 replies; 9+ messages in thread
From: Jonathan Cameron @ 2020-02-08 14:36 UTC (permalink / raw)
  To: Alexandru Tachici; +Cc: linux-iio, linux-kernel, Mircea Caprioru

On Thu, 6 Feb 2020 11:23:13 +0200
Alexandru Tachici <alexandru.tachici@analog.com> wrote:

> Adding dt-bindings documentation for AD5770R DAC.
> 
> Signed-off-by: Mircea Caprioru <mircea.caprioru@analog.com>
> Signed-off-by: Alexandru Tachici <alexandru.tachici@analog.com>

A few minor bits inline.

> ---
>  .../bindings/iio/dac/adi,ad5770r.yaml         | 176 ++++++++++++++++++
>  1 file changed, 176 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml
> 
> diff --git a/Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml b/Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml
> new file mode 100644
> index 000000000000..02b2f6e67583
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml
> @@ -0,0 +1,176 @@
> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
> +# Copyright 2020 Analog Devices Inc.
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/bindings/iio/adc/adi,ad5770r.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Analog Devices AD5770R DAC device driver
> +
> +maintainers:
> +  - Mircea Caprioru <mircea.caprioru@analog.com>
> +
> +description: |
> +  Bindings for the Analog Devices AD5770R current DAC device. Datasheet can be
> +  found here:
> +    https://www.analog.com/media/en/technical-documentation/data-sheets/AD5770R.pdf
> +
> +properties:
> +  compatible:
> +    enum:
> +      - adi,ad5770r
> +
> +  reg:
> +    maxItems: 1
> +
> +  vref-supply:

Probably need to also mention the main power supply?
Ideally enable it in the driver, but if not should still be in the binding.

> +    description: Specify the voltage of the external reference used.
> +      Available reference options are 1.25 V or 2.5 V. If no
> +      external reference declared then the device will use the
> +      internal reference of 1.25 V.
> +
> +  adi,external-resistor:
> +    description: Specify if an external 2.5k ohm resistor is used. If not
> +      specified the device will use and internal 2.5k ohm resistor.

use an internal...

> +      The precision resistor is used for reference current generation.
> +    type: boolean
> +
> +  reset-gpios:
> +    description: GPIO spec for the RESET pin. If specified, it will be
> +      asserted during driver probe.
> +    maxItems: 1
> +
> +  channel0:
> +    description: Represents an external channel which are
> +      connected to the DAC. Channel 0 can act both as a current
> +      source and sink.
> +    type: object
> +
> +    properties:
> +      num:
> +        description: This represents the channel number.
> +        items:
> +          const: 0
> +
> +      adi,range-microamp:
> +          description: Output range of the channel.
> +          oneOf:
> +            - $ref: /schemas/types.yaml#/definitions/int32-array
> +            - items:
> +                - enum: [0 300000]
> +                - enum: [-60000 0]
> +                - enum: [-60000 300000]
> +
> +  channel1:
> +    description: Represents an external channel which are
> +      connected to the DAC.
> +    type: object
> +
> +    properties:
> +      num:
> +        description: This represents the channel number.
> +        items:
> +          const: 1
> +
> +      adi,range-microamp:
> +          description: Output range of the channel.
> +          allOf:
> +            - $ref: /schemas/types.yaml#/definitions/uint32-array
> +            - items:
> +                - enum: [0 140000]
> +                - enum: [0 250000]
> +
> +  channel2:
> +    description: Represents an external channel which are
> +      connected to the DAC.
> +    type: object
> +
> +    properties:
> +      num:
> +        description: This represents the channel number.
> +        items:
> +          const: 2
> +
> +      adi,range-microamp:
> +          description: Output range of the channel.
> +          allOf:
> +            - $ref: /schemas/types.yaml#/definitions/uint32-array
> +            - items:
> +                - enum: [0 140000]
> +                - enum: [0 250000]
> +
> +patternProperties:
> +  "^channel@([3-5])$":
> +    type: object
> +    description: Represents the external channels which are connected to the DAC.
> +    properties:
> +      num:
> +        description: This represents the channel number.
> +        items:
> +          minimum: 3
> +          maximum: 5
> +
> +      adi,range-microamp:
> +          description: Output range of the channel.
> +          oneOf:
> +            - $ref: /schemas/types.yaml#/definitions/uint32-array
> +            - items:
> +                - enum: [0 45000]
> +                - enum: [0 100000]
> +
> +required:
> +- reg
> +- diff-channels
> +- channel0
> +- channel1
> +- channel2
> +- channel3
> +- channel4
> +- channel5
> +
> +examples:
> +  - |
> +        spi {
> +                #address-cells = <1>;
> +                #size-cells = <0>;
> +
> +                ad5770r@0 {
> +                        compatible = "ad5770r";
> +                        reg = <0>;
> +                        spi-max-frequency = <1000000>;
> +                        vref-supply = <&vref>;
> +                        adi,external-resistor;
> +                        reset-gpios = <&gpio 22 0>;
> +
> +                        channel@0 {
> +                                num = <0>;
> +                                adi,range-microamp = <(-60000) 300000>;
> +                        };
> +
> +                        channel@1 {
> +                                num = <1>;
> +                                adi,range-microamp = <0 140000>;
> +                        };
> +
> +                        channel@2 {
> +                                num = <2>;
> +                                adi,range-microamp = <0 55000>;
> +                        };
> +
> +                        channel@3 {
> +                                num = <3>;
> +                                adi,range-microamp = <0 45000>;
> +                        };
> +
> +                        channel@4 {
> +                                num = <4>;
> +                                adi,range-microamp = <0 45000>;
> +                        };
> +
> +                        channel@5 {
> +                                num = <5>;
> +                                adi,range-microamp = <0 45000>;
> +                        };
> +                };
> +        };
> +...
> \ No newline at end of file
Tidy that up if you are doing a new version.



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 1/2] iio: dac: ad5770r: Add AD5770R support
  2020-02-08 14:33   ` Jonathan Cameron
@ 2020-02-11 13:51     ` Ardelean, Alexandru
  2020-02-14 12:45       ` Ardelean, Alexandru
  0 siblings, 1 reply; 9+ messages in thread
From: Ardelean, Alexandru @ 2020-02-11 13:51 UTC (permalink / raw)
  To: jic23, Tachici, Alexandru; +Cc: linux-kernel, linux-iio, Caprioru, Mircea

On Sat, 2020-02-08 at 14:33 +0000, Jonathan Cameron wrote:
> On Thu, 6 Feb 2020 11:23:12 +0200
> Alexandru Tachici <alexandru.tachici@analog.com> wrote:
> 
> > The AD5770R is a 6-channel, 14-bit resolution, low noise, programmable
> > current output digital-to-analog converter (DAC) for photonics control
> > applications.
> > 
> > It contains five 14-bit resolution current sourcing DAC channels and one
> > 14-bit resolution current sourcing/sinking DAC channel.
> 
> I'd like to see considerably more discussion of the common header.
> What devices do these shared registers apply to?
> Current header naming kind of implies all ADI devices with an
> spi interface which I find very unlikely.

Right.
My bad here, since I don't think I provided AlexandruT the full details on this.

We'll see about a V3 with some reference for these.

Thanks
Alex

> 
> Otherwise one minor comment inline.
> 
> Thanks,
> 
> Jonathan
> 
> 
> > Signed-off-by: Mircea Caprioru <mircea.caprioru@analog.com>
> > Signed-off-by: Alexandru Tachici <alexandru.tachici@analog.com>
> > ---
> >  drivers/iio/dac/Kconfig                 |  10 +
> >  drivers/iio/dac/Makefile                |   1 +
> >  drivers/iio/dac/ad5770r.c               | 662 ++++++++++++++++++++++++
> >  include/linux/iio/common/adi_spi_regs.h |  44 ++
> >  4 files changed, 717 insertions(+)
> >  create mode 100644 drivers/iio/dac/ad5770r.c
> >  create mode 100644 include/linux/iio/common/adi_spi_regs.h
> > 
> > diff --git a/drivers/iio/dac/Kconfig b/drivers/iio/dac/Kconfig
> > index 979070196da9..c74f2151ee42 100644
> > --- a/drivers/iio/dac/Kconfig
> > +++ b/drivers/iio/dac/Kconfig
> > @@ -208,6 +208,16 @@ config AD5764
> >  	  To compile this driver as a module, choose M here: the
> >  	  module will be called ad5764.
> >  
> > +config AD5770R
> > +	tristate "Analog Devices AD5770R IDAC driver"
> > +	depends on SPI_MASTER
> > +	help
> > +	  Say yes here to build support for Analog Devices AD5770R Digital to
> > +	  Analog Converter.
> > +
> > +	  To compile this driver as a module, choose M here: the
> > +	  module will be called ad5770r.
> > +
> >  config AD5791
> >  	tristate "Analog Devices AD5760/AD5780/AD5781/AD5790/AD5791 DAC SPI
> > driver"
> >  	depends on SPI
> > diff --git a/drivers/iio/dac/Makefile b/drivers/iio/dac/Makefile
> > index 1369fa1d2f0e..2fc481167724 100644
> > --- a/drivers/iio/dac/Makefile
> > +++ b/drivers/iio/dac/Makefile
> > @@ -19,6 +19,7 @@ obj-$(CONFIG_AD5755) += ad5755.o
> >  obj-$(CONFIG_AD5755) += ad5758.o
> >  obj-$(CONFIG_AD5761) += ad5761.o
> >  obj-$(CONFIG_AD5764) += ad5764.o
> > +obj-$(CONFIG_AD5770R) += ad5770r.o
> >  obj-$(CONFIG_AD5791) += ad5791.o
> >  obj-$(CONFIG_AD5686) += ad5686.o
> >  obj-$(CONFIG_AD5686_SPI) += ad5686-spi.o
> > diff --git a/drivers/iio/dac/ad5770r.c b/drivers/iio/dac/ad5770r.c
> > new file mode 100644
> > index 000000000000..a8b2c1850ffd
> > --- /dev/null
> > +++ b/drivers/iio/dac/ad5770r.c
> > @@ -0,0 +1,662 @@
> > +// SPDX-License-Identifier: GPL-2.0+
> > +/*
> > + * AD5770R Digital to analog converters driver
> > + *
> > + * Copyright 2018 Analog Devices Inc.
> > + */
> > +
> > +#include <linux/delay.h>
> > +#include <linux/device.h>
> > +#include <linux/gpio/consumer.h>
> > +#include <linux/iio/common/adi_spi_regs.h>
> > +#include <linux/iio/iio.h>
> > +#include <linux/iio/sysfs.h>
> > +#include <linux/kernel.h>
> > +#include <linux/module.h>
> > +#include <linux/property.h>
> > +#include <linux/regmap.h>
> > +#include <linux/regulator/consumer.h>
> > +#include <linux/spi/spi.h>
> > +
> > +/* AD5770R configuration registers */
> > +#define AD5770R_CHANNEL_CONFIG		0x14
> > +#define AD5770R_OUTPUT_RANGE(ch)	(0x15 + (ch))
> > +#define AD5770R_FILTER_RESISTOR(ch)	(0x1D + (ch))
> > +#define AD5770R_REFERENCE		0x1B
> > +#define AD5770R_DAC_LSB(ch)		(0x26 + 2 * (ch))
> > +#define AD5770R_DAC_MSB(ch)		(0x27 + 2 * (ch))
> > +#define AD5770R_CH_SELECT		0x34
> > +#define AD5770R_CH_ENABLE		0x44
> > +
> > +/* AD5770R_CHANNEL_CONFIG */
> > +#define AD5770R_CFG_CH0_SINK_EN(x)		(((x) & 0x1) << 7)
> > +#define AD5770R_CFG_SHUTDOWN_B(x, ch)		(((x) & 0x1) << (ch))
> > +
> > +/* AD5770R_OUTPUT_RANGE */
> > +#define AD5770R_RANGE_OUTPUT_SCALING(x)		(((x) & 0x3F) << 2)
> > +#define AD5770R_RANGE_MODE(x)			((x) & 0x03)
> > +
> > +/* AD5770R_REFERENCE */
> > +#define AD5770R_REF_RESISTOR_SEL(x)		(((x) & 0x1) << 2)
> > +#define AD5770R_REF_SEL(x)			(((x) & 0x3) << 0)
> > +
> > +/* AD5770R_CH_ENABLE */
> > +#define AD5770R_CH_SET(x, channel)		(((x) & 0x1) << (channel))
> > +
> > +#define AD5770R_MAX_CHANNELS	6
> > +#define AD5770R_MAX_CH_MODES	14
> > +#define AD5770R_LOW_VREF_mV	1250
> > +#define AD5770R_HIGH_VREF_mV	2500
> > +
> > +enum ad5770r_ch0_modes {
> > +	AD5770R_CH0_0_300 = 0,
> > +	AD5770R_CH0_NEG_60_0,
> > +	AD5770R_CH0_NEG_60_300
> > +};
> > +
> > +enum ad5770r_ch1_modes {
> > +	AD5770R_CH1_0_140_LOW_HEAD = 1,
> > +	AD5770R_CH1_0_140_LOW_NOISE,
> > +	AD5770R_CH1_0_250
> > +};
> > +
> > +enum ad5770r_ch2_5_modes {
> > +	AD5770R_CH_LOW_RANGE = 0,
> > +	AD5770R_CH_HIGH_RANGE
> > +};
> > +
> > +enum ad5770r_ref_v {
> > +	AD5770R_EXT_2_5_V = 0,
> > +	AD5770R_INT_1_25_V_OUT_ON,
> > +	AD5770R_EXT_1_25_V,
> > +	AD5770R_INT_1_25_V_OUT_OFF
> > +};
> > +
> > +enum ad5770r_output_filter_resistor {
> > +	AD5770R_FILTER_60_OHM = 0x0,
> > +	AD5770R_FILTER_5_6_KOHM = 0x5,
> > +	AD5770R_FILTER_11_2_KOHM,
> > +	AD5770R_FILTER_22_2_KOHM,
> > +	AD5770R_FILTER_44_4_KOHM,
> > +	AD5770R_FILTER_104_KOHM,
> > +};
> > +
> > +struct ad5770r_out_range {
> > +	u8	out_scale;
> > +	u8	out_range_mode;
> > +};
> > +
> > +/**
> > + * struct ad5770R_state - driver instance specific data
> > + * @spi:		spi_device
> > + * @regmap:		regmap
> > + * @vref_reg:		fixed regulator for reference configuration
> > + * @gpio_reset:		gpio descriptor
> > + * @output_mode:		array contains channels output ranges
> > + * @vref:		reference value
> > + * @ch_pwr_down:		powerdown flags
> > + * @internal_ref:	internal reference flag
> > + * @external_res:	external 2.5k resistor flag
> > + */
> > +struct ad5770r_state {
> > +	struct spi_device		*spi;
> > +	struct regmap			*regmap;
> > +	struct regulator		*vref_reg;
> > +	struct gpio_desc		*gpio_reset;
> > +	struct ad5770r_out_range	output_mode[AD5770R_MAX_CHANNELS];
> > +	int				vref;
> > +	bool				ch_pwr_down[AD5770R_MAX_CHANNELS];
> > +	bool				internal_ref;
> > +	bool				external_res;
> > +};
> > +
> > +static const struct regmap_config ad5770r_spi_regmap_config = {
> > +	.reg_bits = 8,
> > +	.val_bits = 8,
> > +	.read_flag_mask = BIT(7),
> > +};
> > +
> > +struct ad5770r_output_modes {
> > +	unsigned int ch;
> > +	u8 mode;
> > +	int min;
> > +	int max;
> > +};
> > +
> > +static struct ad5770r_output_modes ad5770r_rng_tbl[] = {
> > +	{ 0, AD5770R_CH0_0_300, 0, 300 },
> > +	{ 0, AD5770R_CH0_NEG_60_0, -60, 0 },
> > +	{ 0, AD5770R_CH0_NEG_60_300, -60, 300 },
> > +	{ 1, AD5770R_CH1_0_140_LOW_HEAD, 0, 140 },
> > +	{ 1, AD5770R_CH1_0_140_LOW_NOISE, 0, 140 },
> > +	{ 1, AD5770R_CH1_0_250, 0, 250 },
> > +	{ 2, AD5770R_CH_LOW_RANGE, 0, 55 },
> > +	{ 2, AD5770R_CH_HIGH_RANGE, 0, 150 },
> > +	{ 3, AD5770R_CH_LOW_RANGE, 0, 45 },
> > +	{ 3, AD5770R_CH_HIGH_RANGE, 0, 100 },
> > +	{ 4, AD5770R_CH_LOW_RANGE, 0, 45 },
> > +	{ 4, AD5770R_CH_HIGH_RANGE, 0, 100 },
> > +	{ 5, AD5770R_CH_LOW_RANGE, 0, 45 },
> > +	{ 5, AD5770R_CH_HIGH_RANGE, 0, 100 },
> > +};
> > +
> > +static const unsigned int ad5770r_filter_freqs[] = {
> > +	153, 357, 715, 1400, 2800, 262000,
> > +};
> > +
> > +static const unsigned int ad5770r_filter_reg_vals[] = {
> > +	AD5770R_FILTER_104_KOHM,
> > +	AD5770R_FILTER_44_4_KOHM,
> > +	AD5770R_FILTER_22_2_KOHM,
> > +	AD5770R_FILTER_11_2_KOHM,
> > +	AD5770R_FILTER_5_6_KOHM,
> > +	AD5770R_FILTER_60_OHM
> > +};
> > +
> > +static int ad5770r_set_output_mode(struct ad5770r_state *st,
> > +				   const struct ad5770r_out_range *out_mode,
> > +				   int channel)
> > +{
> > +	unsigned int regval;
> > +
> > +	regval = AD5770R_RANGE_OUTPUT_SCALING(out_mode->out_scale) |
> > +		 AD5770R_RANGE_MODE(out_mode->out_range_mode);
> > +
> > +	return regmap_write(st->regmap,
> > +			    AD5770R_OUTPUT_RANGE(channel), regval);
> > +}
> > +
> > +static int ad5770r_set_reference(struct ad5770r_state *st)
> > +{
> > +	unsigned int regval = 0;
> 
> It's immediately set to something else, so don't initialize it here.
> 
> > +
> > +	regval = AD5770R_REF_RESISTOR_SEL(st->external_res);
> > +
> > +	if (st->internal_ref) {
> > +		regval |= AD5770R_REF_SEL(AD5770R_INT_1_25_V_OUT_OFF);
> > +	} else {
> > +		switch (st->vref) {
> > +		case AD5770R_LOW_VREF_mV:
> > +			regval |= AD5770R_REF_SEL(AD5770R_EXT_1_25_V);
> > +			break;
> > +		case AD5770R_HIGH_VREF_mV:
> > +			regval |= AD5770R_REF_SEL(AD5770R_EXT_2_5_V);
> > +			break;
> > +		default:
> > +			regval = AD5770R_REF_SEL(AD5770R_INT_1_25_V_OUT_OFF);
> > +			break;
> > +		}
> > +	}
> > +
> > +	return regmap_write(st->regmap, AD5770R_REFERENCE, regval);
> > +}
> > +
> > +static int ad5770r_soft_reset(struct ad5770r_state *st)
> > +{
> > +	return regmap_write(st->regmap, ADI_SPI_IF_CONFIG_A,
> > +			    ADI_SPI_IF_SW_RESET_SEL(1));
> > +}
> > +
> > +static int ad5770r_reset(struct ad5770r_state *st)
> > +{
> > +	/* Perform software reset if no GPIO provided */
> > +	if (!st->gpio_reset)
> > +		return ad5770r_soft_reset(st);
> > +
> > +	gpiod_set_value_cansleep(st->gpio_reset, 0);
> > +	usleep_range(10, 20);
> > +	gpiod_set_value_cansleep(st->gpio_reset, 1);
> > +
> > +	/* data must not be written during reset timeframe */
> > +	usleep_range(100, 200);
> > +
> > +	return 0;
> > +}
> > +
> > +static int ad5770r_get_range(struct ad5770r_state *st,
> > +			     int ch, int *min, int *max)
> > +{
> > +	int i;
> > +	u8 tbl_ch, tbl_mode, out_range;
> > +
> > +	out_range = st->output_mode[ch].out_range_mode;
> > +
> > +	for (i = 0; i < AD5770R_MAX_CH_MODES; i++) {
> > +		tbl_ch = ad5770r_rng_tbl[i].ch;
> > +		tbl_mode = ad5770r_rng_tbl[i].mode;
> > +		if (tbl_ch == ch && tbl_mode == out_range) {
> > +			*min = ad5770r_rng_tbl[i].min;
> > +			*max = ad5770r_rng_tbl[i].max;
> > +			return 0;
> > +		}
> > +	}
> > +
> > +	return -EINVAL;
> > +}
> > +
> > +static int ad5770r_get_filter_freq(struct iio_dev *indio_dev,
> > +				   const struct iio_chan_spec *chan, int *freq)
> > +{
> > +	struct ad5770r_state *st = iio_priv(indio_dev);
> > +	int ret;
> > +	unsigned int regval, i;
> > +
> > +	ret = regmap_read(st->regmap,
> > +			  AD5770R_FILTER_RESISTOR(chan->channel), &regval);
> > +	if (ret < 0)
> > +		return ret;
> > +
> > +	for (i = 0; i < ARRAY_SIZE(ad5770r_filter_reg_vals); i++)
> > +		if (regval == ad5770r_filter_reg_vals[i])
> > +			break;
> > +	if (i == ARRAY_SIZE(ad5770r_filter_reg_vals))
> > +		return -EINVAL;
> > +
> > +	*freq = ad5770r_filter_freqs[i];
> > +
> > +	return IIO_VAL_INT;
> > +}
> > +
> > +static int ad5770r_set_filter_freq(struct iio_dev *indio_dev,
> > +				   const struct iio_chan_spec *chan,
> > +				   unsigned int freq)
> > +{
> > +	struct ad5770r_state *st = iio_priv(indio_dev);
> > +	unsigned int regval, i;
> > +
> > +	for (i = 0; i < ARRAY_SIZE(ad5770r_filter_freqs); i++)
> > +		if (ad5770r_filter_freqs[i] >= freq)
> > +			break;
> > +	if (i == ARRAY_SIZE(ad5770r_filter_freqs))
> > +		return -EINVAL;
> > +
> > +	regval = ad5770r_filter_reg_vals[i];
> > +
> > +	return regmap_write(st->regmap, AD5770R_FILTER_RESISTOR(chan->channel),
> > +			    regval);
> > +}
> > +
> > +static int ad5770r_read_raw(struct iio_dev *indio_dev,
> > +			    struct iio_chan_spec const *chan,
> > +			    int *val, int *val2, long info)
> > +{
> > +	struct ad5770r_state *st = iio_priv(indio_dev);
> > +	int max, min, ret;
> > +	__le16 buf16;
> > +
> > +	switch (info) {
> > +	case IIO_CHAN_INFO_RAW:
> > +		ret = regmap_bulk_read(st->regmap,
> > +				       chan->address,
> > +				       &buf16, 2);
> > +		if (ret)
> > +			return 0;
> > +		*val = le16_to_cpu(buf16) >> 2;
> > +		return IIO_VAL_INT;
> > +	case IIO_CHAN_INFO_SCALE:
> > +		ret = ad5770r_get_range(st, chan->channel, &min, &max);
> > +		if (ret < 0)
> > +			return ret;
> > +		*val = max - min;
> > +		/* There is no sign bit. (negative current is mapped from 0)
> > +		 * (sourced/sinked) current = raw * scale + offset
> > +		 * where offset in case of CH0 can be negative.
> > +		 */
> > +		*val2 = 14;
> > +		return IIO_VAL_FRACTIONAL_LOG2;
> > +	case IIO_CHAN_INFO_LOW_PASS_FILTER_3DB_FREQUENCY:
> > +		return ad5770r_get_filter_freq(indio_dev, chan, val);
> > +	case IIO_CHAN_INFO_OFFSET:
> > +		ret = ad5770r_get_range(st, chan->channel, &min, &max);
> > +		if (ret < 0)
> > +			return ret;
> > +		*val = min;
> > +		return IIO_VAL_INT;
> > +	default:
> > +		return -EINVAL;
> > +	}
> > +}
> > +
> > +static int ad5770r_write_raw(struct iio_dev *indio_dev,
> > +			     struct iio_chan_spec const *chan,
> > +			     int val, int val2, long info)
> > +{
> > +	struct ad5770r_state *st = iio_priv(indio_dev);
> > +	u8 buf[2];
> > +
> > +	switch (info) {
> > +	case IIO_CHAN_INFO_RAW:
> > +		buf[0] = ((u16)val >> 6);
> > +		buf[1] = (val & 0x3F) << 2;
> > +		return regmap_bulk_write(st->regmap, chan->address,
> > +					 buf, ARRAY_SIZE(buf));
> > +	case IIO_CHAN_INFO_LOW_PASS_FILTER_3DB_FREQUENCY:
> > +		return ad5770r_set_filter_freq(indio_dev, chan, val);
> > +	default:
> > +		return -EINVAL;
> > +	}
> > +}
> > +
> > +static int ad5770r_read_freq_avail(struct iio_dev *indio_dev,
> > +				   struct iio_chan_spec const *chan,
> > +				   const int **vals, int *type, int *length,
> > +				   long mask)
> > +{
> > +	switch (mask) {
> > +	case IIO_CHAN_INFO_LOW_PASS_FILTER_3DB_FREQUENCY:
> > +		*type = IIO_VAL_INT;
> > +		*vals = ad5770r_filter_freqs;
> > +		*length = ARRAY_SIZE(ad5770r_filter_freqs);
> > +		return IIO_AVAIL_LIST;
> > +	}
> > +
> > +	return -EINVAL;
> > +}
> > +
> > +static int ad5770r_reg_access(struct iio_dev *indio_dev,
> > +			      unsigned int reg,
> > +			      unsigned int writeval,
> > +			      unsigned int *readval)
> > +{
> > +	struct ad5770r_state *st = iio_priv(indio_dev);
> > +
> > +	if (readval)
> > +		return regmap_read(st->regmap, reg, readval);
> > +	else
> > +		return regmap_write(st->regmap, reg, writeval);
> > +}
> > +
> > +static const struct iio_info ad5770r_info = {
> > +	.read_raw = ad5770r_read_raw,
> > +	.write_raw = ad5770r_write_raw,
> > +	.read_avail = ad5770r_read_freq_avail,
> > +	.debugfs_reg_access = &ad5770r_reg_access,
> > +};
> > +
> > +static int ad5770r_store_output_range(struct ad5770r_state *st,
> > +				      int min, int max, int index)
> > +{
> > +	int i;
> > +
> > +	for (i = 0; i < AD5770R_MAX_CH_MODES; i++) {
> > +		if (ad5770r_rng_tbl[i].ch != index)
> > +			continue;
> > +		if (ad5770r_rng_tbl[i].min != min ||
> > +		    ad5770r_rng_tbl[i].max != max)
> > +			continue;
> > +		st->output_mode[index].out_range_mode = ad5770r_rng_tbl[i].mode;
> > +
> > +		return 0;
> > +	}
> > +
> > +	return -EINVAL;
> > +}
> > +
> > +static ssize_t ad5770r_read_dac_powerdown(struct iio_dev *indio_dev,
> > +					  uintptr_t private,
> > +					  const struct iio_chan_spec *chan,
> > +					  char *buf)
> > +{
> > +	struct ad5770r_state *st = iio_priv(indio_dev);
> > +
> > +	return sprintf(buf, "%d\n", st->ch_pwr_down[chan->channel]);
> > +}
> > +
> > +static ssize_t ad5770r_write_dac_powerdown(struct iio_dev *indio_dev,
> > +					   uintptr_t private,
> > +					   const struct iio_chan_spec *chan,
> > +					   const char *buf, size_t len)
> > +{
> > +	struct ad5770r_state *st = iio_priv(indio_dev);
> > +	unsigned int regval;
> > +	unsigned int mask;
> > +	bool readin;
> > +	int ret;
> > +
> > +	ret = kstrtobool(buf, &readin);
> > +	if (ret)
> > +		return ret;
> > +
> > +	readin = !readin;
> > +
> > +	regval = AD5770R_CFG_SHUTDOWN_B(readin, chan->channel);
> > +	if (chan->channel == 0 &&
> > +	    st->output_mode[0].out_range_mode > AD5770R_CH0_0_300) {
> > +		regval |= AD5770R_CFG_CH0_SINK_EN(readin);
> > +		mask = BIT(chan->channel) + BIT(7);
> > +	} else {
> > +		mask = BIT(chan->channel);
> > +	}
> > +	ret = regmap_update_bits(st->regmap, AD5770R_CHANNEL_CONFIG, mask,
> > +				 regval);
> > +	if (ret)
> > +		return ret;
> > +
> > +	regval = AD5770R_CH_SET(readin, chan->channel);
> > +	ret = regmap_update_bits(st->regmap, AD5770R_CH_ENABLE,
> > +				 BIT(chan->channel), regval);
> > +	if (ret)
> > +		return ret;
> > +
> > +	st->ch_pwr_down[chan->channel] = !readin;
> > +
> > +	return len;
> > +}
> > +
> > +static const struct iio_chan_spec_ext_info ad5770r_ext_info[] = {
> > +	{
> > +		.name = "powerdown",
> > +		.read = ad5770r_read_dac_powerdown,
> > +		.write = ad5770r_write_dac_powerdown,
> > +		.shared = IIO_SEPARATE,
> > +	},
> > +	{ }
> > +};
> > +
> > +#define AD5770R_IDAC_CHANNEL(index, reg) {				\
> > +	.type = IIO_CURRENT,						\
> > +	.address = reg,							\
> > +	.indexed = 1,							\
> > +	.channel = index,						\
> > +	.output = 1,							\
> > +	.info_mask_separate = BIT(IIO_CHAN_INFO_RAW) |			\
> > +		BIT(IIO_CHAN_INFO_SCALE) |				\
> > +		BIT(IIO_CHAN_INFO_OFFSET) |				\
> > +		BIT(IIO_CHAN_INFO_LOW_PASS_FILTER_3DB_FREQUENCY),	\
> > +	.info_mask_shared_by_type_available =				\
> > +		BIT(IIO_CHAN_INFO_LOW_PASS_FILTER_3DB_FREQUENCY),	\
> > +	.ext_info = ad5770r_ext_info,					\
> > +}
> > +
> > +static const struct iio_chan_spec ad5770r_channels[] = {
> > +	AD5770R_IDAC_CHANNEL(0, AD5770R_DAC_MSB(0)),
> > +	AD5770R_IDAC_CHANNEL(1, AD5770R_DAC_MSB(1)),
> > +	AD5770R_IDAC_CHANNEL(2, AD5770R_DAC_MSB(2)),
> > +	AD5770R_IDAC_CHANNEL(3, AD5770R_DAC_MSB(3)),
> > +	AD5770R_IDAC_CHANNEL(4, AD5770R_DAC_MSB(4)),
> > +	AD5770R_IDAC_CHANNEL(5, AD5770R_DAC_MSB(5)),
> > +};
> > +
> > +static int ad5770r_channel_config(struct ad5770r_state *st)
> > +{
> > +	int ret, tmp[2], min, max;
> > +	unsigned int num;
> > +	struct fwnode_handle *child;
> > +
> > +	num = device_get_child_node_count(&st->spi->dev);
> > +	if (num != AD5770R_MAX_CHANNELS)
> > +		return -EINVAL;
> > +
> > +	device_for_each_child_node(&st->spi->dev, child) {
> > +		ret = fwnode_property_read_u32(child, "num", &num);
> > +		if (ret)
> > +			return ret;
> > +		if (num > AD5770R_MAX_CHANNELS)
> > +			return -EINVAL;
> > +
> > +		ret = fwnode_property_read_u32_array(child,
> > +						     "adi,range-microamp",
> > +						     tmp, 2);
> > +		if (ret)
> > +			return ret;
> > +
> > +		min = tmp[0] / 1000;
> > +		max = tmp[1] / 1000;
> > +		ret = ad5770r_store_output_range(st, min, max, num);
> > +		if (ret)
> > +			return ret;
> > +	}
> > +
> > +	return ret;
> > +}
> > +
> > +static int ad5770r_init(struct ad5770r_state *st)
> > +{
> > +	int ret, i;
> > +
> > +	st->gpio_reset = devm_gpiod_get_optional(&st->spi->dev, "reset",
> > +						 GPIOD_OUT_HIGH);
> > +	if (IS_ERR(st->gpio_reset))
> > +		return PTR_ERR(st->gpio_reset);
> > +
> > +	/* Perform a reset */
> > +	ret = ad5770r_reset(st);
> > +	if (ret)
> > +		return ret;
> > +
> > +	/* Set output range */
> > +	ret = ad5770r_channel_config(st);
> > +	if (ret)
> > +		return ret;
> > +
> > +	for (i = 0; i < AD5770R_MAX_CHANNELS; i++) {
> > +		ret = ad5770r_set_output_mode(st,  &st->output_mode[i], i);
> > +		if (ret)
> > +			return ret;
> > +	}
> > +
> > +	st->external_res = fwnode_property_read_bool(st->spi->dev.fwnode,
> > +						     "adi,external-resistor");
> > +
> > +	ret = ad5770r_set_reference(st);
> > +	if (ret)
> > +		return ret;
> > +
> > +	/* Set outputs off */
> > +	ret = regmap_write(st->regmap, AD5770R_CHANNEL_CONFIG, 0x00);
> > +	if (ret)
> > +		return ret;
> > +
> > +	ret = regmap_write(st->regmap, AD5770R_CH_ENABLE, 0x00);
> > +	if (ret)
> > +		return ret;
> > +
> > +	for (i = 0; i < AD5770R_MAX_CHANNELS; i++)
> > +		st->ch_pwr_down[i] = true;
> > +
> > +	return ret;
> > +}
> > +
> > +static void ad5770r_disable_regulator(void *data)
> > +{
> > +	struct spi_device *spi = (struct spi_device *)data;
> > +	struct iio_dev *indio_dev = spi_get_drvdata(spi);
> > +	struct ad5770r_state *st = iio_priv(indio_dev);
> > +
> > +	if (st->vref_reg)
> > +		regulator_disable(st->vref_reg);
> > +}
> > +
> > +static int ad5770r_probe(struct spi_device *spi)
> > +{
> > +	struct ad5770r_state *st;
> > +	struct iio_dev *indio_dev;
> > +	struct regmap *regmap;
> > +	int ret;
> > +
> > +	indio_dev = devm_iio_device_alloc(&spi->dev, sizeof(*st));
> > +	if (!indio_dev)
> > +		return -ENOMEM;
> > +
> > +	st = iio_priv(indio_dev);
> > +	spi_set_drvdata(spi, indio_dev);
> > +
> > +	st->spi = spi;
> > +
> > +	regmap = devm_regmap_init_spi(spi, &ad5770r_spi_regmap_config);
> > +	if (IS_ERR(regmap)) {
> > +		dev_err(&spi->dev, "Error initializing spi regmap: %ld\n",
> > +			PTR_ERR(regmap));
> > +		return PTR_ERR(regmap);
> > +	}
> > +	st->regmap = regmap;
> > +
> > +	ret = devm_add_action_or_reset(&spi->dev, ad5770r_disable_regulator,
> > +				       st);
> > +	if (ret < 0)
> > +		return ret;
> > +
> > +	st->vref_reg = devm_regulator_get_optional(&spi->dev, "vref");
> > +	if (!IS_ERR(st->vref_reg)) {
> > +		ret = regulator_enable(st->vref_reg);
> > +		if (ret) {
> > +			dev_err(&spi->dev,
> > +				"Failed to enable vref regulators: %d\n", ret);
> > +			return ret;
> > +		}
> > +
> > +		ret = regulator_get_voltage(st->vref_reg);
> > +		if (ret < 0)
> > +			return ret;
> > +
> > +		st->vref = ret / 1000;
> > +	} else {
> > +		if (PTR_ERR(st->vref_reg) == -ENODEV) {
> > +			st->vref = AD5770R_LOW_VREF_mV;
> > +			st->internal_ref = true;
> > +		} else {
> > +			return PTR_ERR(st->vref_reg);
> > +		}
> > +	}
> > +
> > +	indio_dev->dev.parent = &spi->dev;
> > +	indio_dev->name = spi_get_device_id(spi)->name;
> > +	indio_dev->info = &ad5770r_info;
> > +	indio_dev->modes = INDIO_DIRECT_MODE;
> > +	indio_dev->channels = ad5770r_channels;
> > +	indio_dev->num_channels = ARRAY_SIZE(ad5770r_channels);
> > +
> > +	ret = ad5770r_init(st);
> > +	if (ret < 0) {
> > +		dev_err(&spi->dev, "AD5770R init failed\n");
> > +		return ret;
> > +	}
> > +
> > +	return devm_iio_device_register(&st->spi->dev, indio_dev);
> > +}
> > +
> > +static const struct of_device_id ad5770r_of_id[] = {
> > +	{ .compatible = "adi,ad5770r", },
> > +};
> > +MODULE_DEVICE_TABLE(of, ad5770r_of_id);
> > +
> > +static const struct spi_device_id ad5770r_id[] = {
> > +	{ "ad5770r", 0 },
> > +	{}
> > +};
> > +MODULE_DEVICE_TABLE(spi, ad5770r_id);
> > +
> > +static struct spi_driver ad5770r_driver = {
> > +	.driver = {
> > +		.name = KBUILD_MODNAME,
> > +		.of_match_table = of_match_ptr(ad5770r_of_id),
> > +	},
> > +	.probe = ad5770r_probe,
> > +	.id_table = ad5770r_id,
> > +};
> > +
> > +module_spi_driver(ad5770r_driver);
> > +
> > +MODULE_AUTHOR("Mircea Caprioru <mircea.caprioru@analog.com>");
> > +MODULE_DESCRIPTION("Analog Devices AD5770R IDAC");
> > +MODULE_LICENSE("GPL v2");
> > diff --git a/include/linux/iio/common/adi_spi_regs.h
> > b/include/linux/iio/common/adi_spi_regs.h
> > new file mode 100644
> > index 000000000000..1fc3bfc7845d
> > --- /dev/null
> > +++ b/include/linux/iio/common/adi_spi_regs.h
> > @@ -0,0 +1,44 @@
> > +/* SPDX-License-Identifier: GPL-2.0 */
> > +/*
> > + * Common Analog Devices SPI configuration registers
> > + *
> > + * Copyright 2019 Analog Devices Inc.
> > + */
> > +
> > +#ifndef _ADI_SPI_REGS_H_
> > +#define _ADI_SPI_REGS_H_
> > +
> > +#include <linux/bits.h>
> > +
> > +#define ADI_SPI_IF_CONFIG_A		0x00
> > +#define ADI_SPI_IF_CONFIG_B		0x01
> > +#define ADI_SPI_IF_DEVICE_CONFIG	0x02
> > +#define ADI_SPI_IF_CHIP_TYPE		0x03
> > +#define ADI_SPI_IF_PRODUCT_ID_L		0x04
> > +#define ADI_SPI_IF_PRODUCT_ID_H		0x05
> > +#define ADI_SPI_IF_CHIP_GRADE		0x06
> > +#define ADI_SPI_IF_SCRACTH_PAD		0x0A
> > +#define ADI_SPI_IF_SPI_REVISION		0x0B
> > +#define ADI_SPI_IF_SPI_VENDOR_L		0x0C
> > +#define ADI_SPI_IF_SPI_VENDOR_H		0x0D
> > +#define ADI_SPI_IF_SPI_STREAM_MODE	0x0E
> > +#define ADI_SPI_IF_CONFIG_C		0x10
> > +#define ADI_SPI_IF_STATUS_A		0x11
> > +
> > +/* ADI_SPI_IF_CONFIG_A */
> > +#define ADI_SPI_IF_SW_RESET_MSK		(BIT(0) | BIT(7))
> > +#define ADI_SPI_IF_SW_RESET_SEL(x)	((x) & ADI_SPI_IF_SW_RESET_MSK)
> > +#define ADI_SPI_IF_ADDR_ASC_MSK		(BIT(2) | BIT(5))
> > +#define ADI_SPI_IF_ADDR_ASC_SEL(x)	(((x) << 2) & ADI_SPI_IF_ADDR_ASC_MSK)
> > +
> > +/* ADI_SPI_IF_CONFIG_B */
> > +#define ADI_SPI_IF_SINGLE_INS_MSK	BIT(7)
> > +#define ADI_SPI_IF_SINGLE_INS_SEL(x)	FIELD_PREP(ADI_SPI_IF_SINGLE_INS
> > _MSK, x)
> > +#define ADI_SPI_IF_SHORT_INS_MSK	BIT(7)
> > +#define ADI_SPI_IF_SHORT_INS_SEL(x)	FIELD_PREP(ADI_SPI_IF_SINGLE_INS
> > _MSK, x)
> > +
> > +/* ADI_SPI_IF_CONFIG_C */
> > +#define ADI_SPI_IF_STRICT_REG_MSK	BIT(5)
> > +#define ADI_SPI_IF_STRICT_REG_GET(x)	FIELD_GET(ADI_SPI_IF_STRICT_REG_
> > MSK, x)
> > +
> > +#endif /* _ADI_SPI_REGS_H_ */

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 1/2] iio: dac: ad5770r: Add AD5770R support
  2020-02-11 13:51     ` Ardelean, Alexandru
@ 2020-02-14 12:45       ` Ardelean, Alexandru
  0 siblings, 0 replies; 9+ messages in thread
From: Ardelean, Alexandru @ 2020-02-14 12:45 UTC (permalink / raw)
  To: jic23, Tachici, Alexandru; +Cc: linux-kernel, linux-iio, Caprioru, Mircea

On Tue, 2020-02-11 at 13:51 +0000, Ardelean, Alexandru wrote:
> [External]
> 
> On Sat, 2020-02-08 at 14:33 +0000, Jonathan Cameron wrote:
> > On Thu, 6 Feb 2020 11:23:12 +0200
> > Alexandru Tachici <alexandru.tachici@analog.com> wrote:
> > 
> > > The AD5770R is a 6-channel, 14-bit resolution, low noise, programmable
> > > current output digital-to-analog converter (DAC) for photonics control
> > > applications.
> > > 
> > > It contains five 14-bit resolution current sourcing DAC channels and one
> > > 14-bit resolution current sourcing/sinking DAC channel.
> > 
> > I'd like to see considerably more discussion of the common header.
> > What devices do these shared registers apply to?
> > Current header naming kind of implies all ADI devices with an
> > spi interface which I find very unlikely.
> 
> Right.
> My bad here, since I don't think I provided AlexandruT the full details on
> this.
> 
> We'll see about a V3 with some reference for these.

Update for this: there is some sort of unofficial/semi-official unification for
some SPI regmaps.
I guess, for now we'll drop any 'adi_spi_regs.h' stuff.
But the good part is that this started some sort of internal discussion about
this.
Let's see where this goes.

Thanks again
Alex A

> 
> Thanks
> Alex
> 
> > Otherwise one minor comment inline.
> > 
> > Thanks,
> > 
> > Jonathan
> > 
> > 
> > > Signed-off-by: Mircea Caprioru <mircea.caprioru@analog.com>
> > > Signed-off-by: Alexandru Tachici <alexandru.tachici@analog.com>
> > > ---
> > >  drivers/iio/dac/Kconfig                 |  10 +
> > >  drivers/iio/dac/Makefile                |   1 +
> > >  drivers/iio/dac/ad5770r.c               | 662 ++++++++++++++++++++++++
> > >  include/linux/iio/common/adi_spi_regs.h |  44 ++
> > >  4 files changed, 717 insertions(+)
> > >  create mode 100644 drivers/iio/dac/ad5770r.c
> > >  create mode 100644 include/linux/iio/common/adi_spi_regs.h
> > > 
> > > diff --git a/drivers/iio/dac/Kconfig b/drivers/iio/dac/Kconfig
> > > index 979070196da9..c74f2151ee42 100644
> > > --- a/drivers/iio/dac/Kconfig
> > > +++ b/drivers/iio/dac/Kconfig
> > > @@ -208,6 +208,16 @@ config AD5764
> > >  	  To compile this driver as a module, choose M here: the
> > >  	  module will be called ad5764.
> > >  
> > > +config AD5770R
> > > +	tristate "Analog Devices AD5770R IDAC driver"
> > > +	depends on SPI_MASTER
> > > +	help
> > > +	  Say yes here to build support for Analog Devices AD5770R Digital to
> > > +	  Analog Converter.
> > > +
> > > +	  To compile this driver as a module, choose M here: the
> > > +	  module will be called ad5770r.
> > > +
> > >  config AD5791
> > >  	tristate "Analog Devices AD5760/AD5780/AD5781/AD5790/AD5791 DAC SPI
> > > driver"
> > >  	depends on SPI
> > > diff --git a/drivers/iio/dac/Makefile b/drivers/iio/dac/Makefile
> > > index 1369fa1d2f0e..2fc481167724 100644
> > > --- a/drivers/iio/dac/Makefile
> > > +++ b/drivers/iio/dac/Makefile
> > > @@ -19,6 +19,7 @@ obj-$(CONFIG_AD5755) += ad5755.o
> > >  obj-$(CONFIG_AD5755) += ad5758.o
> > >  obj-$(CONFIG_AD5761) += ad5761.o
> > >  obj-$(CONFIG_AD5764) += ad5764.o
> > > +obj-$(CONFIG_AD5770R) += ad5770r.o
> > >  obj-$(CONFIG_AD5791) += ad5791.o
> > >  obj-$(CONFIG_AD5686) += ad5686.o
> > >  obj-$(CONFIG_AD5686_SPI) += ad5686-spi.o
> > > diff --git a/drivers/iio/dac/ad5770r.c b/drivers/iio/dac/ad5770r.c
> > > new file mode 100644
> > > index 000000000000..a8b2c1850ffd
> > > --- /dev/null
> > > +++ b/drivers/iio/dac/ad5770r.c
> > > @@ -0,0 +1,662 @@
> > > +// SPDX-License-Identifier: GPL-2.0+
> > > +/*
> > > + * AD5770R Digital to analog converters driver
> > > + *
> > > + * Copyright 2018 Analog Devices Inc.
> > > + */
> > > +
> > > +#include <linux/delay.h>
> > > +#include <linux/device.h>
> > > +#include <linux/gpio/consumer.h>
> > > +#include <linux/iio/common/adi_spi_regs.h>
> > > +#include <linux/iio/iio.h>
> > > +#include <linux/iio/sysfs.h>
> > > +#include <linux/kernel.h>
> > > +#include <linux/module.h>
> > > +#include <linux/property.h>
> > > +#include <linux/regmap.h>
> > > +#include <linux/regulator/consumer.h>
> > > +#include <linux/spi/spi.h>
> > > +
> > > +/* AD5770R configuration registers */
> > > +#define AD5770R_CHANNEL_CONFIG		0x14
> > > +#define AD5770R_OUTPUT_RANGE(ch)	(0x15 + (ch))
> > > +#define AD5770R_FILTER_RESISTOR(ch)	(0x1D + (ch))
> > > +#define AD5770R_REFERENCE		0x1B
> > > +#define AD5770R_DAC_LSB(ch)		(0x26 + 2 * (ch))
> > > +#define AD5770R_DAC_MSB(ch)		(0x27 + 2 * (ch))
> > > +#define AD5770R_CH_SELECT		0x34
> > > +#define AD5770R_CH_ENABLE		0x44
> > > +
> > > +/* AD5770R_CHANNEL_CONFIG */
> > > +#define AD5770R_CFG_CH0_SINK_EN(x)		(((x) & 0x1) << 7)
> > > +#define AD5770R_CFG_SHUTDOWN_B(x, ch)		(((x) & 0x1) << (ch))
> > > +
> > > +/* AD5770R_OUTPUT_RANGE */
> > > +#define AD5770R_RANGE_OUTPUT_SCALING(x)		(((x) & 0x3F) << 2)
> > > +#define AD5770R_RANGE_MODE(x)			((x) & 0x03)
> > > +
> > > +/* AD5770R_REFERENCE */
> > > +#define AD5770R_REF_RESISTOR_SEL(x)		(((x) & 0x1) << 2)
> > > +#define AD5770R_REF_SEL(x)			(((x) & 0x3) << 0)
> > > +
> > > +/* AD5770R_CH_ENABLE */
> > > +#define AD5770R_CH_SET(x, channel)		(((x) & 0x1) <<
> > > (channel))
> > > +
> > > +#define AD5770R_MAX_CHANNELS	6
> > > +#define AD5770R_MAX_CH_MODES	14
> > > +#define AD5770R_LOW_VREF_mV	1250
> > > +#define AD5770R_HIGH_VREF_mV	2500
> > > +
> > > +enum ad5770r_ch0_modes {
> > > +	AD5770R_CH0_0_300 = 0,
> > > +	AD5770R_CH0_NEG_60_0,
> > > +	AD5770R_CH0_NEG_60_300
> > > +};
> > > +
> > > +enum ad5770r_ch1_modes {
> > > +	AD5770R_CH1_0_140_LOW_HEAD = 1,
> > > +	AD5770R_CH1_0_140_LOW_NOISE,
> > > +	AD5770R_CH1_0_250
> > > +};
> > > +
> > > +enum ad5770r_ch2_5_modes {
> > > +	AD5770R_CH_LOW_RANGE = 0,
> > > +	AD5770R_CH_HIGH_RANGE
> > > +};
> > > +
> > > +enum ad5770r_ref_v {
> > > +	AD5770R_EXT_2_5_V = 0,
> > > +	AD5770R_INT_1_25_V_OUT_ON,
> > > +	AD5770R_EXT_1_25_V,
> > > +	AD5770R_INT_1_25_V_OUT_OFF
> > > +};
> > > +
> > > +enum ad5770r_output_filter_resistor {
> > > +	AD5770R_FILTER_60_OHM = 0x0,
> > > +	AD5770R_FILTER_5_6_KOHM = 0x5,
> > > +	AD5770R_FILTER_11_2_KOHM,
> > > +	AD5770R_FILTER_22_2_KOHM,
> > > +	AD5770R_FILTER_44_4_KOHM,
> > > +	AD5770R_FILTER_104_KOHM,
> > > +};
> > > +
> > > +struct ad5770r_out_range {
> > > +	u8	out_scale;
> > > +	u8	out_range_mode;
> > > +};
> > > +
> > > +/**
> > > + * struct ad5770R_state - driver instance specific data
> > > + * @spi:		spi_device
> > > + * @regmap:		regmap
> > > + * @vref_reg:		fixed regulator for reference configuration
> > > + * @gpio_reset:		gpio descriptor
> > > + * @output_mode:		array contains channels output ranges
> > > + * @vref:		reference value
> > > + * @ch_pwr_down:		powerdown flags
> > > + * @internal_ref:	internal reference flag
> > > + * @external_res:	external 2.5k resistor flag
> > > + */
> > > +struct ad5770r_state {
> > > +	struct spi_device		*spi;
> > > +	struct regmap			*regmap;
> > > +	struct regulator		*vref_reg;
> > > +	struct gpio_desc		*gpio_reset;
> > > +	struct ad5770r_out_range	output_mode[AD5770R_MAX_CHANNELS];
> > > +	int				vref;
> > > +	bool				ch_pwr_down[AD5770R_MAX_CHANNELS];
> > > +	bool				internal_ref;
> > > +	bool				external_res;
> > > +};
> > > +
> > > +static const struct regmap_config ad5770r_spi_regmap_config = {
> > > +	.reg_bits = 8,
> > > +	.val_bits = 8,
> > > +	.read_flag_mask = BIT(7),
> > > +};
> > > +
> > > +struct ad5770r_output_modes {
> > > +	unsigned int ch;
> > > +	u8 mode;
> > > +	int min;
> > > +	int max;
> > > +};
> > > +
> > > +static struct ad5770r_output_modes ad5770r_rng_tbl[] = {
> > > +	{ 0, AD5770R_CH0_0_300, 0, 300 },
> > > +	{ 0, AD5770R_CH0_NEG_60_0, -60, 0 },
> > > +	{ 0, AD5770R_CH0_NEG_60_300, -60, 300 },
> > > +	{ 1, AD5770R_CH1_0_140_LOW_HEAD, 0, 140 },
> > > +	{ 1, AD5770R_CH1_0_140_LOW_NOISE, 0, 140 },
> > > +	{ 1, AD5770R_CH1_0_250, 0, 250 },
> > > +	{ 2, AD5770R_CH_LOW_RANGE, 0, 55 },
> > > +	{ 2, AD5770R_CH_HIGH_RANGE, 0, 150 },
> > > +	{ 3, AD5770R_CH_LOW_RANGE, 0, 45 },
> > > +	{ 3, AD5770R_CH_HIGH_RANGE, 0, 100 },
> > > +	{ 4, AD5770R_CH_LOW_RANGE, 0, 45 },
> > > +	{ 4, AD5770R_CH_HIGH_RANGE, 0, 100 },
> > > +	{ 5, AD5770R_CH_LOW_RANGE, 0, 45 },
> > > +	{ 5, AD5770R_CH_HIGH_RANGE, 0, 100 },
> > > +};
> > > +
> > > +static const unsigned int ad5770r_filter_freqs[] = {
> > > +	153, 357, 715, 1400, 2800, 262000,
> > > +};
> > > +
> > > +static const unsigned int ad5770r_filter_reg_vals[] = {
> > > +	AD5770R_FILTER_104_KOHM,
> > > +	AD5770R_FILTER_44_4_KOHM,
> > > +	AD5770R_FILTER_22_2_KOHM,
> > > +	AD5770R_FILTER_11_2_KOHM,
> > > +	AD5770R_FILTER_5_6_KOHM,
> > > +	AD5770R_FILTER_60_OHM
> > > +};
> > > +
> > > +static int ad5770r_set_output_mode(struct ad5770r_state *st,
> > > +				   const struct ad5770r_out_range *out_mode,
> > > +				   int channel)
> > > +{
> > > +	unsigned int regval;
> > > +
> > > +	regval = AD5770R_RANGE_OUTPUT_SCALING(out_mode->out_scale) |
> > > +		 AD5770R_RANGE_MODE(out_mode->out_range_mode);
> > > +
> > > +	return regmap_write(st->regmap,
> > > +			    AD5770R_OUTPUT_RANGE(channel), regval);
> > > +}
> > > +
> > > +static int ad5770r_set_reference(struct ad5770r_state *st)
> > > +{
> > > +	unsigned int regval = 0;
> > 
> > It's immediately set to something else, so don't initialize it here.
> > 
> > > +
> > > +	regval = AD5770R_REF_RESISTOR_SEL(st->external_res);
> > > +
> > > +	if (st->internal_ref) {
> > > +		regval |= AD5770R_REF_SEL(AD5770R_INT_1_25_V_OUT_OFF);
> > > +	} else {
> > > +		switch (st->vref) {
> > > +		case AD5770R_LOW_VREF_mV:
> > > +			regval |= AD5770R_REF_SEL(AD5770R_EXT_1_25_V);
> > > +			break;
> > > +		case AD5770R_HIGH_VREF_mV:
> > > +			regval |= AD5770R_REF_SEL(AD5770R_EXT_2_5_V);
> > > +			break;
> > > +		default:
> > > +			regval = AD5770R_REF_SEL(AD5770R_INT_1_25_V_OUT_OFF);
> > > +			break;
> > > +		}
> > > +	}
> > > +
> > > +	return regmap_write(st->regmap, AD5770R_REFERENCE, regval);
> > > +}
> > > +
> > > +static int ad5770r_soft_reset(struct ad5770r_state *st)
> > > +{
> > > +	return regmap_write(st->regmap, ADI_SPI_IF_CONFIG_A,
> > > +			    ADI_SPI_IF_SW_RESET_SEL(1));
> > > +}
> > > +
> > > +static int ad5770r_reset(struct ad5770r_state *st)
> > > +{
> > > +	/* Perform software reset if no GPIO provided */
> > > +	if (!st->gpio_reset)
> > > +		return ad5770r_soft_reset(st);
> > > +
> > > +	gpiod_set_value_cansleep(st->gpio_reset, 0);
> > > +	usleep_range(10, 20);
> > > +	gpiod_set_value_cansleep(st->gpio_reset, 1);
> > > +
> > > +	/* data must not be written during reset timeframe */
> > > +	usleep_range(100, 200);
> > > +
> > > +	return 0;
> > > +}
> > > +
> > > +static int ad5770r_get_range(struct ad5770r_state *st,
> > > +			     int ch, int *min, int *max)
> > > +{
> > > +	int i;
> > > +	u8 tbl_ch, tbl_mode, out_range;
> > > +
> > > +	out_range = st->output_mode[ch].out_range_mode;
> > > +
> > > +	for (i = 0; i < AD5770R_MAX_CH_MODES; i++) {
> > > +		tbl_ch = ad5770r_rng_tbl[i].ch;
> > > +		tbl_mode = ad5770r_rng_tbl[i].mode;
> > > +		if (tbl_ch == ch && tbl_mode == out_range) {
> > > +			*min = ad5770r_rng_tbl[i].min;
> > > +			*max = ad5770r_rng_tbl[i].max;
> > > +			return 0;
> > > +		}
> > > +	}
> > > +
> > > +	return -EINVAL;
> > > +}
> > > +
> > > +static int ad5770r_get_filter_freq(struct iio_dev *indio_dev,
> > > +				   const struct iio_chan_spec *chan, int *freq)
> > > +{
> > > +	struct ad5770r_state *st = iio_priv(indio_dev);
> > > +	int ret;
> > > +	unsigned int regval, i;
> > > +
> > > +	ret = regmap_read(st->regmap,
> > > +			  AD5770R_FILTER_RESISTOR(chan->channel), &regval);
> > > +	if (ret < 0)
> > > +		return ret;
> > > +
> > > +	for (i = 0; i < ARRAY_SIZE(ad5770r_filter_reg_vals); i++)
> > > +		if (regval == ad5770r_filter_reg_vals[i])
> > > +			break;
> > > +	if (i == ARRAY_SIZE(ad5770r_filter_reg_vals))
> > > +		return -EINVAL;
> > > +
> > > +	*freq = ad5770r_filter_freqs[i];
> > > +
> > > +	return IIO_VAL_INT;
> > > +}
> > > +
> > > +static int ad5770r_set_filter_freq(struct iio_dev *indio_dev,
> > > +				   const struct iio_chan_spec *chan,
> > > +				   unsigned int freq)
> > > +{
> > > +	struct ad5770r_state *st = iio_priv(indio_dev);
> > > +	unsigned int regval, i;
> > > +
> > > +	for (i = 0; i < ARRAY_SIZE(ad5770r_filter_freqs); i++)
> > > +		if (ad5770r_filter_freqs[i] >= freq)
> > > +			break;
> > > +	if (i == ARRAY_SIZE(ad5770r_filter_freqs))
> > > +		return -EINVAL;
> > > +
> > > +	regval = ad5770r_filter_reg_vals[i];
> > > +
> > > +	return regmap_write(st->regmap, AD5770R_FILTER_RESISTOR(chan->channel),
> > > +			    regval);
> > > +}
> > > +
> > > +static int ad5770r_read_raw(struct iio_dev *indio_dev,
> > > +			    struct iio_chan_spec const *chan,
> > > +			    int *val, int *val2, long info)
> > > +{
> > > +	struct ad5770r_state *st = iio_priv(indio_dev);
> > > +	int max, min, ret;
> > > +	__le16 buf16;
> > > +
> > > +	switch (info) {
> > > +	case IIO_CHAN_INFO_RAW:
> > > +		ret = regmap_bulk_read(st->regmap,
> > > +				       chan->address,
> > > +				       &buf16, 2);
> > > +		if (ret)
> > > +			return 0;
> > > +		*val = le16_to_cpu(buf16) >> 2;
> > > +		return IIO_VAL_INT;
> > > +	case IIO_CHAN_INFO_SCALE:
> > > +		ret = ad5770r_get_range(st, chan->channel, &min, &max);
> > > +		if (ret < 0)
> > > +			return ret;
> > > +		*val = max - min;
> > > +		/* There is no sign bit. (negative current is mapped from 0)
> > > +		 * (sourced/sinked) current = raw * scale + offset
> > > +		 * where offset in case of CH0 can be negative.
> > > +		 */
> > > +		*val2 = 14;
> > > +		return IIO_VAL_FRACTIONAL_LOG2;
> > > +	case IIO_CHAN_INFO_LOW_PASS_FILTER_3DB_FREQUENCY:
> > > +		return ad5770r_get_filter_freq(indio_dev, chan, val);
> > > +	case IIO_CHAN_INFO_OFFSET:
> > > +		ret = ad5770r_get_range(st, chan->channel, &min, &max);
> > > +		if (ret < 0)
> > > +			return ret;
> > > +		*val = min;
> > > +		return IIO_VAL_INT;
> > > +	default:
> > > +		return -EINVAL;
> > > +	}
> > > +}
> > > +
> > > +static int ad5770r_write_raw(struct iio_dev *indio_dev,
> > > +			     struct iio_chan_spec const *chan,
> > > +			     int val, int val2, long info)
> > > +{
> > > +	struct ad5770r_state *st = iio_priv(indio_dev);
> > > +	u8 buf[2];
> > > +
> > > +	switch (info) {
> > > +	case IIO_CHAN_INFO_RAW:
> > > +		buf[0] = ((u16)val >> 6);
> > > +		buf[1] = (val & 0x3F) << 2;
> > > +		return regmap_bulk_write(st->regmap, chan->address,
> > > +					 buf, ARRAY_SIZE(buf));
> > > +	case IIO_CHAN_INFO_LOW_PASS_FILTER_3DB_FREQUENCY:
> > > +		return ad5770r_set_filter_freq(indio_dev, chan, val);
> > > +	default:
> > > +		return -EINVAL;
> > > +	}
> > > +}
> > > +
> > > +static int ad5770r_read_freq_avail(struct iio_dev *indio_dev,
> > > +				   struct iio_chan_spec const *chan,
> > > +				   const int **vals, int *type, int *length,
> > > +				   long mask)
> > > +{
> > > +	switch (mask) {
> > > +	case IIO_CHAN_INFO_LOW_PASS_FILTER_3DB_FREQUENCY:
> > > +		*type = IIO_VAL_INT;
> > > +		*vals = ad5770r_filter_freqs;
> > > +		*length = ARRAY_SIZE(ad5770r_filter_freqs);
> > > +		return IIO_AVAIL_LIST;
> > > +	}
> > > +
> > > +	return -EINVAL;
> > > +}
> > > +
> > > +static int ad5770r_reg_access(struct iio_dev *indio_dev,
> > > +			      unsigned int reg,
> > > +			      unsigned int writeval,
> > > +			      unsigned int *readval)
> > > +{
> > > +	struct ad5770r_state *st = iio_priv(indio_dev);
> > > +
> > > +	if (readval)
> > > +		return regmap_read(st->regmap, reg, readval);
> > > +	else
> > > +		return regmap_write(st->regmap, reg, writeval);
> > > +}
> > > +
> > > +static const struct iio_info ad5770r_info = {
> > > +	.read_raw = ad5770r_read_raw,
> > > +	.write_raw = ad5770r_write_raw,
> > > +	.read_avail = ad5770r_read_freq_avail,
> > > +	.debugfs_reg_access = &ad5770r_reg_access,
> > > +};
> > > +
> > > +static int ad5770r_store_output_range(struct ad5770r_state *st,
> > > +				      int min, int max, int index)
> > > +{
> > > +	int i;
> > > +
> > > +	for (i = 0; i < AD5770R_MAX_CH_MODES; i++) {
> > > +		if (ad5770r_rng_tbl[i].ch != index)
> > > +			continue;
> > > +		if (ad5770r_rng_tbl[i].min != min ||
> > > +		    ad5770r_rng_tbl[i].max != max)
> > > +			continue;
> > > +		st->output_mode[index].out_range_mode = ad5770r_rng_tbl[i].mode;
> > > +
> > > +		return 0;
> > > +	}
> > > +
> > > +	return -EINVAL;
> > > +}
> > > +
> > > +static ssize_t ad5770r_read_dac_powerdown(struct iio_dev *indio_dev,
> > > +					  uintptr_t private,
> > > +					  const struct iio_chan_spec *chan,
> > > +					  char *buf)
> > > +{
> > > +	struct ad5770r_state *st = iio_priv(indio_dev);
> > > +
> > > +	return sprintf(buf, "%d\n", st->ch_pwr_down[chan->channel]);
> > > +}
> > > +
> > > +static ssize_t ad5770r_write_dac_powerdown(struct iio_dev *indio_dev,
> > > +					   uintptr_t private,
> > > +					   const struct iio_chan_spec *chan,
> > > +					   const char *buf, size_t len)
> > > +{
> > > +	struct ad5770r_state *st = iio_priv(indio_dev);
> > > +	unsigned int regval;
> > > +	unsigned int mask;
> > > +	bool readin;
> > > +	int ret;
> > > +
> > > +	ret = kstrtobool(buf, &readin);
> > > +	if (ret)
> > > +		return ret;
> > > +
> > > +	readin = !readin;
> > > +
> > > +	regval = AD5770R_CFG_SHUTDOWN_B(readin, chan->channel);
> > > +	if (chan->channel == 0 &&
> > > +	    st->output_mode[0].out_range_mode > AD5770R_CH0_0_300) {
> > > +		regval |= AD5770R_CFG_CH0_SINK_EN(readin);
> > > +		mask = BIT(chan->channel) + BIT(7);
> > > +	} else {
> > > +		mask = BIT(chan->channel);
> > > +	}
> > > +	ret = regmap_update_bits(st->regmap, AD5770R_CHANNEL_CONFIG, mask,
> > > +				 regval);
> > > +	if (ret)
> > > +		return ret;
> > > +
> > > +	regval = AD5770R_CH_SET(readin, chan->channel);
> > > +	ret = regmap_update_bits(st->regmap, AD5770R_CH_ENABLE,
> > > +				 BIT(chan->channel), regval);
> > > +	if (ret)
> > > +		return ret;
> > > +
> > > +	st->ch_pwr_down[chan->channel] = !readin;
> > > +
> > > +	return len;
> > > +}
> > > +
> > > +static const struct iio_chan_spec_ext_info ad5770r_ext_info[] = {
> > > +	{
> > > +		.name = "powerdown",
> > > +		.read = ad5770r_read_dac_powerdown,
> > > +		.write = ad5770r_write_dac_powerdown,
> > > +		.shared = IIO_SEPARATE,
> > > +	},
> > > +	{ }
> > > +};
> > > +
> > > +#define AD5770R_IDAC_CHANNEL(index, reg) {				
> > > \
> > > +	.type = IIO_CURRENT,						\
> > > +	.address = reg,							\
> > > +	.indexed = 1,							\
> > > +	.channel = index,						\
> > > +	.output = 1,							\
> > > +	.info_mask_separate = BIT(IIO_CHAN_INFO_RAW) |			\
> > > +		BIT(IIO_CHAN_INFO_SCALE) |				\
> > > +		BIT(IIO_CHAN_INFO_OFFSET) |				\
> > > +		BIT(IIO_CHAN_INFO_LOW_PASS_FILTER_3DB_FREQUENCY),	\
> > > +	.info_mask_shared_by_type_available =				\
> > > +		BIT(IIO_CHAN_INFO_LOW_PASS_FILTER_3DB_FREQUENCY),	\
> > > +	.ext_info = ad5770r_ext_info,					\
> > > +}
> > > +
> > > +static const struct iio_chan_spec ad5770r_channels[] = {
> > > +	AD5770R_IDAC_CHANNEL(0, AD5770R_DAC_MSB(0)),
> > > +	AD5770R_IDAC_CHANNEL(1, AD5770R_DAC_MSB(1)),
> > > +	AD5770R_IDAC_CHANNEL(2, AD5770R_DAC_MSB(2)),
> > > +	AD5770R_IDAC_CHANNEL(3, AD5770R_DAC_MSB(3)),
> > > +	AD5770R_IDAC_CHANNEL(4, AD5770R_DAC_MSB(4)),
> > > +	AD5770R_IDAC_CHANNEL(5, AD5770R_DAC_MSB(5)),
> > > +};
> > > +
> > > +static int ad5770r_channel_config(struct ad5770r_state *st)
> > > +{
> > > +	int ret, tmp[2], min, max;
> > > +	unsigned int num;
> > > +	struct fwnode_handle *child;
> > > +
> > > +	num = device_get_child_node_count(&st->spi->dev);
> > > +	if (num != AD5770R_MAX_CHANNELS)
> > > +		return -EINVAL;
> > > +
> > > +	device_for_each_child_node(&st->spi->dev, child) {
> > > +		ret = fwnode_property_read_u32(child, "num", &num);
> > > +		if (ret)
> > > +			return ret;
> > > +		if (num > AD5770R_MAX_CHANNELS)
> > > +			return -EINVAL;
> > > +
> > > +		ret = fwnode_property_read_u32_array(child,
> > > +						     "adi,range-microamp",
> > > +						     tmp, 2);
> > > +		if (ret)
> > > +			return ret;
> > > +
> > > +		min = tmp[0] / 1000;
> > > +		max = tmp[1] / 1000;
> > > +		ret = ad5770r_store_output_range(st, min, max, num);
> > > +		if (ret)
> > > +			return ret;
> > > +	}
> > > +
> > > +	return ret;
> > > +}
> > > +
> > > +static int ad5770r_init(struct ad5770r_state *st)
> > > +{
> > > +	int ret, i;
> > > +
> > > +	st->gpio_reset = devm_gpiod_get_optional(&st->spi->dev, "reset",
> > > +						 GPIOD_OUT_HIGH);
> > > +	if (IS_ERR(st->gpio_reset))
> > > +		return PTR_ERR(st->gpio_reset);
> > > +
> > > +	/* Perform a reset */
> > > +	ret = ad5770r_reset(st);
> > > +	if (ret)
> > > +		return ret;
> > > +
> > > +	/* Set output range */
> > > +	ret = ad5770r_channel_config(st);
> > > +	if (ret)
> > > +		return ret;
> > > +
> > > +	for (i = 0; i < AD5770R_MAX_CHANNELS; i++) {
> > > +		ret = ad5770r_set_output_mode(st,  &st->output_mode[i], i);
> > > +		if (ret)
> > > +			return ret;
> > > +	}
> > > +
> > > +	st->external_res = fwnode_property_read_bool(st->spi->dev.fwnode,
> > > +						     "adi,external-resistor");
> > > +
> > > +	ret = ad5770r_set_reference(st);
> > > +	if (ret)
> > > +		return ret;
> > > +
> > > +	/* Set outputs off */
> > > +	ret = regmap_write(st->regmap, AD5770R_CHANNEL_CONFIG, 0x00);
> > > +	if (ret)
> > > +		return ret;
> > > +
> > > +	ret = regmap_write(st->regmap, AD5770R_CH_ENABLE, 0x00);
> > > +	if (ret)
> > > +		return ret;
> > > +
> > > +	for (i = 0; i < AD5770R_MAX_CHANNELS; i++)
> > > +		st->ch_pwr_down[i] = true;
> > > +
> > > +	return ret;
> > > +}
> > > +
> > > +static void ad5770r_disable_regulator(void *data)
> > > +{
> > > +	struct spi_device *spi = (struct spi_device *)data;
> > > +	struct iio_dev *indio_dev = spi_get_drvdata(spi);
> > > +	struct ad5770r_state *st = iio_priv(indio_dev);
> > > +
> > > +	if (st->vref_reg)
> > > +		regulator_disable(st->vref_reg);
> > > +}
> > > +
> > > +static int ad5770r_probe(struct spi_device *spi)
> > > +{
> > > +	struct ad5770r_state *st;
> > > +	struct iio_dev *indio_dev;
> > > +	struct regmap *regmap;
> > > +	int ret;
> > > +
> > > +	indio_dev = devm_iio_device_alloc(&spi->dev, sizeof(*st));
> > > +	if (!indio_dev)
> > > +		return -ENOMEM;
> > > +
> > > +	st = iio_priv(indio_dev);
> > > +	spi_set_drvdata(spi, indio_dev);
> > > +
> > > +	st->spi = spi;
> > > +
> > > +	regmap = devm_regmap_init_spi(spi, &ad5770r_spi_regmap_config);
> > > +	if (IS_ERR(regmap)) {
> > > +		dev_err(&spi->dev, "Error initializing spi regmap: %ld\n",
> > > +			PTR_ERR(regmap));
> > > +		return PTR_ERR(regmap);
> > > +	}
> > > +	st->regmap = regmap;
> > > +
> > > +	ret = devm_add_action_or_reset(&spi->dev, ad5770r_disable_regulator,
> > > +				       st);
> > > +	if (ret < 0)
> > > +		return ret;
> > > +
> > > +	st->vref_reg = devm_regulator_get_optional(&spi->dev, "vref");
> > > +	if (!IS_ERR(st->vref_reg)) {
> > > +		ret = regulator_enable(st->vref_reg);
> > > +		if (ret) {
> > > +			dev_err(&spi->dev,
> > > +				"Failed to enable vref regulators: %d\n", ret);
> > > +			return ret;
> > > +		}
> > > +
> > > +		ret = regulator_get_voltage(st->vref_reg);
> > > +		if (ret < 0)
> > > +			return ret;
> > > +
> > > +		st->vref = ret / 1000;
> > > +	} else {
> > > +		if (PTR_ERR(st->vref_reg) == -ENODEV) {
> > > +			st->vref = AD5770R_LOW_VREF_mV;
> > > +			st->internal_ref = true;
> > > +		} else {
> > > +			return PTR_ERR(st->vref_reg);
> > > +		}
> > > +	}
> > > +
> > > +	indio_dev->dev.parent = &spi->dev;
> > > +	indio_dev->name = spi_get_device_id(spi)->name;
> > > +	indio_dev->info = &ad5770r_info;
> > > +	indio_dev->modes = INDIO_DIRECT_MODE;
> > > +	indio_dev->channels = ad5770r_channels;
> > > +	indio_dev->num_channels = ARRAY_SIZE(ad5770r_channels);
> > > +
> > > +	ret = ad5770r_init(st);
> > > +	if (ret < 0) {
> > > +		dev_err(&spi->dev, "AD5770R init failed\n");
> > > +		return ret;
> > > +	}
> > > +
> > > +	return devm_iio_device_register(&st->spi->dev, indio_dev);
> > > +}
> > > +
> > > +static const struct of_device_id ad5770r_of_id[] = {
> > > +	{ .compatible = "adi,ad5770r", },
> > > +};
> > > +MODULE_DEVICE_TABLE(of, ad5770r_of_id);
> > > +
> > > +static const struct spi_device_id ad5770r_id[] = {
> > > +	{ "ad5770r", 0 },
> > > +	{}
> > > +};
> > > +MODULE_DEVICE_TABLE(spi, ad5770r_id);
> > > +
> > > +static struct spi_driver ad5770r_driver = {
> > > +	.driver = {
> > > +		.name = KBUILD_MODNAME,
> > > +		.of_match_table = of_match_ptr(ad5770r_of_id),
> > > +	},
> > > +	.probe = ad5770r_probe,
> > > +	.id_table = ad5770r_id,
> > > +};
> > > +
> > > +module_spi_driver(ad5770r_driver);
> > > +
> > > +MODULE_AUTHOR("Mircea Caprioru <mircea.caprioru@analog.com>");
> > > +MODULE_DESCRIPTION("Analog Devices AD5770R IDAC");
> > > +MODULE_LICENSE("GPL v2");
> > > diff --git a/include/linux/iio/common/adi_spi_regs.h
> > > b/include/linux/iio/common/adi_spi_regs.h
> > > new file mode 100644
> > > index 000000000000..1fc3bfc7845d
> > > --- /dev/null
> > > +++ b/include/linux/iio/common/adi_spi_regs.h
> > > @@ -0,0 +1,44 @@
> > > +/* SPDX-License-Identifier: GPL-2.0 */
> > > +/*
> > > + * Common Analog Devices SPI configuration registers
> > > + *
> > > + * Copyright 2019 Analog Devices Inc.
> > > + */
> > > +
> > > +#ifndef _ADI_SPI_REGS_H_
> > > +#define _ADI_SPI_REGS_H_
> > > +
> > > +#include <linux/bits.h>
> > > +
> > > +#define ADI_SPI_IF_CONFIG_A		0x00
> > > +#define ADI_SPI_IF_CONFIG_B		0x01
> > > +#define ADI_SPI_IF_DEVICE_CONFIG	0x02
> > > +#define ADI_SPI_IF_CHIP_TYPE		0x03
> > > +#define ADI_SPI_IF_PRODUCT_ID_L		0x04
> > > +#define ADI_SPI_IF_PRODUCT_ID_H		0x05
> > > +#define ADI_SPI_IF_CHIP_GRADE		0x06
> > > +#define ADI_SPI_IF_SCRACTH_PAD		0x0A
> > > +#define ADI_SPI_IF_SPI_REVISION		0x0B
> > > +#define ADI_SPI_IF_SPI_VENDOR_L		0x0C
> > > +#define ADI_SPI_IF_SPI_VENDOR_H		0x0D
> > > +#define ADI_SPI_IF_SPI_STREAM_MODE	0x0E
> > > +#define ADI_SPI_IF_CONFIG_C		0x10
> > > +#define ADI_SPI_IF_STATUS_A		0x11
> > > +
> > > +/* ADI_SPI_IF_CONFIG_A */
> > > +#define ADI_SPI_IF_SW_RESET_MSK		(BIT(0) | BIT(7))
> > > +#define ADI_SPI_IF_SW_RESET_SEL(x)	((x) & ADI_SPI_IF_SW_RESET_MSK)
> > > +#define ADI_SPI_IF_ADDR_ASC_MSK		(BIT(2) | BIT(5))
> > > +#define ADI_SPI_IF_ADDR_ASC_SEL(x)	(((x) << 2) &
> > > ADI_SPI_IF_ADDR_ASC_MSK)
> > > +
> > > +/* ADI_SPI_IF_CONFIG_B */
> > > +#define ADI_SPI_IF_SINGLE_INS_MSK	BIT(7)
> > > +#define ADI_SPI_IF_SINGLE_INS_SEL(x)	FIELD_PREP(ADI_SPI_IF_SINGLE_INS
> > > _MSK, x)
> > > +#define ADI_SPI_IF_SHORT_INS_MSK	BIT(7)
> > > +#define ADI_SPI_IF_SHORT_INS_SEL(x)	FIELD_PREP(ADI_SPI_IF_SINGLE_INS
> > > _MSK, x)
> > > +
> > > +/* ADI_SPI_IF_CONFIG_C */
> > > +#define ADI_SPI_IF_STRICT_REG_MSK	BIT(5)
> > > +#define ADI_SPI_IF_STRICT_REG_GET(x)	FIELD_GET(ADI_SPI_IF_STRICT_REG_
> > > MSK, x)
> > > +
> > > +#endif /* _ADI_SPI_REGS_H_ */

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2020-02-14 12:45 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-02-06  9:23 [PATCH 0/2] iio: dac: AD5770R: Add support Alexandru Tachici
2020-02-06  9:23 ` [PATCH 1/2] iio: dac: ad5770r: Add AD5770R support Alexandru Tachici
2020-02-08  9:05   ` kbuild test robot
2020-02-08 14:33   ` Jonathan Cameron
2020-02-11 13:51     ` Ardelean, Alexandru
2020-02-14 12:45       ` Ardelean, Alexandru
2020-02-06  9:23 ` [PATCH 2/2] dt-bindings: iio: dac: Add docs for AD5770R DAC Alexandru Tachici
2020-02-08 14:36   ` Jonathan Cameron
2020-02-08 14:24 ` [PATCH 0/2] iio: dac: AD5770R: Add support Jonathan Cameron

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).