From: Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxx> On Mon, 10 Feb 2025 16:33:26 -0600, David Lechner wrote: > This series was inspired by some minor annoyance I have experienced a > few times in recent reviews. > > Calling gpiod_set_array_value_cansleep() can be quite verbose due to > having so many parameters. In most cases, we already have a struct > gpio_descs that contains the first 3 parameters so we end up with 3 (or > often even 6) pointer indirections at each call site. Also, people have > a tendency to want to hard-code the first argument instead of using > struct gpio_descs.ndescs, often without checking that ndescs >= the > hard-coded value. > > [...] Applied, thanks! [01/15] gpiolib: add gpiod_multi_set_value_cansleep() commit: 91931af18bd22437e08e2471f5484d6fbdd8ab93 Best regards, -- Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxx>