From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1030216AbcBZPyi (ORCPT ); Fri, 26 Feb 2016 10:54:38 -0500 Received: from xavier.telenet-ops.be ([195.130.132.52]:36705 "EHLO xavier.telenet-ops.be" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754700AbcBZPyg (ORCPT ); Fri, 26 Feb 2016 10:54:36 -0500 From: Geert Uytterhoeven To: Michael Turquette , Stephen Boyd Cc: Ramesh Shanmugasundaram , linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org, Geert Uytterhoeven Subject: [PATCH] clk: Ignore disabled DT clock providers Date: Fri, 26 Feb 2016 16:54:31 +0100 Message-Id: <1456502071-13516-1-git-send-email-geert+renesas@glider.be> X-Mailer: git-send-email 1.9.1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 Signed-off-by: Geert Uytterhoeven --- drivers/clk/clk.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c index a1bc8db1c00a064f..4c0cb112390986b2 100644 --- a/drivers/clk/clk.c +++ b/drivers/clk/clk.c @@ -3247,6 +3247,9 @@ void __init of_clk_init(const struct of_device_id *matches) for_each_matching_node_and_match(np, matches, &match) { struct clock_provider *parent; + if (!of_device_is_available(np)) + continue; + parent = kzalloc(sizeof(*parent), GFP_KERNEL); if (!parent) { list_for_each_entry_safe(clk_provider, next, -- 1.9.1