periodic_event: Don't use RTC for 1s event.

It complicates the code and really doesn't add a anything.
jakkra · Sep 12, 2024 · 2205f3a · 2205f3a
1 parent 50f6aed
commit 2205f3a
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 57 deletions.
diff --git a/app/lvgl_resources b/app/lvgl_resources
diff --git a/app/src/events/zsw_periodic_event.c b/app/src/events/zsw_periodic_event.c
@@ -6,21 +6,12 @@
 #include "events/periodic_event.h"
 
 #define PERIODIC_FAST_INTERVAL_MS 100
-#define PERIODIC_SLOW_INTERVAL_MS 10000
-
-#if CONFIG_RTC_UPDATE
-#include <zephyr/drivers/rtc.h>
-
-static const struct device *const rtc = DEVICE_DT_GET(DT_ALIAS(rtc));
-
-static void handle_mid_timeout(const struct device *dev, void *user_data);
-#else
 #define PERIODIC_MID_INTERVAL_MS 1000
-#endif
+#define PERIODIC_SLOW_INTERVAL_MS 10000
 
+ZBUS_CHAN_DECLARE(periodic_event_100ms_chan);
 ZBUS_CHAN_DECLARE(periodic_event_1s_chan);
 ZBUS_CHAN_DECLARE(periodic_event_10s_chan);
-ZBUS_CHAN_DECLARE(periodic_event_100ms_chan);
 
 int zsw_periodic_chan_add_obs(const struct zbus_channel *chan, const struct zbus_observer *obs)
 {
@@ -36,15 +27,12 @@ int zsw_periodic_chan_add_obs(const struct zbus_channel *chan, const struct zbus
     zbus_chan_claim(chan, K_FOREVER);
     work = (struct k_work_delayable *)zbus_chan_user_data(chan);
     __ASSERT(work != NULL, "Invalid channel");
+
     if (!k_work_delayable_is_pending(work)) {
         if (chan == &periodic_event_10s_chan) {
             ret =  k_work_reschedule(work, K_MSEC(PERIODIC_SLOW_INTERVAL_MS));
         } else if (chan == &periodic_event_1s_chan) {
-#if CONFIG_RTC_UPDATE
-            ret = rtc_update_set_callback(rtc, handle_mid_timeout, NULL);
-#else
             ret =  k_work_reschedule(work, K_MSEC(PERIODIC_MID_INTERVAL_MS));
-#endif
         } else if (chan == &periodic_event_100ms_chan) {
             ret =  k_work_reschedule(work, K_MSEC(PERIODIC_FAST_INTERVAL_MS));
         } else {
@@ -58,19 +46,12 @@ int zsw_periodic_chan_add_obs(const struct zbus_channel *chan, const struct zbus
 
 int zsw_periodic_chan_rm_obs(const struct zbus_channel *chan, const struct zbus_observer *obs)
 {
+    struct k_work_delayable *work = NULL;
     int ret = zbus_chan_rm_obs(chan, obs, K_MSEC(100));
     if (ret == 0 && sys_slist_is_empty(&chan->data->observers)) {
-#if CONFIG_RTC_UPDATE
-        if (chan == &periodic_event_1s_chan) {
-            ret = rtc_update_set_callback(rtc, NULL, NULL);
-        } else
-#endif
-        {
-            struct k_work_delayable *work = NULL;
-            work = (struct k_work_delayable *)zbus_chan_user_data(chan);
-            __ASSERT(k_work_delayable_is_pending(work), "Periodic slow work is not pending");
-            ret = k_work_cancel_delayable(work);
-        }
+        work = (struct k_work_delayable *)zbus_chan_user_data(chan);
+        __ASSERT(k_work_delayable_is_pending(work), "Periodic slow work is not pending");
+        ret = k_work_cancel_delayable(work);
     }
     return ret;
 }
@@ -88,15 +69,6 @@ static void handle_slow_timeout(struct k_work *item)
     zbus_chan_pub(&periodic_event_10s_chan, &evt, K_MSEC(250));
 }
 
-#if CONFIG_RTC_UPDATE
-static void handle_mid_timeout(const struct device *dev, void *user_data)
-{
-    struct periodic_event evt = {
-    };
-
-    zbus_chan_pub(&periodic_event_1s_chan, &evt, K_MSEC(250));
-}
-#else
 static void handle_mid_timeout(struct k_work *item)
 {
     struct periodic_event evt = {
@@ -109,7 +81,6 @@ static void handle_mid_timeout(struct k_work *item)
 
     zbus_chan_pub(&periodic_event_1s_chan, &evt, K_MSEC(250));
 }
-#endif
 
 static void handle_fast_timeout(struct k_work *item)
 {
@@ -131,10 +102,8 @@ static int zsw_timer_init(void)
     work = (struct k_work_delayable *)zbus_chan_user_data(&periodic_event_10s_chan);
     k_work_init_delayable(work, handle_slow_timeout);
 
-#ifndef CONFIG_RTC_UPDATE
     work = (struct k_work_delayable *)zbus_chan_user_data(&periodic_event_1s_chan);
     k_work_init_delayable(work, handle_mid_timeout);
-#endif
 
     work = (struct k_work_delayable *)zbus_chan_user_data(&periodic_event_100ms_chan);
     k_work_init_delayable(work, handle_fast_timeout);

diff --git a/app/src/main.c b/app/src/main.c
@@ -287,22 +287,6 @@ static void run_wdt_work(struct k_work *item)
 
 int main(void)
 {
-    /*
-    while (1) {
-        struct i2c_msg msgs[1];
-        uint8_t dst;
-
-        for (uint8_t i = 0; i < 127; i++) {
-            msgs[0].buf = &dst;
-            msgs[0].len = 0U;
-            msgs[0].flags = I2C_MSG_WRITE | I2C_MSG_STOP;
-            if (i2c_transfer(i2c, &msgs[0], 1, i) == 0) {
-                LOG_INF("Found device at 0x%X", i);
-            }
-        }
-
-        k_msleep(1000);
-    }*/
 #ifdef CONFIG_SPI_FLASH_LOADER
     if (bootmode_check(ZSW_BOOT_MODE_RTT_FLASH_LOADER)) {
         LOG_WRN("SPI Flash Loader Boot Mode");

diff --git a/app/src/zsw_clock.h b/app/src/zsw_clock.h
@@ -35,15 +35,13 @@ typedef struct {
     uint32_t    tv_usec;
 } zsw_timeval_t;
 
-
 /**
  * Sets the time of the clock.
  *
  * @param ztm A pointer to a `zsw_timeval_t` structure representing the desired time.
  */
 void zsw_clock_set_time(zsw_timeval_t *ztm);
 
-
 /**
  * @brief Retrieves the current time from the.
  *
@@ -53,7 +51,6 @@ void zsw_clock_set_time(zsw_timeval_t *ztm);
  */
 void zsw_clock_get_time(zsw_timeval_t *ztm);
 
-
 /**
  * @brief Sets the timezone for the ZSW clock.
  *