drivers/sensor: lis2dux12: fix odr and range

The current implementation assumes that sensor odr and range are
always configured in the Device Tree at compile time which might
not be the case.

Instead, application can set odr and range either at compile time
through the DT or using SENSOR_ATTR_SAMPLING_FREQUENCY and
SENSOR_ATTR_FULL_SCALE attributes at runtime, so each driver instance
must keep trace of the latest values set and use them in the sensor
APIs which require them (e.g. lis2dux12_mode_set).

Signed-off-by: Armando Visconti <[email protected]>
(cherry picked from commit cf20aa0)

Author: avisconti

drivers/sensor/st/lis2dux12/lis2dux12.c

@@ -27,10 +27,12 @@ LOG_MODULE_REGISTER(LIS2DUX12, CONFIG_SENSOR_LOG_LEVEL);
 
 static int lis2dux12_set_odr(const struct device *dev, uint8_t odr)
 {
+	struct lis2dux12_data *data = dev->data;
 	const struct lis2dux12_config *cfg = dev->config;
 	stmdev_ctx_t *ctx = (stmdev_ctx_t *)&cfg->ctx;
 	lis2dux12_md_t mode = {.odr = odr};
 
+	data->odr = odr;
 	return lis2dux12_mode_set(ctx, &mode);
 }
 
@@ -40,7 +42,7 @@ static int lis2dux12_set_range(const struct device *dev, uint8_t range)
 	struct lis2dux12_data *data = dev->data;
 	const struct lis2dux12_config *cfg = dev->config;
 	stmdev_ctx_t *ctx = (stmdev_ctx_t *)&cfg->ctx;
-	lis2dux12_md_t val = { .odr = cfg->odr, .fs = range };
+	lis2dux12_md_t val = { .odr = data->odr, .fs = range };
 
 	err = lis2dux12_mode_set(ctx, &val);
 
@@ -66,6 +68,7 @@ static int lis2dux12_set_range(const struct device *dev, uint8_t range)
 		break;
 	}
 
+	data->range = range;
 	return 0;
 }
 
@@ -187,7 +190,7 @@ static int lis2dux12_sample_fetch_accel(const struct device *dev)
 	stmdev_ctx_t *ctx = (stmdev_ctx_t *)&cfg->ctx;
 
 	/* fetch raw data sample */
-	lis2dux12_md_t mode = {.fs = cfg->range};
+	lis2dux12_md_t mode = {.fs = data->range};
 	lis2dux12_xl_data_t xzy_data = {0};
 
 	if (lis2dux12_xl_data_get(ctx, &mode, &xzy_data) < 0) {
@@ -355,11 +358,7 @@ static int lis2dux12_init(const struct device *dev)
 
 	/* set sensor default pm and odr */
 	LOG_DBG("%s: pm: %d, odr: %d", dev->name, cfg->pm, cfg->odr);
-	lis2dux12_md_t mode = {
-		.odr = cfg->odr,
-		.fs = cfg->range,
-	};
-	ret = lis2dux12_mode_set(ctx, &mode);
+	ret = lis2dux12_set_odr(dev, cfg->odr);
 	if (ret < 0) {
 		LOG_ERR("%s: odr init error (12.5 Hz)", dev->name);
 		return ret;

drivers/sensor/st/lis2dux12/lis2dux12.h

@@ -51,6 +51,8 @@ struct lis2dux12_data {
 	int sample_y;
 	int sample_z;
 	float gain;
+	uint8_t range;
+	uint8_t odr;
 
 #ifdef CONFIG_LIS2DUX12_ENABLE_TEMP
 	int sample_temp;