2023-09-12 11:21:21

by Linus Walleij

[permalink] [raw]
Subject: Re: [PATCH 1/3] pinctrl: da9062: add missing include

On Fri, Sep 1, 2023 at 1:29 PM Bartosz Golaszewski <[email protected]> wrote:

> From: Bartosz Golaszewski <[email protected]>
>
> gpiod_is_active_low() is defined in linux/gpio/consumer.h. It's only
> because we're pulling in the gpiolib.h private header that we get this
> declaration implicitly but let's fix it as that is going away soon.
>
> Signed-off-by: Bartosz Golaszewski <[email protected]>

Acked-by: Linus Walleij <[email protected]>
For allt three patches.
Just apply them to the GPIO tree, that's easiest!

Yours,
Linus Walleij


2023-09-14 01:38:08

by Bartosz Golaszewski

[permalink] [raw]
Subject: Re: [PATCH 1/3] pinctrl: da9062: add missing include

On Tue, Sep 12, 2023 at 9:30 AM Linus Walleij <[email protected]> wrote:
>
> On Fri, Sep 1, 2023 at 1:29 PM Bartosz Golaszewski <[email protected]> wrote:
>
> > From: Bartosz Golaszewski <[email protected]>
> >
> > gpiod_is_active_low() is defined in linux/gpio/consumer.h. It's only
> > because we're pulling in the gpiolib.h private header that we get this
> > declaration implicitly but let's fix it as that is going away soon.
> >
> > Signed-off-by: Bartosz Golaszewski <[email protected]>
>
> Acked-by: Linus Walleij <[email protected]>
> For allt three patches.
> Just apply them to the GPIO tree, that's easiest!
>
> Yours,
> Linus Walleij

Ok, queued, thanks!

Bart