* [PATCH] iio: magnetometer: st_magn: fix drdy line configuration for LIS3MDL
@ 2017-08-29 11:45 Lorenzo Bianconi
2017-09-03 16:39 ` Jonathan Cameron
0 siblings, 1 reply; 2+ messages in thread
From: Lorenzo Bianconi @ 2017-08-29 11:45 UTC (permalink / raw)
To: jic23; +Cc: linux-iio, linus.walleij, lorenzo.bianconi
Data-ready line in LIS3MDL is routed to drdy pin and it is not possible
to select a different INT pin. st_sensors_set_dataready_irq() assumes
that if drdy int address is not exported in register map, irq trigger
is not supported by the sensor and hw_irq_trigger is always false.
Based on this configuration st_sensors_irq_thread does not consume
generated interrupt causing an unhandled irq.
Fix this taking into account status register address in
st_sensors_set_dataready_irq()
Fixes: 90efe0556292 (iio: st_sensors: harden interrupt handling)
Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi@st.com>
---
drivers/iio/common/st_sensors/st_sensors_core.c | 11 ++++++++++-
drivers/iio/magnetometer/st_magn_core.c | 4 ++++
2 files changed, 14 insertions(+), 1 deletion(-)
diff --git a/drivers/iio/common/st_sensors/st_sensors_core.c b/drivers/iio/common/st_sensors/st_sensors_core.c
index d99bb1460fe2..02e833b14db0 100644
--- a/drivers/iio/common/st_sensors/st_sensors_core.c
+++ b/drivers/iio/common/st_sensors/st_sensors_core.c
@@ -463,8 +463,17 @@ int st_sensors_set_dataready_irq(struct iio_dev *indio_dev, bool enable)
u8 drdy_mask;
struct st_sensor_data *sdata = iio_priv(indio_dev);
- if (!sdata->sensor_settings->drdy_irq.addr)
+ if (!sdata->sensor_settings->drdy_irq.addr) {
+ /*
+ * there are some devices (e.g. LIS3MDL) where drdy line is
+ * routed to a given pin and it is not possible to select a
+ * different one. Take into account irq status register
+ * to understand if irq trigger can be properly supported
+ */
+ if (sdata->sensor_settings->drdy_irq.addr_stat_drdy)
+ sdata->hw_irq_trigger = enable;
return 0;
+ }
/* Enable/Disable the interrupt generator 1. */
if (sdata->sensor_settings->drdy_irq.ig1.en_addr > 0) {
diff --git a/drivers/iio/magnetometer/st_magn_core.c b/drivers/iio/magnetometer/st_magn_core.c
index 83fdd7230917..f7388d265eb0 100644
--- a/drivers/iio/magnetometer/st_magn_core.c
+++ b/drivers/iio/magnetometer/st_magn_core.c
@@ -315,6 +315,10 @@ static const struct st_sensor_settings st_magn_sensors_settings[] = {
},
},
},
+ .drdy_irq = {
+ /* drdy line is routed drdy pin */
+ .addr_stat_drdy = ST_SENSORS_DEFAULT_STAT_ADDR,
+ },
.multi_read_bit = true,
.bootime = 2,
},
--
2.14.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] iio: magnetometer: st_magn: fix drdy line configuration for LIS3MDL
2017-08-29 11:45 [PATCH] iio: magnetometer: st_magn: fix drdy line configuration for LIS3MDL Lorenzo Bianconi
@ 2017-09-03 16:39 ` Jonathan Cameron
0 siblings, 0 replies; 2+ messages in thread
From: Jonathan Cameron @ 2017-09-03 16:39 UTC (permalink / raw)
To: Lorenzo Bianconi; +Cc: linux-iio, linus.walleij, lorenzo.bianconi
On Tue, 29 Aug 2017 13:45:11 +0200
Lorenzo Bianconi <lorenzo.bianconi83@gmail.com> wrote:
> Data-ready line in LIS3MDL is routed to drdy pin and it is not possible
> to select a different INT pin. st_sensors_set_dataready_irq() assumes
> that if drdy int address is not exported in register map, irq trigger
> is not supported by the sensor and hw_irq_trigger is always false.
> Based on this configuration st_sensors_irq_thread does not consume
> generated interrupt causing an unhandled irq.
> Fix this taking into account status register address in
> st_sensors_set_dataready_irq()
>
> Fixes: 90efe0556292 (iio: st_sensors: harden interrupt handling)
> Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi@st.com>
Applied to the fixes-togreg-post-rc1 branch of iio.git.
Thanks,
Jonathan.
> ---
> drivers/iio/common/st_sensors/st_sensors_core.c | 11 ++++++++++-
> drivers/iio/magnetometer/st_magn_core.c | 4 ++++
> 2 files changed, 14 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/iio/common/st_sensors/st_sensors_core.c b/drivers/iio/common/st_sensors/st_sensors_core.c
> index d99bb1460fe2..02e833b14db0 100644
> --- a/drivers/iio/common/st_sensors/st_sensors_core.c
> +++ b/drivers/iio/common/st_sensors/st_sensors_core.c
> @@ -463,8 +463,17 @@ int st_sensors_set_dataready_irq(struct iio_dev *indio_dev, bool enable)
> u8 drdy_mask;
> struct st_sensor_data *sdata = iio_priv(indio_dev);
>
> - if (!sdata->sensor_settings->drdy_irq.addr)
> + if (!sdata->sensor_settings->drdy_irq.addr) {
> + /*
> + * there are some devices (e.g. LIS3MDL) where drdy line is
> + * routed to a given pin and it is not possible to select a
> + * different one. Take into account irq status register
> + * to understand if irq trigger can be properly supported
> + */
> + if (sdata->sensor_settings->drdy_irq.addr_stat_drdy)
> + sdata->hw_irq_trigger = enable;
> return 0;
> + }
>
> /* Enable/Disable the interrupt generator 1. */
> if (sdata->sensor_settings->drdy_irq.ig1.en_addr > 0) {
> diff --git a/drivers/iio/magnetometer/st_magn_core.c b/drivers/iio/magnetometer/st_magn_core.c
> index 83fdd7230917..f7388d265eb0 100644
> --- a/drivers/iio/magnetometer/st_magn_core.c
> +++ b/drivers/iio/magnetometer/st_magn_core.c
> @@ -315,6 +315,10 @@ static const struct st_sensor_settings st_magn_sensors_settings[] = {
> },
> },
> },
> + .drdy_irq = {
> + /* drdy line is routed drdy pin */
> + .addr_stat_drdy = ST_SENSORS_DEFAULT_STAT_ADDR,
> + },
> .multi_read_bit = true,
> .bootime = 2,
> },
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-09-03 16:39 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-29 11:45 [PATCH] iio: magnetometer: st_magn: fix drdy line configuration for LIS3MDL Lorenzo Bianconi
2017-09-03 16:39 ` Jonathan Cameron
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.