Re: [PATCH v6 05/17] powerpc: define get_cycles macro for arch-override

From: Michael Ellerman
Date: Thu Apr 28 2022 - 01:19:50 EST


"Jason A. Donenfeld" <Jason@xxxxxxxxx> writes:
> PowerPC defines a get_cycles() function, but it forgot to do the usual
> `#define get_cycles get_cycles` dance, making it impossible for generic
> code to see if an arch-specific function was defined.
>
> Cc: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> Cc: Arnd Bergmann <arnd@xxxxxxxx>
> Cc: Michael Ellerman <mpe@xxxxxxxxxxxxxx>
> Cc: Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx>
> Cc: Paul Mackerras <paulus@xxxxxxxxx>
> Signed-off-by: Jason A. Donenfeld <Jason@xxxxxxxxx>
> ---
> arch/powerpc/include/asm/timex.h | 1 +
> 1 file changed, 1 insertion(+)

Acked-by: Michael Ellerman <mpe@xxxxxxxxxxxxxx>

cheers

> diff --git a/arch/powerpc/include/asm/timex.h b/arch/powerpc/include/asm/timex.h
> index fa2e76e4093a..14b4489de52c 100644
> --- a/arch/powerpc/include/asm/timex.h
> +++ b/arch/powerpc/include/asm/timex.h
> @@ -19,6 +19,7 @@ static inline cycles_t get_cycles(void)
> {
> return mftb();
> }
> +#define get_cycles get_cycles
>
> #endif /* __KERNEL__ */
> #endif /* _ASM_POWERPC_TIMEX_H */
> --
> 2.35.1