Package: chrony
Version: 1.24-3.1
Severity: normal
Tags: patch

RTC support for Linux 3.0 is also missing in chrony.
My patch (ignoring spaces) is the following, I hope it's correct because I
haven't checked if there're RTC changes in linux 3.0. I send attached the
full patch.

--- rtc_linux.c.orig    2010-02-04 13:07:19.000000000 +0100
+++ rtc_linux.c 2011-09-20 13:23:05.000000000 +0200
@@ -575,9 +575,11 @@

   /* Obviously this test can get more elaborate when we know about
      more system types. */
-  if (major != 2) {
+  switch (major) {
+    case 1:
     return 0;
-  } else {
+      break;
+    case 2:
     switch (minor) {
       case 0:
         if (patch <= 31) {
@@ -596,6 +598,9 @@
       case 8:
         break; /* OK for all patch levels */
     }
+      break;
+    case 3:
+      break; /* OK for all patch levels */
   }

   /* Setup details depending on configuration options */

Regards,
Roberto Lumbreras
Debian developer
--- rtc_linux.c.orig	2010-02-04 13:07:19.000000000 +0100
+++ rtc_linux.c	2011-09-20 13:23:05.000000000 +0200
@@ -575,27 +575,32 @@
 
   /* Obviously this test can get more elaborate when we know about
      more system types. */
-  if (major != 2) {
-    return 0;
-  } else {
-    switch (minor) {
-      case 0:
-        if (patch <= 31) {
+  switch (major) {
+    case 1:
+      return 0;
+      break;
+    case 2:
+      switch (minor) {
+        case 0:
+          if (patch <= 31) {
+            return 0;
+          }
+          break;
+        case 1:
           return 0;
-        }
-        break;
-      case 1:
-        return 0;
-        break;
-      case 2:
-      case 3:
-      case 4:
-      case 5:
-      case 6:
-      case 7:
-      case 8:
-        break; /* OK for all patch levels */
-    } 
+          break;
+        case 2:
+        case 3:
+        case 4:
+        case 5:
+        case 6:
+        case 7:
+        case 8:
+          break; /* OK for all patch levels */
+      } 
+      break;
+    case 3:
+      break; /* OK for all patch levels */
   }
 
   /* Setup details depending on configuration options */

Reply via email to