accel_list2dw12: Set high perf mode for ODR >= 50z

Until recently we're used it only as lid accel, where no significant
precision is required. However in Winterhold we want to use it for
body detection. In order to reduce the noise and make it usable for that
purpose we need to set it into "high performance" power mode.
ODR is used to recognize the sensor purpose.
For lid accel it's set to 12.5Hz, assume that if it's at least 50Hz
the high performance is wanted.

BUG=b:262680246
BRANCH=none
TEST=Set ODR to 50Hz, verify high performance mode is on

Change-Id: I8c4decc93d2f2cfcc074cf2399f5cb582eb72696
Signed-off-by: Kornel Dulęba <[email protected]>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4379149
Reviewed-by: Gwendal Grignou <[email protected]>

Author: kornelduleba

driver/accel_lis2dw12.c

@@ -360,7 +360,7 @@ static int set_data_rate(const struct motion_sensor_t *s, int rate, int rnd)
 	 * But lis2dw12 needs switch low power mode according to odr value.
 	 */
 	if (!IS_ENABLED(CONFIG_ACCEL_LIS2DWL)) {
-		if (reg_val > LIS2DW12_ODR_200HZ_VAL)
+		if (reg_val >= LIS2DW12_ODR_50HZ_VAL)
 			ret = lis2dw12_set_power_mode(s, LIS2DW12_HIGH_PERF, 0);
 		else
 			ret = lis2dw12_set_power_mode(
@@ -392,7 +392,7 @@ static int get_rms_noise(const struct motion_sensor_t *s)
 	rate = INT_TO_FP(st_get_data_rate(s) / 1000);
 
 	/*
-	 * LIS2DW12: 90ug/sqrt(Hz) when ODR is over 200Hz
+	 * LIS2DW12: 90ug/sqrt(Hz) when ODR is over 50Hz
 	 * When lower, we are in power mode 2, so the noise density does not
 	 * depend on frequency and the RMS at +/-2g is 2.4mg.
 	 *
@@ -401,7 +401,7 @@ static int get_rms_noise(const struct motion_sensor_t *s)
 	 */
 
 	if (!IS_ENABLED(CONFIG_ACCEL_LIS2DWL)) {
-		if (rate < INT_TO_FP(200))
+		if (rate < INT_TO_FP(50))
 			return 2400;
 		noise_density_ug = INT_TO_FP(90);
 	} else {

zephyr/test/drivers/default/src/lis2dw12.c

@@ -337,14 +337,14 @@ ZTEST(lis2dw12, test_lis2dw12_set_rate)
 			test_params[i].expected_reg_val, odr_bits,
 			LIS2DW12_ODR_MAX_VAL);
 
-		/* Check if high performance mode was enabled if rate >
-		 * 200,000mHz
+		/* Check if high performance mode was enabled if rate >=
+		 * 50,000mHz
 		 */
 
 		uint8_t mode_bits = lis2dw12_emul_peek_mode(emul);
 		uint8_t lpmode_bits = lis2dw12_emul_peek_lpmode(emul);
 
-		if (odr_bits > LIS2DW12_ODR_200HZ_VAL) {
+		if (odr_bits >= LIS2DW12_ODR_50HZ_VAL) {
 			/* High performance mode, LP mode immaterial */
 			zassert_equal(mode_bits, LIS2DW12_HIGH_PERF,
 				      "MODE[1:0] should be 0x%x, but got 0x%x",