staging:iio:ad7606: Use GPIO descriptor API
[linux-2.6-block.git] / drivers / staging / iio / adc / ad7606_core.c
index f79ee61851f6024994e9fe201c2ffde1755e3c0f..4ef6cb4a27262f9e09c8de794b8fc96eca44de63 100644 (file)
@@ -13,7 +13,7 @@
 #include <linux/sysfs.h>
 #include <linux/regulator/consumer.h>
 #include <linux/err.h>
-#include <linux/gpio.h>
+#include <linux/gpio/consumer.h>
 #include <linux/delay.h>
 #include <linux/sched.h>
 #include <linux/module.h>
 
 int ad7606_reset(struct ad7606_state *st)
 {
-       if (gpio_is_valid(st->pdata->gpio_reset)) {
-               gpio_set_value(st->pdata->gpio_reset, 1);
+       if (st->gpio_reset) {
+               gpiod_set_value(st->gpio_reset, 1);
                ndelay(100); /* t_reset >= 100ns */
-               gpio_set_value(st->pdata->gpio_reset, 0);
+               gpiod_set_value(st->gpio_reset, 0);
                return 0;
        }
 
        return -ENODEV;
 }
 
+int ad7606_read_samples(struct ad7606_state *st)
+{
+       unsigned int num = st->chip_info->num_channels;
+       u16 *data = st->data;
+       int ret;
+
+       /*
+        * The frstdata signal is set to high while and after reading the sample
+        * of the first channel and low for all other channels. This can be used
+        * to check that the incoming data is correctly aligned. During normal
+        * operation the data should never become unaligned, but some glitch or
+        * electrostatic discharge might cause an extra read or clock cycle.
+        * Monitoring the frstdata signal allows to recover from such failure
+        * situations.
+        */
+
+       if (st->gpio_frstdata) {
+               ret = st->bops->read_block(st->dev, 1, data);
+               if (ret)
+                       return ret;
+
+               if (!gpiod_get_value(st->gpio_frstdata)) {
+                       ad7606_reset(st);
+                       return -EIO;
+               }
+
+               data++;
+               num--;
+       }
+
+       return st->bops->read_block(st->dev, num, data);
+}
+
 static int ad7606_scan_direct(struct iio_dev *indio_dev, unsigned int ch)
 {
        struct ad7606_state *st = iio_priv(indio_dev);
        int ret;
 
        st->done = false;
-       gpio_set_value(st->pdata->gpio_convst, 1);
+       gpiod_set_value(st->gpio_convst, 1);
 
        ret = wait_event_interruptible(st->wq_data_avail, st->done);
        if (ret)
                goto error_ret;
 
-       if (gpio_is_valid(st->pdata->gpio_frstdata)) {
-               ret = st->bops->read_block(st->dev, 1, st->data);
-               if (ret)
-                       goto error_ret;
-               if (!gpio_get_value(st->pdata->gpio_frstdata)) {
-                       /* This should never happen */
-                       ad7606_reset(st);
-                       ret = -EIO;
-                       goto error_ret;
-               }
-               ret = st->bops->read_block(st->dev,
-                       st->chip_info->num_channels - 1, &st->data[1]);
-               if (ret)
-                       goto error_ret;
-       } else {
-               ret = st->bops->read_block(st->dev,
-                       st->chip_info->num_channels, st->data);
-               if (ret)
-                       goto error_ret;
-       }
-
-       ret = st->data[ch];
+       ret = ad7606_read_samples(st);
+       if (ret == 0)
+               ret = st->data[ch];
 
 error_ret:
-       gpio_set_value(st->pdata->gpio_convst, 0);
+       gpiod_set_value(st->gpio_convst, 0);
 
        return ret;
 }
@@ -103,6 +117,9 @@ static int ad7606_read_raw(struct iio_dev *indio_dev,
                *val = st->range * 2;
                *val2 = st->chip_info->channels[0].scan_type.realbits;
                return IIO_VAL_FRACTIONAL_LOG2;
+       case IIO_CHAN_INFO_OVERSAMPLING_RATIO:
+               *val = st->oversampling;
+               return IIO_VAL_INT;
        }
        return -EINVAL;
 }
