drivers/sensor: lis2dux12: add high performance mode

Add driver capability to properly set high performance mode
while setting data rate (thru lis2duxxx_mode_set() API)
based on how power-mode is set into DTS: if it is set to
LIS2DUX12_OPER_MODE_HIGH_PERFORMANCE then configure HP mode,
LP/ULP mode otherwise.

Signed-off-by: Armando Visconti <[email protected]>

Author: avisconti

drivers/sensor/st/lis2dux12/lis2dux12.c

@@ -50,21 +50,33 @@ static const float lis2dux12_odr_map[LIS2DUX12_DT_ODR_END] = {FOREACH_ODR_ENUM(G
 static int lis2dux12_freq_to_odr_val(const struct device *dev, uint16_t freq)
 {
 	const struct lis2dux12_config *cfg = dev->config;
+	int odr;
 
-	/* constrain loop to prevent erroneous power mode/odr combinations */
-	size_t i = (cfg->pm != LIS2DUX12_OPER_MODE_LOW_POWER) ? LIS2DUX12_DT_ODR_6Hz
-							      : LIS2DUX12_DT_ODR_1Hz_ULP;
-	size_t len = (cfg->pm != LIS2DUX12_OPER_MODE_LOW_POWER) ? LIS2DUX12_DT_ODR_END
-								: LIS2DUX12_DT_ODR_6Hz;
+	for (odr = LIS2DUX12_DT_ODR_OFF; odr < LIS2DUX12_DT_ODR_END; odr++) {
+		if (freq <= lis2dux12_odr_map[odr]) {
+			break;
+		}
+	}
 
-	while (i < len) {
-		if (freq <= lis2dux12_odr_map[i]) {
-			return i;
+	if (unlikely(odr == LIS2DUX12_DT_ODR_END)) {
+		/* no valid odr found */
+		return -EINVAL;
+	}
+
+	if (unlikely(odr == LIS2DUX12_DT_ODR_OFF)) {
+		return LIS2DUX12_DT_ODR_OFF;
+	}
+
+	/* handle high performance mode */
+	if (cfg->pm == LIS2DUX12_OPER_MODE_HIGH_PERFORMANCE) {
+		if (odr < LIS2DUX12_DT_ODR_6Hz) {
+			odr = LIS2DUX12_DT_ODR_6Hz;
 		}
-		++i;
+
+		odr |= 0x10;
 	}
 
-	return -EINVAL;
+	return odr;
 }
 
 static int lis2dux12_set_fs(const struct device *dev, int16_t fs)

dts/bindings/sensor/st,lis2dux12-common.yaml

@@ -5,7 +5,7 @@ description: |
     When setting the odr, power-mode, and range properties in a .dts or .dtsi file you may include
     st_lis2dux12.h and use the macros defined there.
     Example:
-    #include <zephyr/dt-bindings/sensor/st_lis2dux12.h>
+    #include <zephyr/dt-bindings/sensor/lis2dux12.h>
     lis2dux12: lis2dux12@0 {
       ...
       power-mode = <LIS2DUX12_OPER_MODE_LOW_POWER>;
@@ -68,8 +68,8 @@ properties:
 
       - 0 # LIS2DUX12_OPER_MODE_POWER_DOWN
       - 1 # LIS2DUX12_OPER_MODE_LOW_POWER
-      - 2 # LIS2DUX12_OPER_MODE_HIGH_RESOLUTION
-      - 3 # LIS2DUX12_OPER_MODE_HIGH_FREQUENCY
+      - 2 # LIS2DUX12_OPER_MODE_HIGH_PERFORMANCE
+      - 3 # LIS2DUX12_OPER_MODE_SINGLE_SHOT
 
     enum: [0, 1, 2, 3]
 

include/zephyr/dt-bindings/sensor/lis2dux12.h

@@ -9,10 +9,10 @@
 #include <zephyr/dt-bindings/dt-util.h>
 
 /* Operating Mode */
-#define LIS2DUX12_OPER_MODE_POWER_DOWN      0
-#define LIS2DUX12_OPER_MODE_LOW_POWER       1
-#define LIS2DUX12_OPER_MODE_HIGH_RESOLUTION 2
-#define LIS2DUX12_OPER_MODE_HIGH_FREQUENCY  3
+#define LIS2DUX12_OPER_MODE_POWER_DOWN        0
+#define LIS2DUX12_OPER_MODE_LOW_POWER         1
+#define LIS2DUX12_OPER_MODE_HIGH_PERFORMANCE  2
+#define LIS2DUX12_OPER_MODE_SINGLE_SHOT       3
 
 /* Data rate */
 #define LIS2DUX12_DT_ODR_OFF      0

tests/drivers/build_all/sensor/i2c.dtsi

@@ -1010,7 +1010,7 @@ test_i2c_lis2dux12: lis2dux12@8a {
 	int2-gpios = <&test_gpio 0 0>;
 	range = <LIS2DUX12_DT_FS_16G>;
 	odr = <LIS2DUX12_DT_ODR_100Hz>;
-	power-mode = <LIS2DUX12_OPER_MODE_HIGH_FREQUENCY>;
+	power-mode = <LIS2DUX12_OPER_MODE_HIGH_PERFORMANCE>;
 	status = "okay";
 };
 
@@ -1227,7 +1227,7 @@ test_i2c_lis2duxs12: lis2duxs12@a7 {
 	int2-gpios = <&test_gpio 0 0>;
 	range = <LIS2DUX12_DT_FS_16G>;
 	odr = <LIS2DUX12_DT_ODR_100Hz>;
-	power-mode = <LIS2DUX12_OPER_MODE_HIGH_FREQUENCY>;
+	power-mode = <LIS2DUX12_OPER_MODE_HIGH_PERFORMANCE>;
 	status = "okay";
 };