drivers: sensor: lsm6dsl: fixed irq_pin for interrupt handling

Yurii Gubin · MaureenHelm · commit f700d7488497 · 2020-11-06T12:48:33.000-06:00
Obtaining irq_pin from dev leads to disabling interrupts on unpredictable pin, as dev points to GPIO, not LSM6DSL. Fixes #29721 Signed-off-by: Yurii Gubin <y.gubin@gmail.com>
diff --git a/drivers/sensor/lsm6dsl/lsm6dsl_trigger.c b/drivers/sensor/lsm6dsl/lsm6dsl_trigger.c
@@ -75,9 +75,9 @@ int lsm6dsl_trigger_set(const struct device *dev,
 static void lsm6dsl_gpio_callback(const struct device *dev,
 				  struct gpio_callback *cb, uint32_t pins)
 {
-	const struct lsm6dsl_config *config = dev->config;
 	struct lsm6dsl_data *drv_data =
 		CONTAINER_OF(cb, struct lsm6dsl_data, gpio_cb);
+	const struct lsm6dsl_config *config = drv_data->dev->config;
 
 	ARG_UNUSED(pins);
 

Original file line number	Diff line number	Diff line change
`@@ -75,9 +75,9 @@ int lsm6dsl_trigger_set(const struct device *dev,`
`75`	`75`	`static void lsm6dsl_gpio_callback(const struct device *dev,`
`76`	`76`	`struct gpio_callback *cb, uint32_t pins)`
`77`	`77`	`{`
`78`		`- const struct lsm6dsl_config *config = dev->config;`
`79`	`78`	`struct lsm6dsl_data *drv_data =`
`80`	`79`	`CONTAINER_OF(cb, struct lsm6dsl_data, gpio_cb);`
	`80`	`+ const struct lsm6dsl_config *config = drv_data->dev->config;`
`81`	`81`
`82`	`82`	`ARG_UNUSED(pins);`
`83`	`83`