iio: accel: bma180: Add support for bma023

The bma023 chip is similar enough to the bma180 and bma25x that the
same driver can support all of them.  The biggest differences are
the lack of a temperature channel and no low power but still working
mode.

The bma150 is a close relative of the bma023, but it does have a
temperature channel so support is not added for it.

Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Jonathan Bakker <xc-racer2@live.ca>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

diff --git a/drivers/iio/accel/Kconfig b/drivers/iio/accel/Kconfig
index 67c35b6..3fe7a74 100644 (file)
--- a/drivers/iio/accel/Kconfig
+++ b/drivers/iio/accel/Kconfig
@@ -89,13 +89,13 @@ config ADXL372_I2C
          module will be called adxl372_i2c.
 
 config BMA180
-       tristate "Bosch BMA180/BMA25x 3-Axis Accelerometer Driver"
+       tristate "Bosch BMA023/BMA180/BMA25x 3-Axis Accelerometer Driver"
        depends on I2C && INPUT_BMA150=n
        select IIO_BUFFER
        select IIO_TRIGGERED_BUFFER
        help
-         Say Y here if you want to build a driver for the Bosch BMA180 or
-         BMA25x triaxial acceleration sensor.
+         Say Y here if you want to build a driver for the Bosch BMA023, BMA180
+         or BMA25x triaxial acceleration sensor.
 
          To compile this driver as a module, choose M here: the
          module will be called bma180.

diff --git a/drivers/iio/accel/bma180.c b/drivers/iio/accel/bma180.c
index 75440dd..19d4f17 100644 (file)
--- a/drivers/iio/accel/bma180.c
+++ b/drivers/iio/accel/bma180.c
@@ -7,6 +7,7 @@
  * Support for BMA250 (c) Peter Meerwald <pmeerw@pmeerw.net>
  *
  * SPI is not supported by driver
+ * BMA023: 7-bit I2C slave address 0x38
  * BMA180: 7-bit I2C slave address 0x40 or 0x41
  * BMA250: 7-bit I2C slave address 0x18 or 0x19
  * BMA254: 7-bit I2C slave address 0x18 or 0x19
