Dear Ezequiel Garcia,
On Wed, 22 Oct 2014 10:34:43 -0300, Ezequiel Garcia wrote:
> +static void __init armada_375_timer_init(struct device_node *np)
> +{
> + struct clk *clk;
> +
> + clk = of_clk_get_by_name(np, "fixed");
> + if (!IS_ERR(clk)) {
> + clk_prepare_enable(clk);
> + timer_clk = clk_get_rate(clk);
> + } else {
> +
> + /*
> + * This fallback is required in order to retain proper
> + * devicetree backwards compatibility.
> + */
> + clk = of_clk_get(np, 0);
> +
> + /* Must have at least a clock */
> + BUG_ON(IS_ERR(clk));
> + clk_prepare_enable(clk);
> + timer_clk = clk_get_rate(clk) / TIMER_DIVIDER;
> + timer25Mhz = false;
> + }
Maybe we could re-use a bit more existing code, don't know if it makes
things clearer though:
clk = of_clk_get_by_name(np, "fixed");
if (!IS_ERR(clk))
armada_xp_timer_init(np);
else
armada_370_timer_init(np);
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
