]> git.ipfire.org Git - thirdparty/util-linux.git/commitdiff
hwclock: improve RTC epoch messages
authorJ William Piggott <elseifthen@gmx.com>
Sat, 15 Jul 2017 19:44:20 +0000 (15:44 -0400)
committerJ William Piggott <elseifthen@gmx.com>
Sun, 16 Jul 2017 12:41:54 +0000 (08:41 -0400)
Make consistent and improve messages in get_epoch_rtc()
and set_epoch_rtc().

Signed-off-by: J William Piggott <elseifthen@gmx.com>
sys-utils/hwclock-rtc.c

index 580356117a9f5c0e21a847b5094c3d3b46bc7b44..2af2666d2057bca677ed48010a26ceb1b4ee62db 100644 (file)
@@ -403,24 +403,21 @@ int get_epoch_rtc(const struct hwclock_control *ctl, unsigned long *epoch_p)
        rtc_fd = open_rtc(ctl);
        if (rtc_fd < 0) {
                if (errno == ENOENT)
-                       warnx(_
-                             ("To manipulate the epoch value in the kernel, we must "
-                              "access the Linux 'rtc' device driver via the device special "
-                              "file.  This file does not exist on this system."));
+                       warnx(_("%s does not exist."), rtc_dev_name);
                else
-                       warn(_("cannot open rtc device"));
+                       warn(_("cannot open %s"), rtc_dev_name);
                return 1;
        }
 
        if (ioctl(rtc_fd, RTC_EPOCH_READ, epoch_p) == -1) {
-               warn(_("ioctl(RTC_EPOCH_READ) to %s failed"), rtc_dev_name);
+               warn(_("ioctl(%d, RTC_EPOCH_READ, epoch_p) to %s failed"),
+                    rtc_fd, rtc_dev_name);
                return 1;
        }
 
        if (ctl->debug)
-               printf(_("we have read epoch %lu from %s "
-                        "with RTC_EPOCH_READ ioctl.\n"), *epoch_p,
-                      rtc_dev_name);
+               printf(_("ioctl(%d, RTC_EPOCH_READ, epoch_p) to %s succeeded.\n"),
+                      rtc_fd, rtc_dev_name);
 
        return 0;
 }
@@ -444,24 +441,22 @@ int set_epoch_rtc(const struct hwclock_control *ctl)
        rtc_fd = open_rtc(ctl);
        if (rtc_fd < 0) {
                if (errno == ENOENT)
-                       warnx(_
-                             ("To manipulate the epoch value in the kernel, we must "
-                              "access the Linux 'rtc' device driver via the device special "
-                              "file.  This file does not exist on this system."));
+                       warnx(_("%s does not exist."), rtc_dev_name);
                else
-                       warn(_("cannot open rtc device"));
+                       warn(_("cannot open %s"), rtc_dev_name);
                return 1;
        }
 
-       if (ctl->debug)
-               printf(_("setting epoch to %lu "
-                        "with RTC_EPOCH_SET ioctl to %s.\n"), epoch,
-                      rtc_dev_name);
        if (ioctl(rtc_fd, RTC_EPOCH_SET, epoch) == -1) {
-               warn(_("ioctl(RTC_EPOCH_SET) to %s failed"), rtc_dev_name);
+               warn(_("ioctl(%d, RTC_EPOCH_SET, %lu) to %s failed"),
+                    rtc_fd, epoch, rtc_dev_name);
                return 1;
        }
 
+       if (ctl->debug)
+               printf(_("ioctl(%d, RTC_EPOCH_SET, %lu) to %s succeeded.\n"),
+                      rtc_fd, epoch, rtc_dev_name);
+
        return 0;
 }
 #endif /* __alpha__ */