diff mbox series

[-next] clocksource/drivers/imx-tpm: Move tpm_read_sched_clock() under CONFIG_ARM

Message ID 20220303184212.2356245-1-nathan@kernel.org (mailing list archive)
State New, archived
Headers show
Series [-next] clocksource/drivers/imx-tpm: Move tpm_read_sched_clock() under CONFIG_ARM | expand

Commit Message

Nathan Chancellor March 3, 2022, 6:42 p.m. UTC
When building this driver for an architecture other than ARCH=arm:

  drivers/clocksource/timer-imx-tpm.c:78:20: error: unused function 'tpm_read_sched_clock' [-Werror,-Wunused-function]
  static u64 notrace tpm_read_sched_clock(void)
                     ^
  1 error generated.

Move the function definition under the existing CONFIG_ARM section so
there is no more warning.

Fixes: 10720e120e2b ("clocksource/drivers/imx-tpm: Exclude sched clock for ARM64")
Signed-off-by: Nathan Chancellor <nathan@kernel.org>
---
 drivers/clocksource/timer-imx-tpm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


base-commit: b22a1c270f533e30bc5d5d4cab6199a2cbb07b07

Comments

Peng Fan March 4, 2022, 12:56 a.m. UTC | #1
> Subject: [PATCH -next] clocksource/drivers/imx-tpm: Move
> tpm_read_sched_clock() under CONFIG_ARM
> 
> When building this driver for an architecture other than ARCH=arm:
> 
>   drivers/clocksource/timer-imx-tpm.c:78:20: error: unused function
> 'tpm_read_sched_clock' [-Werror,-Wunused-function]
>   static u64 notrace tpm_read_sched_clock(void)
>                      ^
>   1 error generated.
> 
> Move the function definition under the existing CONFIG_ARM section so there
> is no more warning.
> 
> Fixes: 10720e120e2b ("clocksource/drivers/imx-tpm: Exclude sched clock for
> ARM64")
> Signed-off-by: Nathan Chancellor <nathan@kernel.org>

Reviewed-by: Peng Fan <peng.fan@nxp.com>

> ---
>  drivers/clocksource/timer-imx-tpm.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/clocksource/timer-imx-tpm.c
> b/drivers/clocksource/timer-imx-tpm.c
> index 60cefc247b71..bd64a8a8427f 100644
> --- a/drivers/clocksource/timer-imx-tpm.c
> +++ b/drivers/clocksource/timer-imx-tpm.c
> @@ -73,12 +73,12 @@ static unsigned long tpm_read_current_timer(void)
> {
>  	return tpm_read_counter();
>  }
> -#endif
> 
>  static u64 notrace tpm_read_sched_clock(void)  {
>  	return tpm_read_counter();
>  }
> +#endif
> 
>  static int tpm_set_next_event(unsigned long delta,
>  				struct clock_event_device *evt)
> 
> base-commit: b22a1c270f533e30bc5d5d4cab6199a2cbb07b07
> --
> 2.35.1
Daniel Lezcano March 4, 2022, 8:18 a.m. UTC | #2
On 03/03/2022 19:42, Nathan Chancellor wrote:
> When building this driver for an architecture other than ARCH=arm:
> 
>    drivers/clocksource/timer-imx-tpm.c:78:20: error: unused function 'tpm_read_sched_clock' [-Werror,-Wunused-function]
>    static u64 notrace tpm_read_sched_clock(void)
>                       ^
>    1 error generated.
> 
> Move the function definition under the existing CONFIG_ARM section so
> there is no more warning.
> 
> Fixes: 10720e120e2b ("clocksource/drivers/imx-tpm: Exclude sched clock for ARM64")
> Signed-off-by: Nathan Chancellor <nathan@kernel.org>
> ---

Applied, thx
diff mbox series

Patch

diff --git a/drivers/clocksource/timer-imx-tpm.c b/drivers/clocksource/timer-imx-tpm.c
index 60cefc247b71..bd64a8a8427f 100644
--- a/drivers/clocksource/timer-imx-tpm.c
+++ b/drivers/clocksource/timer-imx-tpm.c
@@ -73,12 +73,12 @@  static unsigned long tpm_read_current_timer(void)
 {
 	return tpm_read_counter();
 }
-#endif
 
 static u64 notrace tpm_read_sched_clock(void)
 {
 	return tpm_read_counter();
 }
+#endif
 
 static int tpm_set_next_event(unsigned long delta,
 				struct clock_event_device *evt)