iio:pressure: device settings are set immediately during probe
authorDenis Ciocca <denis.ciocca@st.com>
Thu, 18 Jul 2019 22:53:51 +0000 (15:53 -0700)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Sat, 27 Jul 2019 21:52:05 +0000 (22:52 +0100)
This patch set pressure settings right after probe start. This is
done in preparation of regmap that needs different configuration
based on multiread bit value.

Signed-off-by: Denis Ciocca <denis.ciocca@st.com>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/pressure/st_pressure_i2c.c
drivers/iio/pressure/st_pressure_spi.c

index b7d9ba706abc52ce0c6d05abd1ae5b3eb6034764..466e7dde5eae234130618df4f7ce48a2553e1158 100644 (file)
@@ -78,18 +78,13 @@ static const struct i2c_device_id st_press_id_table[] = {
 MODULE_DEVICE_TABLE(i2c, st_press_id_table);
 
 static int st_press_i2c_probe(struct i2c_client *client,
-                                               const struct i2c_device_id *id)
+                             const struct i2c_device_id *id)
 {
-       struct iio_dev *indio_dev;
+       const struct st_sensor_settings *settings;
        struct st_sensor_data *press_data;
+       struct iio_dev *indio_dev;
        int ret;
 
-       indio_dev = devm_iio_device_alloc(&client->dev, sizeof(*press_data));
-       if (!indio_dev)
-               return -ENOMEM;
-
-       press_data = iio_priv(indio_dev);
-
        if (client->dev.of_node) {
                st_sensors_of_name_probe(&client->dev, st_press_of_match,
                                         client->name, sizeof(client->name));
@@ -99,10 +94,24 @@ static int st_press_i2c_probe(struct i2c_client *client,
                        return -ENODEV;
 
                strlcpy(client->name, st_press_id_table[ret].name,
-                               sizeof(client->name));
+                       sizeof(client->name));
        } else if (!id)
                return -ENODEV;
 
+       settings = st_press_get_settings(client->name);
+       if (!settings) {
+               dev_err(&client->dev, "device name %s not recognized.\n",
+                       client->name);
+               return -ENODEV;
+       }
+
+       indio_dev = devm_iio_device_alloc(&client->dev, sizeof(*press_data));
+       if (!indio_dev)
+               return -ENOMEM;
+
+       press_data = iio_priv(indio_dev);
+       press_data->sensor_settings = (struct st_sensor_settings *)settings;
+
        st_sensors_i2c_configure(indio_dev, client, press_data);
 
        ret = st_press_common_probe(indio_dev);
index ef61401c41d396318ed02c4ca849402149265574..3e8c1ffe001e3f2b332afd775f31cf2afed889cd 100644 (file)
@@ -61,18 +61,28 @@ MODULE_DEVICE_TABLE(of, st_press_of_match);
 
 static int st_press_spi_probe(struct spi_device *spi)
 {
-       struct iio_dev *indio_dev;
+       const struct st_sensor_settings *settings;
        struct st_sensor_data *press_data;
+       struct iio_dev *indio_dev;
        int err;
 
+       st_sensors_of_name_probe(&spi->dev, st_press_of_match,
+                                spi->modalias, sizeof(spi->modalias));
+
+       settings = st_press_get_settings(spi->modalias);
+       if (!settings) {
+               dev_err(&spi->dev, "device name %s not recognized.\n",
+                       spi->modalias);
+               return -ENODEV;
+       }
+
        indio_dev = devm_iio_device_alloc(&spi->dev, sizeof(*press_data));
-       if (indio_dev == NULL)
+       if (!indio_dev)
                return -ENOMEM;
 
        press_data = iio_priv(indio_dev);
+       press_data->sensor_settings = (struct st_sensor_settings *)settings;
 
-       st_sensors_of_name_probe(&spi->dev, st_press_of_match,
-                                spi->modalias, sizeof(spi->modalias));
        st_sensors_spi_configure(indio_dev, spi, press_data);
 
        err = st_press_common_probe(indio_dev);