Re: [PATCH 00/13] gpiolib: add gpiods_set_array_value_cansleep

From: Jonathan Cameron
Date: Sat Feb 01 2025 - 07:46:23 EST


On Fri, 31 Jan 2025 14:51:52 -0600
David Lechner <dlechner@xxxxxxxxxxxx> wrote:

> On 1/31/25 2:38 PM, Andrew Lunn wrote:
> >> So I'm proposing that we add a gpiods_set_array_value_cansleep()
> >> function that is a wrapper around gpiod_set_array_value_cansleep()
> >> that has struct gpio_descs as the first parameter to make it a bit
> >> easier to read the code and avoid the hard-coding temptation.
> >
> > This looks reasonable.
> >
> > How do you plan to get it merged, since you cross a lot of subsystems
> > here.
> >
> > Andrew
>
> Since these are mostly small changes and most of the touched drivers aren't
> seeing much action, I think it would be OK for as much as possible to go through
> the GPIO tree.
>
> We might need an immutable branch from that though since I know that iio: adc:
> ad7606 is currently being actively worked on.
Looks good to me (subject to requested name change from Bartosz)
I'd suggest an immutable with patch 1 then up to each subsystem
maintainer to pick that up or wait for next cycle.

Always hard to predict what else will get worked on at this stage
of a cycle.

Jonathan


>
> If there are any patches leftover that don't get acked to go through the GPIO
> tree, I can resubmit them after the next kernel release cycle since none of
> this is urgent anyway.
>