* [PATCH 0/2] enable LIR and clear_on_read for st_lsm6dsx @ 2019-09-11 6:50 Lorenzo Bianconi 2019-09-11 6:50 ` [PATCH 1/2] iio: imu: st_lsm6dsx: enable LIR for sensor events Lorenzo Bianconi 2019-09-11 6:50 ` [PATCH 2/2] iio: imu: st_lsm6dsx: enable clear on read for latched interrupts Lorenzo Bianconi 0 siblings, 2 replies; 7+ messages in thread From: Lorenzo Bianconi @ 2019-09-11 6:50 UTC (permalink / raw) To: jic23 Cc: sean, lorenzo.bianconi, linux-iio, mario.tesi, armando.visconti, denis.ciocca Enable by default Latched interrupts and clear_on_read (when available) for sensor events. @Sean: could you please rebase your series on top of this one in order to double check if it fixes the reported issue between buffered reading and wakeup events? Lorenzo Bianconi (2): iio: imu: st_lsm6dsx: enable LIR for sensor events iio: imu: st_lsm6dsx: enable clear on read for latched interrupts drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h | 4 ++ drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c | 59 ++++++++++++++++++++ 2 files changed, 63 insertions(+) -- 2.21.0 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/2] iio: imu: st_lsm6dsx: enable LIR for sensor events 2019-09-11 6:50 [PATCH 0/2] enable LIR and clear_on_read for st_lsm6dsx Lorenzo Bianconi @ 2019-09-11 6:50 ` Lorenzo Bianconi 2019-09-11 12:37 ` Sean Nyekjaer 2019-09-11 6:50 ` [PATCH 2/2] iio: imu: st_lsm6dsx: enable clear on read for latched interrupts Lorenzo Bianconi 1 sibling, 1 reply; 7+ messages in thread From: Lorenzo Bianconi @ 2019-09-11 6:50 UTC (permalink / raw) To: jic23 Cc: sean, lorenzo.bianconi, linux-iio, mario.tesi, armando.visconti, denis.ciocca Enable Latched interrupt by default for sensor events Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org> --- drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h | 2 ++ drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c | 35 ++++++++++++++++++++ 2 files changed, 37 insertions(+) diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h index 5e3cd96b0059..3ea0dc13d101 100644 --- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h +++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h @@ -215,6 +215,7 @@ struct st_lsm6dsx_ext_dev_settings { * @fs_table: Hw sensors gain table (gain + val). * @decimator: List of decimator register info (addr + mask). * @batch: List of FIFO batching register info (addr + mask). + * @lir: Latched interrupt register info (addr + mask). * @fifo_ops: Sensor hw FIFO parameters. * @ts_settings: Hw timer related settings. * @shub_settings: i2c controller related settings. @@ -237,6 +238,7 @@ struct st_lsm6dsx_settings { struct st_lsm6dsx_fs_table_entry fs_table[2]; struct st_lsm6dsx_reg decimator[ST_LSM6DSX_MAX_ID]; struct st_lsm6dsx_reg batch[ST_LSM6DSX_MAX_ID]; + struct st_lsm6dsx_reg lir; struct st_lsm6dsx_fifo_ops fifo_ops; struct st_lsm6dsx_hw_ts_settings ts_settings; struct st_lsm6dsx_shub_settings shub_settings; diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c index 2d3495560136..a208da865efe 100644 --- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c +++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c @@ -237,6 +237,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { .mask = GENMASK(5, 3), }, }, + .lir = { + .addr = 0x58, + .mask = BIT(0), + }, .fifo_ops = { .update_fifo = st_lsm6dsx_update_fifo, .read_fifo = st_lsm6dsx_read_fifo, @@ -349,6 +353,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { .mask = GENMASK(5, 3), }, }, + .lir = { + .addr = 0x58, + .mask = BIT(0), + }, .fifo_ops = { .update_fifo = st_lsm6dsx_update_fifo, .read_fifo = st_lsm6dsx_read_fifo, @@ -470,6 +478,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { .mask = GENMASK(5, 3), }, }, + .lir = { + .addr = 0x58, + .mask = BIT(0), + }, .fifo_ops = { .update_fifo = st_lsm6dsx_update_fifo, .read_fifo = st_lsm6dsx_read_fifo, @@ -585,6 +597,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { .mask = GENMASK(7, 4), }, }, + .lir = { + .addr = 0x56, + .mask = BIT(0), + }, .fifo_ops = { .update_fifo = st_lsm6dsx_update_fifo, .read_fifo = st_lsm6dsx_read_tagged_fifo, @@ -715,6 +731,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { .mask = GENMASK(7, 4), }, }, + .lir = { + .addr = 0x56, + .mask = BIT(0), + }, .fifo_ops = { .update_fifo = st_lsm6dsx_update_fifo, .read_fifo = st_lsm6dsx_read_tagged_fifo, @@ -822,6 +842,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { .mask = GENMASK(7, 4), }, }, + .lir = { + .addr = 0x56, + .mask = BIT(0), + }, .fifo_ops = { .update_fifo = st_lsm6dsx_update_fifo, .read_fifo = st_lsm6dsx_read_tagged_fifo, @@ -1416,6 +1440,17 @@ static int st_lsm6dsx_init_device(struct st_lsm6dsx_hw *hw) if (err < 0) return err; + /* enable Latched interrupts for device events */ + if (hw->settings->lir.addr) { + unsigned int data; + + data = ST_LSM6DSX_SHIFT_VAL(1, hw->settings->lir.mask); + err = regmap_update_bits(hw->regmap, hw->settings->lir.addr, + hw->settings->lir.mask, data); + if (err < 0) + return err; + } + err = st_lsm6dsx_init_shub(hw); if (err < 0) return err; -- 2.21.0 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] iio: imu: st_lsm6dsx: enable LIR for sensor events 2019-09-11 6:50 ` [PATCH 1/2] iio: imu: st_lsm6dsx: enable LIR for sensor events Lorenzo Bianconi @ 2019-09-11 12:37 ` Sean Nyekjaer 2019-09-15 10:57 ` Jonathan Cameron 0 siblings, 1 reply; 7+ messages in thread From: Sean Nyekjaer @ 2019-09-11 12:37 UTC (permalink / raw) To: Lorenzo Bianconi, jic23 Cc: lorenzo.bianconi, linux-iio, mario.tesi, armando.visconti, denis.ciocca On 11/09/2019 08.50, Lorenzo Bianconi wrote: > Enable Latched interrupt by default for sensor events > > Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org> Tested-by: Sean Nyekjaer <sean@geanix.com> > --- > drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h | 2 ++ > drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c | 35 ++++++++++++++++++++ > 2 files changed, 37 insertions(+) > > diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h > index 5e3cd96b0059..3ea0dc13d101 100644 > --- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h > +++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h > @@ -215,6 +215,7 @@ struct st_lsm6dsx_ext_dev_settings { > * @fs_table: Hw sensors gain table (gain + val). > * @decimator: List of decimator register info (addr + mask). > * @batch: List of FIFO batching register info (addr + mask). > + * @lir: Latched interrupt register info (addr + mask). > * @fifo_ops: Sensor hw FIFO parameters. > * @ts_settings: Hw timer related settings. > * @shub_settings: i2c controller related settings. > @@ -237,6 +238,7 @@ struct st_lsm6dsx_settings { > struct st_lsm6dsx_fs_table_entry fs_table[2]; > struct st_lsm6dsx_reg decimator[ST_LSM6DSX_MAX_ID]; > struct st_lsm6dsx_reg batch[ST_LSM6DSX_MAX_ID]; > + struct st_lsm6dsx_reg lir; > struct st_lsm6dsx_fifo_ops fifo_ops; > struct st_lsm6dsx_hw_ts_settings ts_settings; > struct st_lsm6dsx_shub_settings shub_settings; > diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c > index 2d3495560136..a208da865efe 100644 > --- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c > +++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c > @@ -237,6 +237,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { > .mask = GENMASK(5, 3), > }, > }, > + .lir = { > + .addr = 0x58, > + .mask = BIT(0), > + }, > .fifo_ops = { > .update_fifo = st_lsm6dsx_update_fifo, > .read_fifo = st_lsm6dsx_read_fifo, > @@ -349,6 +353,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { > .mask = GENMASK(5, 3), > }, > }, > + .lir = { > + .addr = 0x58, > + .mask = BIT(0), > + }, > .fifo_ops = { > .update_fifo = st_lsm6dsx_update_fifo, > .read_fifo = st_lsm6dsx_read_fifo, > @@ -470,6 +478,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { > .mask = GENMASK(5, 3), > }, > }, > + .lir = { > + .addr = 0x58, > + .mask = BIT(0), > + }, > .fifo_ops = { > .update_fifo = st_lsm6dsx_update_fifo, > .read_fifo = st_lsm6dsx_read_fifo, > @@ -585,6 +597,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { > .mask = GENMASK(7, 4), > }, > }, > + .lir = { > + .addr = 0x56, > + .mask = BIT(0), > + }, > .fifo_ops = { > .update_fifo = st_lsm6dsx_update_fifo, > .read_fifo = st_lsm6dsx_read_tagged_fifo, > @@ -715,6 +731,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { > .mask = GENMASK(7, 4), > }, > }, > + .lir = { > + .addr = 0x56, > + .mask = BIT(0), > + }, > .fifo_ops = { > .update_fifo = st_lsm6dsx_update_fifo, > .read_fifo = st_lsm6dsx_read_tagged_fifo, > @@ -822,6 +842,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { > .mask = GENMASK(7, 4), > }, > }, > + .lir = { > + .addr = 0x56, > + .mask = BIT(0), > + }, > .fifo_ops = { > .update_fifo = st_lsm6dsx_update_fifo, > .read_fifo = st_lsm6dsx_read_tagged_fifo, > @@ -1416,6 +1440,17 @@ static int st_lsm6dsx_init_device(struct st_lsm6dsx_hw *hw) > if (err < 0) > return err; > > + /* enable Latched interrupts for device events */ > + if (hw->settings->lir.addr) { > + unsigned int data; > + > + data = ST_LSM6DSX_SHIFT_VAL(1, hw->settings->lir.mask); > + err = regmap_update_bits(hw->regmap, hw->settings->lir.addr, > + hw->settings->lir.mask, data); > + if (err < 0) > + return err; > + } > + > err = st_lsm6dsx_init_shub(hw); > if (err < 0) > return err; > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] iio: imu: st_lsm6dsx: enable LIR for sensor events 2019-09-11 12:37 ` Sean Nyekjaer @ 2019-09-15 10:57 ` Jonathan Cameron 0 siblings, 0 replies; 7+ messages in thread From: Jonathan Cameron @ 2019-09-15 10:57 UTC (permalink / raw) To: Sean Nyekjaer Cc: Lorenzo Bianconi, lorenzo.bianconi, linux-iio, mario.tesi, armando.visconti, denis.ciocca On Wed, 11 Sep 2019 14:37:23 +0200 Sean Nyekjaer <sean@geanix.com> wrote: > On 11/09/2019 08.50, Lorenzo Bianconi wrote: > > Enable Latched interrupt by default for sensor events > > > > Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org> > Tested-by: Sean Nyekjaer <sean@geanix.com> Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to play with it. Thanks, Jonathan > > --- > > drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h | 2 ++ > > drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c | 35 ++++++++++++++++++++ > > 2 files changed, 37 insertions(+) > > > > diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h > > index 5e3cd96b0059..3ea0dc13d101 100644 > > --- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h > > +++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h > > @@ -215,6 +215,7 @@ struct st_lsm6dsx_ext_dev_settings { > > * @fs_table: Hw sensors gain table (gain + val). > > * @decimator: List of decimator register info (addr + mask). > > * @batch: List of FIFO batching register info (addr + mask). > > + * @lir: Latched interrupt register info (addr + mask). > > * @fifo_ops: Sensor hw FIFO parameters. > > * @ts_settings: Hw timer related settings. > > * @shub_settings: i2c controller related settings. > > @@ -237,6 +238,7 @@ struct st_lsm6dsx_settings { > > struct st_lsm6dsx_fs_table_entry fs_table[2]; > > struct st_lsm6dsx_reg decimator[ST_LSM6DSX_MAX_ID]; > > struct st_lsm6dsx_reg batch[ST_LSM6DSX_MAX_ID]; > > + struct st_lsm6dsx_reg lir; > > struct st_lsm6dsx_fifo_ops fifo_ops; > > struct st_lsm6dsx_hw_ts_settings ts_settings; > > struct st_lsm6dsx_shub_settings shub_settings; > > diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c > > index 2d3495560136..a208da865efe 100644 > > --- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c > > +++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c > > @@ -237,6 +237,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { > > .mask = GENMASK(5, 3), > > }, > > }, > > + .lir = { > > + .addr = 0x58, > > + .mask = BIT(0), > > + }, > > .fifo_ops = { > > .update_fifo = st_lsm6dsx_update_fifo, > > .read_fifo = st_lsm6dsx_read_fifo, > > @@ -349,6 +353,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { > > .mask = GENMASK(5, 3), > > }, > > }, > > + .lir = { > > + .addr = 0x58, > > + .mask = BIT(0), > > + }, > > .fifo_ops = { > > .update_fifo = st_lsm6dsx_update_fifo, > > .read_fifo = st_lsm6dsx_read_fifo, > > @@ -470,6 +478,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { > > .mask = GENMASK(5, 3), > > }, > > }, > > + .lir = { > > + .addr = 0x58, > > + .mask = BIT(0), > > + }, > > .fifo_ops = { > > .update_fifo = st_lsm6dsx_update_fifo, > > .read_fifo = st_lsm6dsx_read_fifo, > > @@ -585,6 +597,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { > > .mask = GENMASK(7, 4), > > }, > > }, > > + .lir = { > > + .addr = 0x56, > > + .mask = BIT(0), > > + }, > > .fifo_ops = { > > .update_fifo = st_lsm6dsx_update_fifo, > > .read_fifo = st_lsm6dsx_read_tagged_fifo, > > @@ -715,6 +731,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { > > .mask = GENMASK(7, 4), > > }, > > }, > > + .lir = { > > + .addr = 0x56, > > + .mask = BIT(0), > > + }, > > .fifo_ops = { > > .update_fifo = st_lsm6dsx_update_fifo, > > .read_fifo = st_lsm6dsx_read_tagged_fifo, > > @@ -822,6 +842,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { > > .mask = GENMASK(7, 4), > > }, > > }, > > + .lir = { > > + .addr = 0x56, > > + .mask = BIT(0), > > + }, > > .fifo_ops = { > > .update_fifo = st_lsm6dsx_update_fifo, > > .read_fifo = st_lsm6dsx_read_tagged_fifo, > > @@ -1416,6 +1440,17 @@ static int st_lsm6dsx_init_device(struct st_lsm6dsx_hw *hw) > > if (err < 0) > > return err; > > > > + /* enable Latched interrupts for device events */ > > + if (hw->settings->lir.addr) { > > + unsigned int data; > > + > > + data = ST_LSM6DSX_SHIFT_VAL(1, hw->settings->lir.mask); > > + err = regmap_update_bits(hw->regmap, hw->settings->lir.addr, > > + hw->settings->lir.mask, data); > > + if (err < 0) > > + return err; > > + } > > + > > err = st_lsm6dsx_init_shub(hw); > > if (err < 0) > > return err; > > ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 2/2] iio: imu: st_lsm6dsx: enable clear on read for latched interrupts 2019-09-11 6:50 [PATCH 0/2] enable LIR and clear_on_read for st_lsm6dsx Lorenzo Bianconi 2019-09-11 6:50 ` [PATCH 1/2] iio: imu: st_lsm6dsx: enable LIR for sensor events Lorenzo Bianconi @ 2019-09-11 6:50 ` Lorenzo Bianconi 2019-09-11 12:37 ` Sean Nyekjaer 1 sibling, 1 reply; 7+ messages in thread From: Lorenzo Bianconi @ 2019-09-11 6:50 UTC (permalink / raw) To: jic23 Cc: sean, lorenzo.bianconi, linux-iio, mario.tesi, armando.visconti, denis.ciocca Enable clear on read feature for latched interrupts. This bit allows immediately clearing the latched interrupts of an event detection upon the read of the corresponding status register. It must be set to 1 together with LIR. This feature is available just on LSM6DS0/LSM6DSR/ASM330LHH Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org> --- drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h | 2 ++ drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c | 24 ++++++++++++++++++++ 2 files changed, 26 insertions(+) diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h index 3ea0dc13d101..fefd9042590a 100644 --- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h +++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h @@ -216,6 +216,7 @@ struct st_lsm6dsx_ext_dev_settings { * @decimator: List of decimator register info (addr + mask). * @batch: List of FIFO batching register info (addr + mask). * @lir: Latched interrupt register info (addr + mask). + * @clear_on_read: Clear on read register info (addr + mask). * @fifo_ops: Sensor hw FIFO parameters. * @ts_settings: Hw timer related settings. * @shub_settings: i2c controller related settings. @@ -239,6 +240,7 @@ struct st_lsm6dsx_settings { struct st_lsm6dsx_reg decimator[ST_LSM6DSX_MAX_ID]; struct st_lsm6dsx_reg batch[ST_LSM6DSX_MAX_ID]; struct st_lsm6dsx_reg lir; + struct st_lsm6dsx_reg clear_on_read; struct st_lsm6dsx_fifo_ops fifo_ops; struct st_lsm6dsx_hw_ts_settings ts_settings; struct st_lsm6dsx_shub_settings shub_settings; diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c index a208da865efe..b65a6ca775e0 100644 --- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c +++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c @@ -601,6 +601,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { .addr = 0x56, .mask = BIT(0), }, + .clear_on_read = { + .addr = 0x56, + .mask = BIT(6), + }, .fifo_ops = { .update_fifo = st_lsm6dsx_update_fifo, .read_fifo = st_lsm6dsx_read_tagged_fifo, @@ -735,6 +739,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { .addr = 0x56, .mask = BIT(0), }, + .clear_on_read = { + .addr = 0x56, + .mask = BIT(6), + }, .fifo_ops = { .update_fifo = st_lsm6dsx_update_fifo, .read_fifo = st_lsm6dsx_read_tagged_fifo, @@ -846,6 +854,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { .addr = 0x56, .mask = BIT(0), }, + .clear_on_read = { + .addr = 0x56, + .mask = BIT(6), + }, .fifo_ops = { .update_fifo = st_lsm6dsx_update_fifo, .read_fifo = st_lsm6dsx_read_tagged_fifo, @@ -1449,6 +1461,18 @@ static int st_lsm6dsx_init_device(struct st_lsm6dsx_hw *hw) hw->settings->lir.mask, data); if (err < 0) return err; + + /* enable clear on read for latched interrupts */ + if (hw->settings->clear_on_read.addr) { + data = ST_LSM6DSX_SHIFT_VAL(1, + hw->settings->clear_on_read.mask); + err = regmap_update_bits(hw->regmap, + hw->settings->clear_on_read.addr, + hw->settings->clear_on_read.mask, + data); + if (err < 0) + return err; + } } err = st_lsm6dsx_init_shub(hw); -- 2.21.0 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] iio: imu: st_lsm6dsx: enable clear on read for latched interrupts 2019-09-11 6:50 ` [PATCH 2/2] iio: imu: st_lsm6dsx: enable clear on read for latched interrupts Lorenzo Bianconi @ 2019-09-11 12:37 ` Sean Nyekjaer 2019-09-15 11:03 ` Jonathan Cameron 0 siblings, 1 reply; 7+ messages in thread From: Sean Nyekjaer @ 2019-09-11 12:37 UTC (permalink / raw) To: Lorenzo Bianconi, jic23 Cc: lorenzo.bianconi, linux-iio, mario.tesi, armando.visconti, denis.ciocca On 11/09/2019 08.50, Lorenzo Bianconi wrote: > Enable clear on read feature for latched interrupts. This bit allows > immediately clearing the latched interrupts of an event detection upon > the read of the corresponding status register. > It must be set to 1 together with LIR. > This feature is available just on LSM6DS0/LSM6DSR/ASM330LHH > > Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org> Tested-by: Sean Nyekjaer <sean@geanix.com> > --- > drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h | 2 ++ > drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c | 24 ++++++++++++++++++++ > 2 files changed, 26 insertions(+) > > diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h > index 3ea0dc13d101..fefd9042590a 100644 > --- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h > +++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h > @@ -216,6 +216,7 @@ struct st_lsm6dsx_ext_dev_settings { > * @decimator: List of decimator register info (addr + mask). > * @batch: List of FIFO batching register info (addr + mask). > * @lir: Latched interrupt register info (addr + mask). > + * @clear_on_read: Clear on read register info (addr + mask). > * @fifo_ops: Sensor hw FIFO parameters. > * @ts_settings: Hw timer related settings. > * @shub_settings: i2c controller related settings. > @@ -239,6 +240,7 @@ struct st_lsm6dsx_settings { > struct st_lsm6dsx_reg decimator[ST_LSM6DSX_MAX_ID]; > struct st_lsm6dsx_reg batch[ST_LSM6DSX_MAX_ID]; > struct st_lsm6dsx_reg lir; > + struct st_lsm6dsx_reg clear_on_read; > struct st_lsm6dsx_fifo_ops fifo_ops; > struct st_lsm6dsx_hw_ts_settings ts_settings; > struct st_lsm6dsx_shub_settings shub_settings; > diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c > index a208da865efe..b65a6ca775e0 100644 > --- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c > +++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c > @@ -601,6 +601,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { > .addr = 0x56, > .mask = BIT(0), > }, > + .clear_on_read = { > + .addr = 0x56, > + .mask = BIT(6), > + }, > .fifo_ops = { > .update_fifo = st_lsm6dsx_update_fifo, > .read_fifo = st_lsm6dsx_read_tagged_fifo, > @@ -735,6 +739,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { > .addr = 0x56, > .mask = BIT(0), > }, > + .clear_on_read = { > + .addr = 0x56, > + .mask = BIT(6), > + }, > .fifo_ops = { > .update_fifo = st_lsm6dsx_update_fifo, > .read_fifo = st_lsm6dsx_read_tagged_fifo, > @@ -846,6 +854,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { > .addr = 0x56, > .mask = BIT(0), > }, > + .clear_on_read = { > + .addr = 0x56, > + .mask = BIT(6), > + }, > .fifo_ops = { > .update_fifo = st_lsm6dsx_update_fifo, > .read_fifo = st_lsm6dsx_read_tagged_fifo, > @@ -1449,6 +1461,18 @@ static int st_lsm6dsx_init_device(struct st_lsm6dsx_hw *hw) > hw->settings->lir.mask, data); > if (err < 0) > return err; > + > + /* enable clear on read for latched interrupts */ > + if (hw->settings->clear_on_read.addr) { > + data = ST_LSM6DSX_SHIFT_VAL(1, > + hw->settings->clear_on_read.mask); > + err = regmap_update_bits(hw->regmap, > + hw->settings->clear_on_read.addr, > + hw->settings->clear_on_read.mask, > + data); > + if (err < 0) > + return err; > + } > } > > err = st_lsm6dsx_init_shub(hw); > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] iio: imu: st_lsm6dsx: enable clear on read for latched interrupts 2019-09-11 12:37 ` Sean Nyekjaer @ 2019-09-15 11:03 ` Jonathan Cameron 0 siblings, 0 replies; 7+ messages in thread From: Jonathan Cameron @ 2019-09-15 11:03 UTC (permalink / raw) To: Sean Nyekjaer Cc: Lorenzo Bianconi, lorenzo.bianconi, linux-iio, mario.tesi, armando.visconti, denis.ciocca On Wed, 11 Sep 2019 14:37:16 +0200 Sean Nyekjaer <sean@geanix.com> wrote: > On 11/09/2019 08.50, Lorenzo Bianconi wrote: > > Enable clear on read feature for latched interrupts. This bit allows > > immediately clearing the latched interrupts of an event detection upon > > the read of the corresponding status register. > > It must be set to 1 together with LIR. > > This feature is available just on LSM6DS0/LSM6DSR/ASM330LHH > > > > Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org> > Tested-by: Sean Nyekjaer <sean@geanix.com> Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to play with it. Thanks, Jonathan > > --- > > drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h | 2 ++ > > drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c | 24 ++++++++++++++++++++ > > 2 files changed, 26 insertions(+) > > > > diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h > > index 3ea0dc13d101..fefd9042590a 100644 > > --- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h > > +++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h > > @@ -216,6 +216,7 @@ struct st_lsm6dsx_ext_dev_settings { > > * @decimator: List of decimator register info (addr + mask). > > * @batch: List of FIFO batching register info (addr + mask). > > * @lir: Latched interrupt register info (addr + mask). > > + * @clear_on_read: Clear on read register info (addr + mask). > > * @fifo_ops: Sensor hw FIFO parameters. > > * @ts_settings: Hw timer related settings. > > * @shub_settings: i2c controller related settings. > > @@ -239,6 +240,7 @@ struct st_lsm6dsx_settings { > > struct st_lsm6dsx_reg decimator[ST_LSM6DSX_MAX_ID]; > > struct st_lsm6dsx_reg batch[ST_LSM6DSX_MAX_ID]; > > struct st_lsm6dsx_reg lir; > > + struct st_lsm6dsx_reg clear_on_read; > > struct st_lsm6dsx_fifo_ops fifo_ops; > > struct st_lsm6dsx_hw_ts_settings ts_settings; > > struct st_lsm6dsx_shub_settings shub_settings; > > diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c > > index a208da865efe..b65a6ca775e0 100644 > > --- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c > > +++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c > > @@ -601,6 +601,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { > > .addr = 0x56, > > .mask = BIT(0), > > }, > > + .clear_on_read = { > > + .addr = 0x56, > > + .mask = BIT(6), > > + }, > > .fifo_ops = { > > .update_fifo = st_lsm6dsx_update_fifo, > > .read_fifo = st_lsm6dsx_read_tagged_fifo, > > @@ -735,6 +739,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { > > .addr = 0x56, > > .mask = BIT(0), > > }, > > + .clear_on_read = { > > + .addr = 0x56, > > + .mask = BIT(6), > > + }, > > .fifo_ops = { > > .update_fifo = st_lsm6dsx_update_fifo, > > .read_fifo = st_lsm6dsx_read_tagged_fifo, > > @@ -846,6 +854,10 @@ static const struct st_lsm6dsx_settings st_lsm6dsx_sensor_settings[] = { > > .addr = 0x56, > > .mask = BIT(0), > > }, > > + .clear_on_read = { > > + .addr = 0x56, > > + .mask = BIT(6), > > + }, > > .fifo_ops = { > > .update_fifo = st_lsm6dsx_update_fifo, > > .read_fifo = st_lsm6dsx_read_tagged_fifo, > > @@ -1449,6 +1461,18 @@ static int st_lsm6dsx_init_device(struct st_lsm6dsx_hw *hw) > > hw->settings->lir.mask, data); > > if (err < 0) > > return err; > > + > > + /* enable clear on read for latched interrupts */ > > + if (hw->settings->clear_on_read.addr) { > > + data = ST_LSM6DSX_SHIFT_VAL(1, > > + hw->settings->clear_on_read.mask); > > + err = regmap_update_bits(hw->regmap, > > + hw->settings->clear_on_read.addr, > > + hw->settings->clear_on_read.mask, > > + data); > > + if (err < 0) > > + return err; > > + } > > } > > > > err = st_lsm6dsx_init_shub(hw); > > ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2019-09-15 11:03 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2019-09-11 6:50 [PATCH 0/2] enable LIR and clear_on_read for st_lsm6dsx Lorenzo Bianconi 2019-09-11 6:50 ` [PATCH 1/2] iio: imu: st_lsm6dsx: enable LIR for sensor events Lorenzo Bianconi 2019-09-11 12:37 ` Sean Nyekjaer 2019-09-15 10:57 ` Jonathan Cameron 2019-09-11 6:50 ` [PATCH 2/2] iio: imu: st_lsm6dsx: enable clear on read for latched interrupts Lorenzo Bianconi 2019-09-11 12:37 ` Sean Nyekjaer 2019-09-15 11:03 ` Jonathan Cameron
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox