userspace/libwr: bugfix: printout of time adjustment was meaningless

a long-long item wants llabs(), not abs(). Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

userspace/libwr: bugfix: printout of time adjustment was meaningless
a long-long item wants llabs(), not abs(). Signed-off-by: Alessandro Rubini <rubini@gnudd.com>
5127702b · Alessandro Rubini · 26f5d737 · 5127702b
Commit 5127702b authored Dec 04, 2014 by Alessandro Rubini
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 5 deletions

pps_gen.c userspace/libwr/pps_gen.c +4 -5

No files found.
--- a/userspace/libwr/pps_gen.c
+++ b/userspace/libwr/pps_gen.c
@@ -47,16 +47,15 @@ int shw_pps_gen_init()
 /* Adjusts the nanosecond (refclk cycle) counter by atomically adding (how_much) cycles. */
 int shw_pps_gen_adjust(int counter, int64_t how_much)
 {
-	TRACE(TRACE_INFO, "Adjust: counter = %s [%c%lld]",
+	TRACE(TRACE_INFO, "Adjust: counter = %s [%+lld]",
 	      counter == PPSG_ADJUST_SEC ? "seconds" : "nanoseconds",
-	      how_much < 0 ? '-' : '+', abs(how_much));
+	      llabs(how_much));

 	if (counter == PPSG_ADJUST_NSEC) {
 		ppsg_write(ADJ_UTCLO, 0);
 		ppsg_write(ADJ_UTCHI, 0);
-		ppsg_write(ADJ_NSEC,
-			   (int32_t) ((int64_t) how_much * 1000LL /
-				      (int64_t) REF_CLOCK_PERIOD_PS));
+		ppsg_write(ADJ_NSEC, /* convert to pico for conversion */
+			   (int32_t) (how_much * 1000 / REF_CLOCK_PERIOD_PS));
 	} else {
 		ppsg_write(ADJ_UTCLO, (uint32_t) (how_much & 0xffffffffLL));
 		ppsg_write(ADJ_UTCHI, (uint32_t) (how_much >> 32) & 0xff);