From 2614daa834d3e561313177041b987095a376a888 Mon Sep 17 00:00:00 2001 From: Armando Visconti Date: Thu, 29 Jun 2023 10:54:54 +0200 Subject: [PATCH] drivers/sensor: lis2dh: move h/w reg debug print right after it is read Move LOG_DBG print just after the printed h/w register is read, to avoid coverity complaining about uninitialized variable. Fix: Coverity-CID: 316407 (issue #58591) Signed-off-by: Armando Visconti --- drivers/sensor/lis2dh/lis2dh_trigger.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/sensor/lis2dh/lis2dh_trigger.c b/drivers/sensor/lis2dh/lis2dh_trigger.c index a30536f037a662..774cd884a69723 100644 --- a/drivers/sensor/lis2dh/lis2dh_trigger.c +++ b/drivers/sensor/lis2dh/lis2dh_trigger.c @@ -387,6 +387,8 @@ static void lis2dh_thread_cb(const struct device *dev) LOG_ERR("clearing interrupt 2 failed: %d", status); return; } + + LOG_DBG("@tick=%u int2_src=0x%x", k_cycle_get_32(), reg_val); } if (likely(lis2dh->handler_anymotion != NULL)) { @@ -400,9 +402,6 @@ static void lis2dh_thread_cb(const struct device *dev) setup_int2(dev, true); } - LOG_DBG("@tick=%u int2_src=0x%x", k_cycle_get_32(), - reg_val); - return; } }