2023-07-21 13:52:20

by Andy Shevchenko

[permalink] [raw]
Subject: Re: [PATCH v3 03/42] clk: ep93xx: add DT support for Cirrus EP93xx

On Thu, Jul 20, 2023 at 04:27:45PM -0700, Stephen Boyd wrote:
> Quoting Nikita Shubin via B4 Relay (2023-07-20 04:29:03)

...

> > +static bool is_best(unsigned long rate, unsigned long now,
> > + unsigned long best)
> > +{
> > + return abs(rate - now) < abs(rate - best);
>
> Another case where we need abs_diff() so that it doesn't get confused
> when trying to do signed comparison.

Here you are: Message-Id: <[email protected]>

--
With Best Regards,
Andy Shevchenko