From: Ahmad Fatoum <a.fatoum@pengutronix.de>
Date: Wed, 17 Jul 2024 11:00:39 +0000 (+0200)
Subject: rtc: factor out RTC_Linux_ReadTimeAfterInterrupt
X-Git-Tag: 4.7-pre1~55
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=924199ef69fce4c343e2cce88fc04e68ba83f019;p=thirdparty%2Fchrony.git

rtc: factor out RTC_Linux_ReadTimeAfterInterrupt

We have code to read time after an RTC's UIE interrupt, which we
currently use as part of maintaining the correction file.

In a later commit, we will need the same functionality for using the RTC
as reference clock, so export the function and give it a descriptive
name appropriate for a globally visible function.
---

diff --git a/rtc_linux.c b/rtc_linux.c
index 06e19308..7e3b904d 100644
--- a/rtc_linux.c
+++ b/rtc_linux.c
@@ -785,11 +785,31 @@ RTC_Linux_CheckInterrupt(int fd)
   return (data & RTC_UF) == RTC_UF;
 }
 
+time_t
+RTC_Linux_ReadTimeAfterInterrupt(int fd, int utc, struct timespec *sys_time_cooked)
+{
+  int status;
+  struct rtc_time rtc_raw;
+
+  /* Read RTC time, sandwiched between two polls of the system clock
+     so we can bound any error */
+
+  SCH_GetLastEventTime(sys_time_cooked, NULL, NULL);
+
+  status = ioctl(fd, RTC_RD_TIME, &rtc_raw);
+  if (status < 0) {
+    LOG(LOGS_ERR, "Could not read time from %s : %s", CNF_GetRtcDevice(), strerror(errno));
+    return -1;
+  }
+
+  /* Convert RTC time into a struct timespec */
+  return t_from_rtc(&rtc_raw, utc);
+}
+
 static void
 read_from_device(int fd_, int event, void *any)
 {
   struct timespec sys_time;
-  struct rtc_time rtc_raw;
   int status, error = 0;
   time_t rtc_t;
 
@@ -801,22 +821,12 @@ read_from_device(int fd_, int event, void *any)
     fd = -1;
     return;
   } else if (status == 0) {
+    /* Wait for the next interrupt, this one may be bogus */
     return;
   }
 
-  SCH_GetLastEventTime(&sys_time, NULL, NULL);
-
-  status = ioctl(fd, RTC_RD_TIME, &rtc_raw);
-  if (status < 0) {
-    LOG(LOGS_ERR, "Could not read time from %s : %s", CNF_GetRtcDevice(), strerror(errno));
-    error = 1;
-    goto turn_off_interrupt;
-  }
-
-  /* Convert RTC time into a struct timespec */
-  rtc_t = t_from_rtc(&rtc_raw, rtc_on_utc);
-
-  if (rtc_t == (time_t)(-1)) {
+  rtc_t = RTC_Linux_ReadTimeAfterInterrupt(fd, rtc_on_utc, &sys_time);
+  if (rtc_t == (time_t)-1) {
     error = 1;
     goto turn_off_interrupt;
   }
diff --git a/rtc_linux.h b/rtc_linux.h
index 1f67ba80..952736d9 100644
--- a/rtc_linux.h
+++ b/rtc_linux.h
@@ -44,5 +44,7 @@ extern void RTC_Linux_CycleLogFile(void);
 
 extern int RTC_Linux_SwitchInterrupt(int fd, int on_off);
 extern int RTC_Linux_CheckInterrupt(int fd);
+extern time_t RTC_Linux_ReadTimeAfterInterrupt(int fd, int utc,
+                                               struct timespec *sys_time_cooked);
 
 #endif /* _GOT_RTC_LINUX_H */