On Tue, Apr 05, 2022 at 02:21:42PM +0200, Bartosz Golaszewski wrote: > On Wed, Mar 30, 2022 at 4:59 PM Andy Shevchenko > <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote: ... > > -#define for_each_gpio_desc_with_flag(i, gc, desc, flag) \ > > +#define for_each_gpio_desc(i, gc, desc) \ > > While at it: how about declaring the i variable in the loop definition? It will require changes in the users, so, I will do it as a prerequisite separate change. > > for (i = 0, desc = gpiochip_get_desc(gc, i); \ > > i < gc->ngpio; \ > > - i++, desc = gpiochip_get_desc(gc, i)) \ > > + i++, desc = gpiochip_get_desc(gc, i)) -- With Best Regards, Andy Shevchenko