Merge tag 'iio-fixes-for-5.2b' of git://git.kernel.org/pub/scm/linux/kernel/git/jic23...
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 17 Jun 2019 20:28:29 +0000 (22:28 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 17 Jun 2019 20:28:29 +0000 (22:28 +0200)
Jonathan writes:

Second set of IIO fixes for the 5.2 cycle.

* ad7150
  - sense of bit for controlling adaptive vs fixed threshold was flipped.
* adt7316
  - Fix a build issue due to wrong headers for gpio usage.
* lsm6dsx
  - correctly suspend / resume i2c slaves when the host goes to sleep.
* mlx90632
  - relax a compatability check to allow for newer devices.

Also one counters fix

* counter/ftm-quaddec
  - missing dependencies in Kconfig.

* tag 'iio-fixes-for-5.2b' of git://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio:
  counter/ftm-quaddec: Add missing dependencies in Kconfig
  staging: iio: adt7316: Fix build errors when GPIOLIB is not set
  iio: temperature: mlx90632 Relax the compatibility check
  iio: imu: st_lsm6dsx: fix PM support for st_lsm6dsx i2c controller
  staging:iio:ad7150: fix threshold mode config bit

drivers/counter/Kconfig
drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h
drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c
drivers/iio/temperature/mlx90632.c
drivers/staging/iio/addac/adt7316.c
drivers/staging/iio/cdc/ad7150.c

index 138ecd8..2967d0a 100644 (file)
@@ -51,6 +51,7 @@ config STM32_LPTIMER_CNT
 
 config FTM_QUADDEC
        tristate "Flex Timer Module Quadrature decoder driver"
+       depends on HAS_IOMEM && OF
        help
          Select this option to enable the Flex Timer Quadrature decoder
          driver.
index 72f7205..5e461f0 100644 (file)
@@ -270,6 +270,7 @@ struct st_lsm6dsx_sensor {
  * @conf_lock: Mutex to prevent concurrent FIFO configuration update.
  * @page_lock: Mutex to prevent concurrent memory page configuration.
  * @fifo_mode: FIFO operating mode supported by the device.
+ * @suspend_mask: Suspended sensor bitmask.
  * @enable_mask: Enabled sensor bitmask.
  * @ts_sip: Total number of timestamp samples in a given pattern.
  * @sip: Total number of samples (acc/gyro/ts) in a given pattern.
@@ -287,6 +288,7 @@ struct st_lsm6dsx_hw {
        struct mutex page_lock;
 
        enum st_lsm6dsx_fifo_mode fifo_mode;
+       u8 suspend_mask;
        u8 enable_mask;
        u8 ts_sip;
        u8 sip;
index b6edc98..fd95d92 100644 (file)
@@ -1109,8 +1109,6 @@ static int __maybe_unused st_lsm6dsx_suspend(struct device *dev)
 {
        struct st_lsm6dsx_hw *hw = dev_get_drvdata(dev);
        struct st_lsm6dsx_sensor *sensor;
-       const struct st_lsm6dsx_reg *reg;
-       unsigned int data;
        int i, err = 0;
 
        for (i = 0; i < ST_LSM6DSX_ID_MAX; i++) {
@@ -1121,12 +1119,16 @@ static int __maybe_unused st_lsm6dsx_suspend(struct device *dev)
                if (!(hw->enable_mask & BIT(sensor->id)))
                        continue;
 
-               reg = &st_lsm6dsx_odr_table[sensor->id].reg;
-               data = ST_LSM6DSX_SHIFT_VAL(0, reg->mask);
-               err = st_lsm6dsx_update_bits_locked(hw, reg->addr, reg->mask,
-                                                   data);
+               if (sensor->id == ST_LSM6DSX_ID_EXT0 ||
+                   sensor->id == ST_LSM6DSX_ID_EXT1 ||
+                   sensor->id == ST_LSM6DSX_ID_EXT2)
+                       err = st_lsm6dsx_shub_set_enable(sensor, false);
+               else
+                       err = st_lsm6dsx_sensor_set_enable(sensor, false);
                if (err < 0)
                        return err;
+
+               hw->suspend_mask |= BIT(sensor->id);
        }
 
        if (hw->fifo_mode != ST_LSM6DSX_FIFO_BYPASS)
@@ -1146,12 +1148,19 @@ static int __maybe_unused st_lsm6dsx_resume(struct device *dev)
                        continue;
 
                sensor = iio_priv(hw->iio_devs[i]);
-               if (!(hw->enable_mask & BIT(sensor->id)))
+               if (!(hw->suspend_mask & BIT(sensor->id)))
                        continue;
 
-               err = st_lsm6dsx_set_odr(sensor, sensor->odr);
+               if (sensor->id == ST_LSM6DSX_ID_EXT0 ||
+                   sensor->id == ST_LSM6DSX_ID_EXT1 ||
+                   sensor->id == ST_LSM6DSX_ID_EXT2)
+                       err = st_lsm6dsx_shub_set_enable(sensor, true);
+               else
+                       err = st_lsm6dsx_sensor_set_enable(sensor, true);
                if (err < 0)
                        return err;
+
+               hw->suspend_mask &= ~BIT(sensor->id);
        }
 
        if (hw->enable_mask)
index be03be7..eaca6ba 100644 (file)
@@ -81,6 +81,8 @@
 /* Magic constants */
 #define MLX90632_ID_MEDICAL    0x0105 /* EEPROM DSPv5 Medical device id */
 #define MLX90632_ID_CONSUMER   0x0205 /* EEPROM DSPv5 Consumer device id */
+#define MLX90632_DSP_VERSION   5 /* DSP version */
+#define MLX90632_DSP_MASK      GENMASK(7, 0) /* DSP version in EE_VERSION */
 #define MLX90632_RESET_CMD     0x0006 /* Reset sensor (address or global) */
 #define MLX90632_REF_12                12LL /**< ResCtrlRef value of Ch 1 or Ch 2 */
 #define MLX90632_REF_3         12LL /**< ResCtrlRef value of Channel 3 */
@@ -667,10 +669,13 @@ static int mlx90632_probe(struct i2c_client *client,
        } else if (read == MLX90632_ID_CONSUMER) {
                dev_dbg(&client->dev,
                        "Detected Consumer EEPROM calibration %x\n", read);
+       } else if ((read & MLX90632_DSP_MASK) == MLX90632_DSP_VERSION) {
+               dev_dbg(&client->dev,
+                       "Detected Unknown EEPROM calibration %x\n", read);      
        } else {
                dev_err(&client->dev,
-                       "EEPROM version mismatch %x (expected %x or %x)\n",
-                       read, MLX90632_ID_CONSUMER, MLX90632_ID_MEDICAL);
+                       "Wrong DSP version %x (expected %x)\n",
+                       read, MLX90632_DSP_VERSION);
                return -EPROTONOSUPPORT;
        }
 
index b6a65ee..dc8c25d 100644 (file)
@@ -6,7 +6,8 @@
  */
 
 #include <linux/interrupt.h>
-#include <linux/gpio.h>
+#include <linux/gpio/consumer.h>
+#include <linux/irq.h>
 #include <linux/workqueue.h>
 #include <linux/device.h>
 #include <linux/kernel.h>
index dd7fcab..e075244 100644 (file)
@@ -5,6 +5,7 @@
  * Copyright 2010-2011 Analog Devices Inc.
  */
 
+#include <linux/bitfield.h>
 #include <linux/interrupt.h>
 #include <linux/device.h>
 #include <linux/kernel.h>
@@ -130,7 +131,7 @@ static int ad7150_read_event_config(struct iio_dev *indio_dev,
 {
        int ret;
        u8 threshtype;
-       bool adaptive;
+       bool thrfixed;
        struct ad7150_chip_info *chip = iio_priv(indio_dev);
 
        ret = i2c_smbus_read_byte_data(chip->client, AD7150_CFG);
@@ -138,21 +139,23 @@ static int ad7150_read_event_config(struct iio_dev *indio_dev,
                return ret;
 
        threshtype = (ret >> 5) & 0x03;
-       adaptive = !!(ret & 0x80);
+
+       /*check if threshold mode is fixed or adaptive*/
+       thrfixed = FIELD_GET(AD7150_CFG_FIX, ret);
 
        switch (type) {
        case IIO_EV_TYPE_MAG_ADAPTIVE:
                if (dir == IIO_EV_DIR_RISING)
-                       return adaptive && (threshtype == 0x1);
-               return adaptive && (threshtype == 0x0);
+                       return !thrfixed && (threshtype == 0x1);
+               return !thrfixed && (threshtype == 0x0);
        case IIO_EV_TYPE_THRESH_ADAPTIVE:
                if (dir == IIO_EV_DIR_RISING)
-                       return adaptive && (threshtype == 0x3);
-               return adaptive && (threshtype == 0x2);
+                       return !thrfixed && (threshtype == 0x3);
+               return !thrfixed && (threshtype == 0x2);
        case IIO_EV_TYPE_THRESH:
                if (dir == IIO_EV_DIR_RISING)
-                       return !adaptive && (threshtype == 0x1);
-               return !adaptive && (threshtype == 0x0);
+                       return thrfixed && (threshtype == 0x1);
+               return thrfixed && (threshtype == 0x0);
        default:
                break;
        }