Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S966751Ab3DQUo3 (ORCPT ); Wed, 17 Apr 2013 16:44:29 -0400 Received: from wolverine01.qualcomm.com ([199.106.114.254]:1027 "EHLO wolverine01.qualcomm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965067Ab3DQUo1 (ORCPT ); Wed, 17 Apr 2013 16:44:27 -0400 X-IronPort-AV: E=Sophos;i="4.87,495,1363158000"; d="scan'208";a="39198197" Message-ID: <516F09AA.2000902@codeaurora.org> Date: Wed, 17 Apr 2013 13:44:26 -0700 From: Stephen Boyd User-Agent: Mozilla/5.0 (X11; Linux i686 on x86_64; rv:17.0) Gecko/20130328 Thunderbird/17.0.5 MIME-Version: 1.0 To: Thomas Gleixner , "arm@kernel.org" CC: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, Mark Rutland , Marc Zyngier , Kukjin Kim , Barry Song , Daniel Walker , Bryan Huntsman , Tony Lindgren , John Stultz , "Brown, David" Subject: Re: [PATCHv4 00/11] Remove ARM local timer API References: <1365456453-16297-1-git-send-email-sboyd@codeaurora.org> In-Reply-To: <1365456453-16297-1-git-send-email-sboyd@codeaurora.org> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4177 Lines: 99 Thomas, Would you be able to pick up patches 1 and 2 from this series and publish it as a stable branch? Preferably based on rc5 but I can always merge it to rc5 and apply the rest of the patches. Arnd/Olof, Would you be able to merge in the rest of the patches to arm-soc? I can provide a branch on 3.9-rc5 and a merge resolution with for-next in arm-soc that you can pull to look at. If not I will have to resend the series. Thanks, Stephen On 04/08/13 14:27, Stephen Boyd wrote: > In light of Mark Rutland's recent work on divorcing the ARM architected > timers from the ARM local timer API and introducing a generic arch hook for > broadcast it seems that we should remove the local timer API entirely. > Doing so will reduce the architecture dependencies of our timer drivers, > reduce code in ARM core, and simplify timer drivers because they no longer > go through an architecture layer that is essentially a hotplug notifier. > > Previous attempts have been made[1] unsuccessfully. I'm hoping this can > be accepted now so that we can clean up the timer drivers that are > used in both UP and SMP situations. Right now these drivers have to ignore > the timer setup callback on the boot CPU to avoid registering clockevents > twice. This is not very symmetric and causes convuluted code that does > the same thing in two places. > > Patches based on v3.9-rc5. I'm still looking for Acks/Tested-by > on EXYNOS and PRIMA2. > > [1] http://article.gmane.org/gmane.linux.ports.arm.kernel/145705 > > Note: A hotplug notifier is used by both x86 for the apb_timer (see > apbt_cpuhp_notify) and by metag (see arch_timer_cpu_notify in > metag_generic.c) so this is not new. > > Changes since v3: > * New patch to fix SMP with dummy timers registered after a global timer > * Push this_cpu_ptr lower to avoid preemptible false positive warnings > * Collected acks/tested-bys > > Changes since v2: > * Bug fixes in smp_twd from Tony Lindgren's testing > * Move smp_twd to use late_time_init hook > * Collected Acks > > Changes since v1: > * Picked up Mark's generic dummy timer driver > * Split out omap changes into new patch > > Mark Rutland (1): > clocksource: add generic dummy timer driver > > Stephen Boyd (10): > clockevents: Prefer CPU local devices over global devices > ARM: smp: Remove duplicate dummy timer implementation > ARM: smp_twd: Divorce smp_twd from local timer API > ARM: OMAP2+: Divorce from local timer API > ARM: EXYNOS4: Divorce mct from local timer API > ARM: PRIMA2: Divorce timer-marco from local timer API > ARM: msm: Divorce msm_timer from local timer API > clocksource: time-armada-370-xp: Fix sparse warning > clocksource: time-armada-370-xp: Divorce from local timer API > ARM: smp: Remove local timer API > > arch/arm/Kconfig | 12 +-- > arch/arm/include/asm/localtimer.h | 34 --------- > arch/arm/kernel/smp.c | 87 --------------------- > arch/arm/kernel/smp_twd.c | 64 +++++++++++----- > arch/arm/mach-exynos/mct.c | 60 ++++++++++----- > arch/arm/mach-msm/timer.c | 127 +++++++++++++++++-------------- > arch/arm/mach-omap2/Kconfig | 1 - > arch/arm/mach-omap2/timer.c | 7 -- > arch/arm/mach-prima2/timer-marco.c | 100 +++++++++++++----------- > drivers/clocksource/Makefile | 1 + > drivers/clocksource/dummy_timer.c | 69 +++++++++++++++++ > drivers/clocksource/time-armada-370-xp.c | 92 +++++++++++----------- > include/linux/time-armada-370-xp.h | 4 +- > kernel/time/tick-common.c | 5 +- > 14 files changed, 327 insertions(+), 336 deletions(-) > delete mode 100644 arch/arm/include/asm/localtimer.h > create mode 100644 drivers/clocksource/dummy_timer.c > -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/