From patchwork Thu Oct 8 15:46:38 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 11823379 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 6EE24175A for ; Thu, 8 Oct 2020 15:48:25 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 5439D20782 for ; Thu, 8 Oct 2020 15:48:25 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729506AbgJHPsY (ORCPT ); Thu, 8 Oct 2020 11:48:24 -0400 Received: from mout.kundenserver.de ([217.72.192.74]:36163 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726165AbgJHPsV (ORCPT ); Thu, 8 Oct 2020 11:48:21 -0400 Received: from localhost.localdomain ([192.30.34.233]) by mrelayeu.kundenserver.de (mreue107 [212.227.15.145]) with ESMTPA (Nemesis) id 1M7KKA-1kIpyQ2KzM-007kBr; Thu, 08 Oct 2020 17:47:15 +0200 From: Arnd Bergmann To: linux-kernel@vger.kernel.org Cc: Arnd Bergmann , Russell King , Tony Luck , Fenghua Yu , Greg Ungerer , Geert Uytterhoeven , Finn Thain , Philip Blundell , Joshua Thompson , Sam Creasey , "James E.J. Bottomley" , Helge Deller , Thomas Gleixner , Daniel Lezcano , John Stultz , Stephen Boyd , Linus Walleij , linux-ia64@vger.kernel.org, linux-parisc@vger.kernel.org, linux-m68k@lists.linux-m68k.org, linux-arm-kernel@lists.infradead.org Subject: [PATCH 00/13] Clean up legacy clock tick users Date: Thu, 8 Oct 2020 17:46:38 +0200 Message-Id: <20201008154651.1901126-1-arnd@arndb.de> X-Mailer: git-send-email 2.27.0 MIME-Version: 1.0 X-Provags-ID: V03:K1:ToCg0sXzfof6vBLW0FBjjVXywAK/gLyirWECWdzB4HS8Fcsvfc3 KICej28tXkzfQ4JRGwmkABm7jfD+Q1lU1BJw7T0Lwpe5f+9kcKbUsHe20Hl/PaNGaJm7f64 vzJ+b6TvkgNmTNmKF4q0otuZPBYaKl04cVbqXwwhtpd4LjYBTPzdyr5BMwVWRMEtd03OY9O 9R1EBRW62aNmSfzzkZ7hQ== X-Spam-Flag: NO X-UI-Out-Filterresults: notjunk:1;V03:K0:XzC6G7gG190=:Cqm3PvhpJLmyzbv8maC2Qh dnY6yR+InSfpI2H7h2qKLgqXY0N/nDpx+LyKwsDfWCfy30pv1/X179z3M18wDXjMbMsUpzZ6F 4B5NNcDcFVMTUCUPlRfy2d3qzVZht/dx1zUdmL1xXlE9E6pFkyUCwoA1J8rrxWA6pnghYo8Ah 0QSFCJMZvvnYyDhAejrhQIiHSmxWuUyB1azRxHYtMm8V4BzxflkuuFqtzYNFyr5b7P1Avsz1e vjXufHhD9qmSPjaXhc5RcMXIkJAI+tb9RmveUbqRO0gARB+S5JRJAOPu55715IQVquUYc+Y5C N5gF9BoV3MEe9uywKU5G6DC9AoyVV+MojMMrN4HTf3AH28dnEbcQyqhAAvff+j+KTTMO4Hukl xIaBIKOo129/+r06dLt9e+ttV49J6xE04WgPvF5KAVvx7q18DrfiB7sQ+FWwkxcvLNlkgfqbV p3QRrIypBFfQvLQYH2yK74by+sk2YmiWpDsQOvIMimQy5W7CyoB4zE8YWuxKhPxD6hec9R/yc DqXtXjbeI22nK835/2cu3jL/ee7OYxq153PK5R+xpWaWMUEfgHODmzl2dQKvdlKKK0EUA6ude Kx5FnaKQM2AQen2RtvDCMpnrIY6mfmE7JyR1UYiwOBxq94oNhfYrqF51lwlEk+2IqfKP8p1pq 2Qpt//BK9nvrV4/feo5U6lbqA3jqVxd4+3SBbYvE4zeNuixTzu60EDSpJLikef7pvt3YItvJw F9NzxPuwnT+XRGwnAbbJtv/0NyKifs6ul5oBFvy4znZCAaP/bkNaai/drzAigtOY0YH+Cevib CIPjT954uXTsZoTu+YGOFhXj/r9QB5+3/FxlRKqwpWINpJmbBojGsvZ5XZYaHlnDgvrQbsx Precedence: bulk List-ID: X-Mailing-List: linux-parisc@vger.kernel.org When I created the patch removing CONFIG_ARCH_GETTIMEOFFSET, I also had a look at CONFIG_GENERIC_CLOCKEVENTS, which is selected by most, but not all, platforms today, each of the ones that lack it doing the timer tick slightly differently. The cleanups here make the old platforms a bit more consistent, in multiple ways: - rather than selecting GENERIC_CLOCKEVENTS on modern platforms, select LEGACY_TIMER_TICK on the old ones. - Hide some more of the internal implementation and only provide a single common entry point for the timer tick, which also makes the behavior more consistent. - Remove the m68k and arm specific infrastructure pieces and call the common helper directly from each timer interrupt function. I tested the series on m68k with the qemu q800 target platform, both with the legacy_timer_tick() implementation and after converting that to a periodic clockevent driver. Arnd Arnd Bergmann (13): timekeeping: add CONFIG_LEGACY_TIMER_TICK ia64: convert to legacy_timer_tick ARM: rpc: use legacy_timer_tick parisc: use legacy_timer_tick m68k: coldfire: use legacy_timer_tick() m68k: split heartbeat out of timer function m68k: sun3/sun3c: use legacy_timer_tick m68k: m68328: use legacy_timer_tick() m68k: change remaining timers to legacy_timer_tick m68k: remove timer_interrupt() function timekeeping: remove xtime_update timekeeping: default GENERIC_CLOCKEVENTS to enabled [RFC] m68k: mac: convert to generic clockevent .../time/clockevents/arch-support.txt | 8 ++-- arch/alpha/Kconfig | 1 - arch/arc/Kconfig | 1 - arch/arm/Kconfig | 13 +----- arch/arm/include/asm/mach/time.h | 2 - arch/arm/kernel/time.c | 14 ------ arch/arm/mach-rpc/time.c | 2 +- arch/arm64/Kconfig | 1 - arch/arm64/Kconfig.platforms | 1 - arch/c6x/Kconfig | 1 - arch/csky/Kconfig | 1 - arch/h8300/Kconfig | 1 - arch/hexagon/Kconfig | 1 - arch/ia64/Kconfig | 1 + arch/ia64/kernel/time.c | 36 ++++++--------- arch/m68k/68000/timers.c | 7 ++- arch/m68k/Kconfig.cpu | 34 +++++++++++--- arch/m68k/Kconfig.machine | 15 +++++++ arch/m68k/amiga/config.c | 11 +++-- arch/m68k/apollo/config.c | 11 +++-- arch/m68k/atari/config.c | 2 +- arch/m68k/atari/time.c | 9 ++-- arch/m68k/bvme6000/config.c | 9 ++-- arch/m68k/coldfire/Makefile | 32 ++++++++------ arch/m68k/coldfire/pit.c | 2 +- arch/m68k/coldfire/sltimers.c | 8 ++-- arch/m68k/coldfire/timers.c | 8 ++-- arch/m68k/hp300/time.c | 8 ++-- arch/m68k/hp300/time.h | 2 +- arch/m68k/include/asm/machdep.h | 12 +++-- arch/m68k/kernel/setup_mm.c | 2 +- arch/m68k/kernel/setup_no.c | 2 +- arch/m68k/kernel/time.c | 18 ++------ arch/m68k/mac/config.c | 6 +-- arch/m68k/mac/via.c | 44 +++++++++++++++---- arch/m68k/mvme147/config.c | 9 ++-- arch/m68k/mvme16x/config.c | 9 ++-- arch/m68k/q40/config.c | 2 +- arch/m68k/q40/q40ints.c | 10 ++--- arch/m68k/sun3/config.c | 4 +- arch/m68k/sun3/sun3ints.c | 3 +- arch/m68k/sun3x/time.c | 5 +-- arch/m68k/sun3x/time.h | 2 +- arch/microblaze/Kconfig | 1 - arch/mips/Kconfig | 1 - arch/nds32/Kconfig | 1 - arch/nios2/Kconfig | 1 - arch/openrisc/Kconfig | 1 - arch/parisc/Kconfig | 2 +- arch/parisc/kernel/time.c | 9 ++-- arch/powerpc/Kconfig | 1 - arch/riscv/Kconfig | 1 - arch/s390/Kconfig | 1 - arch/sh/Kconfig | 1 - arch/sparc/Kconfig | 1 - arch/um/Kconfig | 1 - arch/x86/Kconfig | 1 - arch/xtensa/Kconfig | 1 - include/linux/timekeeping.h | 5 +-- kernel/time/Kconfig | 9 +++- kernel/time/Makefile | 1 + kernel/time/tick-legacy.c | 37 ++++++++++++++++ kernel/time/timekeeping.c | 16 ------- kernel/time/timekeeping.h | 1 + 64 files changed, 242 insertions(+), 221 deletions(-) create mode 100644 kernel/time/tick-legacy.c Cc: Russell King Cc: Tony Luck Cc: Fenghua Yu Cc: Greg Ungerer Cc: Geert Uytterhoeven Cc: Finn Thain Cc: Philip Blundell Cc: Joshua Thompson Cc: Sam Creasey Cc: "James E.J. Bottomley" Cc: Helge Deller Cc: Thomas Gleixner Cc: Daniel Lezcano Cc: John Stultz Cc: Stephen Boyd Cc: Linus Walleij Cc: linux-kernel@vger.kernel.org Cc: linux-ia64@vger.kernel.org Cc: linux-parisc@vger.kernel.org Cc: linux-m68k@lists.linux-m68k.org Cc: linux-arm-kernel@lists.infradead.org Reviewed-by: Linus Walleij