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 3BC4CC982D8 for ; Fri, 18 Sep 2026 17:59:20 +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-Type: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:Cc:To:From:Date: Reply-To:Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date :Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=O/Ou2Ux6cMxYP7KlhKwAj8zvjvPXYoncUXz8Uc3tuQU=; b=zmEpTMiKHwPkxj0V6yn+u6V8Yx ygej1u7jkgJ8h97zw25NLI2f00dZy0n1rQ+9y6pTNcvGj+33FR7KrxhLFBN/KVN2aRLRXlSZucT3A uxkzWDpMOY782HEbYqFyvPe+1PLChcUb4zM/H4qW1Ns4DDZV0fzqyBDJGEKDiBDGWRYNuOK7aduuk CiSTBFTvH0MZnfKYe317y3P7qkSsra15sJ4HZcIso4odabdvVbR8a9CJ5L88Eah1AYyxFc51ODxe0 Xb4hBeUjCirc3NuPSRiwuWlrNPFYrydmgVWVDqfGvLmbz/Z47OKhZcot8zc1rdn/wvisnvEjKbRj/ LmyuHNCA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1x7crl-0000000FEx4-1ab8; Fri, 18 Sep 2026 17:59:17 +0000 Received: from sender6-op-o11.zoho.com ([165.173.180.11]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1x7cri-0000000FEwG-3Ins for linux-rockchip@lists.infradead.org; Fri, 18 Sep 2026 17:59:16 +0000 ARC-Seal: i=1; a=rsa-sha256; t=1789754351; cv=none; d=zohomail.com; s=zohoarc; b=mlB7qU5gcehEKzGxFzgUo0KtBETvwoRtLpQW/IKGcV5xe1eNrFxE9ZPAmDGwbNbo00jU32OeN7gTF2CF6Jfp8luTKkw/XRz2z0STvrYPFFL95+t6gDoUItAzRKGCLnYs3h/75df4PvqutUt538E/84628zQZ5VFUhZik6NV6yY8= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1789754351; h=Content-Type:Cc:Cc:Date:Date:From:From:In-Reply-To:MIME-Version:Message-ID:Subject:Subject:To:To:Message-Id:Reply-To; bh=IXnlYsOw284Jns76QNX5PPz8/Szr6hkOSfn2OKsgiLI=; b=h/Y1olcmYtssgpRHtlLC4N/VrPektyfBQE6Cjjoo+TT4ltr/XOlSP6fFmHVoD31EIupWzv/AGGHgbtaICQfBUx+jtpeljOG32fMWiJvWVPehLREazBKJrr95IBy2lF7jWTj1gPA9pHNKqXOD2wheaGdtKIbuNa8Xf4ch1hccSBM= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass header.i=collabora.com; spf=pass smtp.mailfrom=sebastian.reichel@collabora.com; dmarc=pass header.from= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1789754351; s=zohomail; d=collabora.com; i=sebastian.reichel@collabora.com; h=Date:Date:From:From:To:To:Cc:Cc:Subject:Subject:Message-ID:MIME-Version:Content-Type:In-Reply-To:Message-Id:Reply-To; bh=IXnlYsOw284Jns76QNX5PPz8/Szr6hkOSfn2OKsgiLI=; b=W+2RgyfSi0Qz+urepmJOA+qOijOidXtWLk1ckKOLGFDIeC+FLE+kici3HS2qhwnH 3cTTkRjMXWP401Kq4y1F4CH9LVSC9fPRgEfXB3NUkXaUkElDkBdrWCJ/4S1jUu/ryQV p9sXg8EhPRQLJPNj/2L+lQswf8BbWEmhJF6O5rIo= Received: by smtp.zohomail.com with SMTPS id 1789754351047914.3784682567832; Fri, 18 Sep 2026 10:59:11 -0700 (PDT) Received: by venus (Postfix, from userid 1000) id 62314180608; Fri, 18 Sep 2026 19:59:08 +0200 (CEST) Date: Fri, 18 Sep 2026 19:59:08 +0200 From: Sebastian Reichel To: Shawn Lin Cc: Ulf Hansson , Heiko Stuebner , linux-pm@vger.kernel.org, linux-rockchip@lists.infradead.org Subject: Re: [PATCH 3/4] pmdomain: rockchip: don't ignore clock lookup errors on attach Message-ID: References: <1789733258-10133-1-git-send-email-shawn.lin@rock-chips.com> <1789733258-10133-4-git-send-email-shawn.lin@rock-chips.com> MIME-Version: 1.0 In-Reply-To: <1789733258-10133-4-git-send-email-shawn.lin@rock-chips.com> X-Zoho-Virus-Status: 1 X-Zoho-AV-Stamp: zmail-av-0.2.13.1.5.4/289.731.60 X-ZohoMailClient: External X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260918_105914_874183_63DCB42D X-CRM114-Status: GOOD ( 21.19 ) X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: multipart/mixed; boundary="===============6477064007475694695==" Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org --===============6477064007475694695== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="5zmlbusiipz75m4r" Content-Disposition: inline --5zmlbusiipz75m4r Content-Type: text/plain; protected-headers=v1; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Subject: Re: [PATCH 3/4] pmdomain: rockchip: don't ignore clock lookup errors on attach MIME-Version: 1.0 Hi, On Fri, Sep 18, 2026 at 08:07:37PM +0800, Shawn Lin wrote: > rockchip_pd_attach_dev() treated every of_clk_get() error as the end > of the clock list, so a clock that exists but is not available yet > (e.g. -EPROBE_DEFER) was silently skipped and the device ended up > without pm_clk management. Only -ENOENT means there are no more > clocks. >=20 > Fixes: 7c696693a4f5 ("soc: rockchip: power-domain: Add power domain drive= r") > Signed-off-by: Shawn Lin > --- Reviewed-by: Sebastian Reichel -- Sebastian > drivers/pmdomain/rockchip/pm-domains.c | 14 +++++++++++++- > 1 file changed, 13 insertions(+), 1 deletion(-) >=20 > diff --git a/drivers/pmdomain/rockchip/pm-domains.c b/drivers/pmdomain/ro= ckchip/pm-domains.c > index b6b099b..e193f97 100644 > --- a/drivers/pmdomain/rockchip/pm-domains.c > +++ b/drivers/pmdomain/rockchip/pm-domains.c > @@ -759,7 +759,19 @@ static int rockchip_pd_attach_dev(struct generic_pm_= domain *genpd, > } > =20 > i =3D 0; > - while ((clk =3D of_clk_get(dev->of_node, i++)) && !IS_ERR(clk)) { > + while (1) { > + clk =3D of_clk_get(dev->of_node, i++); > + if (IS_ERR(clk)) { > + error =3D PTR_ERR(clk); > + if (error =3D=3D -ENOENT) > + break; > + > + dev_err(dev, "failed to get clock %d: %d\n", i - 1, > + error); > + pm_clk_destroy(dev); > + return error; > + } > + > dev_dbg(dev, "adding clock '%pC' to list of PM clocks\n", clk); > error =3D pm_clk_add_clk(dev, clk); > if (error) { > --=20 > 2.7.4 >=20 >=20 --5zmlbusiipz75m4r Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEE72YNB0Y/i3JqeVQT2O7X88g7+poFAmqte+sACgkQ2O7X88g7 +pp/rg/9E0K5Y3juU+WAyHN7VwCZTa/CXCWLdA2WG7AEvCX5FVLMV3zttcfHAImp ejiJdooJY4jlBq0SL/PfPXWu1DUH2kjUl//7+vUJSI8IosuiecUlbKl1LuASXoSE rY5omU+uFfBrydSKH2jrM9ztezETs/6RbZBcL5lFA8hwMSXLWfeXs0SXDOvedo9N ZkZRVKVG8GffcdiaK42eT10AONWeerInHK1PAHjCHwdzpbd+HiZoOMYRJvIv2cZd guCCZ3uf0nbLKpr/1oQeK3KPmG0Oh8sRZV4YdLNesdOp9CCno32XPmqBRPjEXCEM lkV1GLWSmm2H7gQExCTLed5iDoYGrkJ+oGrmoil0sbpv32ReAZeJeg4YrBC8JCBi XCU+EXehkZ4jZYNjEbqHQuKRobpA3PKU7VHTJbDW7St5k3Zqlw4hq8SuktiofNyL Es9gAYXSjdhgPciyD+po19kokDHLooVWdIwTcWEuXbf/6IynTBq7jAMfO86D6KCO RzeQXdeS2ldmiXwfTBxbc4ZUngOLlinP9mc+TVOgf077QDgx05/wgJmNUcGw6fO/ QfLvFGODmoh6I7FsxfTgF+OxaJdJuyfup1eIsJ+lJG3slj/0O517A1KpOzANGc6V XdNLHFLDvzgrhzM8qCaeO855UyfNAcQCLchXwB8snaY07kWxBuU= =/dh9 -----END PGP SIGNATURE----- --5zmlbusiipz75m4r-- --===============6477064007475694695== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip --===============6477064007475694695==--