[PATCH] lib: sbi_timer: Drop unnecessary get_platform_ticks wrapper

[Anup Patel]

On Sun, May 29, 2022 at 10:31 PM Samuel Holland <samuel at sholland.org> wrote:
>
> The device's timer_value callback is already the right prototype to use
> for the get_time_val function pointer.
>
> Signed-off-by: Samuel Holland <samuel at sholland.org>

Looks good to me.

Reviewed-by: Anup Patel <anup at brainfault.org>

Regards,
Anup

> ---
>
>  lib/sbi/sbi_timer.c | 7 +------
>  1 file changed, 1 insertion(+), 6 deletions(-)
>
> diff --git a/lib/sbi/sbi_timer.c b/lib/sbi/sbi_timer.c
> index 353886f..b0a6e63 100644
> --- a/lib/sbi/sbi_timer.c
> +++ b/lib/sbi/sbi_timer.c
> @@ -44,11 +44,6 @@ static u64 get_ticks(void)
>  }
>  #endif
>
> -static u64 get_platform_ticks(void)
> -{
> -       return timer_dev->timer_value();
> -}
> -
>  static void nop_delay_fn(void *opaque)
>  {
>         cpu_relax();
> @@ -167,7 +162,7 @@ void sbi_timer_set_device(const struct sbi_timer_device *dev)
>
>         timer_dev = dev;
>         if (!get_time_val && timer_dev->timer_value)
> -               get_time_val = get_platform_ticks;
> +               get_time_val = timer_dev->timer_value;
>  }
>
>  int sbi_timer_init(struct sbi_scratch *scratch, bool cold_boot)
> --
> 2.35.1
>
>
> --
> opensbi mailing list
> opensbi at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/opensbi