Messages in this thread | | | Date | Fri, 20 Jan 2023 14:09:35 +0000 | From | Conor Dooley <> | Subject | Re: [PATCH v2] clocksource/drivers/riscv: Get rid of clocksource_arch_init() callback |
| |
On Thu, Dec 29, 2022 at 10:46:01PM +0000, Prabhakar wrote: > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > > Having a clocksource_arch_init() callback always sets vdso_clock_mode to > VDSO_CLOCKMODE_ARCHTIMER if GENERIC_GETTIMEOFDAY is enabled, this is > required for the riscv-timer. > > This works for platforms where just riscv-timer clocksource is present. > On platforms where other clock sources are available we want them to > register with vdso_clock_mode set to VDSO_CLOCKMODE_NONE. > > On the Renesas RZ/Five SoC OSTM block can be used as clocksource [0], to > avoid multiple clock sources being registered as VDSO_CLOCKMODE_ARCHTIMER > move setting of vdso_clock_mode in the riscv-timer driver instead of doing > this in clocksource_arch_init() callback as done similarly for ARM/64 > architecture. > > [0] drivers/clocksource/renesas-ostm.c > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > Tested-by: Samuel Holland <samuel@sholland.org> > --- > RFC -> v2 > * Dropped vdso_default static global variable
> * Used IS_ENABLED() macro
Not sure it really makes much difference here either way, but increased coverage is always good I suppose.
rv32 is the user for the !GENERIC_GETTIMEOFDAY patch IIRC and nothing seemed obviously amiss there to me either.
Reviewed-by: Conor Dooley <conor.dooley@microchip.com>
[unhandled content-type:application/pgp-signature] | |