From 0f32118e70c097f54fe706cb34adeaf9010a6a7a Mon Sep 17 00:00:00 2001 From: Sami Kerola Date: Sun, 17 Jul 2016 16:14:16 +0100 Subject: hwclock: use symbolic magic values passed in between functions The manipulate_clock() is seeing return value from busywait_for_rtc_clock_tick(). And the get_permissions_cmos() can see i386_iopl() return value. Reviewed-by: J William Piggott Signed-off-by: Sami Kerola --- sys-utils/hwclock-rtc.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'sys-utils/hwclock-rtc.c') diff --git a/sys-utils/hwclock-rtc.c b/sys-utils/hwclock-rtc.c index b974cc2aa..48a6f4d58 100644 --- a/sys-utils/hwclock-rtc.c +++ b/sys-utils/hwclock-rtc.c @@ -208,7 +208,7 @@ static int busywait_for_rtc_clock_tick(const struct hwclock_control *ctl, rc = do_rtc_read_ioctl(rtc_fd, &start_time); if (rc) - return 1; + return RTC_BUSYWAIT_FAILED; /* * Wait for change. Should be within a second, but in case @@ -223,13 +223,13 @@ static int busywait_for_rtc_clock_tick(const struct hwclock_control *ctl, gettimeofday(&now, NULL); if (time_diff(now, begin) > 1.5) { warnx(_("Timed out waiting for time change.")); - return 2; + return RTC_BUSYWAIT_TIMEOUT; } } while (1); if (rc) - return 3; - return 0; + return RTC_BUSYWAIT_FAILED; + return RTC_BUSYWAIT_OK; } /* -- cgit v1.2.3-55-g7522