[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <42585223-2d92-5a9e-3707-d5e70fe6937b@amazon.com>
Date: Tue, 20 Dec 2022 21:43:25 +0200
From: "Hawa, Hanna" <hhhawa@...zon.com>
To: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
CC: <wsa@...nel.org>, <jarkko.nikula@...ux.intel.com>,
<mika.westerberg@...ux.intel.com>, <jsd@...ihalf.com>,
<linus.walleij@...aro.org>, <ben-linux@...ff.org>,
<linux-i2c@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
<dwmw@...zon.co.uk>, <benh@...zon.com>, <ronenk@...zon.com>,
<talel@...zon.com>, <jonnyc@...zon.com>, <hanochu@...zon.com>,
<farbere@...zon.com>, <itamark@...zon.com>,
Lareine Khawaly <lareine@...zon.com>
Subject: Re: [PATCH v4 1/1] i2c: designware: use casting of u64 in clock
multiplication to avoid overflow
On 12/20/2022 9:27 PM, Andy Shevchenko wrote:
> Thinking more on this, I would probably replace the order of arguments to make
> it ' + offset - N' in each case. Since plus will be on the previous line and
> become first it will be easier to parse the arithmetical expression.
Do you want this change in the same patch? i don't think it's related
here.. I can push separated change
Thanks,
Hanna
Powered by blists - more mailing lists