@@ -129,12 +146,11 @@ static ssize_t ad7606_store_range(struct device *dev,
        if (ret)
                return ret;
 
-       if (!(lval == 5000 || lval == 10000)) {
-               dev_err(dev, "range is not supported\n");
+       if (!(lval == 5000 || lval == 10000))
                return -EINVAL;
-       }
+
        mutex_lock(&indio_dev->mlock);
-       gpio_set_value(st->pdata->gpio_range, lval == 10000);
+       gpiod_set_value(st->gpio_range, lval == 10000);
        st->range = lval;
        mutex_unlock(&indio_dev->mlock);
 
@@ -145,19 +161,9 @@ static IIO_DEVICE_ATTR(in_voltage_range, S_IRUGO | S_IWUSR,
                       ad7606_show_range, ad7606_store_range, 0);
 static IIO_CONST_ATTR(in_voltage_range_available, "5000 10000");
 
-static ssize_t ad7606_show_oversampling_ratio(struct device *dev,
-                                             struct device_attribute *attr,
-                                             char *buf)
-{
-       struct iio_dev *indio_dev = dev_to_iio_dev(dev);
-       struct ad7606_state *st = iio_priv(indio_dev);
-
-       return sprintf(buf, "%u\n", st->oversampling);
-}
-
 static int ad7606_oversampling_get_index(unsigned int val)
 {
-       unsigned char supported[] = {0, 2, 4, 8, 16, 32, 64};
+       unsigned char supported[] = {1, 2, 4, 8, 16, 32, 64};
        int i;
 
        for (i = 0; i < ARRAY_SIZE(supported); i++)
@@ -167,44 +173,45 @@ static int ad7606_oversampling_get_index(unsigned int val)
        return -EINVAL;
 }
 
-static ssize_t ad7606_store_oversampling_ratio(struct device *dev,
-                                              struct device_attribute *attr,
-                                              const char *buf, size_t count)
+static int ad7606_write_raw(struct iio_dev *indio_dev,
+                           struct iio_chan_spec const *chan,
+                           int val,
+                           int val2,
+                           long mask)
 {
-       struct iio_dev *indio_dev = dev_to_iio_dev(dev);
        struct ad7606_state *st = iio_priv(indio_dev);
-       unsigned long lval;
+       int values[3];
        int ret;
 
-       ret = kstrtoul(buf, 10, &lval);
-       if (ret)
-               return ret;
+       switch (mask) {
+       case IIO_CHAN_INFO_OVERSAMPLING_RATIO:
+               if (val2)
+                       return -EINVAL;
+               ret = ad7606_oversampling_get_index(val);
+               if (ret < 0)
+                       return ret;
 
-       ret = ad7606_oversampling_get_index(lval);
-       if (ret < 0) {
-               dev_err(dev, "oversampling %lu is not supported\n", lval);
-               return ret;
-       }
+               values[0] = (ret >> 0) & 1;
+               values[1] = (ret >> 1) & 1;
+               values[2] = (ret >> 2) & 1;
 
-       mutex_lock(&indio_dev->mlock);
-       gpio_set_value(st->pdata->gpio_os0, (ret >> 0) & 1);
-       gpio_set_value(st->pdata->gpio_os1, (ret >> 1) & 1);
-       gpio_set_value(st->pdata->gpio_os1, (ret >> 2) & 1);
-       st->oversampling = lval;
-       mutex_unlock(&indio_dev->mlock);
+               mutex_lock(&indio_dev->mlock);
+               gpiod_set_array_value(ARRAY_SIZE(values), st->gpio_os->desc,
+                                     values);
+               st->oversampling = val;
+               mutex_unlock(&indio_dev->mlock);
 
-       return count;
+               return 0;
+       default:
+               return -EINVAL;
+       }
 }
 
-static IIO_DEVICE_ATTR(oversampling_ratio, S_IRUGO | S_IWUSR,
-                      ad7606_show_oversampling_ratio,
-                      ad7606_store_oversampling_ratio, 0);
-static IIO_CONST_ATTR(oversampling_ratio_available, "0 2 4 8 16 32 64");
+static IIO_CONST_ATTR(oversampling_ratio_available, "1 2 4 8 16 32 64");
 
 static struct attribute *ad7606_attributes_os_and_range[] = {
        &iio_dev_attr_in_voltage_range.dev_attr.attr,
        &iio_const_attr_in_voltage_range_available.dev_attr.attr,
-       &iio_dev_attr_oversampling_ratio.dev_attr.attr,
        &iio_const_attr_oversampling_ratio_available.dev_attr.attr,
        NULL,
 };
@@ -214,7 +221,6 @@ static const struct attribute_group ad7606_attribute_group_os_and_range = {
 };
 
 static struct attribute *ad7606_attributes_os[] = {
-       &iio_dev_attr_oversampling_ratio.dev_attr.attr,
        &iio_const_attr_oversampling_ratio_available.dev_attr.attr,
        NULL,
 };
@@ -241,6 +247,8 @@ static const struct attribute_group ad7606_attribute_group_range = {
                .address = num,                                 \
                .info_mask_separate = BIT(IIO_CHAN_INFO_RAW),   \
                .info_mask_shared_by_type = BIT(IIO_CHAN_INFO_SCALE),\
+               .info_mask_shared_by_all =                      \
+                       BIT(IIO_CHAN_INFO_OVERSAMPLING_RATIO),  \
                .scan_index = num,                              \
                .scan_type = {                                  \
                        .sign = 's',                            \
@@ -267,20 +275,14 @@ static const struct ad7606_chip_info ad7606_chip_info_tbl[] = {
         * More devices added in future
         */
        [ID_AD7606_8] = {
-               .name = "ad7606",
-               .int_vref_mv = 2500,
                .channels = ad7606_channels,
                .num_channels = 9,
        },
        [ID_AD7606_6] = {
-               .name = "ad7606-6",
-               .int_vref_mv = 2500,
                .channels = ad7606_channels,
                .num_channels = 7,
        },
        [ID_AD7606_4] = {
-               .name = "ad7606-4",
-               .int_vref_mv = 2500,
                .channels = ad7606_channels,
                .num_channels = 5,
        },
@@ -288,119 +290,37 @@ static const struct ad7606_chip_info ad7606_chip_info_tbl[] = {
 
 static int ad7606_request_gpios(struct ad7606_state *st)
 {
-       struct gpio gpio_array[3] = {
-               [0] = {
-                       .gpio =  st->pdata->gpio_os0,
-                       .flags = GPIOF_DIR_OUT | ((st->oversampling & 1) ?
-                                GPIOF_INIT_HIGH : GPIOF_INIT_LOW),
-                       .label = "AD7606_OS0",
-               },
-               [1] = {
-                       .gpio =  st->pdata->gpio_os1,
-                       .flags = GPIOF_DIR_OUT | ((st->oversampling & 2) ?
-                                GPIOF_INIT_HIGH : GPIOF_INIT_LOW),
-                       .label = "AD7606_OS1",
-               },
-               [2] = {
-                       .gpio =  st->pdata->gpio_os2,
-                       .flags = GPIOF_DIR_OUT | ((st->oversampling & 4) ?
-                                GPIOF_INIT_HIGH : GPIOF_INIT_LOW),
-                       .label = "AD7606_OS2",
-               },
-       };
-       int ret;
+       struct device *dev = st->dev;
 
-       if (gpio_is_valid(st->pdata->gpio_convst)) {
-               ret = gpio_request_one(st->pdata->gpio_convst,
-                                      GPIOF_OUT_INIT_LOW,
-                                      "AD7606_CONVST");
-               if (ret) {
-                       dev_err(st->dev, "failed to request GPIO CONVST\n");
-                       goto error_ret;
-               }
-       } else {
-               ret = -EIO;
-               goto error_ret;
-       }
+       st->gpio_convst = devm_gpiod_get(dev, "conversion-start",
+                                        GPIOD_OUT_LOW);
+       if (IS_ERR(st->gpio_convst))
+               return PTR_ERR(st->gpio_convst);
 
-       if (gpio_is_valid(st->pdata->gpio_os0) &&
-           gpio_is_valid(st->pdata->gpio_os1) &&
-           gpio_is_valid(st->pdata->gpio_os2)) {
-               ret = gpio_request_array(gpio_array, ARRAY_SIZE(gpio_array));
-               if (ret < 0)
-                       goto error_free_convst;
-       }
+       st->gpio_reset = devm_gpiod_get_optional(dev, "reset", GPIOD_OUT_LOW);
+       if (IS_ERR(st->gpio_reset))
+               return PTR_ERR(st->gpio_reset);
 
-       if (gpio_is_valid(st->pdata->gpio_reset)) {
-               ret = gpio_request_one(st->pdata->gpio_reset,
-                                      GPIOF_OUT_INIT_LOW,
-                                      "AD7606_RESET");
-               if (ret < 0)
-                       goto error_free_os;
-       }
+       st->gpio_range = devm_gpiod_get_optional(dev, "range", GPIOD_OUT_LOW);
+       if (IS_ERR(st->gpio_range))
+               return PTR_ERR(st->gpio_range);
 
-       if (gpio_is_valid(st->pdata->gpio_range)) {
-               ret = gpio_request_one(st->pdata->gpio_range, GPIOF_DIR_OUT |
-                                      ((st->range == 10000) ? GPIOF_INIT_HIGH :
-                                       GPIOF_INIT_LOW), "AD7606_RANGE");
-               if (ret < 0)
-                       goto error_free_reset;
-       }
-       if (gpio_is_valid(st->pdata->gpio_stby)) {
-               ret = gpio_request_one(st->pdata->gpio_stby,
-                                      GPIOF_OUT_INIT_HIGH,
-                                      "AD7606_STBY");
-               if (ret < 0)
-                       goto error_free_range;
-       }
+       st->gpio_standby = devm_gpiod_get_optional(dev, "standby",
+                                                  GPIOD_OUT_HIGH);
+       if (IS_ERR(st->gpio_standby))
+               return PTR_ERR(st->gpio_standby);
 
-       if (gpio_is_valid(st->pdata->gpio_frstdata)) {
-               ret = gpio_request_one(st->pdata->gpio_frstdata, GPIOF_IN,
-                                      "AD7606_FRSTDATA");
-               if (ret < 0)
-                       goto error_free_stby;
-       }
+       st->gpio_frstdata = devm_gpiod_get_optional(dev, "first-data",
+                                                   GPIOD_IN);
+       if (IS_ERR(st->gpio_frstdata))
+               return PTR_ERR(st->gpio_frstdata);
 
-       return 0;
+       st->gpio_os = devm_gpiod_get_array_optional(dev, "oversampling-ratio",
+                       GPIOD_OUT_LOW);
+       if (IS_ERR(st->gpio_os))
+               return PTR_ERR(st->gpio_os);
 
-error_free_stby:
-       if (gpio_is_valid(st->pdata->gpio_stby))
-               gpio_free(st->pdata->gpio_stby);
-error_free_range:
-       if (gpio_is_valid(st->pdata->gpio_range))
-               gpio_free(st->pdata->gpio_range);
-error_free_reset:
-       if (gpio_is_valid(st->pdata->gpio_reset))
-               gpio_free(st->pdata->gpio_reset);
-error_free_os:
-       if (gpio_is_valid(st->pdata->gpio_os0) &&
-           gpio_is_valid(st->pdata->gpio_os1) &&
-           gpio_is_valid(st->pdata->gpio_os2))
-               gpio_free_array(gpio_array, ARRAY_SIZE(gpio_array));
-error_free_convst:
-       gpio_free(st->pdata->gpio_convst);
-error_ret:
-       return ret;
-}
-
-static void ad7606_free_gpios(struct ad7606_state *st)
-{
-       if (gpio_is_valid(st->pdata->gpio_frstdata))
-               gpio_free(st->pdata->gpio_frstdata);
-       if (gpio_is_valid(st->pdata->gpio_stby))
-               gpio_free(st->pdata->gpio_stby);
-       if (gpio_is_valid(st->pdata->gpio_range))
-               gpio_free(st->pdata->gpio_range);
-       if (gpio_is_valid(st->pdata->gpio_reset))
-               gpio_free(st->pdata->gpio_reset);
-       if (gpio_is_valid(st->pdata->gpio_os0) &&
-           gpio_is_valid(st->pdata->gpio_os1) &&
-           gpio_is_valid(st->pdata->gpio_os2)) {
-               gpio_free(st->pdata->gpio_os2);
-               gpio_free(st->pdata->gpio_os1);
-               gpio_free(st->pdata->gpio_os0);
-       }
-       gpio_free(st->pdata->gpio_convst);
+       return 0;
 }
 
 /**
@@ -429,12 +349,14 @@ static const struct iio_info ad7606_info_no_os_or_range = {
 static const struct iio_info ad7606_info_os_and_range = {
        .driver_module = THIS_MODULE,
        .read_raw = &ad7606_read_raw,
+       .write_raw = &ad7606_write_raw,
        .attrs = &ad7606_attribute_group_os_and_range,
 };
 
 static const struct iio_info ad7606_info_os = {
        .driver_module = THIS_MODULE,
        .read_raw = &ad7606_read_raw,
+       .write_raw = &ad7606_write_raw,
        .attrs = &ad7606_attribute_group_os,
 };
 
@@ -444,79 +366,66 @@ static const struct iio_info ad7606_info_range = {
        .attrs = &ad7606_attribute_group_range,
 };
 
-struct iio_dev *ad7606_probe(struct device *dev, int irq,
-                            void __iomem *base_address,
-                            unsigned int id,
-                            const struct ad7606_bus_ops *bops)
+int ad7606_probe(struct device *dev, int irq, void __iomem *base_address,
+                const char *name, unsigned int id,
+                const struct ad7606_bus_ops *bops)
 {
-       struct ad7606_platform_data *pdata = dev->platform_data;
        struct ad7606_state *st;
        int ret;
        struct iio_dev *indio_dev;
 
        indio_dev = devm_iio_device_alloc(dev, sizeof(*st));
        if (!indio_dev)
-               return ERR_PTR(-ENOMEM);
+               return -ENOMEM;
 
        st = iio_priv(indio_dev);
 
        st->dev = dev;
        st->bops = bops;
        st->base_address = base_address;
-       st->range = pdata->default_range == 10000 ? 10000 : 5000;
-
-       ret = ad7606_oversampling_get_index(pdata->default_os);
-       if (ret < 0) {
-               dev_warn(dev, "oversampling %d is not supported\n",
-                        pdata->default_os);
-               st->oversampling = 0;
-       } else {
-               st->oversampling = pdata->default_os;
-       }
+       st->range = 5000;
+       st->oversampling = 1;
 
        st->reg = devm_regulator_get(dev, "vcc");
        if (!IS_ERR(st->reg)) {
                ret = regulator_enable(st->reg);
                if (ret)
-                       return ERR_PTR(ret);
+                       return ret;
        }
 
-       st->pdata = pdata;
+       ret = ad7606_request_gpios(st);
+       if (ret)
+               goto error_disable_reg;
+
        st->chip_info = &ad7606_chip_info_tbl[id];
 
        indio_dev->dev.parent = dev;
-       if (gpio_is_valid(st->pdata->gpio_os0) &&
-           gpio_is_valid(st->pdata->gpio_os1) &&
-           gpio_is_valid(st->pdata->gpio_os2)) {
-               if (gpio_is_valid(st->pdata->gpio_range))
+       if (st->gpio_os) {
+               if (st->gpio_range)
                        indio_dev->info = &ad7606_info_os_and_range;
                else
                        indio_dev->info = &ad7606_info_os;
        } else {
-               if (gpio_is_valid(st->pdata->gpio_range))
+               if (st->gpio_range)
                        indio_dev->info = &ad7606_info_range;
                else
                        indio_dev->info = &ad7606_info_no_os_or_range;
        }
        indio_dev->modes = INDIO_DIRECT_MODE;
-       indio_dev->name = st->chip_info->name;
+       indio_dev->name = name;
        indio_dev->channels = st->chip_info->channels;
        indio_dev->num_channels = st->chip_info->num_channels;
 
        init_waitqueue_head(&st->wq_data_avail);
 
-       ret = ad7606_request_gpios(st);
-       if (ret)
-               goto error_disable_reg;
-
        ret = ad7606_reset(st);
        if (ret)
                dev_warn(st->dev, "failed to RESET: no RESET GPIO specified\n");
 
-       ret = request_irq(irq, ad7606_interrupt,
-                         IRQF_TRIGGER_FALLING, st->chip_info->name, indio_dev);
+       ret = request_irq(irq, ad7606_interrupt, IRQF_TRIGGER_FALLING, name,
+                         indio_dev);
        if (ret)
-               goto error_free_gpios;
+               goto error_disable_reg;
 
        ret = ad7606_register_ring_funcs_and_init(indio_dev);
        if (ret)
@@ -526,25 +435,25 @@ struct iio_dev *ad7606_probe(struct device *dev, int irq,
        if (ret)
                goto error_unregister_ring;
 
-       return indio_dev;
+       dev_set_drvdata(dev, indio_dev);
+
+       return 0;
 error_unregister_ring:
        ad7606_ring_cleanup(indio_dev);
 
 error_free_irq:
        free_irq(irq, indio_dev);
 
-error_free_gpios:
-       ad7606_free_gpios(st);
-
 error_disable_reg:
        if (!IS_ERR(st->reg))
                regulator_disable(st->reg);
-       return ERR_PTR(ret);
+       return ret;
 }
 EXPORT_SYMBOL_GPL(ad7606_probe);
 
-int ad7606_remove(struct iio_dev *indio_dev, int irq)
+int ad7606_remove(struct device *dev, int irq)
 {
+       struct iio_dev *indio_dev = dev_get_drvdata(dev);
        struct ad7606_state *st = iio_priv(indio_dev);
 
        iio_device_unregister(indio_dev);
@@ -554,8 +463,6 @@ int ad7606_remove(struct iio_dev *indio_dev, int irq)
        if (!IS_ERR(st->reg))
                regulator_disable(st->reg);
 
-       ad7606_free_gpios(st);
-
        return 0;
 }
 EXPORT_SYMBOL_GPL(ad7606_remove);
@@ -567,10 +474,9 @@ static int ad7606_suspend(struct device *dev)
        struct iio_dev *indio_dev = dev_get_drvdata(dev);
        struct ad7606_state *st = iio_priv(indio_dev);
 
-       if (gpio_is_valid(st->pdata->gpio_stby)) {
-               if (gpio_is_valid(st->pdata->gpio_range))
-                       gpio_set_value(st->pdata->gpio_range, 1);
-               gpio_set_value(st->pdata->gpio_stby, 0);
+       if (st->gpio_standby) {
+               gpiod_set_value(st->gpio_range, 1);
+               gpiod_set_value(st->gpio_standby, 0);
        }
 
        return 0;
@@ -581,12 +487,9 @@ static int ad7606_resume(struct device *dev)
        struct iio_dev *indio_dev = dev_get_drvdata(dev);
        struct ad7606_state *st = iio_priv(indio_dev);
 
-       if (gpio_is_valid(st->pdata->gpio_stby)) {
-               if (gpio_is_valid(st->pdata->gpio_range))
-                       gpio_set_value(st->pdata->gpio_range,
-                                      st->range == 10000);
-
-               gpio_set_value(st->pdata->gpio_stby, 1);
+       if (st->gpio_standby) {
+               gpiod_set_value(st->gpio_range, st->range == 10000);
+               gpiod_set_value(st->gpio_standby, 1);
                ad7606_reset(st);
        }