staging:iio:ad7793: Use common Sigma Delta library
authorLars-Peter Clausen <lars@metafoo.de>
Fri, 10 Aug 2012 16:36:00 +0000 (17:36 +0100)
committerJonathan Cameron <jic23@kernel.org>
Mon, 27 Aug 2012 17:10:36 +0000 (18:10 +0100)
Convert the ad7793 driver to make use of the new common code for devices from
the Analog Devices Sigma Delta family.

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
drivers/staging/iio/adc/Kconfig
drivers/staging/iio/adc/ad7793.c
drivers/staging/iio/adc/ad7793.h

index d0eb27be381c53135212afe0941b6c4d32ba9e63..f7d7e44af6721390a5b34650dd733ee9728c26ee 100644 (file)
@@ -111,8 +111,7 @@ config AD7780
 config AD7793
        tristate "Analog Devices AD7792 AD7793 ADC driver"
        depends on SPI
-       select IIO_BUFFER
-       select IIO_TRIGGERED_BUFFER
+       select AD_SIGMA_DELTA
        help
          Say yes here to build support for Analog Devices
          AD7792 and AD7793 SPI analog to digital converters (ADC).
index 5c2fcd7d2def94d09099bf1937e493b2ff4bdb9d..90335958d9c9fa5d812f3f5913c9ec448482f639 100644 (file)
@@ -24,6 +24,7 @@
 #include <linux/iio/trigger.h>
 #include <linux/iio/trigger_consumer.h>
 #include <linux/iio/triggered_buffer.h>
