drivers: sensor: iis2iclx: Store sensor trigger as a pointer

Fixes the iis2iclx sensor driver to store the user-supplied sensor
trigger as a pointer rather than a copy. This enables the trigger
handler to use CONTAINER_OF to retrieve a context pointer when the
trigger is embedded in a larger struct.

Signed-off-by: Maureen Helm <[email protected]>

Author: MaureenHelm

drivers/sensor/iis2iclx/iis2iclx.h

@@ -86,7 +86,9 @@ struct iis2iclx_data {
 #ifdef CONFIG_IIS2ICLX_TRIGGER
 	struct gpio_callback gpio_cb;
 	sensor_trigger_handler_t handler_drdy_acc;
+	const struct sensor_trigger *trig_drdy_acc;
 	sensor_trigger_handler_t handler_drdy_temp;
+	const struct sensor_trigger *trig_drdy_temp;
 
 #if defined(CONFIG_IIS2ICLX_TRIGGER_OWN_THREAD)
 	K_KERNEL_STACK_MEMBER(thread_stack, CONFIG_IIS2ICLX_THREAD_STACK_SIZE);

drivers/sensor/iis2iclx/iis2iclx_trigger.c

@@ -101,6 +101,7 @@ int iis2iclx_trigger_set(const struct device *dev,
 
 	if (trig->chan == SENSOR_CHAN_ACCEL_XYZ) {
 		iis2iclx->handler_drdy_acc = handler;
+		iis2iclx->trig_drdy_acc = trig;
 		if (handler) {
 			return iis2iclx_enable_xl_int(dev, IIS2ICLX_EN_BIT);
 		} else {
@@ -110,6 +111,7 @@ int iis2iclx_trigger_set(const struct device *dev,
 #if defined(CONFIG_IIS2ICLX_ENABLE_TEMP)
 	else if (trig->chan == SENSOR_CHAN_DIE_TEMP) {
 		iis2iclx->handler_drdy_temp = handler;
+		iis2iclx->trig_drdy_temp = trig;
 		if (handler) {
 			return iis2iclx_enable_t_int(dev, IIS2ICLX_EN_BIT);
 		} else {
@@ -128,9 +130,6 @@ int iis2iclx_trigger_set(const struct device *dev,
 static void iis2iclx_handle_interrupt(const struct device *dev)
 {
 	struct iis2iclx_data *iis2iclx = dev->data;
-	struct sensor_trigger drdy_trigger = {
-		.type = SENSOR_TRIG_DATA_READY,
-	};
 	const struct iis2iclx_config *cfg = dev->config;
 	iis2iclx_status_reg_t status;
 
@@ -150,12 +149,12 @@ static void iis2iclx_handle_interrupt(const struct device *dev)
 		}
 
 		if ((status.xlda) && (iis2iclx->handler_drdy_acc != NULL)) {
-			iis2iclx->handler_drdy_acc(dev, &drdy_trigger);
+			iis2iclx->handler_drdy_acc(dev, iis2iclx->trig_drdy_acc);
 		}
 
 #if defined(CONFIG_IIS2ICLX_ENABLE_TEMP)
 		if ((status.tda) && (iis2iclx->handler_drdy_temp != NULL)) {
-			iis2iclx->handler_drdy_temp(dev, &drdy_trigger);
+			iis2iclx->handler_drdy_temp(dev, iis2iclx->trig_drdy_temp);
 		}
 #endif
 	}