Re: [PATCH 1/1] i2c: designware: use u64 for clock freq to avoid u32 multiplication overflow
From: Andy Shevchenko
Date: Wed Dec 14 2022 - 06:45:14 EST
On Wed, Dec 14, 2022 at 10:34:18AM +0000, Hanna Hawa wrote:
> From: Lareine Khawaly <lareine@xxxxxxxxxx>
>
> In functions i2c_dw_scl_lcnt() and i2c_dw_scl_hcnt() may have overflow
> by depending on the values of the given parameters including the ic_clk.
> For example in our usecase where ic_clk >= 1000000, multiplication of
It's hard to count 0:s, can you use plain words instead (like million, billion)?
> ic_clk * 4700 will result in 32 bit overflow.
> This commit change the ic_clk to be u64 parameter to avoid the overflow.
Read Submitting Patches about imperative mood in the commit messages.
...
Smells like you are missing the Fixes tag.
--
With Best Regards,
Andy Shevchenko