From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (prime256v1) server-digest SHA256) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8EF87606A329 for ; Fri, 6 Nov 2026 08:35:27 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key, unprotected) header.d=lists.infradead.org header.i=@lists.infradead.org header.a=rsa-sha256 header.s=bombadil.20210309 header.b=u+xCT5VE; dkim=fail reason="signature verification failed" (2048-bit key, unprotected) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256 header.s=k20260515 header.b=XHNjp+vt 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:MIME-Version:Reply-To:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:Date:References: In-Reply-To:Cc:To:Subject:From:Message-ID:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Unsubscribe-Post:List-Owner; bh=rXLJlxeIn+6kgDhw85rRzcVZyp+Mq+Xt206142gYxTE=; b=u+xCT5VEt7HTWibTQHYe1XtSEV ei+LywF1rCZcuI8qra9E08p+TRMXVtQNqioVz5UPW1K5M5cVoa4Jgk8XWm017YwZwCpYlgTiF47yH ZW8m1KvdseaOi1mxzV1YO6dOkdLPsUQMtin3FknNJTfPfEGAc1O8UA7mjbPPDZd+jimdqqPpc/UvZ 6FIJsAlWSZd9SwIklEVx031uBJNgqrddRo+Tv2Nkr6sJhHKZL7VJA2+pOtftd7NBIJIUhU5qV87ww 08pnbtnYY83g+eHZlA3zo2wpNmSioFwZkXMqZZQTOMIh5hGF04ljGAZXlCiU/lWEOn3d3k+8EYUw2 ILaugBJw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.100.1 #2 (Red Hat Linux)) id 1xPFPw-0000000BbR3-0G9y; Fri, 06 Nov 2026 08:35:24 +0000 Received: from tor.source.kernel.org ([172.105.4.254]) by bombadil.infradead.org with esmtps (Exim 4.100.1 #2 (Red Hat Linux)) id 1xPFPu-0000000BbQh-1bDi for linux-amlogic@lists.infradead.org; Fri, 06 Nov 2026 08:35:22 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by tor.source.kernel.org (Postfix) with ESMTP id D0A3860224; Fri, 6 Nov 2026 08:35:21 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 2BB0B1F008A3; Fri, 6 Nov 2026 08:35:21 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1793954121; bh=f3PG1p+JcFz4gGeGrXXjHJetKWIiSij3xc+yaIrEX1o=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=XHNjp+vtPW+YXFY8uhIbZ4nb8t2tISru5MOO0SiWR30+lKr8gYkQQHi8TZbJablBi kWZNcUFEcPUjRBH2BpmWB8O3eXekvqlUxO7bTOy7nVVAxMnwPCHtLvTORNX3SjCAVZ u523B1NKo/fPdCqw95ZSAOnn/VVLuD2Lg+9PJpnH0HQjmpTF0BMv/tcK8lG4CahW2g OhPv/othBFWV47/tPYJD4F4wOEGGQP9kGLYtvWECI3WA6085JFkFoBAdtVBd28AWlL 9fMNHC0YmrHXPXIB6UlVfSqwnpH5waVhcBUfaJipAPaectPQlxN6j3r7EQZNtKbzUF vtdnz/+VS2wog== Message-ID: From: sashiko-bot@kernel.org Subject: Re: [PATCH 2/7] clk: meson: pll: Support PLLs without a pre-divider To: "Carlo Caione" Cc: linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-amlogic@lists.infradead.org, robh@kernel.org, conor+dt@kernel.org, neil.armstrong@linaro.org In-Reply-To: <20261106-upstream-s7-clk-v1-2-7d501d62621f@baylibre.com> References: <20261106-upstream-s7-clk-v1-0-7d501d62621f@baylibre.com> <20261106-upstream-s7-clk-v1-2-7d501d62621f@baylibre.com> Date: Fri, 06 Nov 2026 08:35:20 +0000 X-BeenThere: linux-amlogic@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: sashiko-reviews@lists.linux.dev X-Archived-By: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-amlogic" X-Spam-Status: No, score=-8.9 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_EF,HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=4.0.1 X-Spam-Checker-Version: SpamAssassin 4.0.1 (2024-03-26) on aws-us-west-2-korg-listserv-alma10-1 VGhhbmsgeW91IGZvciB5b3VyIGNvbnRyaWJ1dGlvbiEgU2FzaGlrbyBBSSByZXZpZXcgZm91bmQg bm8gbmV3IGlzc3VlcwppbiB0aGlzIHBhdGNoLCBidXQgbm90aWNlZCAxIHJlbGF0ZWQgcHJlLWV4 aXN0aW5nIGlzc3VlIGl0IGRvZXMgbm90CmludHJvZHVjZToKLSBbSGlnaF0gY2xrOiBtZXNvbjog MzItYml0IGludGVnZXIgb3ZlcmZsb3cgaW4KICBtZXNvbl9jbGtfZ2V0X3BsbF9yYW5nZV9pbmRl eCgpIFsxXQotLQoKY29tbWl0IDI0MTAzNWVjZmRiMjA2YWNkY2YyOTBhNWUyODgxMDRhMDU2YTNm OGIKQXV0aG9yOiBDYXJsbyBDYWlvbmUgPGNjYWlvbmVAYmF5bGlicmUuY29tPgoKY2xrOiBtZXNv bjogcGxsOiBTdXBwb3J0IFBMTHMgd2l0aG91dCBhIHByZS1kaXZpZGVyCgpUaGlzIGNvbW1pdCBt YWtlcyB0aGUgUExMIHByZS1kaXZpZGVyIHBhcmFtZXRlciBvcHRpb25hbCBpbiB0aGUgY29tbW9u IE1lc29uClBMTCBkcml2ZXIsIGRlZmF1bHRpbmcgdGhlIGRpdmlkZXIgdG8gMSB3aGVuIHRoZSBw YXJhbWV0ZXIgaXMgbm90IGFwcGxpY2FibGUuCgo+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2Nsay9t ZXNvbi9jbGstcGxsLmMgYi9kcml2ZXJzL2Nsay9tZXNvbi9jbGstcGxsLmMKPiBpbmRleCAxZWE2 NTc5YTc2MGYzLi40NGIzMzA1MTQwNWY1IDEwMDY0NAo+IC0tLSBhL2RyaXZlcnMvY2xrL21lc29u L2Nsay1wbGwuYwo+ICsrKyBiL2RyaXZlcnMvY2xrL21lc29uL2Nsay1wbGwuYwoKWyAuLi4gXQoK PiBAQCAtMTc0LDcgKzE3Niw3IEBAIHN0YXRpYyBpbnQgbWVzb25fY2xrX2dldF9wbGxfcmFuZ2Vf aW5kZXgodW5zaWduZWQgbG9uZyByYXRlLAo+ICAJKm4gPSBpbmRleCArIDE7Cj4gIAo+ICAJLyog Q2hlY2sgdGhlIHByZWRpdmlkZXIgcmFuZ2UgKi8KPiAtCWlmICgqbiA+PSAoMSA8PCBwbGwtPm4u d2lkdGgpKQo+ICsJaWYgKCpuID49IChNRVNPTl9QQVJNX0FQUExJQ0FCTEUoJnBsbC0+bikgPyAx IDw8IHBsbC0+bi53aWR0aCA6IDIpKQo+ICAJCXJldHVybiAtRUlOVkFMOwo+ICAKPiAgCWlmICgq biA9PSAxKSB7CgpbU2V2ZXJpdHk6IEhpZ2hdClRoaXMgcHJvYmxlbSB3YXNuJ3QgaW50cm9kdWNl ZCBieSB0aGlzIHBhdGNoLCBidXQgY2FuIGBwbGwtPnJhbmdlLT5tYXggKgpwYXJlbnRfcmF0ZWAg aW4gbWVzb25fY2xrX2dldF9wbGxfcmFuZ2VfaW5kZXgoKSBhbmQgYHBhcmVudF9yYXRlICogbWAg aW4KX19wbGxfcGFyYW1zX3dpdGhfZnJhYygpIG92ZXJmbG93IDMyLWJpdCB1bnNpZ25lZCBsb25n IGFyaXRobWV0aWMgWzFdPwoKZHJpdmVycy9jbGsvbWVzb24vY2xrLXBsbC5jOm1lc29uX2Nsa19n ZXRfcGxsX3JhbmdlX2luZGV4KCkgewoJLi4uCglpZiAoKm4gPT0gMSkgewoJCS8qIEdldCB0aGUg Ym91bmRhcmllcyBvdXQgdGhlIHdheSAqLwoJCWlmIChyYXRlIDw9IHBsbC0+cmFuZ2UtPm1pbiAq IHBhcmVudF9yYXRlKSB7CgkJCSptID0gcGxsLT5yYW5nZS0+bWluOwoJCQlyZXR1cm4gLUVOT0RB VEE7CgkJfSBlbHNlIGlmIChyYXRlID49IHBsbC0+cmFuZ2UtPm1heCAqIHBhcmVudF9yYXRlKSB7 CgkJCSptID0gcGxsLT5yYW5nZS0+bWF4OwoJCQlyZXR1cm4gLUVOT0RBVEE7CgkJfQoJfQoJLi4u Cn0KCmRyaXZlcnMvY2xrL21lc29uL2Nsay1wbGwuYzpfX3BsbF9wYXJhbXNfd2l0aF9mcmFjKCkg ewoJLi4uCgkvKiBCYWlsIG91dCBpZiB3ZSBhcmUgYWxyZWFkeSBvdmVyIHRoZSByZXF1ZXN0ZWQg cmF0ZSAqLwoJaWYgKHJhdGUgPCBwYXJlbnRfcmF0ZSAqIG0gLyBuKQoJCXJldHVybiAwOwoJLi4u Cn0KCk9uIDMyLWJpdCBhcmNoaXRlY3R1cmVzLCBtdWx0aXBseWluZyBhIDI0IE1IeiBgcGFyZW50 X3JhdGVgIGJ5IGEgdHlwaWNhbAptdWx0aXBsaWVyIChzdWNoIGFzIGBwbGwtPnJhbmdlLT5tYXhg IG9mIDI1MCkgcmVhY2hlcyA2LDAwMCwwMDAsMDAwIGFuZCB3cmFwcwphcm91bmQgMzItYml0IGBV TE9OR19NQVhgLiAgVGhpcyBjYXVzZXMgdGhlIHJhbmdlIGNoZWNrIGluCm1lc29uX2Nsa19nZXRf cGxsX3JhbmdlX2luZGV4KCkgdG8gbWlzY2FsY3VsYXRlIGBtYCBhbmQgdGhlIGd1YXJkIGluCl9f cGxsX3BhcmFtc193aXRoX2ZyYWMoKSB0byBiZSBieXBhc3NlZCwgdW5kZXJmbG93aW5nIGB2YWwg LT0gbSAqIGZyYWNfbWF4YAphbmQgd3JpdGluZyBjb3JydXB0ZWQgbXVsdGlwbGllciBhbmQgZnJh Y3Rpb25hbCB2YWx1ZXMgdG8gdGhlIFBMTCBoYXJkd2FyZS4KClsxXSBodHRwczovL3Nhc2hpa28u ZGV2LyMvYnVnL2xpbnV4LThhNWVjNDMyLWVmYmQtNGQzZi04M2M3LTJhNTBmYzI4NTBjNwoKLS0g ClNhc2hpa28gQUkgcmV2aWV3IMK3IGh0dHBzOi8vc2FzaGlrby5kZXYvIy9wYXRjaHNldC8yMDI2 MTEwNi11cHN0cmVhbS1zNy1jbGstdjEtMC03ZDUwMWQ2MjYyMWZAYmF5bGlicmUuY29tP3BhcnQ9 MgoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KbGludXgt YW1sb2dpYyBtYWlsaW5nIGxpc3QKbGludXgtYW1sb2dpY0BsaXN0cy5pbmZyYWRlYWQub3JnCmh0 dHA6Ly9saXN0cy5pbmZyYWRlYWQub3JnL21haWxtYW4vbGlzdGluZm8vbGludXgtYW1sb2dpYwo= From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.remote-ip=100.103.45.18 ARC-Seal: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; cv=none; t=1793954122; b=ApS2qyExtsxw1dvUKy2geCGbQj+oLqbmpko/Eo0LTIKpOXFEqEXAkudaPyFm3h1VUttb 37AgRMJ/63pRe5Ur/YN54WoIaxQsQ5IvHQdbWBagUZ+r/XRsUlse4X8uzUHe9Vyvob09I ++4tMcFeMAAqnQL0IyBDM3/jKviw9ifztRONIK8jSt8dpQEkxntJIrHJTDUN57GxjKrLO rSbVRDOOqnb1qJ5lj6aQvN46xhgR78kigtsYYNAh4In/BvD/gmO/Li8vSvvSaLvr8rpbF 4KBFqunilMB5Q0B20sZu+04CmYu+JjLt7b88FTmAQj6YorJ5lFoYIoKxDAOxmhgd8UQ== ARC-Message-Signature: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; c=relaxed/relaxed; t=1793954122; h=Received:DKIM-Signature:Message-ID:From:Subject:Reply-To:To:Cc: In-Reply-To:References:Content-Type:Content-Transfer-Encoding:Date; bh=f3PG1p+JcFz4gGeGrXXjHJetKWIiSij3xc+yaIrEX1o=; b=SSmu+uLhVN+NnFBP3i/4U2+0BibCOvf5xZvvLV2X1HoOaN8DJk8mOV52OfsIQXYHfudc h9BueulOKClnXEXzxYNNDI8fRAhphA92irUbujsCzELjo5zXESjyHlrt579NrAOWZ0U9J C/T5xfOSlm+vNtlpEYp5z0GoUjv8dXb0uZnDSVgJQ01Tbuqhpe1aoL+7YVqwZnhi2/HjJ pkaAbcdpECfCZ1qI2wzui3bUauV5/88E2WtB84CHWGXPyalBAR3GOiOtF1YtMs77qKHij GgIvWx+VPIJpvBDzrKw05CJ4khNFAqk8Wjp8wTASSmKxpCGmG/BijhNEBwdbc0aJMkw== ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256 header.s=k20260515 header.b=XHNjp+vt; arc=none smtp.remote-ip=100.103.45.18 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (prime256v1)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E6CFA606A32C; Fri, 6 Nov 2026 08:35:21 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key, unprotected) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256 header.s=k20260515 header.b=XHNjp+vt Received: by smtp.kernel.org (Postfix) with ESMTPSA id 2BB0B1F008A3; Fri, 6 Nov 2026 08:35:21 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1793954121; bh=f3PG1p+JcFz4gGeGrXXjHJetKWIiSij3xc+yaIrEX1o=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=XHNjp+vtPW+YXFY8uhIbZ4nb8t2tISru5MOO0SiWR30+lKr8gYkQQHi8TZbJablBi kWZNcUFEcPUjRBH2BpmWB8O3eXekvqlUxO7bTOy7nVVAxMnwPCHtLvTORNX3SjCAVZ u523B1NKo/fPdCqw95ZSAOnn/VVLuD2Lg+9PJpnH0HQjmpTF0BMv/tcK8lG4CahW2g OhPv/othBFWV47/tPYJD4F4wOEGGQP9kGLYtvWECI3WA6085JFkFoBAdtVBd28AWlL 9fMNHC0YmrHXPXIB6UlVfSqwnpH5waVhcBUfaJipAPaectPQlxN6j3r7EQZNtKbzUF vtdnz/+VS2wog== Message-ID: From: sashiko-bot@kernel.org Subject: Re: [PATCH 2/7] clk: meson: pll: Support PLLs without a pre-divider Reply-To: sashiko-reviews@lists.linux.dev To: "Carlo Caione" Cc: linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-amlogic@lists.infradead.org, robh@kernel.org, conor+dt@kernel.org, neil.armstrong@linaro.org In-Reply-To: <20261106-upstream-s7-clk-v1-2-7d501d62621f@baylibre.com> References: <20261106-upstream-s7-clk-v1-0-7d501d62621f@baylibre.com> <20261106-upstream-s7-clk-v1-2-7d501d62621f@baylibre.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Date: Fri, 06 Nov 2026 08:35:20 +0000 X-Mailing-List: linux-clk@vger.kernel.org List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list Thank you for your contribution! Sashiko AI review found no new issues in this patch, but noticed 1 related pre-existing issue it does not introduce: - [High] clk: meson: 32-bit integer overflow in meson_clk_get_pll_range_index() [1] -- commit 241035ecfdb206acdcf290a5e288104a056a3f8b Author: Carlo Caione clk: meson: pll: Support PLLs without a pre-divider This commit makes the PLL pre-divider parameter optional in the common Meson PLL driver, defaulting the divider to 1 when the parameter is not applicabl= e. > diff --git a/drivers/clk/meson/clk-pll.c b/drivers/clk/meson/clk-pll.c > index 1ea6579a760f3..44b33051405f5 100644 > --- a/drivers/clk/meson/clk-pll.c > +++ b/drivers/clk/meson/clk-pll.c [ ... ] > @@ -174,7 +176,7 @@ static int meson_clk_get_pll_range_index(unsigned lon= g rate, > *n =3D index + 1; > =20 > /* Check the predivider range */ > - if (*n >=3D (1 << pll->n.width)) > + if (*n >=3D (MESON_PARM_APPLICABLE(&pll->n) ? 1 << pll->n.width : 2)) > return -EINVAL; > =20 > if (*n =3D=3D 1) { [Severity: High] This problem wasn't introduced by this patch, but can `pll->range->max * parent_rate` in meson_clk_get_pll_range_index() and `parent_rate * m` in __pll_params_with_frac() overflow 32-bit unsigned long arithmetic [1]? drivers/clk/meson/clk-pll.c:meson_clk_get_pll_range_index() { ... if (*n =3D=3D 1) { /* Get the boundaries out the way */ if (rate <=3D pll->range->min * parent_rate) { *m =3D pll->range->min; return -ENODATA; } else if (rate >=3D pll->range->max * parent_rate) { *m =3D pll->range->max; return -ENODATA; } } ... } drivers/clk/meson/clk-pll.c:__pll_params_with_frac() { ... /* Bail out if we are already over the requested rate */ if (rate < parent_rate * m / n) return 0; ... } On 32-bit architectures, multiplying a 24 MHz `parent_rate` by a typical multiplier (such as `pll->range->max` of 250) reaches 6,000,000,000 and wra= ps around 32-bit `ULONG_MAX`. This causes the range check in meson_clk_get_pll_range_index() to miscalculate `m` and the guard in __pll_params_with_frac() to be bypassed, underflowing `val -=3D m * frac_ma= x` and writing corrupted multiplier and fractional values to the PLL hardware. [1] https://sashiko.dev/#/bug/linux-8a5ec432-efbd-4d3f-83c7-2a50fc2850c7 --=20 Sashiko AI review =C2=B7 https://sashiko.dev/#/patchset/20261106-upstream-s= 7-clk-v1-0-7d501d62621f@baylibre.com?part=3D2