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 81B6DC5DF7D for ; Fri, 21 Aug 2026 14:06:31 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: 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=IXNJ/14cVvvKroHplPIGuCBeBYfygG4Y7sUaJ45XWQE=; b=Kr0sk+yp+pzOYN7QWeHKnh7e82 oa+DL1N42sS43OvMyFCkc8HibmQnKMODNlax5kHgirHfE+9Bp3biMSkaBwKDym99JOVLyOLTO7moW 9wAk7DWdDwUdXOMC5wG5FGF3WSigWwhNnWWAtT1va92om47RffHCvbUOTVpkWcEYNfOTLp0/v+LAN QzE94v+sjGseGnq9NRqrfnKBaUBQO2o4h7gn4Skf6lKgBlkrRlyhjEWnrZkq+lqrkxIN/W/UP+T7T kMZuum+wdk6SlZrcpTovIkwRLcz2Iblqqzm9RnKzG1ZXr2rcM+TfvHF3HY1JvTLkMxoqwDUsUwF9r 2qBPynWg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1wxPt8-0000000DV1t-0PoK; Fri, 21 Aug 2026 14:06:30 +0000 Received: from desiato.infradead.org ([2001:8b0:10b:1:d65d:64ff:fe57:4e05]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1wxPt3-0000000DV0f-3PXm; Fri, 21 Aug 2026 14:06:27 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=desiato.20200630; h=In-Reply-To:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=IXNJ/14cVvvKroHplPIGuCBeBYfygG4Y7sUaJ45XWQE=; b=ncW4sMreE7q7fQi68KS8NZBpkb 5P1b04VbSJv6aLq7D6n0GdEPABWrOvpMf1sYduFmjxhfntxfhRvc8o5PBbrlaq4mXw+3TMCSP+0pt zE5TJx+WobyJQ5olZIbmGKApS4WxJ0+Gkbr3oCUjIrmSxTuZsy/25mCwWeDgwi+Jf0E3fHpxmj5IW MTsUv8b+Ta2kXj+xNhN2KyeRzHNrBZVvs4SujpUdDwjHwAEoGePbwtYnMQB3CJe3z+lMDcGbxNIK9 0tpKaGA4PLDrvp0QPMlf7XB03cBuHE+l/J/tx54j28MI9AGmeIgD1bfwfcP03wCDq7sv29WXK/KFi cWNE/vlQ==; Received: from sender6-op-o11.zoho.com ([165.173.180.11]) by desiato.infradead.org with esmtps (Exim 4.99.2 #2 (Red Hat Linux)) id 1wxPt0-00000003ot0-0u8U; Fri, 21 Aug 2026 14:06:24 +0000 ARC-Seal: i=1; a=rsa-sha256; t=1787321085; cv=none; d=zohomail.com; s=zohoarc; b=aS3Wxr42/WowGTQMOwQn+Aik7EWu3ZtXAyK7Ka9JjnfmyekKH/X9DF7cl903wIOEJg01lQp+gWpZGWmsspCAx0u4fU8P+aSiUAIfuOxMh2gINvbdCDlWZZYuBuYcc4/Hgvq6u6X2dotGYHOAUIOO1A5l2vx+Nlrvv36qnd4Zo1U= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1787321085; 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=IXNJ/14cVvvKroHplPIGuCBeBYfygG4Y7sUaJ45XWQE=; b=dGSfR9zZz1WQQVWan6t8MG3C7sldyy+btQ1fIcHks35yTU0HKlGgpqkis8Bvw4NE/R8apiMLyVPbZ2hQ/JtlCZybWHRdin/g6Jwx5Uj4lpEPGM+gKQTp/tJmHKoctrb8wHffa2SpJrAL6iyPO9uS8eTa8hjDXKTHy2rgRVXWuH4= 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=1787321085; 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=IXNJ/14cVvvKroHplPIGuCBeBYfygG4Y7sUaJ45XWQE=; b=TTKVytQB8iMxRzpvUXKXpNq1OdRpglO16qtqbszi6lJF4nmFxmWOyOMX5spT+YP3 Xv/lujJhnv14rcillKejZcMdJjESzp+mPU/hITTrVhEacTe4GuC1BIRlXTEL7mp/IW2 dTt5sCybxxEgKjnIOYRF8dHMon4uyWe1V4AxLMQU= Received: by mx.zohomail.com with SMTPS id 1787321084147104.50540849911135; Fri, 21 Aug 2026 07:04:44 -0700 (PDT) Received: by venus (Postfix, from userid 1000) id 52022181F47; Fri, 21 Aug 2026 16:04:34 +0200 (CEST) Date: Fri, 21 Aug 2026 16:04:34 +0200 From: Sebastian Reichel To: Geert Uytterhoeven Cc: Ryan Chen , Joel Stanley , Stephen Boyd , Brian Masney , Jerome Brunet , Andrew Jeffery , Billy Tsai , Nicolas Ferre , Alexandre Belloni , Claudiu Beznea , Ray Jui , Scott Branden , Broadcom internal kernel review list , Taichi Sugaya , Takao Orito , Linus Walleij , Avi Fishman , Tomer Maimon , Tali Perry , Patrick Venture , Nancy Yuen , Benjamin Fair , Sudeep Holla , Cristian Marussi , David Lechner , Abel Vesa , Peng Fan , Frank Li , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , Paul Cercueil , Santosh Shilimkar , Andrew Lunn , Gregory Clement , Sebastian Hesselbarth , Vladimir Zapolskiy , Heiko Stuebner , Krzysztof Kozlowski , Sylwester Nawrocki , Chanwoo Choi , Peter Griffin , Alim Akhtar , Dinh Nguyen , Viresh Kumar , Maxime Coquelin , Alexandre Torgue , Emilio =?utf-8?B?TMOzcGV6?= , Chen-Yu Tsai , Jernej Skrabec , Samuel Holland , Prashant Gaikwad , Thierry Reding , Jonathan Hunter , Kunihiko Hayashi , Masami Hiramatsu , Ulf Hansson , Liviu Dudau , Lorenzo Pieralisi , Michal Simek , Matthias Brugger , linux-clk@vger.kernel.org, linux-aspeed@lists.ozlabs.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, openbmc@lists.ozlabs.org, arm-scmi@vger.kernel.org, imx@lists.linux.dev, linux-mips@vger.kernel.org, linux-renesas-soc@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-samsung-soc@vger.kernel.org, soc@lists.linux.dev, linux-stm32@st-md-mailman.stormreply.com, linux-sunxi@lists.linux.dev, linux-tegra@vger.kernel.org, linux-mediatek@lists.infradead.org Subject: Re: [PATCH 28/45] clk: rockchip: Make sure clk_init_data is fully initialized Message-ID: References: <49127c1f7529acccb17e1408db9f21f6d15aee2b.1787239813.git.geert+renesas@glider.be> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="zybuuvaovsaw7bmk" Content-Disposition: inline In-Reply-To: <49127c1f7529acccb17e1408db9f21f6d15aee2b.1787239813.git.geert+renesas@glider.be> X-Zoho-Virus-Status: 1 X-Zoho-AV-Stamp: zmail-av-0.2.10.1.5.2/287.318.86 X-ZohoMailClient: External X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260821_150622_815966_F6956500 X-CRM114-Status: GOOD ( 24.03 ) X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org --zybuuvaovsaw7bmk Content-Type: text/plain; protected-headers=v1; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Subject: Re: [PATCH 28/45] clk: rockchip: Make sure clk_init_data is fully initialized MIME-Version: 1.0 Hi, On Fri, Aug 21, 2026 at 10:53:38AM +0200, Geert Uytterhoeven wrote: > The clk_init_data structure contains several mutually-exclusive members > for different methods to specify the possible parents of a clock, > prompting drivers to initialize only the members they need. However, > not initializing all members may cause subtle issues, which are only > exposed when CONFIG_INIT_STACK_ALL_PATTERN or CONFIG_INIT_STACK_NONE is > enabled. >=20 > Make sure all members are fully initialized, to avoid such bugs, and to > prevent future breakage when converting drivers to a different method > for specifying the parents. >=20 > Signed-off-by: Geert Uytterhoeven > --- > Compile-tested only. > --- Reviewed-by: Sebastian Reichel Greetings, -- Sebstian > drivers/clk/rockchip/clk-cpu.c | 2 +- > drivers/clk/rockchip/clk-ddr.c | 2 +- > drivers/clk/rockchip/clk-gate-grf.c | 2 +- > drivers/clk/rockchip/clk-inverter.c | 2 +- > drivers/clk/rockchip/clk-mmc-phase.c | 2 +- > drivers/clk/rockchip/clk-muxgrf.c | 2 +- > drivers/clk/rockchip/clk-pll.c | 2 +- > drivers/clk/rockchip/clk.c | 2 +- > 8 files changed, 8 insertions(+), 8 deletions(-) >=20 > diff --git a/drivers/clk/rockchip/clk-cpu.c b/drivers/clk/rockchip/clk-cp= u.c > index 6205af9ba88a8def..eb6b8b878e4dbeea 100644 > --- a/drivers/clk/rockchip/clk-cpu.c > +++ b/drivers/clk/rockchip/clk-cpu.c > @@ -304,7 +304,7 @@ struct clk *rockchip_clk_register_cpuclk(const char *= name, > int nrates, void __iomem *reg_base, spinlock_t *lock) > { > struct rockchip_cpuclk *cpuclk; > - struct clk_init_data init; > + struct clk_init_data init =3D {}; > struct clk *clk, *cclk; > int ret; > =20 > diff --git a/drivers/clk/rockchip/clk-ddr.c b/drivers/clk/rockchip/clk-dd= r.c > index 81ae8a4ee30ed95c..fbb702d0b9d996d5 100644 > --- a/drivers/clk/rockchip/clk-ddr.c > +++ b/drivers/clk/rockchip/clk-ddr.c > @@ -97,7 +97,7 @@ struct clk *rockchip_clk_register_ddrclk(const char *na= me, int flags, > spinlock_t *lock) > { > struct rockchip_ddrclk *ddrclk; > - struct clk_init_data init; > + struct clk_init_data init =3D {}; > struct clk *clk; > =20 > ddrclk =3D kzalloc_obj(*ddrclk); > diff --git a/drivers/clk/rockchip/clk-gate-grf.c b/drivers/clk/rockchip/c= lk-gate-grf.c > index 88e409e91a636aa1..458314a2569181d1 100644 > --- a/drivers/clk/rockchip/clk-gate-grf.c > +++ b/drivers/clk/rockchip/clk-gate-grf.c > @@ -73,7 +73,7 @@ struct clk *rockchip_clk_register_gate_grf(const char *= name, > u8 gate_flags) > { > struct rockchip_gate_grf *gate; > - struct clk_init_data init; > + struct clk_init_data init =3D {}; > struct clk *clk; > =20 > if (IS_ERR(regmap)) { > diff --git a/drivers/clk/rockchip/clk-inverter.c b/drivers/clk/rockchip/c= lk-inverter.c > index bf9072ed9fd4daed..e6c7dad60e966288 100644 > --- a/drivers/clk/rockchip/clk-inverter.c > +++ b/drivers/clk/rockchip/clk-inverter.c > @@ -75,8 +75,8 @@ struct clk *rockchip_clk_register_inverter(const char *= name, > void __iomem *reg, int shift, int flags, > spinlock_t *lock) > { > - struct clk_init_data init; > struct rockchip_inv_clock *inv_clock; > + struct clk_init_data init =3D {}; > struct clk *clk; > =20 > inv_clock =3D kmalloc_obj(*inv_clock); > diff --git a/drivers/clk/rockchip/clk-mmc-phase.c b/drivers/clk/rockchip/= clk-mmc-phase.c > index 4efd22953f3e0533..d92bea75faf73767 100644 > --- a/drivers/clk/rockchip/clk-mmc-phase.c > +++ b/drivers/clk/rockchip/clk-mmc-phase.c > @@ -205,8 +205,8 @@ struct clk *rockchip_clk_register_mmc(const char *nam= e, > struct regmap *grf, int grf_reg, > int shift) > { > - struct clk_init_data init; > struct rockchip_mmc_clock *mmc_clock; > + struct clk_init_data init =3D {}; > struct clk *clk; > int ret; > =20 > diff --git a/drivers/clk/rockchip/clk-muxgrf.c b/drivers/clk/rockchip/clk= -muxgrf.c > index ccbb1379501ab474..ecdc77654813a3c3 100644 > --- a/drivers/clk/rockchip/clk-muxgrf.c > +++ b/drivers/clk/rockchip/clk-muxgrf.c > @@ -59,7 +59,7 @@ struct clk *rockchip_clk_register_muxgrf(const char *na= me, > int shift, int width, int mux_flags) > { > struct rockchip_muxgrf_clock *muxgrf_clock; > - struct clk_init_data init; > + struct clk_init_data init =3D {}; > struct clk *clk; > =20 > if (IS_ERR(regmap)) { > diff --git a/drivers/clk/rockchip/clk-pll.c b/drivers/clk/rockchip/clk-pl= l.c > index 706ca4b344d399a3..7204b09243d483ae 100644 > --- a/drivers/clk/rockchip/clk-pll.c > +++ b/drivers/clk/rockchip/clk-pll.c > @@ -1069,7 +1069,7 @@ struct clk *rockchip_clk_register_pll(struct rockch= ip_clk_provider *ctx, > unsigned long flags, u8 clk_pll_flags) > { > const char *pll_parents[3]; > - struct clk_init_data init; > + struct clk_init_data init =3D {}; > struct rockchip_clk_pll *pll; > struct clk_mux *pll_mux; > struct clk *pll_clk, *mux_clk; > diff --git a/drivers/clk/rockchip/clk.c b/drivers/clk/rockchip/clk.c > index ee8c79b938d3c15b..10df21178ba483e2 100644 > --- a/drivers/clk/rockchip/clk.c > +++ b/drivers/clk/rockchip/clk.c > @@ -257,7 +257,7 @@ static struct clk *rockchip_clk_register_frac_branch( > =20 > if (child) { > struct clk_mux *frac_mux =3D &frac->mux; > - struct clk_init_data init; > + struct clk_init_data init =3D {}; > struct clk *mux_clk; > int ret; > =20 > --=20 > 2.43.0 >=20 >=20 --zybuuvaovsaw7bmk Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEE72YNB0Y/i3JqeVQT2O7X88g7+poFAmqIWusACgkQ2O7X88g7 +poirhAApVi+Y+WU3mrw03dbGLE92vZNTJMxmeK5fzM3D7/o1HO6GGgy5ZBAlAcY AHjbpJgLpOY+bLGDnfv97vri7zP1duirRf2Yw3fhqiFp+tdt8AX6f9hU8bfwu+uy IaquBhUMuFejInZcStNJFlC4BF1UJkkacYSNtxFWAsKFMLlzOJQKu7lnBZwfbFWu C+FZ7xkWGWrSGNvroDobqiM/LDEYmI6OxnSwbCfoeczBF6D1VEaSeZhbDDVfx0hR ja1QtzHWcfm4Y3PLzEaNpmYerMSNE1fZIMFmraZZ3oFaak+uOYWZEWk0UzeZF4cv L5jzi4Cwa2V4TIot/53rF+xlSAjZAxTrBEH99onUWqBf4XyMa4k2DnlE+wPNqG5p Mz4QW3dtELwXRsHfeWqAT+rF2Iv2rsb8mkWeJ9ir0Ll2v8PHl2Ro9gWyag1zeRF5 cahZmOPxJFpLKOVsNZX6AAhgvlsJN8kSZ8T48gpuTj+TjdEzrvm8A8l2yU5Or/C1 kT56IUoCapPk1sZVEqNNPZQZduKm53nfXe9viHXg4BVjz8xXOU2LAZJ/gvrqjqzX XzRHvPAAdb+4q0BOqXmFgI6e4nSDBivR2zl2aidohQnxA++NPmGSG0xjBNZAqSY1 HSFBW4PATNUCcY+1z4MjEfCGtmWtnYk0d23W6+Va0wfJjBcmMi8= =BMUY -----END PGP SIGNATURE----- --zybuuvaovsaw7bmk--