[PATCH v2 06/13] iio: adc: ad7606: use gpiod_multi_set_value_cansleep
Jonathan Cameron
jic23 at kernel.org
Sat Feb 8 05:23:13 PST 2025
On Fri, 7 Feb 2025 14:13:23 +0200
Andy Shevchenko <andy.shevchenko at gmail.com> wrote:
> On Fri, Feb 7, 2025 at 12:48 AM David Lechner <dlechner at baylibre.com> wrote:
> >
> > Reduce verbosity by using gpiod_multi_set_value_cansleep() instead of
> > gpiod_set_array_value().
> >
> > These are not called in an atomic context, so changing to the cansleep
> > variant is fine.
>
> ...
>
> > if (st->gpio_os) {
> > - gpiod_set_array_value(st->gpio_os->ndescs,
> > - st->gpio_os->desc, st->gpio_os->info, os);
> > + gpiod_multi_set_value_cansleep(st->gpio_os, os);
> > }
>
> Now I do not see any justification to leave {}.
>
With that tidied up (good spot!) if this goes through the GPIO tree
Acked-by: Jonathan Cameron <Jonathan.Cameron at huawei.com>
More information about the linux-phy
mailing list