@@ -33,6 +34,7 @@
 #define BMA180_IRQ_NAME "bma180_event"
 
 enum chip_ids {
+       BMA023,
        BMA180,
        BMA250,
        BMA254,
@@ -64,6 +66,18 @@ struct bma180_part_info {
 };
 
 /* Register set */
+#define BMA023_CTRL_REG0       0x0a
+#define BMA023_CTRL_REG1       0x0b
+#define BMA023_CTRL_REG2       0x14
+#define BMA023_CTRL_REG3       0x15
+
+#define BMA023_RANGE_MASK      GENMASK(4, 3) /* Range of accel values */
+#define BMA023_BW_MASK         GENMASK(2, 0) /* Accel bandwidth */
+#define BMA023_SLEEP           BIT(0)
+#define BMA023_INT_RESET_MASK  BIT(6)
+#define BMA023_NEW_DATA_INT    BIT(5) /* Intr every new accel data is ready */
+#define BMA023_RESET_VAL       BIT(1)
+
 #define BMA180_CHIP_ID         0x00 /* Need to distinguish BMA180 from other */
 #define BMA180_ACC_X_LSB       0x02 /* First of 6 registers of accel data */
 #define BMA180_TEMP            0x08
@@ -94,6 +108,7 @@ struct bma180_part_info {
 /* We have to write this value in reset register to do soft reset */
 #define BMA180_RESET_VAL       0xb6
 
+#define BMA023_ID_REG_VAL      0x02
 #define BMA180_ID_REG_VAL      0x03
 #define BMA250_ID_REG_VAL      0x03
 #define BMA254_ID_REG_VAL      0xfa /* 250 decimal */
@@ -156,6 +171,9 @@ enum bma180_chan {
        TEMP
 };
 
+static int bma023_bw_table[] = { 25, 50, 100, 190, 375, 750, 1500 }; /* Hz */
+static int bma023_scale_table[] = { 2452, 4903, 9709, };
+
 static int bma180_bw_table[] = { 10, 20, 40, 75, 150, 300 }; /* Hz */
 static int bma180_scale_table[] = { 1275, 1863, 2452, 3727, 4903, 9709, 19417 };
 
@@ -350,11 +368,28 @@ static int bma180_chip_init(struct bma180_data *data)
         */
        msleep(20);
 
-       ret = bma180_set_new_data_intr_state(data, false);
+       return bma180_set_new_data_intr_state(data, false);
+}
+
+static int bma023_chip_config(struct bma180_data *data)
+{
+       int ret = bma180_chip_init(data);
+
        if (ret)
-               return ret;
+               goto err;
+
+       ret = bma180_set_bw(data, 50); /* 50 Hz */
+       if (ret)
+               goto err;
+       ret = bma180_set_scale(data, 2452); /* 2 G */
+       if (ret)
+               goto err;
 
-       return bma180_set_pmode(data, false);
+       return 0;
+
+err:
+       dev_err(&data->client->dev, "failed to config the chip\n");
+       return ret;
 }
 
 static int bma180_chip_config(struct bma180_data *data)
@@ -363,6 +398,9 @@ static int bma180_chip_config(struct bma180_data *data)
 
        if (ret)
                goto err;
+       ret = bma180_set_pmode(data, false);
+       if (ret)
+               goto err;
        ret = bma180_set_bits(data, BMA180_CTRL_REG0, BMA180_DIS_WAKE_UP, 1);
        if (ret)
                goto err;
@@ -392,6 +430,9 @@ static int bma25x_chip_config(struct bma180_data *data)
 
        if (ret)
                goto err;
+       ret = bma180_set_pmode(data, false);
+       if (ret)
+               goto err;
        ret = bma180_set_bw(data, 16); /* 16 Hz */
        if (ret)
                goto err;
@@ -414,6 +455,17 @@ err:
        return ret;
 }
 
+static void bma023_chip_disable(struct bma180_data *data)
+{
+       if (bma180_set_sleep_state(data, true))
+               goto err;
+
+       return;
+
+err:
+       dev_err(&data->client->dev, "failed to disable the chip\n");
+}
+
 static void bma180_chip_disable(struct bma180_data *data)
 {
        if (bma180_set_new_data_intr_state(data, false))
@@ -610,6 +662,11 @@ static const struct iio_enum bma180_power_mode_enum = {
        .set = bma180_set_power_mode,
 };
 
+static const struct iio_chan_spec_ext_info bma023_ext_info[] = {
+       IIO_MOUNT_MATRIX(IIO_SHARED_BY_DIR, bma180_accel_get_mount_matrix),
+       { }
+};
+
 static const struct iio_chan_spec_ext_info bma180_ext_info[] = {
        IIO_ENUM("power_mode", true, &bma180_power_mode_enum),
        IIO_ENUM_AVAILABLE("power_mode", &bma180_power_mode_enum),
@@ -617,6 +674,23 @@ static const struct iio_chan_spec_ext_info bma180_ext_info[] = {
        { }
 };
 
+#define BMA023_ACC_CHANNEL(_axis, _bits) {                             \
+       .type = IIO_ACCEL,                                              \
+       .modified = 1,                                                  \
+       .channel2 = IIO_MOD_##_axis,                                    \
+       .info_mask_separate = BIT(IIO_CHAN_INFO_RAW),                   \
+       .info_mask_shared_by_type = BIT(IIO_CHAN_INFO_SCALE) |          \
+               BIT(IIO_CHAN_INFO_LOW_PASS_FILTER_3DB_FREQUENCY),       \
+       .scan_index = AXIS_##_axis,                                     \
+       .scan_type = {                                                  \
+               .sign = 's',                                            \
+               .realbits = _bits,                                      \
+               .storagebits = 16,                                      \
+               .shift = 16 - _bits,                                    \
+       },                                                              \
+       .ext_info = bma023_ext_info,                                    \
+}
+
 #define BMA180_ACC_CHANNEL(_axis, _bits) {                             \
        .type = IIO_ACCEL,                                              \
        .modified = 1,                                                  \
@@ -646,6 +720,13 @@ static const struct iio_chan_spec_ext_info bma180_ext_info[] = {
        },                                                              \
 }
 
