Re: [PATCH v5 3/3] m68k: virt: Remove LEGACY_TIMER_TICK

[Arnd Bergmann]

On Thu, Jan 13, 2022 at 11:35 AM Laurent Vivier <laurent@xxxxxxxxx> wrote:
>
> Move virt machine to generic clockevents.
>
> cc: Arnd Bergmann <arnd@xxxxxxxx>
> Signed-off-by: Laurent Vivier <laurent@xxxxxxxxx>

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