Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933190AbcLILQm (ORCPT ); Fri, 9 Dec 2016 06:16:42 -0500 Received: from terminus.zytor.com ([198.137.202.10]:42960 "EHLO terminus.zytor.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932383AbcLILQl (ORCPT ); Fri, 9 Dec 2016 06:16:41 -0500 Date: Fri, 9 Dec 2016 03:14:17 -0800 From: tip-bot for Thomas Gleixner Message-ID: Cc: prarit@redhat.com, liavr@mellanox.com, john.stultz@linaro.org, cmetcalf@mellanox.com, lvivier@redhat.com, tglx@linutronix.de, david@gibson.dropbear.id.au, linux-kernel@vger.kernel.org, christopher.s.hall@intel.com, peterz@infradead.org, richardcochran@gmail.com, mingo@kernel.org, hpa@zytor.com Reply-To: hpa@zytor.com, linux-kernel@vger.kernel.org, christopher.s.hall@intel.com, mingo@kernel.org, richardcochran@gmail.com, peterz@infradead.org, david@gibson.dropbear.id.au, john.stultz@linaro.org, liavr@mellanox.com, prarit@redhat.com, tglx@linutronix.de, cmetcalf@mellanox.com, lvivier@redhat.com In-Reply-To: <20161208204228.844699737@linutronix.de> References: <20161208204228.844699737@linutronix.de> To: linux-tip-commits@vger.kernel.org Subject: [tip:timers/core] timekeeping: Get rid of pointless typecasts Git-Commit-ID: cbd99e3b289e43000c29aa4aa9b94b394cdc68bd X-Mailer: tip-git-log-daemon Robot-ID: Robot-Unsubscribe: Contact to get blacklisted from these emails MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset=UTF-8 Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1936 Lines: 47 Commit-ID: cbd99e3b289e43000c29aa4aa9b94b394cdc68bd Gitweb: http://git.kernel.org/tip/cbd99e3b289e43000c29aa4aa9b94b394cdc68bd Author: Thomas Gleixner AuthorDate: Thu, 8 Dec 2016 20:49:36 +0000 Committer: Thomas Gleixner CommitDate: Fri, 9 Dec 2016 12:06:42 +0100 timekeeping: Get rid of pointless typecasts 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 Reviewed-by: David Gibson Acked-by: Peter Zijlstra (Intel) Cc: Parit Bhargava Cc: Laurent Vivier Cc: "Christopher S. Hall" Cc: Chris Metcalf Cc: Richard Cochran Cc: Liav Rehana Cc: John Stultz Link: http://lkml.kernel.org/r/20161208204228.844699737@linutronix.de Signed-off-by: Thomas Gleixner --- kernel/time/timekeeping.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/kernel/time/timekeeping.c b/kernel/time/timekeeping.c index 5244821..82e1b5c 100644 --- a/kernel/time/timekeeping.c +++ b/kernel/time/timekeeping.c @@ -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) {