On Fri, Jan 31, 2025 at 10:25 PM David Lechner <dlechner@xxxxxxxxxxxx> wrote: > > Reduce verbosity by using gpiods_set_array_value_cansleep() instead of > gpiods_set_array_value_cansleep(). ... > -static void gpiod_set_array_single_value_cansleep(unsigned int ndescs, > - struct gpio_desc **desc, > - struct gpio_array *info, > +static void gpiod_set_array_single_value_cansleep(struct gpio_descs *descs, > int value) This name puts its feet to the GPIO territory. Since you are changing it, can you make sure it has a local namespace and not GPIO one? -- With Best Regards, Andy Shevchenko