Skip to content

Commit d0d29aa

Browse files
anthony289478cfriedt
authored andcommitted
sensor: icm45686: Swap ASSERT to CHECKIF in fifo decode
Change the __ASSERT on unsupported fifo packet header to CHECKIF so the user can choose how to handle the invalid packet. Signed-off-by: Anthony Williams <[email protected]>
1 parent 2df6c36 commit d0d29aa

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

drivers/sensor/tdk/icm45686/icm45686_decoder.c

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
*/
88

99
#include <zephyr/drivers/sensor_clock.h>
10+
#include <zephyr/sys/check.h>
1011

1112
#include "icm45686.h"
1213
#include "icm45686_reg.h"
@@ -489,11 +490,13 @@ static int icm45686_fifo_decode(const uint8_t *buffer,
489490
/** This driver assumes 20-byte fifo packets, with both accel and gyro,
490491
* and no auxiliary sensors.
491492
*/
492-
__ASSERT(!(fdata->header & FIFO_HEADER_EXT_HEADER_EN(true)) &&
493+
CHECKIF(!(!(fdata->header & FIFO_HEADER_EXT_HEADER_EN(true)) &&
493494
(fdata->header & FIFO_HEADER_ACCEL_EN(true)) &&
494495
(fdata->header & FIFO_HEADER_GYRO_EN(true)) &&
495-
(fdata->header & FIFO_HEADER_HIRES_EN(true)),
496-
"Unsupported FIFO packet format 0x%02x", fdata->header);
496+
(fdata->header & FIFO_HEADER_HIRES_EN(true)))) {
497+
LOG_ERR("Unsupported FIFO packet format 0x%02x", fdata->header);
498+
return -ENOTSUP;
499+
}
497500

498501
switch (chan_spec.chan_type) {
499502
case SENSOR_CHAN_ACCEL_XYZ:

0 commit comments

Comments
 (0)