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