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 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 smtp.lore.kernel.org (Postfix) with ESMTPS id 775CEC282DE for ; Thu, 6 Mar 2025 17:28:56 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:References :In-Reply-To:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=Zw5VA9nVTFm95F/fPGU83EEO6FAdHlc2RRO5eaxauBU=; b=XhjMlHF/cL35w8 2eH8J7ocshWWPteedzKPthE65vp3ey4TBJ0Dr1WAcihfumtOSr9f/jmfGaUj56moIpV4kXNiw+il6 EH4N4xyL8gZkDf66wIFreXFgjSwGLclDk9x/cvhkvEHt6syXmgkjD62+KSY3IPXAxSDISJpZSa4NJ n3jTBSVNGHE6Qcx7WWQOzdfgQgAxI4JYk7weaY+7VP1JS6NMLEzsTyLk1SCCa+OH8SHMhL/44DwoS tAlceoPvOEwmgnBGK/HOINNazMPQA4YNpHlitpP82TNgcU0RdbLO779kwkpjJywunZAfHjxpWnCg9 a3s3oJNylJu9MWJfu9mQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tqF1e-0000000Bgop-1X6T; Thu, 06 Mar 2025 17:28:50 +0000 Received: from mail-wm1-x32b.google.com ([2a00:1450:4864:20::32b]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tqEz0-0000000Bgcy-08Hw for linux-amlogic@lists.infradead.org; Thu, 06 Mar 2025 17:26:08 +0000 Received: by mail-wm1-x32b.google.com with SMTP id 5b1f17b1804b1-43bcb1a9890so14856005e9.0 for ; Thu, 06 Mar 2025 09:26:05 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20230601.gappssmtp.com; s=20230601; t=1741281964; x=1741886764; darn=lists.infradead.org; h=mime-version:message-id:date:user-agent:references:in-reply-to :subject:cc:to:from:from:to:cc:subject:date:message-id:reply-to; bh=9gBi13oH2YTOQlVLrmxAry8taY7YW2dmM3vhgwkXBio=; b=2Vlq7H7gIpMMQy4oiKXOo0XeNgbryuKSIccjG19iGNR8+ahg9Tf5twqbdJBLXNYbAs eNcPOauwK2DA5Qj/zEEzbqp+MhLGe+VXl6kuDmBZPLNoY0cPH0xHxQXK2jmdbHq8ckBx GujnOfhL1y6uafcQVuk5GuhLvtBLnHJgSITBSAHUT0oPawE/2l8DewVCh2NQa0hfXkfp tkDQAdOC+vNe4Z1dC4awM0j3GHoHLo4GLFuM80IW819sahVMj/LNap8ZWuon1k1Fv0sL yf53iM2kHsnx9+8AmRrudQobbuVObD2t2M3RU6Xy0ifgsr1xkGxTUhgz/OkNba7GZY5T GGSQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1741281964; x=1741886764; h=mime-version:message-id:date:user-agent:references:in-reply-to :subject:cc:to:from:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=9gBi13oH2YTOQlVLrmxAry8taY7YW2dmM3vhgwkXBio=; b=jPJ0Fm2OpAf73UEZnygVBVhs/m+W3Ok23DKvi1q7Zd5V9ei08luGfvtWqJfnMBRHNp P40U+7nf1qHm8YDFbJKSxWGm3ve+UvwD4Ha1NjYx4NxR7LXBB/Ulxy1rOK0ntc1Q9jVV 2aGwL/WBGDb4EDFkJGozbPJcEf1YWL8a1tWwancMvRqa0Zi+2hlPoR2UlTt+4m/vvQ26 35R8LWUhGgXBJ0KIwSXmNsO9GAAnCg0oKawFWkOPijJxiSgRl/lThEh7xFLmQWrmh8PA mdrBSUI/Ms0UhJKhWgN8dWuUQ5jxVImYmiDkpoOJ/C23Ez5+fKoWS92LQQWm4AISok87 xk1w== X-Forwarded-Encrypted: i=1; AJvYcCWUxJCghssgembrfprQ683koJBl3Xwgbji13typSLKkwJpXNeqIaPwUDZDIKVn39sfdJ7+O5yCuoBnXuQ5h@lists.infradead.org X-Gm-Message-State: AOJu0Yz3i3fbpN+W58+dpw1zs16QIHdQN/FGjLocV8Yv8ABqB6us/yih V+GdsTFCQKimgJIg+1UkAqkZWgi8eV0xY47AOenEkQEtOevMMBzUiWi4Y79ZyoP7TmIlpeMZ+sx m X-Gm-Gg: ASbGncsi95ML4C61ipbL4flfoVDGWmivBomiy/qkGOJJH+tnH++71CJ2u9m8ie4mcvH Kh7z1zkC7Yl/x/1mZsui4p+AMuuQMMTetLg/mnM7oomSs+G/8PkE9vueqHbkgwwDLj9RCYJvaAT md+cc8CIvZtmF307k8v/ll4Nw3COjTUtuQwTdPAle772KRaR6W5W3MpaVanlRMzVjqQMS+VL79F pIO6P54prCMjXaIGBw5CPQhTH/EVjtocQWqhX9aH83+LkwyR/rwCmy6IjoS4qUq1nlZYOd9KDqp KzsuYF5pSpw7ROqovaMBleuJNmSwkpSapwbRrTkHRI0= X-Google-Smtp-Source: AGHT+IHDAt6NYWgVgdYHY+auIYwz15KHwMO0sECwAzb9Y48d4oIdn6zdb/5QFSFzH4BHnl8HwsC49Q== X-Received: by 2002:a5d:5985:0:b0:391:865:5a93 with SMTP id ffacd0b85a97d-3913211b64fmr247337f8f.22.1741281964164; Thu, 06 Mar 2025 09:26:04 -0800 (PST) Received: from localhost ([2a01:e0a:3c5:5fb1:29d4:36d9:5043:acd]) by smtp.gmail.com with UTF8SMTPSA id ffacd0b85a97d-3912c0e4065sm2703591f8f.62.2025.03.06.09.26.03 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 06 Mar 2025 09:26:03 -0800 (PST) From: Jerome Brunet To: Cc: "'Neil Armstrong'" , "'Michael Turquette'" , "'Stephen Boyd'" , "'Kevin Hilman'" , "'Martin Blumenstingl'" , , , , Subject: Re: [PATCH] clk: meson: g12a: Fix kernel warnings when no display attached In-Reply-To: <004801db8eb7$99808e20$cc81aa60$@martijnvandeventer.nl> (linux@martijnvandeventer.nl's message of "Thu, 6 Mar 2025 17:48:33 +0100") References: <20250213221702.606-1-linux@martijnvandeventer.nl> <1jpljkzyf0.fsf@starbuckisacylon.baylibre.com> <003301db888e$8ea84e90$abf8ebb0$@martijnvandeventer.nl> <1jplj3g21q.fsf@starbuckisacylon.baylibre.com> <004801db8eb7$99808e20$cc81aa60$@martijnvandeventer.nl> User-Agent: mu4e 1.12.8; emacs 29.4 Date: Thu, 06 Mar 2025 18:26:03 +0100 Message-ID: <1j4j065avo.fsf@starbuckisacylon.baylibre.com> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250306_092606_280608_7FE4C8CD X-CRM114-Status: GOOD ( 37.87 ) X-BeenThere: linux-amlogic@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-amlogic" Errors-To: linux-amlogic-bounces+linux-amlogic=archiver.kernel.org@lists.infradead.org On Thu 06 Mar 2025 at 17:48, wrote: > Hi Jerome, > >> > >> > Thank you for reviewing, and apologies for my late response due to a >> holiday. >> > >> >> On Thu 13 Feb 2025 at 23:17, Martijn van Deventer >> >> wrote: >> >> >> >> > When booting SM1 or G12A boards without a dislay attached to HDMI, >> >> > the kernel shows the following warning: >> >> > >> >> > [CRTC:46:meson_crtc] vblank wait timed out >> >> > WARNING: CPU: 2 PID: 265 at >> drivers/gpu/drm/drm_atomic_helper.c:1682 >> >> drm_atomic_helper_wait_for_vblanks.part.0+0x240/0x264 >> >> > CPU: 2 UID: 0 PID: 265 Comm: setfont Tainted: G C >> >> > Tainted: [C]=CRAP >> >> > pstate: 60400009 (nZCv daif +PAN -UAO -TCO -DIT -SSBS BTYPE=--) >> >> > pc : drm_atomic_helper_wait_for_vblanks.part.0+0x240/0x264 >> >> > lr : drm_atomic_helper_wait_for_vblanks.part.0+0x240/0x264 >> >> > Call trace: >> >> > drm_atomic_helper_wait_for_vblanks.part.0+0x240/0x264 >> >> > drm_atomic_helper_commit_tail_rpm+0x84/0xa0 >> >> > commit_tail+0xa4/0x18c >> >> > drm_atomic_helper_commit+0x164/0x178 >> >> > drm_atomic_commit+0xb4/0xec >> >> > drm_client_modeset_commit_atomic+0x210/0x270 >> >> > drm_client_modeset_commit_locked+0x5c/0x188 >> >> > drm_fb_helper_pan_display+0xb8/0x1d4 >> >> > fb_pan_display+0x7c/0x120 >> >> > bit_update_start+0x20/0x48 >> >> > fbcon_switch+0x418/0x54c >> >> > el0t_64_sync+0x194/0x198 >> >> > >> >> > This happens when the kernel disables the unused clocks. >> >> > Sometimes this causes the boot to hang. >> >> > >> >> > By (re)adding the flag CLK_IGNORE_UNUSED to the VCLK2 clocks, these >> >> > clocks will not be disabled. >> >> > >> >> > This partially reverts commit b70cb1a21a54 ("clk: meson: g12a: >> >> > make VCLK2 and ENCL clock path configurable by CCF"). >> >> >> >> It looks like DRM needs those clock enabled regardless of connection >> >> status on HDMI. Even with this change applied, you would get the same >> >> problem again if the bootloader does not take of turning the clock on, >> >> which is not a given. >> >> >> >> CLK_IGNORE_UNUSED gives not guarantee a clock will be enabled or stay >> >> enabled at any point. >> >> >> >> A proper fix to this issue should be done in DRM, IMO. >> > >> > I know and I totally agree. Unfortunately, I don't have access to any >> > vendor >> > documentation, nor do I have any real knowledge about the DRM/HDMI >> > subsystem to fix that. >> >> You have identified which clocks are not properly claimed, by what they >> are not claimed and even when. 50% of the job is done. Thanks for this. > > You're welcome, no problem. > >> > >> > And I guess if it were as easy as adding a clock to the DT and calling >> > clk_prepare_enable on it in the probe function, Neil would have done >> that >> > already. >> > >> > So, all I can do, for now, is revert to the previous situation when it >> > did >> work >> > for (probably) most boards. >> >> Maybe so, but it does not make this change appropriate. The problem >> is the DRM driver which does not enable what it needs to properly >> operate. This should be fixed. > > I understand. So I guess that is the end of the line for this patch. > Because this patch will not be accepted and if someone else finds the > time and has the knowledge to fix this the proper way, it will be a > completely different patch. > > Although I, of course, agree with you that it should be fixed properly, > I find it a bit difficult to accept that if we accidentally break something, > while trying to make things better, we are not allowed to revert it > because it was already somewhat broken. Resulting in a more broken > situation than before... Once again, you are encouraged to fix things up ... where fixes are needed. DRM is hardly immutable. If you don't feel like you can do it on your own, you can still engage with the other contributors who may know this better and help you. > > On the other hand, I also understand that if you, as a maintainer, allow > that, chances are it will never see a proper fix. :-) > > Cheers! > >> > >> >> > >> >> > Fixes: b70cb1a21a54 ("clk: meson: g12a: make VCLK2 and ENCL clock >> path >> >> configurable by CCF"). >> >> > Signed-off-by: Martijn van Deventer >> >> > --- >> >> > drivers/clk/meson/g12a.c | 12 ++++++------ >> >> > 1 file changed, 6 insertions(+), 6 deletions(-) >> >> > >> >> > diff --git a/drivers/clk/meson/g12a.c b/drivers/clk/meson/g12a.c >> >> > index cfffd434e998..1651898658f5 100644 >> >> > --- a/drivers/clk/meson/g12a.c >> >> > +++ b/drivers/clk/meson/g12a.c >> >> > @@ -3234,7 +3234,7 @@ static struct clk_regmap g12a_vclk2_div = { -- Jerome _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic