iio: adc: ti-ads1015: Make use of device property API

Make use of device property API in this driver so that both OF based
system and ACPI based system can use this driver.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

diff --git a/drivers/iio/adc/ti-ads1015.c b/drivers/iio/adc/ti-ads1015.c
index 3b123b4..5ea4f45 100644 (file)
--- a/drivers/iio/adc/ti-ads1015.c
+++ b/drivers/iio/adc/ti-ads1015.c
@@ -12,10 +12,10 @@
  */
 
 #include <linux/module.h>
-#include <linux/of_device.h>
 #include <linux/init.h>
 #include <linux/irq.h>
 #include <linux/i2c.h>
+#include <linux/property.h>
 #include <linux/regmap.h>
 #include <linux/pm_runtime.h>
 #include <linux/mutex.h>
@@ -77,6 +77,7 @@
 #define ADS1015_DEFAULT_CHAN           0
 
 enum chip_ids {
+       ADSXXXX = 0,
        ADS1015,
        ADS1115,
 };
@@ -843,65 +844,58 @@ static const struct iio_info ads1115_info = {
        .attrs          = &ads1115_attribute_group,
 };
 
-#ifdef CONFIG_OF
-static int ads1015_get_channels_config_of(struct i2c_client *client)
+static int ads1015_client_get_channels_config(struct i2c_client *client)
 {
        struct iio_dev *indio_dev = i2c_get_clientdata(client);
        struct ads1015_data *data = iio_priv(indio_dev);
-       struct device_node *node;
+       struct device *dev = &client->dev;
+       struct fwnode_handle *node;
+       int i = -1;
 
-       if (!client->dev.of_node ||
-           !of_get_next_child(client->dev.of_node, NULL))
-               return -EINVAL;
-
-       for_each_child_of_node(client->dev.of_node, node) {
+       device_for_each_child_node(dev, node) {
                u32 pval;
                unsigned int channel;
                unsigned int pga = ADS1015_DEFAULT_PGA;
                unsigned int data_rate = ADS1015_DEFAULT_DATA_RATE;
 
-               if (of_property_read_u32(node, "reg", &pval)) {
-                       dev_err(&client->dev, "invalid reg on %pOF\n",
-                               node);
+               if (fwnode_property_read_u32(node, "reg", &pval)) {
+                       dev_err(dev, "invalid reg on %pfw\n", node);
                        continue;
                }
 
                channel = pval;
                if (channel >= ADS1015_CHANNELS) {
-                       dev_err(&client->dev,
-                               "invalid channel index %d on %pOF\n",
+                       dev_err(dev, "invalid channel index %d on %pfw\n",
                                channel, node);
                        continue;
                }
 
-               if (!of_property_read_u32(node, "ti,gain", &pval)) {
+               if (!fwnode_property_read_u32(node, "ti,gain", &pval)) {
                        pga = pval;
                        if (pga > 6) {
-                               dev_err(&client->dev, "invalid gain on %pOF\n",
-                                       node);
-                               of_node_put(node);
+                               dev_err(dev, "invalid gain on %pfw\n", node);
+                               fwnode_handle_put(node);
                                return -EINVAL;
                        }
                }
 
-               if (!of_property_read_u32(node, "ti,datarate", &pval)) {
+               if (!fwnode_property_read_u32(node, "ti,datarate", &pval)) {
                        data_rate = pval;
                        if (data_rate > 7) {
-                               dev_err(&client->dev,
-                                       "invalid data_rate on %pOF\n",
-                                       node);
-                               of_node_put(node);
+                               dev_err(dev, "invalid data_rate on %pfw\n", node);
+                               fwnode_handle_put(node);
                                return -EINVAL;
                        }
                }
 
                data->channel_data[channel].pga = pga;
                data->channel_data[channel].data_rate = data_rate;
+
+               i++;
        }
 
-       return 0;
+       return i < 0 ? -EINVAL : 0;
 }
-#endif
 
 static void ads1015_get_channels_config(struct i2c_client *client)
 {
@@ -910,10 +904,9 @@ static void ads1015_get_channels_config(struct i2c_client *client)
        struct iio_dev *indio_dev = i2c_get_clientdata(client);
        struct ads1015_data *data = iio_priv(indio_dev);
 
-#ifdef CONFIG_OF
-       if (!ads1015_get_channels_config_of(client))
+       if (!ads1015_client_get_channels_config(client))
                return;
-#endif
+
        /* fallback on default configuration */
        for (k = 0; k < ADS1015_CHANNELS; ++k) {
                data->channel_data[k].pga = ADS1015_DEFAULT_PGA;
@@ -951,9 +944,8 @@ static int ads1015_probe(struct i2c_client *client,
        indio_dev->name = ADS1015_DRV_NAME;
        indio_dev->modes = INDIO_DIRECT_MODE;
 
-       if (client->dev.of_node)
-               chip = (enum chip_ids)of_device_get_match_data(&client->dev);
-       else
+       chip = (enum chip_ids)device_get_match_data(&client->dev);
+       if (chip == ADSXXXX)
                chip = id->driver_data;
        switch (chip) {
        case ADS1015:
@@ -968,6 +960,9 @@ static int ads1015_probe(struct i2c_client *client,
                indio_dev->info = &ads1115_info;
                data->data_rate = (unsigned int *) &ads1115_data_rate;
                break;
+       default:
+               dev_err(&client->dev, "Unknown chip %d\n", chip);
+               return -EINVAL;
        }
 
        data->event_channel = ADS1015_CHANNELS;