On 02/09/15 20:02, Linus Walleij wrote:
> This accelerometer accidentally either emits a DRDY signal or an
> IRQ signal. Accidentally I activated the IRQ signal as I thought
> it was analogous to the interrupt generator on other ST
> accelerometers. This was wrong. After this patch generic_buffer
> gives a nice stream of accelerometer readings.
>
> Fixes: 3acddf74f807778f "iio: st-sensors: add support for lis3lv02d accelerometer"
> Cc: Denis CIOCCA <denis.ciocca@xxxxxx>
> Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx>
Can see where that went wrong looking at the datasheets.
Applied to the fixes-togreg branch of iio.git and marked
for stable.
Thanks,
Jonathan
> ---
> drivers/iio/accel/st_accel_core.c | 6 ------
> 1 file changed, 6 deletions(-)
>
> diff --git a/drivers/iio/accel/st_accel_core.c b/drivers/iio/accel/st_accel_core.c
> index 058c87d9f564..6765eebb7c0a 100644
> --- a/drivers/iio/accel/st_accel_core.c
> +++ b/drivers/iio/accel/st_accel_core.c
> @@ -149,8 +149,6 @@
> #define ST_ACCEL_4_BDU_MASK 0x40
> #define ST_ACCEL_4_DRDY_IRQ_ADDR 0x21
> #define ST_ACCEL_4_DRDY_IRQ_INT1_MASK 0x04
> -#define ST_ACCEL_4_IG1_EN_ADDR 0x21
> -#define ST_ACCEL_4_IG1_EN_MASK 0x08
> #define ST_ACCEL_4_MULTIREAD_BIT true
>
> /* CUSTOM VALUES FOR SENSOR 5 */
> @@ -484,10 +482,6 @@ static const struct st_sensor_settings st_accel_sensors_settings[] = {
> .drdy_irq = {
> .addr = ST_ACCEL_4_DRDY_IRQ_ADDR,
> .mask_int1 = ST_ACCEL_4_DRDY_IRQ_INT1_MASK,
> - .ig1 = {
> - .en_addr = ST_ACCEL_4_IG1_EN_ADDR,
> - .en_mask = ST_ACCEL_4_IG1_EN_MASK,
> - },
> },
> .multi_read_bit = ST_ACCEL_4_MULTIREAD_BIT,
> .bootime = 2, /* guess */
>
--
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