+#include <linux/iio/adc/ad_sigma_delta.h>
 
 #include "ad7793.h"
 
  */
 
 struct ad7793_chip_info {
-       struct iio_chan_spec            channel[7];
+       struct iio_chan_spec    channel[7];
 };
 
 struct ad7793_state {
-       struct spi_device               *spi;
-       struct iio_trigger              *trig;
        const struct ad7793_chip_info   *chip_info;
        struct regulator                *reg;
-       wait_queue_head_t               wq_data_avail;
-       bool                            done;
-       bool                            irq_dis;
        u16                             int_vref_mv;
        u16                             mode;
        u16                             conf;
        u32                             scale_avail[8][2];
 
-       /*
-        * DMA (thus cache coherency maintenance) requires the
-        * transfer buffers to live in their own cache lines.
-        */
-       u8                              data[4] ____cacheline_aligned;
+       struct ad_sigma_delta           sd;
+
 };
 
 enum ad7793_supported_device_ids {
@@ -64,169 +57,41 @@ enum ad7793_supported_device_ids {
        ID_AD7793,
 };
 
-static int __ad7793_write_reg(struct ad7793_state *st, bool locked,
-                             bool cs_change, unsigned char reg,
-                             unsigned size, unsigned val)
+static struct ad7793_state *ad_sigma_delta_to_ad7793(struct ad_sigma_delta *sd)
 {
-       u8 *data = st->data;
-       struct spi_transfer t = {
-               .tx_buf         = data,
-               .len            = size + 1,
-               .cs_change      = cs_change,
-       };
-       struct spi_message m;
-
-       data[0] = AD7793_COMM_WRITE | AD7793_COMM_ADDR(reg);
-
-       switch (size) {
-       case 3:
-               data[1] = val >> 16;
-               data[2] = val >> 8;
-               data[3] = val;
-               break;
-       case 2:
-               data[1] = val >> 8;
-               data[2] = val;
-               break;
-       case 1:
-               data[1] = val;
-               break;
-       default:
-               return -EINVAL;
-       }
-
-       spi_message_init(&m);
-       spi_message_add_tail(&t, &m);
-
-       if (locked)
-               return spi_sync_locked(st->spi, &m);
-       else
-               return spi_sync(st->spi, &m);
+       return container_of(sd, struct ad7793_state, sd);
 }
 
-static int ad7793_write_reg(struct ad7793_state *st,
-                           unsigned reg, unsigned size, unsigned val)
+static int ad7793_set_channel(struct ad_sigma_delta *sd, unsigned int channel)
 {
-       return __ad7793_write_reg(st, false, false, reg, size, val);
-}
+       struct ad7793_state *st = ad_sigma_delta_to_ad7793(sd);
 
-static int __ad7793_read_reg(struct ad7793_state *st, bool locked,
-                            bool cs_change, unsigned char reg,
-                            int *val, unsigned size)
-{
-       u8 *data = st->data;
-       int ret;
-       struct spi_transfer t[] = {
-               {
-                       .tx_buf = data,
-                       .len = 1,
-               }, {
-                       .rx_buf = data,
-                       .len = size,
-                       .cs_change = cs_change,
-               },
-       };
-       struct spi_message m;
-
-       data[0] = AD7793_COMM_READ | AD7793_COMM_ADDR(reg);
-
-       spi_message_init(&m);
-       spi_message_add_tail(&t[0], &m);
-       spi_message_add_tail(&t[1], &m);
-
-       if (locked)
-               ret = spi_sync_locked(st->spi, &m);
-       else
-               ret = spi_sync(st->spi, &m);
+       st->conf &= ~AD7793_CONF_CHAN_MASK;
+       st->conf |= AD7793_CONF_CHAN(channel);
 
-       if (ret < 0)
-               return ret;
-
-       switch (size) {
-       case 3:
-               *val = data[0] << 16 | data[1] << 8 | data[2];
-               break;
-       case 2:
-               *val = data[0] << 8 | data[1];
-               break;
-       case 1:
-               *val = data[0];
-               break;
-       default:
-               return -EINVAL;
-       }
-
-       return 0;
+       return ad_sd_write_reg(&st->sd, AD7793_REG_CONF, 2, st->conf);
 }
 
-static int ad7793_read_reg(struct ad7793_state *st,
-                          unsigned reg, int *val, unsigned size)
+static int ad7793_set_mode(struct ad_sigma_delta *sd,
+                          enum ad_sigma_delta_mode mode)
 {
-       return __ad7793_read_reg(st, 0, 0, reg, val, size);
-}
-
-static int ad7793_read(struct ad7793_state *st, unsigned ch,
-                      unsigned len, int *val)
-{
-       int ret;
-       st->conf = (st->conf & ~AD7793_CONF_CHAN(-1)) | AD7793_CONF_CHAN(ch);
-       st->mode = (st->mode & ~AD7793_MODE_SEL(-1)) |
-               AD7793_MODE_SEL(AD7793_MODE_SINGLE);
+       struct ad7793_state *st = ad_sigma_delta_to_ad7793(sd);
 
-       ad7793_write_reg(st, AD7793_REG_CONF, sizeof(st->conf), st->conf);
+       st->mode &= ~AD7793_MODE_SEL_MASK;
+       st->mode |= AD7793_MODE_SEL(mode);
 
-       spi_bus_lock(st->spi->master);
-       st->done = false;
-
-       ret = __ad7793_write_reg(st, 1, 1, AD7793_REG_MODE,
-                                sizeof(st->mode), st->mode);
-       if (ret < 0)
-               goto out;
-
-       st->irq_dis = false;
-       enable_irq(st->spi->irq);
-       wait_event_interruptible(st->wq_data_avail, st->done);
-
-       ret = __ad7793_read_reg(st, 1, 0, AD7793_REG_DATA, val, len);
-out:
-       spi_bus_unlock(st->spi->master);
-
-       return ret;
+       return ad_sd_write_reg(&st->sd, AD7793_REG_MODE, 2, st->mode);
 }
 
-static int ad7793_calibrate(struct ad7793_state *st, unsigned mode, unsigned ch)
-{
-       int ret;
-
-       st->conf = (st->conf & ~AD7793_CONF_CHAN(-1)) | AD7793_CONF_CHAN(ch);
-       st->mode = (st->mode & ~AD7793_MODE_SEL(-1)) | AD7793_MODE_SEL(mode);
-
-       ad7793_write_reg(st, AD7793_REG_CONF, sizeof(st->conf), st->conf);
-
-       spi_bus_lock(st->spi->master);
-       st->done = false;
-
-       ret = __ad7793_write_reg(st, 1, 1, AD7793_REG_MODE,
-                                sizeof(st->mode), st->mode);
-       if (ret < 0)
-               goto out;
-
-       st->irq_dis = false;
-       enable_irq(st->spi->irq);
-       wait_event_interruptible(st->wq_data_avail, st->done);
-
-       st->mode = (st->mode & ~AD7793_MODE_SEL(-1)) |
-               AD7793_MODE_SEL(AD7793_MODE_IDLE);
-
-       ret = __ad7793_write_reg(st, 1, 0, AD7793_REG_MODE,
-                                sizeof(st->mode), st->mode);
-out:
-       spi_bus_unlock(st->spi->master);
-
-       return ret;
-}
+static const struct ad_sigma_delta_info ad7793_sigma_delta_info = {
+       .set_channel = ad7793_set_channel,
+       .set_mode = ad7793_set_mode,
+       .has_registers = true,
+       .addr_shift = 3,
+       .read_mask = BIT(6),
+};
 
-static const u8 ad7793_calib_arr[6][2] = {
+static const struct ad_sd_calib_data ad7793_calib_arr[6] = {
        {AD7793_MODE_CAL_INT_ZERO, AD7793_CH_AIN1P_AIN1M},
        {AD7793_MODE_CAL_INT_FULL, AD7793_CH_AIN1P_AIN1M},
        {AD7793_MODE_CAL_INT_ZERO, AD7793_CH_AIN2P_AIN2M},
@@ -237,59 +102,48 @@ static const u8 ad7793_calib_arr[6][2] = {
 
 static int ad7793_calibrate_all(struct ad7793_state *st)
 {
-       int i, ret;
-
-       for (i = 0; i < ARRAY_SIZE(ad7793_calib_arr); i++) {
-               ret = ad7793_calibrate(st, ad7793_calib_arr[i][0],
-                                      ad7793_calib_arr[i][1]);
-               if (ret)
-                       goto out;
-       }
-
-       return 0;
-out:
-       dev_err(&st->spi->dev, "Calibration failed\n");
-       return ret;
+       return ad_sd_calibrate_all(&st->sd, ad7793_calib_arr,
+                                  ARRAY_SIZE(ad7793_calib_arr));
 }
 
-static int ad7793_setup(struct ad7793_state *st,
+static int ad7793_setup(struct iio_dev *indio_dev,
        const struct ad7793_platform_data *pdata)
 {
+       struct ad7793_state *st = iio_priv(indio_dev);
        int i, ret = -1;
        unsigned long long scale_uv;
        u32 id;
 
        /* reset the serial interface */
-       ret = spi_write(st->spi, (u8 *)&ret, sizeof(ret));
+       ret = spi_write(st->sd.spi, (u8 *)&ret, sizeof(ret));
        if (ret < 0)
                goto out;
        msleep(1); /* Wait for at least 500us */
 
        /* write/read test for device presence */
-       ret = ad7793_read_reg(st, AD7793_REG_ID, &id, 1);
+       ret = ad_sd_read_reg(&st->sd, AD7793_REG_ID, 1, &id);
        if (ret)
                goto out;
 
        id &= AD7793_ID_MASK;
 
        if (!((id == AD7792_ID) || (id == AD7793_ID))) {
-               dev_err(&st->spi->dev, "device ID query failed\n");
+               dev_err(&st->sd.spi->dev, "device ID query failed\n");
                goto out;
        }
 
-       st->mode  = (pdata->mode & ~AD7793_MODE_SEL(-1)) |
-                       AD7793_MODE_SEL(AD7793_MODE_IDLE);
-       st->conf  = pdata->conf & ~AD7793_CONF_CHAN(-1);
+       st->mode = pdata->mode;
+       st->conf = pdata->conf;
 
-       ret = ad7793_write_reg(st, AD7793_REG_MODE, sizeof(st->mode), st->mode);
+       ret = ad7793_set_mode(&st->sd, AD_SD_MODE_IDLE);
        if (ret)
                goto out;
 
-       ret = ad7793_write_reg(st, AD7793_REG_CONF, sizeof(st->conf), st->conf);
+       ret = ad7793_set_channel(&st->sd, 0);
        if (ret)
                goto out;
 
-       ret = ad7793_write_reg(st, AD7793_REG_IO,
+       ret = ad_sd_write_reg(&st->sd, AD7793_REG_IO,
                               sizeof(pdata->io), pdata->io);
        if (ret)
                goto out;
@@ -311,184 +165,10 @@ static int ad7793_setup(struct ad7793_state *st,
 
        return 0;
 out:
-       dev_err(&st->spi->dev, "setup failed\n");
-       return ret;
-}
-
-static int ad7793_ring_preenable(struct iio_dev *indio_dev)
-{
-       struct ad7793_state *st = iio_priv(indio_dev);
-       unsigned channel;
-       int ret;
-
-       if (bitmap_empty(indio_dev->active_scan_mask, indio_dev->masklength))
-               return -EINVAL;
-       ret = iio_sw_buffer_preenable(indio_dev);
-       if (ret < 0)
-               return ret;
-
-       channel = find_first_bit(indio_dev->active_scan_mask,
-                                indio_dev->masklength);
-
-       st->mode  = (st->mode & ~AD7793_MODE_SEL(-1)) |
-                   AD7793_MODE_SEL(AD7793_MODE_CONT);
-       st->conf  = (st->conf & ~AD7793_CONF_CHAN(-1)) |
-                   AD7793_CONF_CHAN(indio_dev->channels[channel].address);
-
-       ad7793_write_reg(st, AD7793_REG_CONF, sizeof(st->conf), st->conf);
-
-       spi_bus_lock(st->spi->master);
-       __ad7793_write_reg(st, 1, 1, AD7793_REG_MODE,
-                          sizeof(st->mode), st->mode);
-
-       st->irq_dis = false;
-       enable_irq(st->spi->irq);
-
-       return 0;
-}
-
-static int ad7793_ring_postdisable(struct iio_dev *indio_dev)
-{
-       struct ad7793_state *st = iio_priv(indio_dev);
-
-       st->mode  = (st->mode & ~AD7793_MODE_SEL(-1)) |
-                   AD7793_MODE_SEL(AD7793_MODE_IDLE);
-
-       st->done = false;
-       wait_event_interruptible(st->wq_data_avail, st->done);
-
-       if (!st->irq_dis)
-               disable_irq_nosync(st->spi->irq);
-
-       __ad7793_write_reg(st, 1, 0, AD7793_REG_MODE,
-                          sizeof(st->mode), st->mode);
-
-       return spi_bus_unlock(st->spi->master);
-}
-
-/**
- * ad7793_trigger_handler() bh of trigger launched polling to ring buffer
- **/
-
-static irqreturn_t ad7793_trigger_handler(int irq, void *p)
-{
-       struct iio_poll_func *pf = p;
-       struct iio_dev *indio_dev = pf->indio_dev;
-       struct iio_buffer *ring = indio_dev->buffer;
-       struct ad7793_state *st = iio_priv(indio_dev);
-       s64 dat64[2];
-       s32 *dat32 = (s32 *)dat64;
-
-       if (!bitmap_empty(indio_dev->active_scan_mask, indio_dev->masklength))
-               __ad7793_read_reg(st, 1, 1, AD7793_REG_DATA,
-                                 dat32,
-                                 indio_dev->channels[0].scan_type.realbits/8);
-
-       /* Guaranteed to be aligned with 8 byte boundary */
-       if (indio_dev->scan_timestamp)
-               dat64[1] = pf->timestamp;
-
-       ring->access->store_to(ring, (u8 *)dat64, pf->timestamp);
-
-       iio_trigger_notify_done(indio_dev->trig);
-       st->irq_dis = false;
-       enable_irq(st->spi->irq);
-
-       return IRQ_HANDLED;
-}
-
-static const struct iio_buffer_setup_ops ad7793_ring_setup_ops = {
-       .preenable = &ad7793_ring_preenable,
-       .postenable = &iio_triggered_buffer_postenable,
-       .predisable = &iio_triggered_buffer_predisable,
-       .postdisable = &ad7793_ring_postdisable,
-       .validate_scan_mask = &iio_validate_scan_mask_onehot,
-};
-
-static int ad7793_register_ring_funcs_and_init(struct iio_dev *indio_dev)
-{
-       return iio_triggered_buffer_setup(indio_dev, &iio_pollfunc_store_time,
-                       &ad7793_trigger_handler, &ad7793_ring_setup_ops);
-}
-
-static void ad7793_ring_cleanup(struct iio_dev *indio_dev)
-{
-       iio_triggered_buffer_cleanup(indio_dev);
-}
-
-/**
- * ad7793_data_rdy_trig_poll() the event handler for the data rdy trig
- **/
-static irqreturn_t ad7793_data_rdy_trig_poll(int irq, void *private)
-{
-       struct ad7793_state *st = iio_priv(private);
-
-       st->done = true;
-       wake_up_interruptible(&st->wq_data_avail);
-       disable_irq_nosync(irq);
-       st->irq_dis = true;
-       iio_trigger_poll(st->trig, iio_get_time_ns());
-
-       return IRQ_HANDLED;
-}
-
-static struct iio_trigger_ops ad7793_trigger_ops = {
-       .owner = THIS_MODULE,
-};
-
-static int ad7793_probe_trigger(struct iio_dev *indio_dev)
-{
-       struct ad7793_state *st = iio_priv(indio_dev);
-       int ret;
-
-       st->trig = iio_trigger_alloc("%s-dev%d",
-                                       spi_get_device_id(st->spi)->name,
-                                       indio_dev->id);
-       if (st->trig == NULL) {
-               ret = -ENOMEM;
-               goto error_ret;
-       }
-       st->trig->ops = &ad7793_trigger_ops;
-
-       ret = request_irq(st->spi->irq,
-                         ad7793_data_rdy_trig_poll,
-                         IRQF_TRIGGER_LOW,
-                         spi_get_device_id(st->spi)->name,
-                         indio_dev);
-       if (ret)
-               goto error_free_trig;
-
-       disable_irq_nosync(st->spi->irq);
-       st->irq_dis = true;
-       st->trig->dev.parent = &st->spi->dev;
-       st->trig->private_data = indio_dev;
-
-       ret = iio_trigger_register(st->trig);
-
-       /* select default trigger */
-       indio_dev->trig = st->trig;
-       if (ret)
-               goto error_free_irq;
-
-       return 0;
-
-error_free_irq:
-       free_irq(st->spi->irq, indio_dev);
-error_free_trig:
-       iio_trigger_free(st->trig);
-error_ret:
+       dev_err(&st->sd.spi->dev, "setup failed\n");
        return ret;
 }
 
-static void ad7793_remove_trigger(struct iio_dev *indio_dev)
-{
-       struct ad7793_state *st = iio_priv(indio_dev);
-
-       iio_trigger_unregister(st->trig);
-       free_irq(st->spi->irq, indio_dev);
-       iio_trigger_free(st->trig);
-}
-
 static const u16 sample_freq_avail[16] = {0, 470, 242, 123, 62, 50, 39, 33, 19,
                                          17, 16, 12, 10, 8, 6, 4};
 
@@ -531,7 +211,7 @@ static ssize_t ad7793_write_frequency(struct device *dev,
                        mutex_lock(&indio_dev->mlock);
                        st->mode &= ~AD7793_MODE_RATE(-1);
                        st->mode |= AD7793_MODE_RATE(i);
-                       ad7793_write_reg(st, AD7793_REG_MODE,
+                       ad_sd_write_reg(&st->sd, AD7793_REG_MODE,
                                         sizeof(st->mode), st->mode);
                        mutex_unlock(&indio_dev->mlock);
                        ret = 0;
@@ -585,26 +265,16 @@ static int ad7793_read_raw(struct iio_dev *indio_dev,
                           long m)
 {
        struct ad7793_state *st = iio_priv(indio_dev);
-       int ret, smpl = 0;
+       int ret;
        unsigned long long scale_uv;
        bool unipolar = !!(st->conf & AD7793_CONF_UNIPOLAR);
 
        switch (m) {
        case IIO_CHAN_INFO_RAW:
-               mutex_lock(&indio_dev->mlock);
-               if (iio_buffer_enabled(indio_dev))
-                       ret = -EBUSY;
-               else
-                       ret = ad7793_read(st, chan->address,
-                                       chan->scan_type.realbits / 8, &smpl);
-               mutex_unlock(&indio_dev->mlock);
-
+               ret = ad_sigma_delta_single_conversion(indio_dev, chan, val);
                if (ret < 0)
                        return ret;
 
-               *val = (smpl >> chan->scan_type.shift) &
-                       ((1 << (chan->scan_type.realbits)) - 1);
-
                return IIO_VAL_INT;
 
        case IIO_CHAN_INFO_SCALE:
@@ -675,17 +345,18 @@ static int ad7793_write_raw(struct iio_dev *indio_dev,
                ret = -EINVAL;
                for (i = 0; i < ARRAY_SIZE(st->scale_avail); i++)
                        if (val2 == st->scale_avail[i][1]) {
+                               ret = 0;
                                tmp = st->conf;
                                st->conf &= ~AD7793_CONF_GAIN(-1);
                                st->conf |= AD7793_CONF_GAIN(i);
 
-                               if (tmp != st->conf) {
-                                       ad7793_write_reg(st, AD7793_REG_CONF,
-                                                        sizeof(st->conf),
-                                                        st->conf);
-                                       ad7793_calibrate_all(st);
-                               }
-                               ret = 0;
+                               if (tmp == st->conf)
+                                       break;
+
+                               ad_sd_write_reg(&st->sd, AD7793_REG_CONF,
+                                               sizeof(st->conf), st->conf);
+                               ad7793_calibrate_all(st);
+                               break;
                        }
                break;
        default:
@@ -696,15 +367,6 @@ static int ad7793_write_raw(struct iio_dev *indio_dev,
        return ret;
 }
 
-static int ad7793_validate_trigger(struct iio_dev *indio_dev,
-                                  struct iio_trigger *trig)
-{
-       if (indio_dev->trig != trig)
-               return -EINVAL;
-
-       return 0;
-}
-
 static int ad7793_write_raw_get_fmt(struct iio_dev *indio_dev,
                               struct iio_chan_spec const *chan,
                               long mask)
@@ -717,166 +379,32 @@ static const struct iio_info ad7793_info = {
        .write_raw = &ad7793_write_raw,
        .write_raw_get_fmt = &ad7793_write_raw_get_fmt,
        .attrs = &ad7793_attribute_group,
-       .validate_trigger = ad7793_validate_trigger,
+       .validate_trigger = ad_sd_validate_trigger,
        .driver_module = THIS_MODULE,
 };
 
 static const struct ad7793_chip_info ad7793_chip_info_tbl[] = {
        [ID_AD7793] = {
-               .channel[0] = {
-                       .type = IIO_VOLTAGE,
-                       .differential = 1,
-                       .indexed = 1,
-                       .channel = 0,
-                       .channel2 = 0,
-                       .address = AD7793_CH_AIN1P_AIN1M,
-                       .info_mask = IIO_CHAN_INFO_RAW_SEPARATE_BIT |
-                       IIO_CHAN_INFO_SCALE_SHARED_BIT |
-                       IIO_CHAN_INFO_OFFSET_SHARED_BIT,
-                       .scan_index = 0,
-                       .scan_type = IIO_ST('u', 24, 32, 0)
-               },
-               .channel[1] = {
-                       .type = IIO_VOLTAGE,
-                       .differential = 1,
-                       .indexed = 1,
-                       .channel = 1,
-                       .channel2 = 1,
-                       .address = AD7793_CH_AIN2P_AIN2M,
-                       .info_mask = IIO_CHAN_INFO_RAW_SEPARATE_BIT |
-                       IIO_CHAN_INFO_SCALE_SHARED_BIT |
-                       IIO_CHAN_INFO_OFFSET_SHARED_BIT,
-                       .scan_index = 1,
-                       .scan_type = IIO_ST('u', 24, 32, 0)
-               },
-               .channel[2] = {
-                       .type = IIO_VOLTAGE,
-                       .differential = 1,
-                       .indexed = 1,
-                       .channel = 2,
-                       .channel2 = 2,
-                       .address = AD7793_CH_AIN3P_AIN3M,
-                       .info_mask = IIO_CHAN_INFO_RAW_SEPARATE_BIT |
-                       IIO_CHAN_INFO_SCALE_SHARED_BIT |
-                       IIO_CHAN_INFO_OFFSET_SHARED_BIT,
-                       .scan_index = 2,
-                       .scan_type = IIO_ST('u', 24, 32, 0)
-               },
-               .channel[3] = {
-                       .type = IIO_VOLTAGE,
-                       .differential = 1,
-                       .extend_name = "shorted",
-                       .indexed = 1,
-                       .channel = 2,
-                       .channel2 = 2,
-                       .address = AD7793_CH_AIN1M_AIN1M,
-                       .info_mask = IIO_CHAN_INFO_RAW_SEPARATE_BIT |
-                       IIO_CHAN_INFO_SCALE_SHARED_BIT |
-                       IIO_CHAN_INFO_OFFSET_SHARED_BIT,
-                       .scan_index = 3,
-                       .scan_type = IIO_ST('u', 24, 32, 0)
-               },
-               .channel[4] = {
-                       .type = IIO_TEMP,
-                       .indexed = 1,
-                       .channel = 0,
-                       .address = AD7793_CH_TEMP,
-                       .info_mask = IIO_CHAN_INFO_RAW_SEPARATE_BIT |
-                       IIO_CHAN_INFO_SCALE_SEPARATE_BIT,
-                       .scan_index = 4,
-                       .scan_type = IIO_ST('u', 24, 32, 0),
+               .channel = {
+                       AD_SD_DIFF_CHANNEL(0, 0, 0, AD7793_CH_AIN1P_AIN1M, 24, 32, 0),
+                       AD_SD_DIFF_CHANNEL(1, 1, 1, AD7793_CH_AIN2P_AIN2M, 24, 32, 0),
+                       AD_SD_DIFF_CHANNEL(2, 2, 2, AD7793_CH_AIN3P_AIN3M, 24, 32, 0),
+                       AD_SD_SHORTED_CHANNEL(3, 0, AD7793_CH_AIN1M_AIN1M, 24, 32, 0),
+                       AD_SD_TEMP_CHANNEL(4, AD7793_CH_TEMP, 24, 32, 0),
+                       AD_SD_SUPPLY_CHANNEL(5, 3, AD7793_CH_AVDD_MONITOR, 24, 32, 0),
+                       IIO_CHAN_SOFT_TIMESTAMP(6),
                },
-               .channel[5] = {
-                       .type = IIO_VOLTAGE,
-                       .extend_name = "supply",
-                       .indexed = 1,
-                       .channel = 4,
-                       .address = AD7793_CH_AVDD_MONITOR,
-                       .info_mask = IIO_CHAN_INFO_RAW_SEPARATE_BIT |
-                       IIO_CHAN_INFO_SCALE_SEPARATE_BIT |
-                       IIO_CHAN_INFO_OFFSET_SHARED_BIT,
-                       .scan_index = 5,
-                       .scan_type = IIO_ST('u', 24, 32, 0),
-               },
-               .channel[6] = IIO_CHAN_SOFT_TIMESTAMP(6),
        },
        [ID_AD7792] = {
-               .channel[0] = {
-                       .type = IIO_VOLTAGE,
-                       .differential = 1,
-                       .indexed = 1,
-                       .channel = 0,
-                       .channel2 = 0,
-                       .address = AD7793_CH_AIN1P_AIN1M,
-                       .info_mask = IIO_CHAN_INFO_RAW_SEPARATE_BIT |
-                       IIO_CHAN_INFO_SCALE_SHARED_BIT |
-                       IIO_CHAN_INFO_OFFSET_SHARED_BIT,
-                       .scan_index = 0,
-                       .scan_type = IIO_ST('u', 16, 32, 0)
-               },
-               .channel[1] = {
-                       .type = IIO_VOLTAGE,
-                       .differential = 1,
-                       .indexed = 1,
-                       .channel = 1,
-                       .channel2 = 1,
-                       .address = AD7793_CH_AIN2P_AIN2M,
-                       .info_mask = IIO_CHAN_INFO_RAW_SEPARATE_BIT |
-                       IIO_CHAN_INFO_SCALE_SHARED_BIT |
-                       IIO_CHAN_INFO_OFFSET_SHARED_BIT,
-                       .scan_index = 1,
-                       .scan_type = IIO_ST('u', 16, 32, 0)
-               },
-               .channel[2] = {
-                       .type = IIO_VOLTAGE,
-                       .differential = 1,
-                       .indexed = 1,
-                       .channel = 2,
-                       .channel2 = 2,
-                       .address = AD7793_CH_AIN3P_AIN3M,
-                       .info_mask = IIO_CHAN_INFO_RAW_SEPARATE_BIT |
-                       IIO_CHAN_INFO_SCALE_SHARED_BIT |
-                       IIO_CHAN_INFO_OFFSET_SHARED_BIT,
-                       .scan_index = 2,
-                       .scan_type = IIO_ST('u', 16, 32, 0)
-               },
-               .channel[3] = {
-                       .type = IIO_VOLTAGE,
-                       .differential = 1,
-                       .extend_name = "shorted",
-                       .indexed = 1,
-                       .channel = 2,
-                       .channel2 = 2,
-                       .address = AD7793_CH_AIN1M_AIN1M,
-                       .info_mask = IIO_CHAN_INFO_RAW_SEPARATE_BIT |
-                       IIO_CHAN_INFO_SCALE_SHARED_BIT |
-                       IIO_CHAN_INFO_OFFSET_SHARED_BIT,
-                       .scan_index = 3,
-                       .scan_type = IIO_ST('u', 16, 32, 0)
-               },
-               .channel[4] = {
-                       .type = IIO_TEMP,
-                       .indexed = 1,
-                       .channel = 0,
-                       .address = AD7793_CH_TEMP,
-                       .info_mask = IIO_CHAN_INFO_RAW_SEPARATE_BIT |
-                       IIO_CHAN_INFO_SCALE_SEPARATE_BIT,
-                       .scan_index = 4,
-                       .scan_type = IIO_ST('u', 16, 32, 0),
+               .channel = {
+                       AD_SD_DIFF_CHANNEL(0, 0, 0, AD7793_CH_AIN1P_AIN1M, 16, 32, 0),
+                       AD_SD_DIFF_CHANNEL(1, 1, 1, AD7793_CH_AIN2P_AIN2M, 16, 32, 0),
+                       AD_SD_DIFF_CHANNEL(2, 2, 2, AD7793_CH_AIN3P_AIN3M, 16, 32, 0),
+                       AD_SD_SHORTED_CHANNEL(3, 0, AD7793_CH_AIN1M_AIN1M, 16, 32, 0),
+                       AD_SD_TEMP_CHANNEL(4, AD7793_CH_TEMP, 16, 32, 0),
+                       AD_SD_SUPPLY_CHANNEL(5, 3, AD7793_CH_AVDD_MONITOR, 16, 32, 0),
+                       IIO_CHAN_SOFT_TIMESTAMP(6),
                },
-               .channel[5] = {
-                       .type = IIO_VOLTAGE,
-                       .extend_name = "supply",
-                       .indexed = 1,
-                       .channel = 4,
-                       .address = AD7793_CH_AVDD_MONITOR,
-                       .info_mask = IIO_CHAN_INFO_RAW_SEPARATE_BIT |
-                       IIO_CHAN_INFO_SCALE_SEPARATE_BIT |
-                       IIO_CHAN_INFO_OFFSET_SHARED_BIT,
-                       .scan_index = 5,
-                       .scan_type = IIO_ST('u', 16, 32, 0),
-               },
-               .channel[6] = IIO_CHAN_SOFT_TIMESTAMP(6),
        },
 };
 
@@ -903,6 +431,8 @@ static int __devinit ad7793_probe(struct spi_device *spi)
 
        st = iio_priv(indio_dev);
 
+       ad_sd_init(&st->sd, indio_dev, spi, &ad7793_sigma_delta_info);
+
        st->reg = regulator_get(&spi->dev, "vcc");
        if (!IS_ERR(st->reg)) {
                ret = regulator_enable(st->reg);
@@ -923,7 +453,6 @@ static int __devinit ad7793_probe(struct spi_device *spi)
                st->int_vref_mv = 1170; /* Build-in ref */
 
        spi_set_drvdata(spi, indio_dev);
-       st->spi = spi;
 
        indio_dev->dev.parent = &spi->dev;
        indio_dev->name = spi_get_device_id(spi)->name;
@@ -932,17 +461,11 @@ static int __devinit ad7793_probe(struct spi_device *spi)
        indio_dev->num_channels = 7;
        indio_dev->info = &ad7793_info;
 
-       init_waitqueue_head(&st->wq_data_avail);
-
-       ret = ad7793_register_ring_funcs_and_init(indio_dev);
+       ret = ad_sd_setup_buffer_and_trigger(indio_dev);
        if (ret)
                goto error_disable_reg;
 
-       ret = ad7793_probe_trigger(indio_dev);
-       if (ret)
-               goto error_unreg_ring;
-
-       ret = ad7793_setup(st, pdata);
+       ret = ad7793_setup(indio_dev, pdata);
        if (ret)
                goto error_remove_trigger;
 
@@ -953,9 +476,7 @@ static int __devinit ad7793_probe(struct spi_device *spi)
        return 0;
 
 error_remove_trigger:
-       ad7793_remove_trigger(indio_dev);
-error_unreg_ring:
-       ad7793_ring_cleanup(indio_dev);
+       ad_sd_cleanup_buffer_and_trigger(indio_dev);
 error_disable_reg:
        if (!IS_ERR(st->reg))
                regulator_disable(st->reg);
@@ -974,8 +495,7 @@ static int ad7793_remove(struct spi_device *spi)
        struct ad7793_state *st = iio_priv(indio_dev);
 
        iio_device_unregister(indio_dev);
-       ad7793_remove_trigger(indio_dev);
-       ad7793_ring_cleanup(indio_dev);
+       ad_sd_cleanup_buffer_and_trigger(indio_dev);
 
        if (!IS_ERR(st->reg)) {
                regulator_disable(st->reg);
index 64f7d41dc4538f9616591cd3c4dac8178ed3c5db..0e296d88412b99a563cc205a8a5bfa1935bfebb1 100644 (file)
@@ -41,6 +41,7 @@
 
 /* Mode Register Bit Designations (AD7793_REG_MODE) */
 #define AD7793_MODE_SEL(x)     (((x) & 0x7) << 13) /* Operation Mode Select */
+#define AD7793_MODE_SEL_MASK   (0x7 << 13) /* Operation Mode Select mask */
 #define AD7793_MODE_CLKSRC(x)  (((x) & 0x3) << 6) /* ADC Clock Source Select */
 #define AD7793_MODE_RATE(x)    ((x) & 0xF) /* Filter Update Rate Select */
 
@@ -70,6 +71,7 @@
 #define AD7793_CONF_REFSEL     (1 << 7) /* INT/EXT Reference Select */
 #define AD7793_CONF_BUF                (1 << 4) /* Buffered Mode Enable */
 #define AD7793_CONF_CHAN(x)    ((x) & 0x7) /* Channel select */
+#define AD7793_CONF_CHAN_MASK  0x7 /* Channel select mask */
 
 #define AD7793_CH_AIN1P_AIN1M  0 /* AIN1(+) - AIN1(-) */
 #define AD7793_CH_AIN2P_AIN2M  1 /* AIN2(+) - AIN2(-) */