[RFC part3 PATCH 2/2] ARM64 / clocksource: Use arch_timer_acpi_init()

Linus Walleij linus.walleij at linaro.org
Tue Dec 3 07:27:21 EST 2013


On Tue, Dec 3, 2013 at 12:15 PM, Hanjun Guo <hanjun.guo at linaro.org> wrote:

> +       /* if can't be initialised from DT, try ACPI way */
> +       if (!arch_timer_get_rate())
> +               arch_timer_acpi_init();
> +
>         arch_timer_rate = arch_timer_get_rate();

This looks a bit fragile. Having a call like arch_timer_get_rate()
to check whether there is a DT node for the timer doesn't seem
right, can you refactor the code to provide some
has_arch_timer_node() or similar call instead, so it's a bit easier
to understand & maintain at least?

Yours,
Linus Walleij



More information about the linux-arm-kernel mailing list