From patchwork Fri Nov 29 16:16:58 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jerome Brunet X-Patchwork-Id: 180490 Delivered-To: patch@linaro.org Received: by 2002:a92:3001:0:0:0:0:0 with SMTP id x1csp1781600ile; Fri, 29 Nov 2019 08:17:11 -0800 (PST) X-Google-Smtp-Source: APXvYqw3XChBZ6Njk50DQLT8YDQMgT6YOzBBjjA3Fdon6Mi9pN1S9EnBC9dTL/yFDNZw69wtUANS X-Received: by 2002:aa7:c141:: with SMTP id r1mr16208429edp.126.1575044231112; Fri, 29 Nov 2019 08:17:11 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1575044231; cv=none; d=google.com; s=arc-20160816; b=nrxadfq/DWuC+3DlZQKf223PqXJiXZdmO1yDnLn3akRkta8iGJv4J/nlOay8aT0WzT ee2Cle2BvwPNS4ACdp5FQjElHGFbi8cBWkW0pJiQI/bGKPP5bhLjdgIzqZTaxCtXJScO NZdKMKyHLx0DlSoWATKR3QUZ9fkCP2LZDl4G39MTiiMS3eruT4tpMcXwrGMgyS2OvKtp GNTHZU5nmcLGYS44IW5DYcTWrEYILRAsQf4x1e6rQRPlU5TsaYo3k9R6XGiHIGEcB0RX 5rg7Zyyr769Ni0VUQuA8kKjtP1iCd1I6x2j7UjUAvva1I5k6diFPvB+UWHsvkr6Y2U44 2f2g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=2u4id74AuIDefeyl6HPZTpmm+BjNUkbxmsv8qghnzxE=; b=GdEhd0SwPGSK9otTm8yweOw0xV0CD+8NWvPHbRhAkalYYFxA7Np34TZdZvoP+OkL7M f7xlBA8gEV2tMRhtPH1p/Ot9IntWYlcrzSBvpv0YLAnJo5DYHPDrm1Rv+TSsaeAYbwcP 0YCxk/QgluZ5nKxBgmWV/ilO6Oim9tKdoD74x/22Z7XGdCgH6kMAGYFmM0rPsp0n+l4y IvDQsaco0OjRViLuoZN/oAW6E9S5n3WgE/HXHzLjz/AlsG2TEtyVkH/w4ZS/XuP0Ppf4 vaXteaEedTvqb/g19DsvSGqBTBmy2O2FU1kJxeg9kw2f2Nh/BWLRw3JI0ktTAM/7WWbo 73Mg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@baylibre-com.20150623.gappssmtp.com header.s=20150623 header.b="eg/2w4s0"; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id k26si3095104ejr.356.2019.11.29.08.17.10; Fri, 29 Nov 2019 08:17:11 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@baylibre-com.20150623.gappssmtp.com header.s=20150623 header.b="eg/2w4s0"; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727028AbfK2QRG (ORCPT + 27 others); Fri, 29 Nov 2019 11:17:06 -0500 Received: from mail-wr1-f66.google.com ([209.85.221.66]:44400 "EHLO mail-wr1-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726608AbfK2QRG (ORCPT ); Fri, 29 Nov 2019 11:17:06 -0500 Received: by mail-wr1-f66.google.com with SMTP id i12so35764288wrn.11 for ; Fri, 29 Nov 2019 08:17:03 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=2u4id74AuIDefeyl6HPZTpmm+BjNUkbxmsv8qghnzxE=; b=eg/2w4s0ARwwzJIyvoQV9eZtjhRTG0mLCFM2uJeucVbUwmBbr0EnTlmRi1O5LjCKc4 Rq3YWQ/7YSShDjqGJvy17FsFYn93axz+BWMlrBUPGtj+jkzfcApSZTuKqKwBu1ubSNOb zN24rOxmy4NALD2wKVz+8CZ9A4goebPRfwznoDQ+tGQ7Mc/mM2bAD2xffLeT3Hh0NeYb 28yfMaszMFyX7HAy7MPZaIMlN1W8JROkY/h0Jz4rVDelMYP9585ksfA6Sfls0TaID75E HFOsiJkopxlO2DOarRjQ5qeHmID66xo5V6QdJScr9RDb0oOwILX3eFYwU7ohjicgP/mK tTbw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=2u4id74AuIDefeyl6HPZTpmm+BjNUkbxmsv8qghnzxE=; b=pjerwZz7RfK3J6u8KfsBkEvwWy2UW/nTt/EfiLyszul+/2BL1u7ujHh8AcubfpX1qH pXZH4bSQtCtSwMw3nQl+LYkWAwEBLU0cDSru5NxPrazcEpjKeiHIS7siABhAKvylU4rQ atBhbj7rGBGrAm7EABWlhVXMWUO64moQ7ptUggjLPT+vDKGOCxsxqDQaisyUs+f0gYka jyE6DcNdXMfjIayJFa8TvgmEPv7yj2UaSVUiV7SsOJao68hXdZCaWQwa7YVTjsxZ84O8 a1SKXqKboJrcaF5ukm+hdKcDIcfiEOI9B6Y0EBlIe8caP2U1xkqcPH0frq8K1nE8bTfe dcaA== X-Gm-Message-State: APjAAAUYbEJ0ztPXWVXJsfuzLyl7IGyrMSAmgcbZxheTtoxWwgUyqSjM h+T3gO74g5fXHtn5Taa1jEsy/A== X-Received: by 2002:a5d:46c7:: with SMTP id g7mr20275395wrs.11.1575044223145; Fri, 29 Nov 2019 08:17:03 -0800 (PST) Received: from starbuck.baylibre.local (laubervilliers-658-1-213-31.w90-63.abo.wanadoo.fr. [90.63.244.31]) by smtp.googlemail.com with ESMTPSA id 72sm28730412wrl.73.2019.11.29.08.17.02 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 29 Nov 2019 08:17:02 -0800 (PST) From: Jerome Brunet To: Stephen Boyd , Michael Turquette Cc: Jerome Brunet , Kevin Hilman , linux-amlogic@lists.infradead.org, linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org Subject: [PATCH] clk: walk orphan list on clock provider registration Date: Fri, 29 Nov 2019 17:16:58 +0100 Message-Id: <20191129161658.344517-1-jbrunet@baylibre.com> X-Mailer: git-send-email 2.23.0 MIME-Version: 1.0 X-Patchwork-Bot: notify Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org So far, we walked the orphan list every time a new clock was registered in CCF. This was fine since the clocks were only referenced by name. Now that the clock can be referenced through DT, it is not enough: * Controller A register first a reference clocks from controller B through DT. * Controller B register all its clocks then register the provider. Each time controller B registers a new clock, the orphan list is walked but it can't match since the provider is registered yet. When the provider is finally registered, the orphan list is not walked unless another clock is registered afterward. This can lead to situation where some clocks remain orphaned even if the parent is available. Walking the orphan list on provider registration solves the problem. Fixes: fc0c209c147f ("clk: Allow parents to be specified without string names") Signed-off-by: Jerome Brunet --- drivers/clk/clk.c | 59 +++++++++++++++++++++++++++++------------------ 1 file changed, 37 insertions(+), 22 deletions(-) -- 2.23.0 diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c index ef4416721777..917ba37c3b9d 100644 --- a/drivers/clk/clk.c +++ b/drivers/clk/clk.c @@ -3249,6 +3249,34 @@ static inline void clk_debug_unregister(struct clk_core *core) } #endif +static void __clk_core_reparent_orphan(void) +{ + struct clk_core *orphan; + struct hlist_node *tmp2; + + /* + * walk the list of orphan clocks and reparent any that newly finds a + * parent. + */ + hlist_for_each_entry_safe(orphan, tmp2, &clk_orphan_list, child_node) { + struct clk_core *parent = __clk_init_parent(orphan); + + /* + * We need to use __clk_set_parent_before() and _after() to + * to properly migrate any prepare/enable count of the orphan + * clock. This is important for CLK_IS_CRITICAL clocks, which + * are enabled during init but might not have a parent yet. + */ + if (parent) { + /* update the clk tree topology */ + __clk_set_parent_before(orphan, parent); + __clk_set_parent_after(orphan, parent, NULL); + __clk_recalc_accuracies(orphan); + __clk_recalc_rates(orphan, 0); + } + } +} + /** * __clk_core_init - initialize the data structures in a struct clk_core * @core: clk_core being initialized @@ -3259,8 +3287,6 @@ static inline void clk_debug_unregister(struct clk_core *core) static int __clk_core_init(struct clk_core *core) { int ret; - struct clk_core *orphan; - struct hlist_node *tmp2; unsigned long rate; if (!core) @@ -3416,27 +3442,8 @@ static int __clk_core_init(struct clk_core *core) clk_enable_unlock(flags); } - /* - * walk the list of orphan clocks and reparent any that newly finds a - * parent. - */ - hlist_for_each_entry_safe(orphan, tmp2, &clk_orphan_list, child_node) { - struct clk_core *parent = __clk_init_parent(orphan); + __clk_core_reparent_orphan(); - /* - * We need to use __clk_set_parent_before() and _after() to - * to properly migrate any prepare/enable count of the orphan - * clock. This is important for CLK_IS_CRITICAL clocks, which - * are enabled during init but might not have a parent yet. - */ - if (parent) { - /* update the clk tree topology */ - __clk_set_parent_before(orphan, parent); - __clk_set_parent_after(orphan, parent, NULL); - __clk_recalc_accuracies(orphan); - __clk_recalc_rates(orphan, 0); - } - } kref_init(&core->ref); out: @@ -4288,6 +4295,10 @@ int of_clk_add_provider(struct device_node *np, mutex_unlock(&of_clk_mutex); pr_debug("Added clock from %pOF\n", np); + clk_prepare_lock(); + __clk_core_reparent_orphan(); + clk_prepare_unlock(); + ret = of_clk_set_defaults(np, true); if (ret < 0) of_clk_del_provider(np); @@ -4323,6 +4334,10 @@ int of_clk_add_hw_provider(struct device_node *np, mutex_unlock(&of_clk_mutex); pr_debug("Added clk_hw provider from %pOF\n", np); + clk_prepare_lock(); + __clk_core_reparent_orphan(); + clk_prepare_unlock(); + ret = of_clk_set_defaults(np, true); if (ret < 0) of_clk_del_provider(np);