From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S964839Ab3GEUmF (ORCPT ); Fri, 5 Jul 2013 16:42:05 -0400 Received: from www.linutronix.de ([62.245.132.108]:46440 "EHLO Galois.linutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752417Ab3GEUmD (ORCPT ); Fri, 5 Jul 2013 16:42:03 -0400 Date: Fri, 5 Jul 2013 22:42:03 +0200 (CEST) From: Thomas Gleixner To: =?ISO-8859-15?Q?S=F6ren_Brinkmann?= cc: Baruch Siach , John Stultz , LKML , LAK , Michal Simek , Arnd Bergmann Subject: Re: [PATCH] clocksource/cadence_ttc: Reuse clocksource as sched_clock In-Reply-To: <0030b826-0eb1-4bb8-a0de-1d8dc68bd81a@TX2EHSMHS017.ehs.local> Message-ID: References: <1372888216-10726-1-git-send-email-soren.brinkmann@xilinx.com> <20130704033026.GX4288@tarshish> <0030b826-0eb1-4bb8-a0de-1d8dc68bd81a@TX2EHSMHS017.ehs.local> User-Agent: Alpine 2.02 (DEB 1266 2009-07-14) MIME-Version: 1.0 Content-Type: MULTIPART/MIXED; BOUNDARY="8323329-1413651716-1373056924=:32106" X-Linutronix-Spam-Score: -1.0 X-Linutronix-Spam-Level: - X-Linutronix-Spam-Status: No , -1.0 points, 5.0 required, ALL_TRUSTED=-1,SHORTCIRCUIT=-0.0001,URIBL_BLOCKED=0.001 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This message is in MIME format. The first part should be readable text, while the remaining parts are likely unreadable without MIME-aware tools. --8323329-1413651716-1373056924=:32106 Content-Type: TEXT/PLAIN; charset=utf-8 Content-Transfer-Encoding: 8BIT On Fri, 5 Jul 2013, Sören Brinkmann wrote: > On Thu, Jul 04, 2013 at 06:30:26AM +0300, Baruch Siach wrote: > > Hi Soren, > > > > On Wed, Jul 03, 2013 at 02:50:16PM -0700, Soren Brinkmann wrote: > > > Reuse the TTC clocksource timer as sched clock, too. Since only a single > > > sched clock is supported in Linux, this feature optional and can be > > > selected through Kconfig. > > > > > > Signed-off-by: Soren Brinkmann > > > --- > > > > [...] > > > > > diff --git a/drivers/clocksource/cadence_ttc_timer.c > > > b/drivers/clocksource/cadence_ttc_timer.c > > > index 4cbe28c..ef0f52b 100644 > > > --- a/drivers/clocksource/cadence_ttc_timer.c > > > +++ b/drivers/clocksource/cadence_ttc_timer.c > > > @@ -22,6 +22,7 @@ > > > #include > > > #include > > > #include > > > +#include > > > > This has changed to linux/sched_clock.h in the tip tree. See > > https://git.kernel.org/cgit/linux/kernel/git/tip/tip.git/commit/?h=timers/core&id=38ff87f77af0b5a93fc8581cff1d6e5692ab8970. > Is that change going into Linus' tree during the 3.11 merge window? The > tip tree is missing some changes to the TTC which went in through the > armsoc tree. Yes, this is about to be sent Linuswards. Can we please coordinate that stuff. drivers/clocksource depend on tip timers/* branches and any modification in some other tree is going to be doomed. We have a mechanism for that in place, if stuff goes cross trees. One of the trees provides a set of commit for the other tree to pull, so we do not end up with merge dependencies and conflicts. Why is clocksource stuff going through armsoc unless it contains related modifications in arch/arm/ Grmbl. tglx --8323329-1413651716-1373056924=:32106--