+static const struct iio_chan_spec bma023_channels[] = {
+       BMA023_ACC_CHANNEL(X, 10),
+       BMA023_ACC_CHANNEL(Y, 10),
+       BMA023_ACC_CHANNEL(Z, 10),
+       IIO_CHAN_SOFT_TIMESTAMP(4),
+};
+
 static const struct iio_chan_spec bma180_channels[] = {
        BMA180_ACC_CHANNEL(X, 14),
        BMA180_ACC_CHANNEL(Y, 14),
@@ -671,6 +752,35 @@ static const struct iio_chan_spec bma254_channels[] = {
 };
 
 static const struct bma180_part_info bma180_part_info[] = {
+       [BMA023] = {
+               .chip_id = BMA023_ID_REG_VAL,
+               .channels = bma023_channels,
+               .num_channels = ARRAY_SIZE(bma023_channels),
+               .scale_table = bma023_scale_table,
+               .num_scales = ARRAY_SIZE(bma023_scale_table),
+               .bw_table = bma023_bw_table,
+               .num_bw = ARRAY_SIZE(bma023_bw_table),
+               /* No temperature channel */
+               .center_temp = 0,
+               .int_reset_reg = BMA023_CTRL_REG0,
+               .int_reset_mask = BMA023_INT_RESET_MASK,
+               .sleep_reg = BMA023_CTRL_REG0,
+               .sleep_mask = BMA023_SLEEP,
+               .bw_reg = BMA023_CTRL_REG2,
+               .bw_mask = BMA023_BW_MASK,
+               .scale_reg = BMA023_CTRL_REG2,
+               .scale_mask = BMA023_RANGE_MASK,
+               /* No power mode on bma023 */
+               .power_reg = 0,
+               .power_mask = 0,
+               .lowpower_val = 0,
+               .int_enable_reg = BMA023_CTRL_REG3,
+               .int_enable_mask = BMA023_NEW_DATA_INT,
+               .softreset_reg = BMA023_CTRL_REG0,
+               .softreset_val = BMA023_RESET_VAL,
+               .chip_config = bma023_chip_config,
+               .chip_disable = bma023_chip_disable,
+       },
        [BMA180] = {
                .chip_id = BMA180_ID_REG_VAL,
                .channels = bma180_channels,
@@ -994,6 +1104,7 @@ static SIMPLE_DEV_PM_OPS(bma180_pm_ops, bma180_suspend, bma180_resume);
 #endif
 
 static const struct i2c_device_id bma180_ids[] = {
+       { "bma023", BMA023 },
        { "bma180", BMA180 },
        { "bma250", BMA250 },
        { "bma254", BMA254 },
@@ -1004,6 +1115,10 @@ MODULE_DEVICE_TABLE(i2c, bma180_ids);
 
 static const struct of_device_id bma180_of_match[] = {
        {
+               .compatible = "bosch,bma023",
+               .data = (void *)BMA023
+       },
+       {
                .compatible = "bosch,bma180",
                .data = (void *)BMA180
        },
@@ -1034,5 +1149,5 @@ module_i2c_driver(bma180_driver);
 
 MODULE_AUTHOR("Kravchenko Oleksandr <x0199363@ti.com>");
 MODULE_AUTHOR("Texas Instruments, Inc.");
-MODULE_DESCRIPTION("Bosch BMA180/BMA25x triaxial acceleration sensor");
+MODULE_DESCRIPTION("Bosch BMA023/BMA180/BMA25x triaxial acceleration sensor");
 MODULE_LICENSE("GPL");