Re: [v2 PATCH] PCI: aardvark: switch to using devm_gpiod_get_optional()

From: Dmitry Torokhov
Date: Wed Dec 07 2022 - 16:09:25 EST


On Wed, Dec 07, 2022 at 02:18:07PM -0600, Bjorn Helgaas wrote:
> On Wed, Dec 07, 2022 at 10:55:06AM -0800, Dmitry Torokhov wrote:
> > ...
> > I wonder if you could also consider picking up the mvebu patch:
> >
> > https://lore.kernel.org/all/Y3KbhIi4ZsSO7+Cl@xxxxxxxxxx/
> >
> > pci-mvebu.c is the very last user of of_get_named_gpio_flags() in the
> > next tree, which we also want to stop exporting.
>
> Sure, but the 0-day bot found a couple issues, which I haven't
> bothered to look into:
>
> https://lore.kernel.org/all/202211151503.HLlq2Z4B-lkp@xxxxxxxxx/

Ugh, indeed, I asumed this was an existing issue and of course I was
wrong. linux/irqchip/chained_irq.h was previously included indirectly
via linux/of_gpio.h and linux/gpio/driver.h.

I sent out v3 tha compiles cleanly.

Thanks.

--
Dmitry