Received-SPF: pass (google.com: domain of linux-kernel+bounces-168671-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
From: Peng Fan <peng.fan@nxp.com>
To: Francesco Dolcini <francesco@dolcini.it>, "Peng Fan (OSS)"
	<peng.fan@oss.nxp.com>
CC: Abel Vesa <abelvesa@kernel.org>, Michael Turquette
	<mturquette@baylibre.com>, Stephen Boyd <sboyd@kernel.org>, Shawn Guo
	<shawnguo@kernel.org>, Sascha Hauer <s.hauer@pengutronix.de>, Pengutronix
 Kernel Team <kernel@pengutronix.de>, Fabio Estevam <festevam@gmail.com>,
	Jacky Bai <ping.bai@nxp.com>, Ye Li <ye.li@nxp.com>, Aisheng Dong
	<aisheng.dong@nxp.com>, "linux-clk@vger.kernel.org"
	<linux-clk@vger.kernel.org>, "imx@lists.linux.dev" <imx@lists.linux.dev>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>, "linux-kernel@vger.kernel.org"
	<linux-kernel@vger.kernel.org>, Abel Vesa <abel.vesa@linaro.org>, Florin
 Pavelescu <florin.pavelescu@nxp.com>, Viorel Suman <viorel.suman@nxp.com>
Subject: RE: [PATCH 07/18] clk: imx: pll14xx: potential integer overflow
 eliminated by casting to u64
Thread-Topic: [PATCH 07/18] clk: imx: pll14xx: potential integer overflow
 eliminated by casting to u64
Thread-Index: AQHanbvU3iJph3bBxUSOuV1ImO0GJbGG+UgAgAAUv1A=
Date: Sat, 4 May 2024 13:13:17 +0000
Message-ID:
 <DU0PR04MB941771866E735F5EC83DD46E881E2@DU0PR04MB9417.eurprd04.prod.outlook.com>
References: <20240504-imx-clk-v1-0-f7915489d58d@nxp.com>
 <20240504-imx-clk-v1-7-f7915489d58d@nxp.com>
 <20240504115828.GA3666@francesco-nb>
In-Reply-To: <20240504115828.GA3666@francesco-nb>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0:
 =?us-ascii?Q?gI1d2cWOjU3BzXl8sSdSEtMu4aWOD9CvvndrKx2Wb++DhhDVI8U4tbkBMmYw?=
 =?us-ascii?Q?mExhx7AKSnRg5VpMhwL/+C+2p2Y9C1IPGFGRfD0tGW3ugprD/XwSnrPluSF5?=
 =?us-ascii?Q?rsCDKu8I6xZbFoCRdpS03w0A5T8p1+OdeCSam5F5d/tsU/z+RZfPOYdgnIuT?=
 =?us-ascii?Q?kYReGzHGzqG815PXY1QUg3dnzxkNp6BYfP69qHeJWJmt0bG0xovq8rE23ce6?=
 =?us-ascii?Q?iL3nGkVExXd0G2HK4mfpHsMsb1iB6sga/RQqhWeZCbV0wqEXZpwxI7fdoPja?=
 =?us-ascii?Q?j+I7mAKQBy0vy7Up+2fTHXzwssFSLxrzms5JZpJLMLrn5+tRhhqx3gpgfLfg?=
 =?us-ascii?Q?sggckgkbrJJzFAgxVtb/rdEFj6Du54iUXKdcxAAA0mNqDxEmzNT2D/5fXVr+?=
 =?us-ascii?Q?D0mxRjeRqtfjy3mvPyWo+SFbnkjkPV3SDv7rjkN3Di8yW5YH5Vfx0IhQKNbS?=
 =?us-ascii?Q?IFaIQNuwC9/USuhxBHu7ZPniuNOpUs4AlDsollZJZvnRHeN+bsnE+fMqiEft?=
 =?us-ascii?Q?AervDAm4Dz0zA0LYAdvzGaqiGPZejM3hVsR7bNZB559HxgJJD2GuSPpQHT+r?=
 =?us-ascii?Q?hp99yZt0GAsmSK70rAOp3L+pXYFje5SxWXiMDXWupMlXb6l6n1qd12wS9NfV?=
 =?us-ascii?Q?KYg1bYeTBruHcvoAfFeWI1Vj9ftQNmeYmzRVV4WJxnVuIyRjzWD6QyTlB3ZV?=
 =?us-ascii?Q?gNdWiwOMYq66PyUsgCQ1gJ+qEjktovZ2xjiQIDbKmNAcuddwfoBuhbxQBUWO?=
 =?us-ascii?Q?FuEVrhlo0YWkuCcoPnb09mKtJf0vIhvjWGMIV/G3aIO+ePO2yY5nFj5hA/4h?=
 =?us-ascii?Q?btzDBfHON3kIjM00wgQZcU34Sod7C33Z9u5Rv2VLfi1oXaCfCERCeUR1yMEz?=
 =?us-ascii?Q?qbcUG+i1Yivy5nitu9ylB8XISbodNc7clzZD54jMANSGezI5stik/ETppYRf?=
 =?us-ascii?Q?BzxQ6iygJ5arOlPAR2NNNxr303cdLYFHY9gVqs11XpEHKusUbkYtwCWb2Y1j?=
 =?us-ascii?Q?Juq2aLoqBwA8PfuwlQLacIJaVi76UvUMs1uFeuy2pCQ61jqdJbAeszV+M9IS?=
 =?us-ascii?Q?k/zkYCnwQzd+9NTf3pc3Lr0ATJcTqq1Mrz6kqWRSvCmg7lgf1dPsM0eGSXq1?=
 =?us-ascii?Q?jHHMUMBi4FulI7NKEqKoTdaQ9jISoVBoRdVaHL5yZmJGVJK7GFgMX6ROgoq8?=
 =?us-ascii?Q?zJCjN4QN/V+Sa8ggHVVNlUeAxEMvKMaj+E1p/QFD1VhO7A9vnf0aLu8567+U?=
 =?us-ascii?Q?6+CecB+6ZiHMAqxTeUkO3yOpL4NdMj1AYrS2oAyZTepPcQbKlEv+FIcqFEIi?=
 =?us-ascii?Q?YDG+Me35kPAfnm1QghJse+4HSME5s3dKlUSYWL9mtY9QBjYg2zHtSDZY/lwL?=
 =?us-ascii?Q?lR7MqRzjHkvpZ2Q+xRkgBW9lT7ncE9HVZuDNe5jBby+V3R409rnCVv60yONP?=
 =?us-ascii?Q?X5cu/Oabui6y7kOh9LT3rEcfPmhg49xpA/6fQYDbKHJjC+gdkCFHwiSWmHYW?=
 =?us-ascii?Q?beN/bCysHvTwTEqMICFZ85V0uShmEUBKyZRykD4aoU3IrO0WmYjGlg6kx+m2?=
 =?us-ascii?Q?nN90N+bLusNfZwCBbFM=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
X-Mailing-List: linux-kernel@vger.kernel.org
List-Id: <linux-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org>
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DU0PR04MB9417.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 31e05e8f-6870-458c-5da1-08dc6c3bf64e
X-MS-Exchange-CrossTenant-originalarrivaltime: 04 May 2024 13:13:17.0590
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: TjNP/Iok4kIX33GGQUpJx2JUDkUXqW/sJQjCezipgZ5g318Cyq4aW6fxjtB9WILhtt5DjtFk7akjCgOmGx+5Uw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAXPR04MB8174

> Subject: Re: [PATCH 07/18] clk: imx: pll14xx: potential integer overflow
> eliminated by casting to u64
>=20
> Hello Peng,
>=20
> On Sat, May 04, 2024 at 08:49:00AM +0800, Peng Fan (OSS) wrote:
> > From: Florin Pavelescu <florin.pavelescu@nxp.com>
> >
> > Cast to u64, so that multiplications and additions will be done in
> > 64-bit arithmetic (and the results will also be 64-bit) and the
> > possibility of integer overflow is eliminated.
> >
> > Fix Unintentional integer overflow reported by Coverity
>=20
> Fixes tag if this is a fix?
>=20
> It would be interesting in general if this can be trigger in real life an=
d with
> which side effects or if we are just silencing the tool

This is just to silencing the tool, maybe this patch is no needed.

Thanks,
Peng.