On 08/06/15 16:37, Denis CIOCCA wrote: > Hi Jonathan, > > Yes my ack was for the all series...Sorry! > > Acked-by: Denis Ciocca <denis.ciocca@xxxxxx> > Cool, added it. Jonathan > Denis > > Sent from my phone > >> On Jun 8, 2015, at 01:01, Jonathan Cameron <jic23@xxxxxxxxxx> wrote: >> >>> On 19/05/15 14:37, Linus Walleij wrote: >>> This adds support for the LIS331DL sensor version. This is >>> a simple 8bit-only accelerometer. >>> >>> Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx> >> Applied to the togreg branch of iio.git - pushed out as testing for >> the autobuilders to play with it. >> >> Dennis, if your ack covered this as well, then I can still add it >> until I push out properly. >> >> Thanks, >> >> Jonathan >>> --- >>> ChangeLog v1->v2: >>> - Change the full-scale sensitivity settings, apparently >>> this is the mg/digit setting multiplied by 1000, so >>> becomes microgram per digit, 18000 and 72000 respectively. >>> --- >>> .../devicetree/bindings/iio/st-sensors.txt | 1 + >>> drivers/iio/accel/st_accel.h | 1 + >>> drivers/iio/accel/st_accel_core.c | 70 ++++++++++++++++++++++ >>> drivers/iio/accel/st_accel_i2c.c | 4 ++ >>> 4 files changed, 76 insertions(+) >>> >>> diff --git a/Documentation/devicetree/bindings/iio/st-sensors.txt b/Documentation/devicetree/bindings/iio/st-sensors.txt >>> index fb5e0c2d18b5..8a6be3bdf267 100644 >>> --- a/Documentation/devicetree/bindings/iio/st-sensors.txt >>> +++ b/Documentation/devicetree/bindings/iio/st-sensors.txt >>> @@ -30,6 +30,7 @@ Accelerometers: >>> - st,lsm330d-accel >>> - st,lsm330dl-accel >>> - st,lsm330dlc-accel >>> +- st,lis331dl-accel >>> - st,lis331dlh-accel >>> - st,lsm303dl-accel >>> - st,lsm303dlm-accel >>> diff --git a/drivers/iio/accel/st_accel.h b/drivers/iio/accel/st_accel.h >>> index 7ee9724b1428..aa1001931d0c 100644 >>> --- a/drivers/iio/accel/st_accel.h >>> +++ b/drivers/iio/accel/st_accel.h >>> @@ -20,6 +20,7 @@ >>> #define LSM330D_ACCEL_DEV_NAME "lsm330d_accel" >>> #define LSM330DL_ACCEL_DEV_NAME "lsm330dl_accel" >>> #define LSM330DLC_ACCEL_DEV_NAME "lsm330dlc_accel" >>> +#define LIS331DL_ACCEL_DEV_NAME "lis331dl_accel" >>> #define LIS331DLH_ACCEL_DEV_NAME "lis331dlh" >>> #define LSM303DL_ACCEL_DEV_NAME "lsm303dl_accel" >>> #define LSM303DLH_ACCEL_DEV_NAME "lsm303dlh_accel" >>> diff --git a/drivers/iio/accel/st_accel_core.c b/drivers/iio/accel/st_accel_core.c >>> index ad19fb4304df..970e9f904dde 100644 >>> --- a/drivers/iio/accel/st_accel_core.c >>> +++ b/drivers/iio/accel/st_accel_core.c >>> @@ -153,6 +153,28 @@ >>> #define ST_ACCEL_4_IG1_EN_MASK 0x08 >>> #define ST_ACCEL_4_MULTIREAD_BIT true >>> >>> +/* CUSTOM VALUES FOR SENSOR 5 */ >>> +#define ST_ACCEL_5_WAI_EXP 0x3b >>> +#define ST_ACCEL_5_ODR_ADDR 0x20 >>> +#define ST_ACCEL_5_ODR_MASK 0x80 >>> +#define ST_ACCEL_5_ODR_AVL_100HZ_VAL 0x00 >>> +#define ST_ACCEL_5_ODR_AVL_400HZ_VAL 0x01 >>> +#define ST_ACCEL_5_PW_ADDR 0x20 >>> +#define ST_ACCEL_5_PW_MASK 0x40 >>> +#define ST_ACCEL_5_FS_ADDR 0x20 >>> +#define ST_ACCEL_5_FS_MASK 0x20 >>> +#define ST_ACCEL_5_FS_AVL_2_VAL 0X00 >>> +#define ST_ACCEL_5_FS_AVL_8_VAL 0X01 >>> +/* TODO: check these resulting gain settings, these are not in the datsheet */ >>> +#define ST_ACCEL_5_FS_AVL_2_GAIN IIO_G_TO_M_S_2(18000) >>> +#define ST_ACCEL_5_FS_AVL_8_GAIN IIO_G_TO_M_S_2(72000) >>> +#define ST_ACCEL_5_DRDY_IRQ_ADDR 0x22 >>> +#define ST_ACCEL_5_DRDY_IRQ_INT1_MASK 0x04 >>> +#define ST_ACCEL_5_DRDY_IRQ_INT2_MASK 0x20 >>> +#define ST_ACCEL_5_IG1_EN_ADDR 0x21 >>> +#define ST_ACCEL_5_IG1_EN_MASK 0x08 >>> +#define ST_ACCEL_5_MULTIREAD_BIT false >>> + >>> static const struct iio_chan_spec st_accel_8bit_channels[] = { >>> ST_SENSORS_LSM_CHANNELS(IIO_ACCEL, >>> BIT(IIO_CHAN_INFO_RAW) | BIT(IIO_CHAN_INFO_SCALE), >>> @@ -470,6 +492,54 @@ static const struct st_sensor_settings st_accel_sensors_settings[] = { >>> .multi_read_bit = ST_ACCEL_4_MULTIREAD_BIT, >>> .bootime = 2, /* guess */ >>> }, >>> + { >>> + .wai = ST_ACCEL_5_WAI_EXP, >>> + .sensors_supported = { >>> + [0] = LIS331DL_ACCEL_DEV_NAME, >>> + }, >>> + .ch = (struct iio_chan_spec *)st_accel_8bit_channels, >>> + .odr = { >>> + .addr = ST_ACCEL_5_ODR_ADDR, >>> + .mask = ST_ACCEL_5_ODR_MASK, >>> + .odr_avl = { >>> + { 100, ST_ACCEL_5_ODR_AVL_100HZ_VAL }, >>> + { 400, ST_ACCEL_5_ODR_AVL_400HZ_VAL, }, >>> + }, >>> + }, >>> + .pw = { >>> + .addr = ST_ACCEL_5_PW_ADDR, >>> + .mask = ST_ACCEL_5_PW_MASK, >>> + .value_on = ST_SENSORS_DEFAULT_POWER_ON_VALUE, >>> + .value_off = ST_SENSORS_DEFAULT_POWER_OFF_VALUE, >>> + }, >>> + .enable_axis = { >>> + .addr = ST_SENSORS_DEFAULT_AXIS_ADDR, >>> + .mask = ST_SENSORS_DEFAULT_AXIS_MASK, >>> + }, >>> + .fs = { >>> + .addr = ST_ACCEL_5_FS_ADDR, >>> + .mask = ST_ACCEL_5_FS_MASK, >>> + .fs_avl = { >>> + [0] = { >>> + .num = ST_ACCEL_FS_AVL_2G, >>> + .value = ST_ACCEL_5_FS_AVL_2_VAL, >>> + .gain = ST_ACCEL_5_FS_AVL_2_GAIN, >>> + }, >>> + [1] = { >>> + .num = ST_ACCEL_FS_AVL_8G, >>> + .value = ST_ACCEL_5_FS_AVL_8_VAL, >>> + .gain = ST_ACCEL_5_FS_AVL_8_GAIN, >>> + }, >>> + }, >>> + }, >>> + .drdy_irq = { >>> + .addr = ST_ACCEL_5_DRDY_IRQ_ADDR, >>> + .mask_int1 = ST_ACCEL_5_DRDY_IRQ_INT1_MASK, >>> + .mask_int2 = ST_ACCEL_5_DRDY_IRQ_INT2_MASK, >>> + }, >>> + .multi_read_bit = ST_ACCEL_5_MULTIREAD_BIT, >>> + .bootime = 2, /* guess */ >>> + }, >>> }; >>> >>> static int st_accel_read_raw(struct iio_dev *indio_dev, >>> diff --git a/drivers/iio/accel/st_accel_i2c.c b/drivers/iio/accel/st_accel_i2c.c >>> index 6b720c190b2d..d4ad72ca4a3d 100644 >>> --- a/drivers/iio/accel/st_accel_i2c.c >>> +++ b/drivers/iio/accel/st_accel_i2c.c >>> @@ -49,6 +49,10 @@ static const struct of_device_id st_accel_of_match[] = { >>> .data = LSM330DLC_ACCEL_DEV_NAME, >>> }, >>> { >>> + .compatible = "st,lis331dl-accel", >>> + .data = LIS331DL_ACCEL_DEV_NAME, >>> + }, >>> + { >>> .compatible = "st,lis331dlh-accel", >>> .data = LIS331DLH_ACCEL_DEV_NAME, >>> }, >> > -- > To unsubscribe from this list: send the line "unsubscribe linux-iio" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html