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=-17.0 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,INCLUDES_CR_TRAILER,INCLUDES_PATCH,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=ham 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 47F2BC64E7B for ; Mon, 30 Nov 2020 13:55:44 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (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 C90EE2084C for ; Mon, 30 Nov 2020 13:55:43 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="Gzh1aWKu"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="REI+paNk" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org C90EE2084C Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+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=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=ERWoV7XaMUSdCdHYS9zp4utZVfAalJO4FD8CHhFQlNM=; b=Gzh1aWKuPPeLGv6JRFn2ykowL +EghP4r9Beq9ipfufO0Ei7Eh40zrd8PsMofGCZz9rrFoVM7rntB5UozEHfb6h1x9d7aWPlSZb8VPo uBaYYui4lzEmT3dTASIVfJwLIshnSV5DsPLiU4VBiuocc8gM2OlnHaj0/mKSHF6o0wBzkueZZW4o/ jk+wks0u0rva4Dr3vlohmMNGnmwmNvIp7LIU2fTKKrqA1P2qiOP5+9ywM8T1fUfCWVCnp3HHDasVZ wHJ3BF5PilY0Q0ZtNzLS8QANzzsT1RN7qVQbXGE1x6yOBcQstmUIWkJLXY5YHTOHqz+mFjBGVRGa7 joNrsgqXw==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kjjdO-0007oq-PD; Mon, 30 Nov 2020 13:54:30 +0000 Received: from mail.kernel.org ([198.145.29.99]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kjjdK-0007nu-Lr for linux-arm-kernel@lists.infradead.org; Mon, 30 Nov 2020 13:54:27 +0000 Received: from dragon (80.251.214.228.16clouds.com [80.251.214.228]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 43EDD2084C; Mon, 30 Nov 2020 13:54:24 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1606744465; bh=SFo+dD030Rr4VH/OCsfPF3+P8SWL/XdTtRWZSdtzCTM=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=REI+paNkmZWCiqV3O/EMvRyyN0tUeXBIVNnIIT76V4zrGG2lxAZZiFOASMnXyO1Lb nH4igZ7EtWUo08O6rmiSqc9A6q1enbNhXimC8CSMsYLX4K/83eVcTiipvbZGe2SmwK PSs3qhj/XJpPIqnw1sdqbNa49KgXTjHFK1c/AXu8= Date: Mon, 30 Nov 2020 21:54:19 +0800 From: Shawn Guo To: Dong Aisheng Subject: Re: [PATCH 1/1] clk: imx: scu: remove the calling of device_is_bound Message-ID: <20201130135419.GI4072@dragon> References: <20201119114302.26263-1-aisheng.dong@nxp.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20201119114302.26263-1-aisheng.dong@nxp.com> User-Agent: Mutt/1.9.4 (2018-02-28) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20201130_085426_820444_5E040EE7 X-CRM114-Status: GOOD ( 24.18 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Stephen Boyd , linux-clk@vger.kernel.org, linux-imx@nxp.com, Sascha Hauer , Sudip Mukherjee , linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Thu, Nov 19, 2020 at 07:43:02PM +0800, Dong Aisheng wrote: > The device_is_bound() is unvisable to drivers when built as modules. s/unvisable/invisible? I fixed it up and applied the patch. Shawn > It's also not aimed to be used by drivers according to Greg K.H. > Let's remove it from clk-scu driver and find another way to do proper > driver loading sequence. > > Cc: Shawn Guo > Cc: Sascha Hauer > Cc: Stephen Boyd > Fixes: 77d8f3068c63 ("clk: imx: scu: add two cells binding support") > Reported-by: Sudip Mukherjee > Signed-off-by: Dong Aisheng > --- > drivers/clk/imx/clk-scu.c | 15 ++++----------- > 1 file changed, 4 insertions(+), 11 deletions(-) > > diff --git a/drivers/clk/imx/clk-scu.c b/drivers/clk/imx/clk-scu.c > index d10f60e48ece..1f5518b7ab39 100644 > --- a/drivers/clk/imx/clk-scu.c > +++ b/drivers/clk/imx/clk-scu.c > @@ -153,7 +153,6 @@ static inline struct clk_scu *to_clk_scu(struct clk_hw *hw) > > int imx_clk_scu_init(struct device_node *np) > { > - struct platform_device *pd_dev; > u32 clk_cells; > int ret, i; > > @@ -166,17 +165,11 @@ int imx_clk_scu_init(struct device_node *np) > if (clk_cells == 2) { > for (i = 0; i < IMX_SC_R_LAST; i++) > INIT_LIST_HEAD(&imx_scu_clks[i]); > - /* > - * Note: SCU clock driver depends on SCU power domain to be ready > - * first. As there're no power domains under scu clock node in dts, > - * we can't use PROBE_DEFER automatically. > - */ > + > + /* pd_np will be used to attach power domains later */ > pd_np = of_find_compatible_node(NULL, NULL, "fsl,scu-pd"); > - pd_dev = of_find_device_by_node(pd_np); > - if (!pd_dev || !device_is_bound(&pd_dev->dev)) { > - of_node_put(pd_np); > - return -EPROBE_DEFER; > - } > + if (!pd_np) > + return -EINVAL; > } > > return platform_driver_register(&imx_clk_scu_driver); > -- > 2.23.0 > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel