Re: [PATCH 5/8] lib/cpumask: change return types to unsigned where appropriate

From: Andy Shevchenko
Date: Sat Jul 02 2022 - 08:00:50 EST


On Fri, Jul 01, 2022 at 05:54:27AM -0700, Yury Norov wrote:
> Switch return types to unsigned int where return values cannot be negative.

...

> -int cpumask_any_and_distribute(const struct cpumask *src1p,
> +unsigned int cpumask_any_and_distribute(const struct cpumask *src1p,
> const struct cpumask *src2p);

It breaks indentation of the second line.

...

> -int cpumask_next_and(int n, const struct cpumask *src1p,
> +unsigned int cpumask_next_and(int n, const struct cpumask *src1p,
> const struct cpumask *src2p)

Ditto.

And seems a lot of cases in the series like this.

--
With Best Regards,
Andy Shevchenko