lkml.org 
[lkml]   [2022]   [Jan]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH v5 3/3] m68k: virt: Remove LEGACY_TIMER_TICK
On Thu, Jan 13, 2022 at 11:35 AM Laurent Vivier <laurent@vivier.eu> wrote:
>
> Move virt machine to generic clockevents.
>
> cc: Arnd Bergmann <arnd@arndb.de>
> Signed-off-by: Laurent Vivier <laurent@vivier.eu>

The change looks good, but it appears that you only just add the legacy code
in the same series, and it would be easier to just add the correct version
first.

> diff --git a/arch/m68k/virt/timer.c b/arch/m68k/virt/timer.c
> index 843bf6ed7e1a..767b01f75abb 100644
> --- a/arch/m68k/virt/timer.c
> +++ b/arch/m68k/virt/timer.c

How about moving the entire file to drivers/clocksource/timer-goldfish.c?
It shouldn't even be architecture specific any more at this point. It probably
still is in practice, but that could be addressed when another architecture
wants to share the implementation.

Arnd

\
 
 \ /
  Last update: 2022-01-13 12:20    [W:1.260 / U:0.204 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site