From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.1 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 01856C43381 for ; Fri, 22 Feb 2019 10:55:10 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id C4FDC2075A for ; Fri, 22 Feb 2019 10:55:09 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="Rk/wz0WO" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org C4FDC2075A Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=pengutronix.de Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Date:To:From:Subject:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=l4A1LSTg1PEsy1Lbn5xHUM9oLeir+nZnLsP44x0aC8U=; b=Rk/wz0WO5WLMyr jZ7AYcbxicl2dZWT4KuLXjgYs6KoYRt+vJp6BGvja5a4J4s1JS7Z5oCDo/yTTAPwH8iQi0MbKYGkG 1SFu7StW40Zp9GjeexE2/kPgwY1DHXE7v18GYISElHDgeHLXDT4KndeLnDFEUPXwNdSKJXBVM3Zoz yTpR+nn2OFfjvVO7eMZCNARkZ6LEy2N/7BARCB5sacMrFSzcLSzWx+qZYQPwlcaKXcGl+i7YAb6ZL //zl7th2sKBLHC1VAfBEXUtsipbjBvvCrz/2Ci/3HHtnDKXx0DeV+XROaQnTlgGP7BQJR2TcDYBX2 DZgplbL84WTpgPoF6FmQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1gx8Ty-0007RA-V5; Fri, 22 Feb 2019 10:55:06 +0000 Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1gx8Tv-0006fo-CD for linux-arm-kernel@lists.infradead.org; Fri, 22 Feb 2019 10:55:05 +0000 Received: from gallifrey.ext.pengutronix.de ([2001:67c:670:201:5054:ff:fe8d:eefb] helo=localhost) by metis.ext.pengutronix.de with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.89) (envelope-from ) id 1gx8Tk-0000oK-0I; Fri, 22 Feb 2019 11:54:52 +0100 Message-ID: <66e1685e5cc3fdcaba6ab55b3d030628c37ef8e2.camel@pengutronix.de> Subject: Re: [PATCH 1/3] clk: imx8mq: initialize clock tree earlier From: Lucas Stach To: Anson Huang , "robh+dt@kernel.org" , "mark.rutland@arm.com" , "shawnguo@kernel.org" , "s.hauer@pengutronix.de" , "kernel@pengutronix.de" , "festevam@gmail.com" , "mturquette@baylibre.com" , "sboyd@kernel.org" , Abel Vesa , "ccaione@baylibre.com" , "agx@sigxcpu.org" , "devicetree@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "linux-clk@vger.kernel.org" Date: Fri, 22 Feb 2019 11:54:47 +0100 In-Reply-To: <1550828264-9962-1-git-send-email-Anson.Huang@nxp.com> References: <1550828264-9962-1-git-send-email-Anson.Huang@nxp.com> User-Agent: Evolution 3.30.5 (3.30.5-1.fc29) MIME-Version: 1.0 X-SA-Exim-Connect-IP: 2001:67c:670:201:5054:ff:fe8d:eefb X-SA-Exim-Mail-From: l.stach@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-arm-kernel@lists.infradead.org X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190222_025503_582333_AB922C91 X-CRM114-Status: GOOD ( 21.36 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: dl-linux-imx Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org Am Freitag, den 22.02.2019, 09:42 +0000 schrieb Anson Huang: > Currently on i.MX8MQ platform, clock driver is probed > later than GPIO driver, and GPIO driver does NOT have > defer probe mechanism since the GPIO clock is optional, So this is what should be fixed. If there is a clock reference in the DT, having the clock driver ready is not optional. Optional to the GPIO driver just means it won't fail if there is no clock reference at all. If there is and the clock provider is not yet there, it needs to do a PROBE_DEFER. So that's a NACK on this patch. Regards, Lucas > some platforms have GPIO clocks and some are NOT. So > it is an issue that on i.MX8MQ platform, there are GPIO > clocks defined, but due to clock tree is NOT ready during > GPIO driver probe, the GPIO clock management will fail > and cause system hang if GPIO clocks are OFF by default. > > This patch changes the i.MX8MQ clock tree initialization > using CLK_OF_DECLARE instead of platform driver model to > make clock tree ready earlier than GPIO driver. > > Signed-off-by: Anson Huang > --- > drivers/clk/imx/clk-imx8mq.c | 33 +++++++-------------------------- > 1 file changed, 7 insertions(+), 26 deletions(-) > > diff --git a/drivers/clk/imx/clk-imx8mq.c b/drivers/clk/imx/clk- > imx8mq.c > index 26b57f4..2df1575 100644 > --- a/drivers/clk/imx/clk-imx8mq.c > +++ b/drivers/clk/imx/clk-imx8mq.c > @@ -269,10 +269,9 @@ static const char *imx8mq_clko2_sels[] = > {"osc_25m", "sys2_pll_200m", "sys1_pll_ > > static struct clk_onecell_data clk_data; > > -static int imx8mq_clocks_probe(struct platform_device *pdev) > +static void __init imx8mq_clocks_init(struct device_node *np) > { > - struct device *dev = &pdev->dev; > - struct device_node *np = dev->of_node; > + struct device_node *anatop_np; > void __iomem *base; > int err; > int i; > @@ -286,10 +285,10 @@ static int imx8mq_clocks_probe(struct > platform_device *pdev) > clks[IMX8MQ_CLK_EXT3] = of_clk_get_by_name(np, "clk_ext3"); > clks[IMX8MQ_CLK_EXT4] = of_clk_get_by_name(np, "clk_ext4"); > > - np = of_find_compatible_node(NULL, NULL, "fsl,imx8mq-anatop"); > - base = of_iomap(np, 0); > + anatop_np = of_find_compatible_node(NULL, NULL, "fsl,imx8mq- > anatop"); > + base = of_iomap(anatop_np, 0); > if (WARN_ON(!base)) > - return -ENOMEM; > + return; > > clks[IMX8MQ_ARM_PLL_REF_SEL] = imx_clk_mux("arm_pll_ref_sel", > base + 0x28, 16, 2, pll_ref_sels, ARRAY_SIZE(pll_ref_sels)); > clks[IMX8MQ_GPU_PLL_REF_SEL] = imx_clk_mux("gpu_pll_ref_sel", > base + 0x18, 16, 2, pll_ref_sels, ARRAY_SIZE(pll_ref_sels)); > @@ -389,10 +388,9 @@ static int imx8mq_clocks_probe(struct > platform_device *pdev) > clks[IMX8MQ_SYS2_PLL_500M] = > imx_clk_fixed_factor("sys2_pll_500m", "sys2_pll_out", 1, 2); > clks[IMX8MQ_SYS2_PLL_1000M] = > imx_clk_fixed_factor("sys2_pll_1000m", "sys2_pll_out", 1, 1); > > - np = dev->of_node; > base = of_iomap(np, 0); > if (WARN_ON(!base)) > - return -ENOMEM; > + return; > > /* CORE */ > clks[IMX8MQ_CLK_A53_SRC] = imx_clk_mux2("arm_a53_src", base + > 0x8000, 24, 3, imx8mq_a53_sels, ARRAY_SIZE(imx8mq_a53_sels)); > @@ -568,22 +566,5 @@ static int imx8mq_clocks_probe(struct > platform_device *pdev) > > err = of_clk_add_provider(np, of_clk_src_onecell_get, > &clk_data); > WARN_ON(err); > - > - return err; > } > - > -static const struct of_device_id imx8mq_clk_of_match[] = { > - { .compatible = "fsl,imx8mq-ccm" }, > - { /* Sentinel */ }, > -}; > -MODULE_DEVICE_TABLE(of, imx8mq_clk_of_match); > - > - > -static struct platform_driver imx8mq_clk_driver = { > - .probe = imx8mq_clocks_probe, > - .driver = { > - .name = "imx8mq-ccm", > - .of_match_table = of_match_ptr(imx8mq_clk_of_match), > - }, > -}; > -module_platform_driver(imx8mq_clk_driver); > +CLK_OF_DECLARE(imx8mq, "fsl,imx8mq-ccm", imx8mq_clocks_init); _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel