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=-5.2 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=no 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 54820C433E0 for ; Thu, 21 Jan 2021 09:57:31 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 0D3A623975 for ; Thu, 21 Jan 2021 09:57:31 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728800AbhAUJ5P (ORCPT ); Thu, 21 Jan 2021 04:57:15 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49978 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728760AbhAUJ5K (ORCPT ); Thu, 21 Jan 2021 04:57:10 -0500 Received: from metis.ext.pengutronix.de (metis.ext.pengutronix.de [IPv6:2001:67c:670:201:290:27ff:fe1d:cc33]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C1C52C061575 for ; Thu, 21 Jan 2021 01:56:28 -0800 (PST) Received: from ptx.hi.pengutronix.de ([2001:67c:670:100:1d::c0]) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1l2WhO-0000L5-KO; Thu, 21 Jan 2021 10:56:18 +0100 Received: from sha by ptx.hi.pengutronix.de with local (Exim 4.92) (envelope-from ) id 1l2WhN-0007FY-8h; Thu, 21 Jan 2021 10:56:17 +0100 Date: Thu, 21 Jan 2021 10:56:17 +0100 From: Sascha Hauer To: Abel Vesa Cc: Adam Ford , arm-soc , Adam Ford-BE , Aisheng Dong , Michael Turquette , Stephen Boyd , Shawn Guo , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Linus Walleij , Jerome Brunet , linux-clk , Linux Kernel Mailing List Subject: Re: [PATCH V3] clk: imx: Fix reparenting of UARTs not associated with sdout Message-ID: <20210121095617.GI19063@pengutronix.de> References: <20210115182909.314756-1-aford173@gmail.com> <20210118125204.hxsanoohwvdtdvym@fsr-ub1664-175> <20210120144454.f6b72lnasw4q3bde@fsr-ub1664-175> <20210120151305.GC19063@pengutronix.de> <20210120152813.x2pbs5vprevkly23@fsr-ub1664-175> <20210120155001.GD19063@pengutronix.de> <20210120161421.h3yng57m3fetwwih@fsr-ub1664-175> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210120161421.h3yng57m3fetwwih@fsr-ub1664-175> X-Sent-From: Pengutronix Hildesheim X-URL: http://www.pengutronix.de/ X-IRC: #ptxdist @freenode X-Accept-Language: de,en X-Accept-Content-Type: text/plain X-Uptime: 08:01:56 up 49 days, 19:28, 71 users, load average: 0.77, 0.38, 0.19 User-Agent: Mutt/1.10.1 (2018-07-13) X-SA-Exim-Connect-IP: 2001:67c:670:100:1d::c0 X-SA-Exim-Mail-From: sha@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-clk@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-clk@vger.kernel.org On Wed, Jan 20, 2021 at 06:14:21PM +0200, Abel Vesa wrote: > On 21-01-20 16:50:01, Sascha Hauer wrote: > > On Wed, Jan 20, 2021 at 05:28:13PM +0200, Abel Vesa wrote: > > > On 21-01-20 16:13:05, Sascha Hauer wrote: > > > > Hi Abel, > > > > > > > > On Wed, Jan 20, 2021 at 04:44:54PM +0200, Abel Vesa wrote: > > > > > On 21-01-18 08:00:43, Adam Ford wrote: > > > > > > On Mon, Jan 18, 2021 at 6:52 AM Abel Vesa wrote: > > > > > > ... > > > > > > > > > > > > > > > > > > > > > > > TBH, I'm against the idea of having to call consumer API from a clock provider driver. > > > > > > > I'm still investigating a way of moving the uart clock control calls in drivers/serial/imx, > > > > > > > where they belong. > > > > > > > > > > > > That makes sense. > > > > > > > > > > > > > > > > Just a thought. The uart clock used for console remains on from u-boot, > > > > > so maybe it's enough to just add the CLK_IGNORE_UNUSED flag to all the > > > > > uart root clocks and remove the prepare/enable calls for uart clocks > > > > > for good. I don't really have a way to test it right now, but maybe > > > > > you could give it a try. > > > > > > > > That would mean that UART clocks will never be disabled, regardless of > > > > whether they are used for console or not. That doesn't sound very > > > > appealing. > > > > > > AFAIK, the only uart clock that is enabled by u-boot is the one used for > > > the console. Later on, when the serial driver probes, it will enable it itself. > > > > > > Unless I'm missing something, this is exactly what we need. > > > > It might enable it, but with CLK_IGNORE_UNUSED the clock won't be > > disabled again when a port is closed after usage > > OK, tell me what I'm getting wrong in the following scenario: > > U-boot leaves the console uart clock enabled. All the other ones are disabled. > > Kernel i.MX clk driver registers the uart clocks with flag CLK_IGNORE_UNUSED. I was wrong at that point. I originally thought the kernel will never disable these clocks, but in fact it only leaves them enabled during the clk_disable_unused call. However, when CLK_IGNORE_UNUSED becomes relevant it's too late already. I just chatted with Lucas and he told me what the original problem was that his patch solved. The problem comes when an unrelated device and the earlycon UART have the same parent clocks. The parent clock is enabled, but it's reference count is zero. Now when the unrelated device probes and toggles its clocks then the shared parent clock will be disabled due to the reference count being zero. Next time earlycon prints a character the system hangs because the UART gates are still enabled, but their parent clocks no longer are. Overall I think Lucas' patches are still valid and relevant and with Adams patches we even no longer have to enable all UART clocks, but only the ones which are actually needed. Sascha -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |