timekeeping: Get rid of pointless typecasts
authorThomas Gleixner <tglx@linutronix.de>
Thu, 8 Dec 2016 20:49:36 +0000 (20:49 +0000)
committerThomas Gleixner <tglx@linutronix.de>
Fri, 9 Dec 2016 11:06:42 +0000 (12:06 +0100)
cycle_t is defined as u64, so casting it to u64 is a pointless and
confusing exercise. cycle_t should simply go away and be replaced with a
plain u64 to avoid further confusion.

Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Reviewed-by: David Gibson <david@gibson.dropbear.id.au>
Acked-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Cc: Parit Bhargava <prarit@redhat.com>
Cc: Laurent Vivier <lvivier@redhat.com>
Cc: "Christopher S. Hall" <christopher.s.hall@intel.com>
Cc: Chris Metcalf <cmetcalf@mellanox.com>
Cc: Richard Cochran <richardcochran@gmail.com>
Cc: Liav Rehana <liavr@mellanox.com>
Cc: John Stultz <john.stultz@linaro.org>
Link: http://lkml.kernel.org/r/20161208204228.844699737@linutronix.de
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
kernel/time/timekeeping.c

index 5244821643a4292af480aecadd81d07d76ce2a04..82e1b5cbebbbe38d15658b2ca9757a8862662e5c 100644 (file)
@@ -258,10 +258,9 @@ static void tk_setup_internals(struct timekeeper *tk, struct clocksource *clock)
        tk->cycle_interval = interval;
 
        /* Go back from cycles -> shifted ns */
-       tk->xtime_interval = (u64) interval * clock->mult;
+       tk->xtime_interval = interval * clock->mult;
        tk->xtime_remainder = ntpinterval - tk->xtime_interval;
-       tk->raw_interval =
-               ((u64) interval * clock->mult) >> clock->shift;
+       tk->raw_interval = (interval * clock->mult) >> clock->shift;
 
         /* if changing clocks, convert xtime_nsec shift units */
        if (old_clock) {