Re: [PATCH] clk: Ignore disabled DT clock providers

From: Stephen Boyd
Date: Fri Feb 26 2016 - 12:42:21 EST


On 02/26, Geert Uytterhoeven wrote:
> of_clk_init() uses for_each_matching_node_and_match() to find clock
> providers, which returns all matching device nodes, whether they are
> enabled or not. Hence clock providers that are disabled explicitly in DT
> using e.g.
>
> "status = "disabled";
>
> are still activated.
>
> Add a check to ignore device nodes that are not enabled, like
> of_irq_init() does.
>
> Reported-by: Ramesh Shanmugasundaram <ramesh.shanmugasundaram@xxxxxxxxxxxxxx>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> ---

Applied to clk-next

Too bad there isn't a for_each_available_matching_node_and_match()
macro to make this all one line. It would be a mouthful though.

--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project