linux-kernel - Re: [PATCH] clk: Ignore disabled DT clock providers

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20160226174214.GR28849@codeaurora.org>
Date:	Fri, 26 Feb 2016 09:42:14 -0800
From:	Stephen Boyd <sboyd@...eaurora.org>
To:	Geert Uytterhoeven <geert+renesas@...der.be>
Cc:	Michael Turquette <mturquette@...libre.com>,
	Ramesh Shanmugasundaram <ramesh.shanmugasundaram@...renesas.com>,
	linux-clk@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] clk: Ignore disabled DT clock providers

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@...renesas.com>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@...der.be>
> ---

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